[hackers] Re: [dwm][PATCH] Use sigaction(SA_NOCLDWAIT) for SIGCHLD handling

From: NRK <nrk_AT_disroot.org>
Date: Wed, 27 Jul 2022 03:22:57 +0600

On Tue, Jul 26, 2022 at 10:08:19PM +0100, Chris Down wrote:
> Pretty sure that comment means "[set up the handler to] clean up any zombies
> immediately" -- I don't see anything else it could be.

Looking at `git blame` takes me to commit 33fe200b :

        + /* clean up any zombies immediately */
        + signal(SIGCHLD, sigchld);
        + sigchld(0);

It sets up the sighandler and then immediately calls `sigchld`. So it
seems like the intention was to *actaully* reap some zombies and not
just setup the sighandler.

This might be some weird edge case (?) Though I do agree with you, I
don't see where a zombie would come from at this stage either.

- NRK
Received on Tue Jul 26 2022 - 23:22:57 CEST