sysbus: Supply missing va_end()
authorMarkus Armbruster <armbru@redhat.com>
Fri, 28 Oct 2011 08:52:25 +0000 (10:52 +0200)
committerStefan Hajnoczi <stefanha@linux.vnet.ibm.com>
Tue, 1 Nov 2011 06:23:48 +0000 (06:23 +0000)
C99 7.15.1: Each invocation of the va_start and va_copy macros shall
be matched by a corresponding invocation of the va_end macro in the
same function.

Spotted by Coverity.  Harmless on the (common) systems where va_end()
does nothing.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
hw/sysbus.c

index 4fab5a41b25e72adee7315107e617ebb1d4d8486..fd2fc6a51df15568a8d2ac703d1a8c8757956a31 100644 (file)
@@ -198,6 +198,7 @@ DeviceState *sysbus_create_varargs(const char *name,
         sysbus_connect_irq(s, n, irq);
         n++;
     }
+    va_end(va);
     return dev;
 }
 
@@ -229,6 +230,7 @@ DeviceState *sysbus_try_create_varargs(const char *name,
         sysbus_connect_irq(s, n, irq);
         n++;
     }
+    va_end(va);
     return dev;
 }