[SSSD] [PATCH] exit original process after sssd is initialized

Jakub Hrozek jhrozek at redhat.com
Fri Nov 2 18:11:45 UTC 2012


On Fri, Nov 02, 2012 at 01:53:52PM -0400, Simo Sorce wrote:
> On Thu, 2012-11-01 at 17:36 +0100, Pavel Březina wrote:
> > +        if (ctx->is_daemon && ctx->parent_pid > 0
> > +                && ctx->parent_pid == getppid()) {
> > +            if (ctx->parent_pid <= 0 || ctx->parent_pid != getppid())
> > {
> > +                /* the parent process was already terminated */
> > +                DEBUG(SSSDBG_MINOR_FAILURE, ("Invalid parent pid
> > \n"));
> > +                goto done;
> > +            }
> > + 
> 
> Isn't the second if condition redundant here ?
> we can't even test it unless it is already true according to the
> previous if () or am I missing something ?
> Did you intend to use some || in the previous condition ?
> 

Oops, I think you're right, thank you Simo.

To reiterate the discussion we had with Pavel in person[1], we wanted to
make sure we're really signaling the original parent and not e.g. init
in the odd case the parent died before we got to this point.

[1] This is a little off-topic, but as many of the SSSD developers are now in
the same office, it is very convenient for us to talk in person. Obviously
this has the drawback of excluding the other developers and the community
from the discussion. We should try harder to document the thoughts in more
detail even though it might seem redundant.



More information about the sssd-devel mailing list