Skip to content

Commit

Permalink
tests: fix crumple/recursive leak
Browse files Browse the repository at this point in the history
Spotted by ASAN:

=================================================================
==27907==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 4120 byte(s) in 1 object(s) allocated from:
    #0 0x7f913458ce50 in calloc (/lib64/libasan.so.5+0xeee50)
    my-unikernel#1 0x7f9133fd641d in g_malloc0 (/lib64/libglib-2.0.so.0+0x5241d)
    qemu#2 0x5561c6643c95 in qdict_crumple_test_recursive /home/elmarco/src/qq/tests/check-block-qdict.c:438
    qemu#3 0x7f9133ff7c49  (/lib64/libglib-2.0.so.0+0x73c49)

Signed-off-by: Marc-André Lureau <[email protected]>
Message-Id: <[email protected]>
[Screwed up in commit 2860b2b]
Reviewed-by: Markus Armbruster <[email protected]>
Signed-off-by: Markus Armbruster <[email protected]>
  • Loading branch information
elmarco authored and Markus Armbruster committed Aug 15, 2018
1 parent b736e25 commit 42478da
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions tests/check-block-qdict.c
Original file line number Diff line number Diff line change
Expand Up @@ -491,6 +491,7 @@ static void qdict_crumple_test_recursive(void)
empty_list_0 = qobject_to(QDict, qlist_pop(empty_list));
g_assert(empty_list_0);
g_assert_cmpint(qdict_size(empty_list_0), ==, 0);
qobject_unref(empty_list_0);

qobject_unref(src);
qobject_unref(dst);
Expand Down

0 comments on commit 42478da

Please sign in to comment.