Message ID | 1449087634-30846-3-git-send-email-armbru@redhat.com |
---|---|
State | New |
Headers | show |
On 12/02/2015 01:20 PM, Markus Armbruster wrote: > From: Marc-André Lureau <marcandre.lureau@redhat.com> > > Before this patch ASAN reported: > SUMMARY: AddressSanitizer: 677165875 byte(s) leaked in 1272437 allocation(s) > > After this patch: > SUMMARY: AddressSanitizer: 465 byte(s) leaked in 32 allocation(s) > > Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> > Message-Id: <1448551895-871-1-git-send-email-marcandre.lureau@redhat.com> > [Straightforwardly rebased onto the previous patch] > Signed-off-by: Markus Armbruster <armbru@redhat.com> > --- > tests/qom-test.c | 21 ++++++++++++++------- > 1 file changed, 14 insertions(+), 7 deletions(-) > Reviewed-by: Eric Blake <eblake@redhat.com>
Am 02.12.2015 um 21:44 schrieb Eric Blake: > On 12/02/2015 01:20 PM, Markus Armbruster wrote: >> From: Marc-André Lureau <marcandre.lureau@redhat.com> >> >> Before this patch ASAN reported: >> SUMMARY: AddressSanitizer: 677165875 byte(s) leaked in 1272437 allocation(s) >> >> After this patch: >> SUMMARY: AddressSanitizer: 465 byte(s) leaked in 32 allocation(s) >> >> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> >> Message-Id: <1448551895-871-1-git-send-email-marcandre.lureau@redhat.com> >> [Straightforwardly rebased onto the previous patch] >> Signed-off-by: Markus Armbruster <armbru@redhat.com> >> --- >> tests/qom-test.c | 21 ++++++++++++++------- >> 1 file changed, 14 insertions(+), 7 deletions(-) >> > > Reviewed-by: Eric Blake <eblake@redhat.com> Thanks, both applied to qom-next. Andreas
diff --git a/tests/qom-test.c b/tests/qom-test.c index fde04e7..3e5e873 100644 --- a/tests/qom-test.c +++ b/tests/qom-test.c @@ -47,7 +47,7 @@ static bool is_blacklisted(const char *arch, const char *mach) static void test_properties(const char *path, bool recurse) { char *child_path; - QDict *response, *tuple; + QDict *response, *tuple, *tmp; QList *list; QListEntry *entry; @@ -57,6 +57,7 @@ static void test_properties(const char *path, bool recurse) g_assert(response); if (!recurse) { + QDECREF(response); return; } @@ -75,14 +76,16 @@ static void test_properties(const char *path, bool recurse) } else { const char *prop = qdict_get_str(tuple, "name"); g_test_message("Testing property %s.%s", path, prop); - response = qmp("{ 'execute': 'qom-get'," - " 'arguments': { 'path': %s," - " 'property': %s } }", - path, prop); + tmp = qmp("{ 'execute': 'qom-get'," + " 'arguments': { 'path': %s," + " 'property': %s } }", + path, prop); /* qom-get may fail but should not, e.g., segfault. */ - g_assert(response); + g_assert(tmp); + QDECREF(tmp); } } + QDECREF(response); } static void test_machine(gconstpointer data) @@ -98,9 +101,11 @@ static void test_machine(gconstpointer data) response = qmp("{ 'execute': 'quit' }"); g_assert(qdict_haskey(response, "return")); + QDECREF(response); qtest_end(); g_free(args); + g_free((void *)machine); } static void add_machine_test_cases(void) @@ -129,10 +134,12 @@ static void add_machine_test_cases(void) mname = qstring_get_str(qstr); if (!is_blacklisted(arch, mname)) { path = g_strdup_printf("qom/%s", mname); - qtest_add_data_func(path, mname, test_machine); + qtest_add_data_func(path, g_strdup(mname), test_machine); } } + qtest_end(); + QDECREF(response); } int main(int argc, char **argv)