Differential D51956

vfs: Assert that st_rdev is NODEV for non-devices
ClosedPublic
Actions

Authored by des on Aug 17 2025, 3:55 PM.

Tags

None

Referenced Files

	F133638328: D51956.id160471.diff
	Mon, Oct 27, 5:03 AM

	Unknown Object (File)
	Fri, Oct 24, 12:47 AM

	Unknown Object (File)
	Sat, Oct 11, 11:24 PM

	Unknown Object (File)
	Sat, Oct 11, 11:24 PM

	Unknown Object (File)
	Sat, Oct 11, 2:06 PM

	Unknown Object (File)
	Sat, Oct 11, 2:06 PM

	Unknown Object (File)
	Sat, Oct 11, 2:06 PM

	Unknown Object (File)
	Sat, Oct 11, 5:47 AM

View All 26 Files

Subscribers

imp

olce

Details

Reviewers

kib

Commits

rGf9ff1e48cc1b: vfs: Assert that st_rdev is NODEV for non-devices

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.Aug 17 2025, 3:55 PM

Herald added subscribers: olce, imp. · View Herald TranscriptAug 17 2025, 3:55 PM

des requested review of this revision.Aug 17 2025, 3:55 PM

Harbormaster completed remote builds in B66307: Diff 160471.Aug 17 2025, 3:55 PM

des added parent revisions: D51955: ext2fs: Set st_rdev to NODEV when not a device, D51954: zfs: Set st_rdev to NODEV, not 0, when not a device, D51947: vfs: Introduce VN_ISDEV() macro.Aug 17 2025, 3:56 PM

I am sure there would be several missed cases.

This revision is now accepted and ready to land.Aug 17 2025, 4:20 PM

Closed by commit rGf9ff1e48cc1b: vfs: Assert that st_rdev is NODEV for non-devices (authored by des). · Explain WhyAug 17 2025, 5:31 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGf9ff1e48cc1b: vfs: Assert that st_rdev is NODEV for non-devices.

des added a reverting change: rG2bc115a6c287: Revert "vfs: Assert that st_rdev is NODEV for non-devices".Aug 17 2025, 11:56 PM

Revision Contents
Changeset List

Path

Size

sys/

kern/

vfs_syscalls.c

2 lines

Diff 160481

View Options

vfs: Assert that st_rdev is NODEV for non-devicesClosedPublicActions