[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 1/5] test-keyval: fix leaks
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v5 1/5] test-keyval: fix leaks |
Date: |
Wed, 3 May 2017 20:06:03 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.0 |
On 05/03/2017 05:38 PM, Marc-André Lureau wrote:
> Spotted by ASAN.
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> tests/test-keyval.c | 4 ++++
> 1 file changed, 4 insertions(+)
Reviewed-by: Eric blake <address@hidden>
>
> diff --git a/tests/test-keyval.c b/tests/test-keyval.c
> index ba19560a22..c556b1b117 100644
> --- a/tests/test-keyval.c
> +++ b/tests/test-keyval.c
> @@ -628,6 +628,7 @@ static void test_keyval_visit_alternate(void)
> visit_type_AltNumStr(v, "a", &ans, &error_abort);
> g_assert_cmpint(ans->type, ==, QTYPE_QSTRING);
> g_assert_cmpstr(ans->u.s, ==, "1");
> + qapi_free_AltNumStr(ans);
> visit_type_AltNumInt(v, "a", &ani, &err);
> error_free_or_abort(&err);
> visit_end_struct(v, NULL);
> @@ -651,9 +652,12 @@ static void test_keyval_visit_any(void)
> g_assert(qlist);
> qstr = qobject_to_qstring(qlist_pop(qlist));
> g_assert_cmpstr(qstring_get_str(qstr), ==, "null");
> + QDECREF(qstr);
> qstr = qobject_to_qstring(qlist_pop(qlist));
> g_assert_cmpstr(qstring_get_str(qstr), ==, "1");
> g_assert(qlist_empty(qlist));
> + QDECREF(qstr);
> + qobject_decref(any);
> visit_check_struct(v, &error_abort);
> visit_end_struct(v, NULL);
> visit_free(v);
>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v5 0/5] Memory leak fixes, Marc-André Lureau, 2017/05/03
- [Qemu-devel] [PATCH v5 2/5] audio: fix capture buffer leaks, Marc-André Lureau, 2017/05/03
- [Qemu-devel] [PATCH v5 3/5] audio: fix WAVState leak, Marc-André Lureau, 2017/05/03
- [Qemu-devel] [PATCH v5 4/5] slirp: fix leak, Marc-André Lureau, 2017/05/03
- [Qemu-devel] [PATCH v5 5/5] dump: fix memory_mapping_filter leak, Marc-André Lureau, 2017/05/03