| Message ID | 20180104160523.22995-12-marcandre.lureau@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | Various build-sys and sanitizer related fixes | expand |
Marc-Andre Lureau <marcandre.lureau@redhat.com> wrote: > Direct leak of 12 byte(s) in 2 object(s) allocated from: > #0 0x7f50d403c850 in malloc (/lib64/libasan.so.4+0xde850) > #1 0x7f50d1ddf98f in vasprintf (/lib64/libc.so.6+0x8098f) > > Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> > Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com> > Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> > --- > tests/migration-test.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tests/migration-test.c b/tests/migration-test.c > index be598d3257..799e24ebc6 100644 > --- a/tests/migration-test.c > +++ b/tests/migration-test.c > @@ -358,13 +358,14 @@ static void migrate_check_parameter(QTestState *who, const char *parameter, > const char *value) > { > QDict *rsp, *rsp_return; > - const char *result; > + char *result; > > rsp = wait_command(who, "{ 'execute': 'query-migrate-parameters' }"); > rsp_return = qdict_get_qdict(rsp, "return"); > result = g_strdup_printf("%" PRId64, > qdict_get_try_int(rsp_return, parameter, -1)); > g_assert_cmpstr(result, ==, value); > + g_free(result); > QDECREF(rsp); > } I got the same patch, it is already on the migration pull requset. Thanks.
diff --git a/tests/migration-test.c b/tests/migration-test.c index be598d3257..799e24ebc6 100644 --- a/tests/migration-test.c +++ b/tests/migration-test.c @@ -358,13 +358,14 @@ static void migrate_check_parameter(QTestState *who, const char *parameter, const char *value) { QDict *rsp, *rsp_return; - const char *result; + char *result; rsp = wait_command(who, "{ 'execute': 'query-migrate-parameters' }"); rsp_return = qdict_get_qdict(rsp, "return"); result = g_strdup_printf("%" PRId64, qdict_get_try_int(rsp_return, parameter, -1)); g_assert_cmpstr(result, ==, value); + g_free(result); QDECREF(rsp); }
Direct leak of 12 byte(s) in 2 object(s) allocated from: #0 0x7f50d403c850 in malloc (/lib64/libasan.so.4+0xde850) #1 0x7f50d1ddf98f in vasprintf (/lib64/libc.so.6+0x8098f) Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> --- tests/migration-test.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)