Add more accurate directory entries check
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F132558937: D33374.diff
	Fri, Oct 17, 11:31 PM

	Unknown Object (File)
	Sat, Oct 11, 4:33 AM

	Unknown Object (File)
	Thu, Oct 9, 6:56 PM

	Unknown Object (File)
	Thu, Oct 9, 11:35 AM

	Unknown Object (File)
	Wed, Oct 8, 9:28 AM

	Unknown Object (File)
	Tue, Oct 7, 8:33 AM

	Unknown Object (File)
	Fri, Oct 3, 2:58 PM

	Unknown Object (File)
	Mon, Sep 29, 4:49 PM

View All 70 Files

Subscribers

imp

Details

Reviewers

pfg
nc

Commits

rGbb9f1ba4b55c: Add more accurate directory entries check

Summary

Rename ext2_dirbadentry() to ext2_check_direntry(). Add directory entry inode value check, and call ext2_check_direntry() in all cases. The dirchk sysctl is removed.

This PR should close next BZ issues:
259024 (https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=259024)
259041 (https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=259041)

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 Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

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

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

LGTM

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

Closed by commit rGbb9f1ba4b55c: Add more accurate directory entries check (authored by fsu). · Explain WhyDec 30 2021, 6:15 AM

This revision was automatically updated to reflect the committed changes.

fsu added a commit: rGbb9f1ba4b55c: Add more accurate directory entries check.

Revision Contents
Changeset List

Path

Size

sys/

fs/

ext2fs/

ext2_lookup.c

39 lines

Diff 100746

View Options

sys/fs/ext2fs/ext2_lookup.c

Add more accurate directory entries checkClosedPublicActions