From: Markus Armbruster <armbru@redhat.com>
Date: Wed, 28 May 2014 09:17:03 +0000 (+0200)
Subject: block/qapi: Plug memory leak in dump_qobject() case QTYPE_QERROR
X-Git-Tag: v1.7.2~30
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=7267e51b324dd23837b244261417c7002054430f;p=thirdparty%2Fqemu.git

block/qapi: Plug memory leak in dump_qobject() case QTYPE_QERROR

Introduced in commit a8d8ecb.  Spotted by Coverity.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Benoit Canet <benoit@irqsave.net>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
(cherry picked from commit f25391c2a6ef1674384204265429520ea50e82bc)
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
---

diff --git a/block/qapi.c b/block/qapi.c
index 5880b3e42b8..e9007c9ae57 100644
--- a/block/qapi.c
+++ b/block/qapi.c
@@ -471,6 +471,7 @@ static void dump_qobject(fprintf_function func_fprintf, void *f,
         case QTYPE_QERROR: {
             QString *value = qerror_human((QError *)obj);
             func_fprintf(f, "%s", qstring_get_str(value));
+            QDECREF(value);
             break;
         }
         case QTYPE_NONE: