Improve extents verification logic
ClosedPublic
Actions

Authored by fsu on Dec 10 2021, 5:23 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, May 23, 6:35 PM

	Unknown Object (File)
	Fri, May 17, 11:27 PM

	Unknown Object (File)
	Apr 29 2024, 6:33 AM

	Unknown Object (File)
	Apr 26 2024, 9:09 AM

	Unknown Object (File)
	Apr 26 2024, 5:53 AM

	Unknown Object (File)
	Jan 24 2024, 3:12 AM

	Unknown Object (File)
	Dec 22 2023, 9:43 PM

	Unknown Object (File)
	Dec 16 2023, 3:37 PM

View All 29 Files

Subscribers

imp

Details

Reviewers

pfg
nc

Commits

rGf1d5e2c862ef: Improve extents verification logic

Summary

Add functionality for extents validation inside the filesystem extents block.
The main logic is implemented under ext4_validate_extent_entries() function, which verifies extents or extents indexes depending of extent depth value.

This PR is more robust fix for issue introduced in:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=259112

Test Plan

Tested using next testing scripts:
https://github.com/thedrgreenthumb/Jenkins/tree/pjenkins/Target/ext2fs_test_all

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

fsu created this revision.Dec 10 2021, 5:23 PM

Herald added a subscriber: imp. · View Herald TranscriptDec 10 2021, 5:23 PM

fsu requested review of this revision.Dec 10 2021, 5:23 PM

pfg accepted this revision.Dec 11 2021, 12:37 AM

This revision is now accepted and ready to land.Dec 11 2021, 12:37 AM

Closed by commit rGf1d5e2c862ef: Improve extents verification logic (authored by fsu). · Explain WhyDec 30 2021, 6:15 AM

This revision was automatically updated to reflect the committed changes.

fsu added a commit: rGf1d5e2c862ef: Improve extents verification logic.

Revision Contents
Changeset List

Path

Size

sys/

fs/

ext2fs/

ext2_extents.c

291 lines

ext2_inode_cnv.c

8 lines

Diff 99825

View Options

sys/fs/ext2fs/ext2_extents.c