[PATCH] os: Initialize the set of signals to be suppressed during our handler.
Alan Coopersmith
alan.coopersmith at oracle.com
Fri Apr 4 10:41:14 PDT 2014
On 04/ 4/14 12:38 AM, Eric Anholt wrote:
> Fixes a valgrind complaint:
>
> ==8805== Syscall param rt_sigaction(act->sa_mask) points to uninitialised byte(s)
> ==8805== at 0x5EB8315: __libc_sigaction (sigaction.c:66)
> ==8805== by 0x5B13DA: busfault_init (busfault.c:145)
> ==8805== by 0x5A60A2: OsInit (osinit.c:191)
> ==8805== by 0x46EBA2: dix_main (main.c:163)
>
> Signed-off-by: Eric Anholt <eric at anholt.net>
> ---
> os/busfault.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/os/busfault.c b/os/busfault.c
> index 43bb6ea..ac0268f 100644
> --- a/os/busfault.c
> +++ b/os/busfault.c
> @@ -142,6 +142,7 @@ busfault_init(void)
>
> act.sa_sigaction = busfault_sigaction;
> act.sa_flags = SA_SIGINFO;
> + sigemptyset(&act.sa_mask);
> if (sigaction(SIGBUS, &act, &old_act) < 0)
> return FALSE;
> previous_busfault_sigaction = old_act.sa_sigaction;
>
Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>
--
-Alan Coopersmith- alan.coopersmith at oracle.com
Oracle Solaris Engineering - http://blogs.oracle.com/alanc
More information about the xorg-devel
mailing list