qom: fix leak for objects created with -object
authorPaolo Bonzini <pbonzini@redhat.com>
Fri, 20 Dec 2013 22:21:07 +0000 (23:21 +0100)
committerLuiz Capitulino <lcapitulino@redhat.com>
Mon, 6 Jan 2014 18:45:47 +0000 (13:45 -0500)
The object must be unref-ed when its variable goes out of scope.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Tested-by: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
vl.c

diff --git a/vl.c b/vl.c
index 0f6754561e36ae671ae12643f336bb736564bbc2..2170a5e94ce2951f0aaf9b0af56c62dbcab41ef0 100644 (file)
--- a/vl.c
+++ b/vl.c
@@ -2810,12 +2810,13 @@ static int object_create(QemuOpts *opts, void *opaque)
 
     obj = object_new(type);
     if (qemu_opt_foreach(opts, object_set_property, obj, 1) < 0) {
+        object_unref(obj);
         return -1;
     }
 
     object_property_add_child(container_get(object_get_root(), "/objects"),
                               id, obj, NULL);
-
+    object_unref(obj);
     return 0;
 }