Page MenuHomeFreeBSD
Differential D19322

Make superblock reading logic more strict.
ClosedPublic
Actions

Authored by fsu on Feb 24 2019, 1:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 6, 3:01 PM
Unknown Object (File)
Sun, Sep 6, 2:41 PM
Unknown Object (File)
Sun, Sep 6, 12:38 AM
Unknown Object (File)
Sat, Sep 5, 12:36 PM
Unknown Object (File)
Sat, Sep 5, 1:48 AM
Unknown Object (File)
Fri, Sep 4, 2:05 AM
Unknown Object (File)
Thu, Sep 3, 12:17 PM
Unknown Object (File)
Thu, Sep 3, 10:45 AM
View All Files
Subscribers
imp

Details

Reviewers
pfg
Commits
rS345265: MFC r344751:
rS344751: Make superblock reading logic more strict.
Summary

Now the comput_sb_data() function checks on-disk superblock data fields more carefully. Also, the cylinder group validation function is added.
It should decrease ability to mount filesystems with corrupted superblock data.

Test Plan

Typical fsx/fstorture tests to avoid regression.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

fsu created this revision.Feb 24 2019, 1:13 PM
Herald added a subscriber: imp. · View Herald TranscriptFeb 24 2019, 1:13 PM
pfg accepted this revision.Feb 27 2019, 2:43 PM
pfg added inline comments.
sys/fs/ext2fs/ext2fs.h
398 ↗(On Diff #54285)

Use a TAB after the

#define
This revision is now accepted and ready to land.Feb 27 2019, 2:43 PM
Closed by commit rS344751: Make superblock reading logic more strict. (authored by fsu). · Explain WhyMar 4 2019, 10:42 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
fs/
ext2fs/
10 lines
3 lines
266 lines
5 lines

Diff 54668

View Options

head/sys/fs/ext2fs/ext2_alloc.c

Loading...
View Options

head/sys/fs/ext2fs/ext2_extern.h

Loading...
View Options

head/sys/fs/ext2fs/ext2_vfsops.c

Loading...
View Options

head/sys/fs/ext2fs/ext2fs.h

Loading...