Re: [PATCH v2 2/2] connect.c: replace signal() with sigaction()

Chris,

On Sat, May 31, 2014 at 10:39:39PM +1200, Chris Packham wrote:

On 31/05/14 08:58, Jeremiah Mahler wrote:

quoted

From signal(2) man page:

...

quoted

-	signal(SIGCHLD, SIG_DFL);
+	memset(&sa, 0, sizeof(sa));
+	sa.sa_handler = SIG_DFL;
+	sigaction(SIGCHLD, &sa, 0);

I think this got lost in the wash with v1 but
Documentation/CodingGuidelines says to use NULL here instead of 0.

Got it.  Will be in to the next revision.

  sigaction(SIGCHLD, &sa, NULL);

Thanks,
-- 
Jeremiah Mahler
jmmahler@gmail.com
http://github.com/jmahler