]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
MINOR: wdt: always clear sigev_value to make valgrind happy
authorWilly Tarreau <w@1wt.eu>
Wed, 26 Feb 2020 13:03:05 +0000 (14:03 +0100)
committerWilly Tarreau <w@1wt.eu>
Wed, 26 Feb 2020 13:05:20 +0000 (14:05 +0100)
In issue #471 it was reported that valgrind sometimes complains about
timer_create() being called with uninitialized bytes. These are in fact
the bits from sigev_value.sival_ptr that are not part of sival_int that
are tagged as such, as valgrind has no way to know we're using the int
instead of the ptr in the union. It's cheap to initialize the field so
let's do it.

src/wdt.c

index 8f31e94920975a5cb27ec515e90529069b521583..8ccdb04054cc2443914174169ffd3007676c45d6 100644 (file)
--- a/src/wdt.c
+++ b/src/wdt.c
@@ -129,7 +129,7 @@ void wdt_handler(int sig, siginfo_t *si, void *arg)
 
 int init_wdt_per_thread()
 {
-       struct sigevent sev;
+       struct sigevent sev = { };
        sigset_t set;
 
        /* unblock the WDTSIG signal we intend to use */