uml: don't free an uninitialized variable

* src/uml_driver.c (umlDomainShutdown): Initialize "info".
Patch and bug report from Ron Yorston <rmy@tigress.co.uk>.

diff --git a/ChangeLog b/ChangeLog
index 870e5c793e177b13dc48914abe9301569b90a0f2..9c3446b6a0e57ea7da7e84ca045f69b7a110c1d0 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+Mon, 1 Dec 2008 10:30:10 +0100 Jim Meyering <meyering@redhat.com>
+
+       uml: don't free an uninitialized variable
+       * src/uml_driver.c (umlDomainShutdown): Initialize "info".
+       Patch and bug report from Ron Yorston <rmy@tigress.co.uk>.
+
 Sun Nov 30 18:36:33 GMT 2008 Daniel P. Berrange <berrange@redhat.com>
 
        * src/xen_inotify.c: Close directory handle once done

diff --git a/src/uml_driver.c b/src/uml_driver.c
index 1ce1d781a14dd17d5b77365cfcff1864b9b811a6..dcf3f66081596866f03d09e76fa358bb96afc204 100644 (file)
--- a/src/uml_driver.c
+++ b/src/uml_driver.c
@@ -1163,7 +1163,7 @@ static virDomainPtr umlDomainCreate(virConnectPtr conn, const char *xml,
 static int umlDomainShutdown(virDomainPtr dom) {
     struct uml_driver *driver = (struct uml_driver *)dom->conn->privateData;
     virDomainObjPtr vm = virDomainFindByID(&driver->domains, dom->id);
-    char* info;
+    char *info = NULL;
 
     if (!vm) {
         umlReportError(dom->conn, dom, NULL, VIR_ERR_INVALID_DOMAIN,
@@ -1672,4 +1672,3 @@ int umlRegister(void) {
     virRegisterStateDriver(&umlStateDriver);
     return 0;
 }
-