Page MenuHomeFreeBSD

Prevent SEGV on corrupted msdos directories
AbandonedPublic

Authored by sevan on Apr 28 2017, 6:01 PM.
Tags
None
Referenced Files
F172427887: D10539.id27836.diff
Fri, Sep 18, 9:53 AM
F172387916: D10539.id.diff
Fri, Sep 18, 3:37 AM
Unknown Object (File)
Aug 17 2026, 1:33 PM
Unknown Object (File)
Aug 10 2026, 3:08 AM
Unknown Object (File)
Aug 2 2026, 3:58 AM
Unknown Object (File)
Jul 28 2026, 3:19 PM
Unknown Object (File)
Jul 22 2026, 4:40 AM
Unknown Object (File)
Jul 2 2026, 4:21 AM
Subscribers

Details

Reviewers
bcr
emaste
Summary

By Veo Zhang via NetBSD
Tested it builds as an individual component (not as part of build world) but
haven't performed any tests on a file system.

Diff Detail

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

Event Timeline

You need to find someone more knowledgeable in that area (i.e. src-committer) to look over that change. Maybe someone who recently made some change to the same code area: https://svnweb.freebsd.org/base/head/sbin/fsck_msdosfs/

It is not incorrect but it looks somewhat weird: Perhaps it's just an off-by-one and changing k >=0 to k > 0 in line 620 solves it more cleanly.

I'll think of it for a while :).