From 72e5368aedd7eb964c1dcc14f11fa3188e9368c6 Mon Sep 17 00:00:00 2001 From: Jay Fenlason Date: Thu, 19 Nov 2009 15:00:02 -0500 Subject: [PATCH] Initialize unused fields in ioctl arguments This change is essentially cosmetic: Set fields of structs passed to the kernel via ioctl so that valgrind will not complain about them. Signed-off-by: Jay Fenlason Signed-off-by: Stefan Richter (changelog, comments) --- src/fw-iso.c | 2 ++ src/fw.c | 1 + 2 files changed, 3 insertions(+) diff --git a/src/fw-iso.c b/src/fw-iso.c index 466b638..677b98d 100644 --- a/src/fw-iso.c +++ b/src/fw-iso.c @@ -140,6 +140,8 @@ int fw_iso_xmit_start(raw1394handle_t handle, int start_on_cycle, return -1; if (fwhandle->iso.prebuffer <= fwhandle->iso.packet_count) { + start_iso.sync = 0; /* unused */ + start_iso.tags = 0; /* unused */ start_iso.cycle = start_on_cycle; start_iso.handle = 0; diff --git a/src/fw.c b/src/fw.c index d63d0da..7277253 100644 --- a/src/fw.c +++ b/src/fw.c @@ -1406,6 +1406,7 @@ iso_resource_modify(raw1394handle_t handle, unsigned int bandwidth, resource.closure = ptr_to_u64(&event); resource.channels = channel >= 0 ? 1ULL << channel : 0; resource.bandwidth = bandwidth; + resource.handle = 0; /* unused */ ioctl_nr = mode == RAW1394_MODIFY_ALLOC ? FW_CDEV_IOC_ALLOCATE_ISO_RESOURCE_ONCE :