[SSSD] [PATCH] Check for existing pidfile before starting the providers

Jakub Hrozek jhrozek at redhat.com
Thu Oct 4 09:19:40 UTC 2012


On Thu, Oct 04, 2012 at 11:08:47AM +0200, Pavel Březina wrote:
> On 10/04/2012 10:58 AM, Jakub Hrozek wrote:
> >On Thu, Oct 04, 2012 at 09:49:12AM +0200, Pavel Březina wrote:
> >>On 10/03/2012 04:29 PM, Jakub Hrozek wrote:
> >>>After we switched to writing pidfile after the responders started, we forgot
> >>>that starting a second SSSD instance would first overwrite the pipes and
> >>>sockets and only then the SSSD would find out there already is a pidfile.
> >>
> >>Nice catch.
> >>
> >>When you're touching monitor_cleanup(), can you change the debug
> >>level to its macro representation please? Also ret = errno is not
> >>used anywhere.
> >
> >The error handled should have said return ret instead of return errno.
> >
> >New patch is attached.
> 
> Ack.

Pushed to master.



More information about the sssd-devel mailing list