[Removed Yann.]
On Wed, 2015-03-18 at 18:16 +0100, Johannes Berg wrote:
> From: Johannes Berg <johannes.berg@xxxxxxxxx>
>
> This was pointed out by static checker analysis, but is likely
> otherwise harmless. Add it anyway to confuse the reader less.
>
> Signed-off-by: Johannes Berg <johannes.berg@xxxxxxxxx>
> ---
> scripts/kconfig/confdata.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/scripts/kconfig/confdata.c b/scripts/kconfig/confdata.c
> index f88d90f20228..28df18dd1147 100644
> --- a/scripts/kconfig/confdata.c
> +++ b/scripts/kconfig/confdata.c
> @@ -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)
This is fixed since v4.0-rc1 with commit b6a2ab2cd473 ("kconfig: use
va_end to match corresponding va_start"). Did that checker find anything
else noteworthy in scripts/kconfig?
Thanks,
Paul Bolle
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
