On Tue, May 10, 2011 at 09:21:32PM +0200, Vincent Bernat wrote:
> OoO Lors de la soirée naissante du mardi 10 mai 2011, vers 17:15,
> Alexander Wirt <[email protected]> disait :
>
> >> Readwrite permissions to the pidfile of a daemon is a really bad idea. a
> >> umask of 000 is probably never a good idea. So I think removing that lines
> >> from keepalived/check/check_daemon.c, keepalived/core/daemon.c and
> >> keepalived/vrrp/vrrp_daemon.c.
> > After thinking a little bit longer about it, it maybe makes sense to clear a
> > users umask. But if this is the case, we should explicitly set permissions
> > for the pidfile (and take care we did that for every open..).
>
> Yes. Here is a patch. I have checked other invocations of open/fopen and
> this was the sole invocation which was faulty.
What's the status? It's been nearly half a year.
Cheers,
Moritz
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
