Robert Foley <robert.fo...@linaro.org> writes:
> After freeing the logfilename, we set logfilename to NULL, in case of an
> error which returns without setting logfilename.
>
> Signed-off-by: Robert Foley <robert.fo...@linaro.org>
As this fixes an existing bug I would put this at the start of the
series. Otherwise:
Reviewed-by: Alex Bennée <alex.ben...@linaro.org>
> ---
> v1
> - This is new in the patch v1.
> ---
> util/log.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/util/log.c b/util/log.c
> index 802b8de42e..1eed74788c 100644
> --- a/util/log.c
> +++ b/util/log.c
> @@ -148,6 +148,7 @@ void qemu_set_log_filename(const char *filename, Error
> **errp)
> char *pidstr;
>
> g_free(logfilename);
> + logfilename = NULL;
>
> pidstr = strstr(filename, "%");
> if (pidstr) {
--
Alex Bennée
