* Marc-André Lureau ([email protected]) wrote:
> This leak was introduced in commit
> 581f08bac22bdd5e081ae07f68071a0fc3c5c2c7.
> 
> (it stands out quickly with ASAN once the rest of the leaks are also
> removed from make check with this series)

Oops yes.

Reviewed-by: Dr. David Alan Gilbert <[email protected]>

> 
> Cc: Dr. David Alan Gilbert <[email protected]>
> Cc: Juan Quintela <[email protected]>
> Signed-off-by: Marc-André Lureau <[email protected]>
> ---
>  migration/savevm.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/migration/savevm.c b/migration/savevm.c
> index 5ecd264134..87c7a00832 100644
> --- a/migration/savevm.c
> +++ b/migration/savevm.c
> @@ -688,6 +688,7 @@ int vmstate_register_with_alias_id(DeviceState *dev, int 
> instance_id,
>  
>                  return -1;
>              }
> +            g_free(id);
>  
>              se->compat = g_new0(CompatEntry, 1);
>              pstrcpy(se->compat->idstr, sizeof(se->compat->idstr), 
> vmsd->name);
> -- 
> 2.11.0.295.gd7dffce1c.dirty
> 
--
Dr. David Alan Gilbert / [email protected] / Manchester, UK

Reply via email to