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

Pavel Březina pbrezina at redhat.com
Thu Oct 4 09:08:47 UTC 2012


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.





More information about the sssd-devel mailing list