Page MenuHomeFreeBSD
Differential D43456

kern: tty: recanonicalize the buffer on ICANON/VEOF/VEOL changes
ClosedPublic
Actions

Authored by kevans on Jan 15 2024, 5:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Aug 28, 7:21 PM
Unknown Object (File)
Fri, Aug 28, 1:18 PM
Unknown Object (File)
Fri, Aug 28, 12:38 PM
Unknown Object (File)
Fri, Aug 21, 3:23 PM
Unknown Object (File)
Tue, Aug 18, 4:23 AM
Unknown Object (File)
Sun, Aug 16, 6:24 AM
Unknown Object (File)
Thu, Aug 13, 5:14 AM
Unknown Object (File)
Wed, Aug 12, 3:29 PM
View All Files
Subscribers
None

Details

Reviewers
cy
imp
kib
Commits
rG522083ffbd1a: kern: tty: recanonicalize the buffer on ICANON/VEOF/VEOL changes
Summary

Before this change, we would canonicalize any partial input if the new
local mode is not ICANON, but that's about it. If we were switching
from -ICANON -> ICANON, or if VEOF/VEOL changes, then our internal canon
accounting would be wrong.

The main consequence of this is that in ICANON mode, we would potentially
hang a read(2) longer if the new VEOF/VEOL appears later in the buffer,
and FIONREAD would be similarly wrong as a result.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
sys/
kern/
22 lines
49 lines
22 lines
sys/
1 line
1 line

Diff 132804

View Options

sys/kern/tty.c

Loading...
View Options

sys/kern/tty_inq.c

Loading...
View Options

sys/kern/tty_ttydisc.c

Loading...
View Options

sys/sys/ttydisc.h

Loading...
View Options

sys/sys/ttyqueue.h

Loading...