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)
Tue, Mar 31, 9:40 AM
Unknown Object (File)
Mon, Mar 30, 10:19 PM
Unknown Object (File)
Fri, Mar 27, 7:19 AM
Unknown Object (File)
Sat, Mar 21, 12:10 PM
Unknown Object (File)
Fri, Mar 13, 5:36 PM
Unknown Object (File)
Wed, Mar 11, 4:55 PM
Unknown Object (File)
Wed, Mar 11, 2:52 AM
Unknown Object (File)
Mon, Mar 9, 9:27 PM
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...