kconfig: use va_end to match corresponding va_start
authorColin Ian King <colin.king@canonical.com>
Mon, 12 Jan 2015 13:18:26 +0000 (13:18 +0000)
committerChristian Lamparter <chunkeey@googlemail.com>
Wed, 20 May 2015 12:07:26 +0000 (14:07 +0200)
Although on some systems va_end is a no-op, it is good practice
to use va_end, especially since the manual states:

"Each invocation of va_start() must be matched by a corresponding
invocation of va_end() in the same function."

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Michal Marek <mmarek@suse.cz>
Signed-off-by: Christian Lamparter <chunkeey@googlemail.com>
config/confdata.c

index 58ac13f0d98b01ad60ef842f37ed85052041e1c0..8028a9d3fdaf3a1136265f026682f1bc0bf8d80f 100644 (file)
@@ -59,6 +59,7 @@ static void conf_message(const char *fmt, ...)
        va_start(ap, fmt);
        if (conf_message_callback)
                conf_message_callback(fmt, ap);
+       va_end(ap);
 }
 
 const char *conf_get_configname(void)