linux-kernel - Re: Moving tty->stopped logic to ldisc (pty: fix data loss when stopped (^S/^Q))

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, 11 Aug 2009 13:00:10 -0600
From:	Joe Peterson <joe@...rush.com>
To:	Greg KH <gregkh@...e.de>
CC:	Artur Skawina <art.08.09@...il.com>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Alan Cox <alan@...rguk.ukuu.org.uk>,
	Linux Kernel <linux-kernel@...r.kernel.org>
Subject: Re: Moving tty->stopped logic to ldisc (pty: fix data loss when stopped
 (^S/^Q))

Greg KH wrote:
> On Tue, Aug 11, 2009 at 12:16:39PM -0600, Joe Peterson wrote:
>> [Please include my email address in replies]
>>
>> Hi all,
>>
>> This is in regard to the problem that the recent patch ("pty: fix data
>> loss when stopped (^S/^Q)") fixes.  The issue is something I've been
>> looking at recently as well after noticing (I noticed it first as loss
>> of echoes, since I was re-testing what my echo buffer code originally
>> fixed).
> 
> Hm, is this resolved by git id 85dfd81dc57e8183a277ddd7a56aa65c96f3f487
> that Linus commited to the tree yesterday?

Well, that patch "fixes" the issue of the lost chars in that it resumes
previous behavior, but it does not address the two points I mentioned in
this post - ones that are hard to fix without moving checks for
tty->stopped to the ldisc or somehow locking the state of tty->stopped
(which seems a bit scary).  In other words, there still exist windows
for loss of chars.

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