]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commitdiff
libvirtd: avoid leak on failure
authorEric Blake <eblake@redhat.com>
Thu, 2 Jun 2011 21:56:32 +0000 (15:56 -0600)
committerEric Blake <eblake@redhat.com>
Fri, 3 Jun 2011 14:11:42 +0000 (08:11 -0600)
Spotted by Coverity.  Only possible on an OOM condition, so
unlikely to bite in the wild.

* daemon/libvirtd.c (qemudSetLogging): Don't leak memory.

daemon/libvirtd.c

index aec81cf3f94aa5f30b7ef8d638e213dec33279d8..728031f0c2d2df37a8f5830bff83428db77d2390 100644 (file)
@@ -2779,8 +2779,10 @@ qemudSetLogging(struct qemud_server *server, virConfPtr conf,
                     goto free_and_fail;
 
                 if (virAsprintf(&tmp, "%d:file:%s/.libvirt/libvirtd.log",
-                                virLogGetDefaultPriority(), userdir) == -1)
+                                virLogGetDefaultPriority(), userdir) == -1) {
+                    VIR_FREE(userdir);
                     goto out_of_memory;
+                }
             }
         } else {
             if (virAsprintf(&tmp, "%d:stderr", virLogGetDefaultPriority()) < 0)