Paths

Table of Contentst

Differential D41381

syslogd: Pre-open null file descriptor
ClosedPublic
Actions

Authored by jfree on Aug 8 2023, 4:36 PM.

Tags

None

Referenced Files

	F87266550: D41381.id125909.diff
	Mon, Jul 1, 12:11 AM

	F87266271: D41381.id125736.diff
	Mon, Jul 1, 12:06 AM

	F87265870: D41381.id125987.diff
	Sun, Jun 30, 11:57 PM

	F87265369: D41381.id126279.diff
	Sun, Jun 30, 11:47 PM

	F87264697: D41381.id127949.diff
	Sun, Jun 30, 11:42 PM

	F87226272: D41381.id125736.diff
	Sun, Jun 30, 1:12 PM

	Unknown Object (File)
	Fri, Jun 28, 7:48 PM

	Unknown Object (File)
	Fri, Jun 21, 7:31 AM

View All 49 Files

Subscribers

Details

Reviewers

markj
emaste

Commits

rG7428ebfe7e25: syslogd: Pre-open null file descriptor

Summary

Open _PATH_DEVNULL before entering capability mode. Access to /dev/null
is needed for dup2() when piping commands.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jfree created this revision.Aug 8 2023, 4:36 PM

Herald added a subscriber: imp. · View Herald TranscriptAug 8 2023, 4:36 PM

jfree requested review of this revision.Aug 8 2023, 4:36 PM

jfree added a parent revision: D41380: syslogd: Do not open console descriptor.

markj accepted this revision.Aug 11 2023, 3:06 PM

markj added inline comments.

usr.sbin/syslogd/syslogd.c
871	I think you could reasonably abort the process if /dev/null isn't accessible. Then you don't need those `nulldesc >= 0` checks.
3611–3612	There is a pre-existing buglet here in that pfd[0] and [1] are not closed if pdfork() fails.

This revision is now accepted and ready to land.Aug 11 2023, 3:06 PM

Update after rebase and address Mark's comments

This revision now requires review to proceed.Aug 12 2023, 3:50 AM

Remove some rebasing errors (consfile got declared twice)

jfree added a child revision: D41433: syslogd: Ignore signals in sigcatch[].Aug 12 2023, 4:03 AM

markj accepted this revision.Aug 14 2023, 2:26 PM

This revision is now accepted and ready to land.Aug 14 2023, 2:26 PM

The nulldesc descriptor is used in waitdaemon() so open it prior to calling waitdaemon()

This revision now requires review to proceed.Aug 14 2023, 5:52 PM

Update after peerlist rebase, adding pidfile_remove()

markj accepted this revision.Aug 23 2023, 2:39 PM

This revision is now accepted and ready to land.Aug 23 2023, 2:39 PM

Closed by commit rG7428ebfe7e25: syslogd: Pre-open null file descriptor (authored by jfree, committed by markj). · Explain WhySep 28 2023, 3:54 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG7428ebfe7e25: syslogd: Pre-open null file descriptor.

Revision Contents
Changeset List

Path

Size

usr.sbin/

syslogd/

27 lines

Diff 127949

usr.sbin/syslogd/syslogd.c

Loading...