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

	Unknown Object (File)
	Dec 10 2025, 7:25 PM

	Unknown Object (File)
	Dec 4 2025, 1:18 AM

	Unknown Object (File)
	Nov 26 2025, 10:04 AM

	Unknown Object (File)
	Nov 17 2025, 7:07 PM

	Unknown Object (File)
	Nov 16 2025, 11:43 PM

	Unknown Object (File)
	Nov 13 2025, 8:26 PM

	Unknown Object (File)
	Oct 29 2025, 9:10 AM

	Unknown Object (File)
	Oct 29 2025, 8:44 AM

View All 34 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 Skipped

Unit

Tests Skipped

Build Status

Buildable 66307
Build 63190: arc lint + arc unit

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 160471

View Options

vfs: Assert that st_rdev is NODEV for non-devicesClosedPublicActions