diff options
author | Jean-Christophe Dubois <jcd@tribudubois.net> | 2009-05-17 18:41:16 +0200 |
---|---|---|
committer | Anthony Liguori <aliguori@us.ibm.com> | 2009-05-20 09:12:58 -0500 |
commit | 9ed415b28b0c808e8b0fc631902cb9ce277f0245 (patch) | |
tree | f356a9472769fc685bb34d3ec0efea1453b2b79c /vl.c | |
parent | e332340a7799cf5681bffac264e3ce90288c037f (diff) | |
download | qemu-9ed415b28b0c808e8b0fc631902cb9ce277f0245.zip qemu-9ed415b28b0c808e8b0fc631902cb9ce277f0245.tar.gz qemu-9ed415b28b0c808e8b0fc631902cb9ce277f0245.tar.bz2 |
initialize struct sigevent before timer_create
When qemu is run under valgrind, valgrind shows the following output
on exit:
==3648== 1 errors in context 2 of 2:
==3648== Syscall param timer_create(evp) points to uninitialised byte(s)
==3648== at 0x54E936A: timer_create (in /lib/librt-2.9.so)
==3648== by 0x405DCF: dynticks_start_timer (vl.c:1549)
==3648== by 0x40A966: main (vl.c:1726)
==3648== Address 0x7fefffb34 is on thread 1's stack
==3648== Uninitialised value was created by a stack allocation
==3648== at 0x405D60: dynticks_start_timer (vl.c:1534)
This patch is a simple fix to remove this potential problem.
Signed-off-by: Jean-Christophe DUBOIS <jcd@tribudubois.net>
Diffstat (limited to 'vl.c')
-rw-r--r-- | vl.c | 5 |
1 files changed, 5 insertions, 0 deletions
@@ -1542,6 +1542,11 @@ static int dynticks_start_timer(struct qemu_alarm_timer *t) sigaction(SIGALRM, &act, NULL); + /* + * Initialize ev struct to 0 to avoid valgrind complaining + * about uninitialized data in timer_create call + */ + memset(&ev, 0, sizeof(ev)); ev.sigev_value.sival_int = 0; ev.sigev_notify = SIGEV_SIGNAL; ev.sigev_signo = SIGALRM; |