[PATCH] Fix bug 11175 - Lots of winbindd zombie processes on Solaris platform

[Volker Lendecke]

On Mon, Mar 23, 2015 at 02:09:35PM -0700, Jeremy Allison wrote:
>  
> +static void (*tdb_robust_mutex_setup_sigchild(void (*handler)(int)))(int)
               ^                                                      ^

Is this set of braces really necessary?

> -	signal(SIGCHLD, tdb_robust_mutext_old_handler);
> +	(void)tdb_robust_mutex_setup_sigchild(tdb_robust_mutext_old_handler);

And this (void)? Is that necessary?

>  cleanup_sig_child:
> -	signal(SIGCHLD, tdb_robust_mutext_old_handler);
> +	(void)tdb_robust_mutex_setup_sigchild(tdb_robust_mutext_old_handler);

Here too.

Thanks,

Volker

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kontakt at sernet.de