Page MenuHomeFreeBSD
Differential D42564

tty: properly check character position when handling IUTF8 backspaces
ClosedPublic
Actions

Authored by bnovkov on Nov 13 2023, 4:49 PM.
Tags
None
Referenced Files
F102803413: D42564.diff
Sun, Nov 17, 9:34 AM
Unknown Object (File)
Sat, Nov 2, 8:11 AM
Unknown Object (File)
Sat, Oct 19, 9:27 AM
Unknown Object (File)
Oct 6 2024, 3:11 PM
Unknown Object (File)
Oct 4 2024, 11:01 AM
Unknown Object (File)
Oct 3 2024, 12:47 AM
Unknown Object (File)
Oct 1 2024, 10:00 PM
Unknown Object (File)
Oct 1 2024, 5:08 PM
View All 60 Files
Subscribers
imp

Details

Reviewers
christos
Commits
rG31f6cfca851f: tty: properly check character position when handling IUTF8 backspaces
rG8647fe60b8c3: tty: properly check character position when handling IUTF8 backspaces
rGae8387cc818a: tty: properly check character position when handling IUTF8 backspaces
rGc6d7be214811: tty: properly check character position when handling IUTF8 backspaces
Summary

This patch fixes PR 275009.

The tty_rubchar code handling backspaces for UTF8 characters didn't properly check whether the beginning of the current line was reached.
This resulted in a kernel panic in ttyinq_unputchar when prodded with certain malformed UTF8 sequences.

Diff Detail

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

Event Timeline

bnovkov created this revision.Nov 13 2023, 4:49 PM
Herald added a subscriber: imp. · View Herald TranscriptNov 13 2023, 4:49 PM
bnovkov requested review of this revision.Nov 13 2023, 4:49 PM
bnovkov edited the summary of this revision. (Show Details)Nov 13 2023, 5:23 PM
christos accepted this revision.Nov 13 2023, 5:53 PM
This revision is now accepted and ready to land.Nov 13 2023, 5:53 PM
christos added a comment.Nov 13 2023, 5:54 PM

Please provide a full diff.

bnovkov updated this revision to Diff 130025.Nov 13 2023, 5:56 PM

Upload diff with full context.

This revision now requires review to proceed.Nov 13 2023, 5:56 PM
christos accepted this revision.Nov 13 2023, 5:58 PM
This revision is now accepted and ready to land.Nov 13 2023, 5:58 PM
Closed by commit rGc6d7be214811: tty: properly check character position when handling IUTF8 backspaces (authored by bnovkov, committed by christos). · Explain WhyNov 13 2023, 6:05 PM
This revision was automatically updated to reflect the committed changes.
christos added a commit: rGc6d7be214811: tty: properly check character position when handling IUTF8 backspaces.
christos added a commit: rGae8387cc818a: tty: properly check character position when handling IUTF8 backspaces.Nov 20 2023, 4:55 PM
christos added a commit: rG8647fe60b8c3: tty: properly check character position when handling IUTF8 backspaces.Nov 20 2023, 4:58 PM
markj added a commit: rG31f6cfca851f: tty: properly check character position when handling IUTF8 backspaces.Dec 5 2023, 6:28 PM

Revision Contents
Changeset List

PathSize
sys/
kern/
8 lines

Diff 130027

View Options

sys/kern/tty_ttydisc.c

Loading...