Re: [PATCH 2/3] qtest: fix qtest_qmp_device_add leak

[Thomas Huth <thuth@redhat.com>]

On 07/11/2019 20.27, Marc-André Lureau wrote:
> Spotted by ASAN.
> 
> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
>   tests/libqtest.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/tests/libqtest.c b/tests/libqtest.c
> index 3706bccd8d..91e9cb220c 100644
> --- a/tests/libqtest.c
> +++ b/tests/libqtest.c
> @@ -1274,6 +1274,7 @@ void qtest_qmp_device_add(QTestState *qts, const char *driver, const char *id,
>       qdict_put_str(args, "id", id);
>   
>       qtest_qmp_device_add_qdict(qts, driver, args);
> +    qobject_unref(args);
>   }
>   
>   static void device_deleted_cb(void *opaque, const char *name, QDict *data)
> 

Fixes: b4510bb4109f5f ("tests: add qtest_qmp_device_add_qdict() helper")

Reviewed-by: Thomas Huth <thuth@redhat.com>

I can queue this via the qtest tree if nobody else wants to take it.