From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34533) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gA7iY-00027p-Lq for qemu-devel@nongnu.org; Wed, 10 Oct 2018 02:11:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gA7iV-0008F4-Nl for qemu-devel@nongnu.org; Wed, 10 Oct 2018 02:11:34 -0400 Received: from mx1.redhat.com ([209.132.183.28]:60816) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gA7iV-00083l-G9 for qemu-devel@nongnu.org; Wed, 10 Oct 2018 02:11:31 -0400 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id EEA3E30820E1 for ; Wed, 10 Oct 2018 06:11:18 +0000 (UTC) From: Markus Armbruster Date: Wed, 10 Oct 2018 08:11:09 +0200 Message-Id: <20181010061113.6057-2-armbru@redhat.com> In-Reply-To: <20181010061113.6057-1-armbru@redhat.com> References: <20181010061113.6057-1-armbru@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PULL 1/5] tests/check-qjson: fix a leak List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= From: Marc-Andr=C3=A9 Lureau Spotted by ASAN: =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D =3D=3D11893=3D=3DERROR: LeakSanitizer: detected memory leaks Direct leak of 1120 byte(s) in 28 object(s) allocated from: #0 0x7fd0515b0c48 in malloc (/lib64/libasan.so.5+0xeec48) #1 0x7fd050ffa3c5 in g_malloc (/lib64/libglib-2.0.so.0+0x523c5) #2 0x559e708b56a4 in qstring_from_str /home/elmarco/src/qq/qobject/qs= tring.c:66 #3 0x559e708b4fe0 in qstring_new /home/elmarco/src/qq/qobject/qstring= .c:23 #4 0x559e708bda7d in parse_string /home/elmarco/src/qq/qobject/json-p= arser.c:143 #5 0x559e708c1009 in parse_literal /home/elmarco/src/qq/qobject/json-= parser.c:484 #6 0x559e708c1627 in parse_value /home/elmarco/src/qq/qobject/json-pa= rser.c:547 #7 0x559e708c1c67 in json_parser_parse /home/elmarco/src/qq/qobject/j= son-parser.c:573 #8 0x559e708bc0ff in json_message_process_token /home/elmarco/src/qq/= qobject/json-streamer.c:92 #9 0x559e708d1655 in json_lexer_feed_char /home/elmarco/src/qq/qobjec= t/json-lexer.c:292 #10 0x559e708d1fe1 in json_lexer_feed /home/elmarco/src/qq/qobject/js= on-lexer.c:339 #11 0x559e708bc856 in json_message_parser_feed /home/elmarco/src/qq/q= object/json-streamer.c:121 #12 0x559e708b8b4b in qobject_from_jsonv /home/elmarco/src/qq/qobject= /qjson.c:69 #13 0x559e708b8d02 in qobject_from_json /home/elmarco/src/qq/qobject/= qjson.c:83 #14 0x559e708a74ae in from_json_str /home/elmarco/src/qq/tests/check-= qjson.c:30 #15 0x559e708a9f83 in utf8_string /home/elmarco/src/qq/tests/check-qj= son.c:781 #16 0x7fd05101bc49 in test_case_run gtestutils.c:2255 #17 0x7fd05101bc49 in g_test_run_suite_internal gtestutils.c:2339 Signed-off-by: Marc-Andr=C3=A9 Lureau Message-Id: <20180901211917.10372-1-marcandre.lureau@redhat.com> Reviewed-by: Markus Armbruster Signed-off-by: Markus Armbruster --- tests/check-qjson.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/check-qjson.c b/tests/check-qjson.c index cc13f3d41e..d876a7a96e 100644 --- a/tests/check-qjson.c +++ b/tests/check-qjson.c @@ -780,6 +780,7 @@ static void utf8_string(void) if (!strstr(json_out, "\\uFFFD")) { str =3D from_json_str(json_out, j, &error_abort); g_assert_cmpstr(qstring_get_try_str(str), =3D=3D, utf8_i= n); + qobject_unref(str); } } } --=20 2.17.1