Differential D57447

rc: Bail if /dev/null is not a device
ClosedPublic
Actions

Authored by des on Jun 4 2026, 4:54 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Jul 28, 8:02 AM

	Unknown Object (File)
	Mon, Jul 27, 4:11 PM

	Unknown Object (File)
	Thu, Jul 23, 4:40 AM

	Unknown Object (File)
	Tue, Jul 21, 6:49 PM

	Unknown Object (File)
	Tue, Jul 21, 1:37 AM

	Unknown Object (File)
	Mon, Jul 20, 11:33 AM

	Unknown Object (File)
	Sat, Jul 18, 9:20 AM

	Unknown Object (File)
	Thu, Jul 16, 10:20 AM

View All 39 Files

Subscribers

Details

Reviewers

emaste
jhb

Commits

rGd09b42a876bb: rc: Bail if /dev/null is not a device
rG5c12023beb94: rc: Bail if /dev/null is not a device
rGb5a96894f67a: rc: Bail if /dev/null is not a device

Summary

On startup, check that /dev/null exists and is a character device.
Otherwise, one of two things will happen: either /dev is a writable
directory and we will immediately create /dev/null as a regular file
and dump garbage into it, or it does not and we will spit out a stream
of error messages about failing to create /dev/null.

PR: 295782
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.Jun 4 2026, 4:54 PM

Herald added a subscriber: imp. · View Herald TranscriptJun 4 2026, 4:54 PM

des requested review of this revision.Jun 4 2026, 4:54 PM

Harbormaster completed remote builds in B73674: Diff 179220.Jun 4 2026, 4:54 PM

emaste accepted this revision.Jun 4 2026, 5:15 PM

This revision is now accepted and ready to land.Jun 4 2026, 5:15 PM

man page

This revision now requires review to proceed.Jun 4 2026, 9:16 PM

Herald added a subscriber: ziaee. · View Herald TranscriptJun 4 2026, 9:16 PM

Harbormaster completed remote builds in B73682: Diff 179236.Jun 4 2026, 9:16 PM

jhb accepted this revision.Jun 5 2026, 2:48 PM

This revision is now accepted and ready to land.Jun 5 2026, 2:48 PM

emaste accepted this revision.Jun 5 2026, 2:53 PM

Closed by commit rGb5a96894f67a: rc: Bail if /dev/null is not a device (authored by des). · Explain WhyJun 5 2026, 3:53 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGb5a96894f67a: rc: Bail if /dev/null is not a device.

des added a commit: rG5c12023beb94: rc: Bail if /dev/null is not a device.Jun 8 2026, 11:01 PM

des added a commit: rGd09b42a876bb: rc: Bail if /dev/null is not a device.

Revision Contents
Changeset List

Path

Size

libexec/

rc/

5 lines

share/

man/

man8/

12 lines

Diff 179280

libexec/rc/rc

Loading...

share/man/man8/rc.8

Loading...