diff options
| author | Thomas Huth <thuth@redhat.com> | 2015-04-27 18:59:04 +0200 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2015-04-30 16:55:16 +0200 |
| commit | 03a96b83b539498510e22aab585e41015ba18247 (patch) | |
| tree | c23b9a37748681d96b9e9b5002c8cc0fc24dfee5 | |
| parent | 498147529d1f8e902e6528a0115143b53475791e (diff) | |
| download | focaccia-qemu-03a96b83b539498510e22aab585e41015ba18247.tar.gz focaccia-qemu-03a96b83b539498510e22aab585e41015ba18247.zip | |
kvm: Silence warning from valgrind
valgrind complains here about uninitialized bytes with the following message: ==17814== Syscall param ioctl(generic) points to uninitialised byte(s) ==17814== at 0x466A780: ioctl (in /usr/lib64/power8/libc-2.17.so) ==17814== by 0x100735B7: kvm_vm_ioctl (kvm-all.c:1920) ==17814== by 0x10074583: kvm_set_ioeventfd_mmio (kvm-all.c:574) Let's fix it by using a proper struct initializer in kvm_set_ioeventfd_mmio(). Signed-off-by: Thomas Huth <thuth@redhat.com> Message-Id: <1430153944-24368-1-git-send-email-thuth@redhat.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| -rw-r--r-- | kvm-all.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/kvm-all.c b/kvm-all.c index 3f7061a180..28f4589219 100644 --- a/kvm-all.c +++ b/kvm-all.c @@ -552,13 +552,13 @@ static int kvm_set_ioeventfd_mmio(int fd, hwaddr addr, uint32_t val, bool assign, uint32_t size, bool datamatch) { int ret; - struct kvm_ioeventfd iofd; - - iofd.datamatch = datamatch ? adjust_ioeventfd_endianness(val, size) : 0; - iofd.addr = addr; - iofd.len = size; - iofd.flags = 0; - iofd.fd = fd; + struct kvm_ioeventfd iofd = { + .datamatch = datamatch ? adjust_ioeventfd_endianness(val, size) : 0, + .addr = addr, + .len = size, + .flags = 0, + .fd = fd, + }; if (!kvm_enabled()) { return -ENOSYS; |