From d0bc5bc3aac64be2c0b91f1446ed748444cc3141 Mon Sep 17 00:00:00 2001 From: Markus Armbruster Date: Fri, 28 Oct 2011 10:52:25 +0200 Subject: [PATCH] sysbus: Supply missing va_end() 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 Signed-off-by: Stefan Hajnoczi --- hw/sysbus.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/sysbus.c b/hw/sysbus.c index 4fab5a4..fd2fc6a 100644 --- a/hw/sysbus.c +++ b/hw/sysbus.c @@ -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; } -- 2.7.4