[SSSD] [PATCH] exit original process after sssd is initialized
Pavel Březina
pbrezina at redhat.com
Fri Nov 2 19:55:39 UTC 2012
On 11/02/2012 06:53 PM, 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 ?
Yes, definitely. This is a rebase bug :/
> 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 ?
>
> Simo.
>
More information about the sssd-devel
mailing list