linux-kernel - Re: [PATCH] TTY: export NR_LDISC and N_* line discipline numbers to user-space

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 24 May 2011 16:09:57 +0200
From:	Tilman Schmidt <tilman@...p.cc>
To:	Florian Fainelli <ffainelli@...ebox.fr>
CC:	Greg KH <gregkh@...e.de>, lkml <linux-kernel@...r.kernel.org>,
	Maxime Bizon <mbizon@...ebox.fr>,
	Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [PATCH] TTY: export NR_LDISC and N_* line discipline numbers
 to user-space

Am 24.05.2011 15:17 schrieb Florian Fainelli:
> On Tuesday 24 May 2011 14:40:44 Greg KH wrote:
>> On Tue, May 24, 2011 at 10:43:03AM +0200, Florian Fainelli wrote:
>>> From: Florian Fainelli <ffainelli@...ebox.fr>
>>>
>>> Since commit (4564f9e5: consolidate line discipline number definitions)
>>> the patch moved all line discipline number from a per-architecture
>>> termios.h to a shared one: tty.h. However, prior to this consolidation
>>> work, the line discipline numbers were outside of an ifdef
>>> __KERNEL__/endif block so these numbers used to be exported to
>>> user-space.
>>>
>>> Since such numbers are kernel ABI anyway, and tty.h is already included
>>> for user- space header processing, just move these relevant defines
>>> outside of the ifdef __KERNEL__/endif block in include/linux/tty.h.
[...]
>> Meaning, why has it taken this long for anyone to notice?
> 
> - long standing user-space applications such as pppd got their N_PPP long ago, 
> and do not have to worry about N_PPP being renumbered so they can just define 
> it in pppd's sources it won't break

Also, I just checked on openSUSE 11.2, which comes with kernel
release is 2.6.27, and it still defines the LD numbers in
/usr/include/bits/ioctl-types.h (included from sys/ioctl.h).
I guess other distributions are doing the same, so their users
wouldn't notice the change.

Anyway, the patch is fine by me. Feel free to add:

Acked-by: Tilman Schmidt <tilman@...p.cc>

Thanks,
Tilman

-- 
Tilman Schmidt                    E-Mail: tilman@...p.cc
Bonn, Germany
Diese Nachricht besteht zu 100% aus wiederverwerteten Bits.
Ungeöffnet mindestens haltbar bis: (siehe Rückseite)


Download attachment "signature.asc" of type "application/pgp-signature" (260 bytes)