Page MenuHomeFreeBSD
Differential D57447

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

Authored by des on Thu, Jun 4, 4:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jun 23, 1:57 PM
Unknown Object (File)
Sun, Jun 21, 3:28 PM
Unknown Object (File)
Sun, Jun 21, 3:23 PM
Unknown Object (File)
Sun, Jun 21, 1:17 PM
Unknown Object (File)
Sun, Jun 21, 1:12 PM
Unknown Object (File)
Sun, Jun 21, 8:12 AM
Unknown Object (File)
Sun, Jun 21, 8:10 AM
Unknown Object (File)
Sat, Jun 20, 7:07 PM
View All 26 Files
Subscribers
imp
ziaee

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.Thu, Jun 4, 4:54 PM
Herald added a subscriber: imp. · View Herald TranscriptThu, Jun 4, 4:54 PM
des requested review of this revision.Thu, Jun 4, 4:54 PM
Harbormaster completed remote builds in B73674: Diff 179220.Thu, Jun 4, 4:54 PM
emaste accepted this revision.Thu, Jun 4, 5:15 PM
This revision is now accepted and ready to land.Thu, Jun 4, 5:15 PM
des updated this revision to Diff 179236.Thu, Jun 4, 9:16 PM

man page

This revision now requires review to proceed.Thu, Jun 4, 9:16 PM
Herald added a subscriber: ziaee. · View Herald TranscriptThu, Jun 4, 9:16 PM
Harbormaster completed remote builds in B73682: Diff 179236.Thu, Jun 4, 9:16 PM
jhb accepted this revision.Fri, Jun 5, 2:48 PM
This revision is now accepted and ready to land.Fri, Jun 5, 2:48 PM
emaste accepted this revision.Fri, Jun 5, 2:53 PM
Closed by commit rGb5a96894f67a: rc: Bail if /dev/null is not a device (authored by des). · Explain WhyFri, Jun 5, 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.Mon, Jun 8, 11:01 PM
des added a commit: rGd09b42a876bb: rc: Bail if /dev/null is not a device.

Revision Contents
Changeset List

PathSize
libexec/
rc/
5 lines
share/
man/
man8/
12 lines

Diff 179280

View Options

libexec/rc/rc

Loading...
View Options

share/man/man8/rc.8

Loading...