Page MenuHomeFreeBSD
Differential D42098

uiomove: Add some assertions
ClosedPublic
Actions

Authored by markj on Oct 5 2023, 9:36 PM.
Tags
None
Referenced Files
F172680370: D42098.diff
Sun, Sep 20, 4:21 AM
Unknown Object (File)
Aug 12 2026, 9:49 PM
Unknown Object (File)
Aug 10 2026, 9:05 AM
Unknown Object (File)
Aug 9 2026, 4:41 PM
Unknown Object (File)
Aug 9 2026, 4:40 PM
Unknown Object (File)
Aug 8 2026, 12:12 AM
Unknown Object (File)
Aug 5 2026, 8:07 AM
Unknown Object (File)
Aug 4 2026, 2:33 AM
View All Files
Subscribers
emaste
imp
jhibbits
riscv

Details

Reviewers
kib
glebius
andrew
manu
Commits
rG2d49b111a312: uiomove: Add some assertions
rGeb8923452e1e: uiomove: Add some assertions
rG8fd0ec53deaa: uiomove: Add some assertions
Summary

Make sure that we don't try to copy with a negative resid.

Make sure that we don't walk off the end of the iovec array.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 53861
Build 50752: arc lint + arc unit

Event Timeline

markj created this revision.Oct 5 2023, 9:36 PM
Herald added a reviewer: andrew. · View Herald TranscriptOct 5 2023, 9:36 PM
Herald added a reviewer: andrew. · View Herald Transcript
Herald added a reviewer: manu. · View Herald Transcript
Herald added subscribers: riscv, jhibbits, emaste, imp. · View Herald Transcript
markj requested review of this revision.Oct 5 2023, 9:36 PM
Harbormaster completed remote builds in B53861: Diff 128332.Oct 5 2023, 9:36 PM
markj added a child revision: D42099: ktrace: Handle uio_resid underflow via MSG_TRUNC.Oct 5 2023, 9:36 PM
imp added a comment.Oct 5 2023, 9:50 PM

Any way to trigger these from userland? I'd think high level tests prevent that...

kib accepted this revision.Oct 5 2023, 9:59 PM
This revision is now accepted and ready to land.Oct 5 2023, 9:59 PM
markj added a comment.Oct 5 2023, 10:01 PM
In D42098#960314, @imp wrote:

Any way to trigger these from userland? I'd think high level tests prevent that...

D42099 fixes one bug which can trigger these assertions, yes.

Closed by commit rG8fd0ec53deaa: uiomove: Add some assertions (authored by markj). · Explain WhyOct 17 2023, 1:38 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG8fd0ec53deaa: uiomove: Add some assertions.
markj added a commit: rGeb8923452e1e: uiomove: Add some assertions.Oct 24 2023, 1:38 PM
markj added a commit: rG2d49b111a312: uiomove: Add some assertions.

Revision Contents
Changeset List

PathSize
sys/
amd64/
amd64/
6 lines
arm/
arm/
6 lines
arm64/
arm64/
6 lines
i386/
i386/
6 lines
kern/
5 lines
powerpc/
powerpc/
5 lines
riscv/
riscv/
6 lines

Diff 128332

View Options

sys/amd64/amd64/uio_machdep.c

Loading...
View Options

sys/arm/arm/uio_machdep.c

Loading...
View Options

sys/arm64/arm64/uio_machdep.c

Loading...
View Options

sys/i386/i386/uio_machdep.c

Loading...
View Options

sys/kern/subr_uio.c

Loading...
View Options

sys/powerpc/powerpc/uio_machdep.c

Loading...
View Options

sys/riscv/riscv/uio_machdep.c

Loading...