[SSSD] [PATCH] Add common SIGCHLD handling.

Stephen Gallagher sgallagh at redhat.com
Thu Dec 15 02:32:22 UTC 2011


On Wed, 2011-12-14 at 13:03 -0500, Stephen Gallagher wrote:
> On Wed, 2011-12-14 at 09:15 -0500, Stephen Gallagher wrote:
> > 
> > On Dec 14, 2011, at 9:09 AM, Jakub Hrozek <jhrozek at redhat.com> wrote:
> > > 
> > > Are you going to fix the two comments you had originally in the later
> > > patch? Those were don't consider it fatal if the PID isn't found
> > > in the hash in sss_child_handler() and reordering sss_child_invoke_cb()
> > > 
> > > If so, then ack
> > 
> > 
> > The first part of that I already described above as being an incorrect
> > reading of the source. I missed the second part and I'll fix that
> > shortly.
> 
> 
> New patch attached fixes the missing reordering of the
> sssd_child_invoke_cb() and adds more explicit handling of unknown PIDs
> (with a new DEBUG log message).

I located one very serious bug while working on some dependent patches.
The waitpid() in this patch was using a bitwise AND with the WNOHANG
instead of a bitwise OR (resulting in the WNOHANG being disabled and
causing waitpid() to spin forever in some situations). New patch
attached.

I also moved the child_common routines out of the provider directory and
into util. I did this in a second patch so it's easier to follow the
changes.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-common-SIGCHLD-handling-for-providers.patch
Type: text/x-patch
Size: 13996 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20111214/c4198310/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Move-child_common-routines-to-util.patch
Type: text/x-patch
Size: 11222 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20111214/c4198310/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20111214/c4198310/attachment.sig>


More information about the sssd-devel mailing list