From: ebiederm@xmission.com (Eric W. Biederman)
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: Peter Hurley <peter@hurleysoftware.com>, Greg KH <greg@kroah.com>,
        Jiri Slaby <jslaby@suse.com>,
        Linus Torvalds <torvalds@linux-foundation.org>,
        Aurelien Jarno <aurelien@aurel32.net>,
        Andy Lutomirski <luto@amacapital.net>,
        Florian Weimer <fw@deneb.enyo.de>, Al Viro <viro@zeniv.linux.org.uk>,
        Serge Hallyn <serge.hallyn@ubuntu.com>, Jann Horn <jann@thejh.net>,
        "security\@kernel.org" <security@kernel.org>,
        "security\@ubuntu.com \>\> security" <security@ubuntu.com>,
        security@debian.org, Willy Tarreau <w@1wt.eu>,
        linux-kernel@vger.kernel.org
References: <43AD2BA7-B594-4299-95F3-D86FD38AF21B@zytor.com>
	<87egexpf4o.fsf@x220.int.ebiederm.org>
	<CA+55aFw9Bg+Zh_T4zP487n3ieaxoMHgZ_nNJVdpSR4kQK9gQ9w@mail.gmail.com>
	<1CB621EF-1647-463B-A144-D611DB150E15@zytor.com>
	<20151208223135.GA8352@kroah.com>
	<87oae0h2bo.fsf@x220.int.ebiederm.org> <56677DE3.5040705@zytor.com>
	<20151209012311.GA11794@kroah.com>
	<84B136DF-55E4-476A-9CB2-062B15677EE5@zytor.com>
	<20151209013859.GA12442@kroah.com> <20151209083225.GA30452@1wt.eu>
	<87wpskyds7.fsf_-_@x220.int.ebiederm.org>
	<566F1CD7.20502@hurleysoftware.com>
	<87y4cq6t1j.fsf@x220.int.ebiederm.org>
	<87a8p54v3t.fsf@x220.int.ebiederm.org>
	<5AFEA8FF-BDEA-40AF-8C45-19F3E9CC36D3@zytor.com>
	<87r3ih1mnp.fsf@x220.int.ebiederm.org>
Date: Mon, 21 Dec 2015 16:03:42 -0600
In-Reply-To: <87r3ih1mnp.fsf@x220.int.ebiederm.org> (Eric W. Biederman's
	message of "Sun, 20 Dec 2015 03:42:34 -0600")
Message-ID: <87zix3xxvl.fsf@x220.int.ebiederm.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Subject: Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1158
Lines: 31

ebiederm@xmission.com (Eric W. Biederman) writes:

> "H. Peter Anvin" <hpa@zytor.com> writes:
>
>> Does it matter if it mounts devpts twice?  It seems like a waste of a
>> minuscule amount of memory, and nothing else.

> It breaks system("mknod /tmp/ptmx c 5 2"); open("/tmp/ptmx");

Correction.

It does break the above but that isn't the real reason we need to
support that.  We only have evidence of pople doing:
"mkdir -p dev/pts; mknod c dev/ptmx 5 2; mount -t devpts dev/pts/"
Where the relatives paths would work.

What actually breaks is "echo NNN > /proc/sys/kernel/pty/reserve"
Which allows the primary instance of devpts to have access to more
ptys than any other instance.

Ultimately if we are going to be backwards compatible we need to
preserve as much of the current behavior as possible so we don't forget
something in the analysis and break something we don't intend to break
by accident.

Eric
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/