Page MenuHomeFreeBSD
Differential D21466

Coverity fixes in fusefs(5)
ClosedPublic
Actions

Authored by asomers on Aug 29 2019, 5:06 PM.
Tags
None
Referenced Files
F109705906: D21466.diff
Sat, Feb 8, 1:46 PM
Unknown Object (File)
Tue, Jan 21, 8:27 AM
Unknown Object (File)
Mon, Jan 20, 7:15 AM
Unknown Object (File)
Sat, Jan 18, 7:37 AM
Unknown Object (File)
Jan 6 2025, 1:57 PM
Unknown Object (File)
Jan 6 2025, 1:57 PM
Unknown Object (File)
Jan 6 2025, 1:57 PM
Unknown Object (File)
Jan 6 2025, 12:56 PM
View All 74 Files
Subscribers
emaste
imp

Details

Reviewers
cem
emaste
Commits
rS351961: Coverity fixes in fusefs(5)
Summary

Coverity fixes in fusefs(5)

CID 1404532 fixes a signed vs unsigned comparison error in fuse_vnop_bmap.
It could potentially have resulted in VOP_BMAP reporting too many
consecutive blocks.

CID 1404364 is much worse. It was an array access by an untrusted,
user-provided variable. It could potentially have resulted in a malicious
file system crashing the kernel or worse.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 26181
Build 24689: arc lint + arc unit

Event Timeline

asomers created this revision.Aug 29 2019, 5:06 PM
Herald added a subscriber: imp. · View Herald TranscriptAug 29 2019, 5:06 PM
Harbormaster completed remote builds in B26181: Diff 61459.Aug 29 2019, 5:06 PM
emaste accepted this revision.Sep 6 2019, 2:26 PM
emaste added a subscriber: emaste.
emaste added inline comments.
sys/fs/fuse/fuse_internal.c
393

namelen includes the '\0' presumably

This revision is now accepted and ready to land.Sep 6 2019, 2:26 PM
Closed by commit rS351961: Coverity fixes in fusefs(5) (authored by asomers). · Explain WhySep 6 2019, 7:40 PM
This revision was automatically updated to reflect the committed changes.
asomers added a commit: rS351961: Coverity fixes in fusefs(5).
asomers mentioned this in rS352351: MFC the new fusefs driver.Sep 15 2019, 4:15 AM

Revision Contents
Changeset List

PathSize
sys/
fs/
fuse/
3 lines
2 lines

Diff 61459

View Options

sys/fs/fuse/fuse_internal.c

Loading...
View Options

sys/fs/fuse/fuse_vnops.c

Loading...