Date: Tue, 24 Jan 2012 09:34:21 -0600 From: Guy Helmer <guy.helmer@palisadesystems.com> To: Doug Barton <dougb@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r229667 - head/usr.sbin/daemon Message-ID: <124C24A6-1E2A-4D30-9ED5-7F717C2E2A4D@palisadesystems.com> In-Reply-To: <4F078EC4.2090802@FreeBSD.org> References: <201201052248.q05MmaZk059871@svn.freebsd.org> <4F066340.9010507@FreeBSD.org> <C1CE4875-6FD5-4A3F-AB6A-64C0573C29AE@palisadesystems.com> <4F078EC4.2090802@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Jan 6, 2012, at 6:16 PM, Doug Barton wrote: > On 01/06/2012 08:18, Guy Helmer wrote: >> On Jan 5, 2012, at 8:58 PM, Doug Barton wrote: >>=20 >>> On 01/05/2012 14:48, Guy Helmer wrote: >>>> Allow daemon(8) to run pidfile_open() before relenquishing >>>> privileges so pid files can be written in /var/run when started >>>> as root. >>>=20 >>> I'm not sure how useful this is since when daemon is exiting it >>> won't be able to remove the pid file (unless I'm missing >>> something). >>>=20 >>> Isn't it better to pre-create the pid file with the proper >>> permissions for the unprivileged user? >>>=20 >>=20 >> Would it be OK for daemon to hang around and wait for the child >> process to exit, then remove the pid file? >=20 > Without having given it any kind of careful thought, that sounds Ok = ... > but I don't understand how daemon could remove a pid file written as > root after it's already dropped privileges. (IOW that's the same = problem > I was bringing up.) >=20 >> The only other alternative I see would be to create a subdirectory >> that is writable by the user so the child can create and delete the >> pid file. >=20 > That's functionally equivalent to pre-creating the pid file with the > right permissions, so it would be Ok. Various ports use each of these > approaches. I'm generally in favor of using the pid file only solution > since rc.d/cleanvar will clean all that stuff up at boot, and it's > preferable to not leave stale directories around for stuff that is no > longer running and/or installed. Having thought about it for a while, I plan to revert the change to = daemon.c that was suggested in the PR, and instead add this note to the = man page: Index: daemon.8 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D --- daemon.8 (revision 230510) +++ daemon.8 (working copy) @@ -59,6 +59,10 @@ using the .Xr pidfile 3 functionality. +If the +.Fl u +option is used, the directory to contain the pidfile must be writable +by the specified user. Note, that the file will be created shortly before the process is actually executed, and will remain after the process exits (although it will be removed if the execution fails). Guy -------- This message has been scanned by ComplianceSafe, powered by Palisade's PacketSure.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?124C24A6-1E2A-4D30-9ED5-7F717C2E2A4D>