remove useless tests before xmlFree

* src/qemu_conf.c (qemudParseDiskXML, qemudParseInterfaceXML):
(qemudParseInputXML, qemudParseDhcpRangesXML):
* src/remote_internal.c (doRemoteOpen):
* src/storage_conf.c (virStoragePoolDefParseDoc):
* src/xm_internal.c (xenXMParseXMLDisk, xenXMParseXMLVif):
(xenXMParseXMLToConfig, xenXMAttachInterface):
* src/xml.c (virDomainParseXMLDiskDesc, virDomainParseXMLIfDesc):
(virDomainXMLDevID):

diff --git a/ChangeLog b/ChangeLog
index 79d26e0400473bdd549874251a3d0a7a63c09f21..43a6f19f80835ce72cf422a74a69d5cbda37cb6d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+Tue Apr 29 21:43:28 CEST 2008 Jim Meyering <meyering@redhat.com>
+
+       remove useless tests before xmlFree
+       * src/qemu_conf.c (qemudParseDiskXML, qemudParseInterfaceXML):
+       (qemudParseInputXML, qemudParseDhcpRangesXML):
+       * src/remote_internal.c (doRemoteOpen):
+       * src/storage_conf.c (virStoragePoolDefParseDoc):
+       * src/xm_internal.c (xenXMParseXMLDisk, xenXMParseXMLVif):
+       (xenXMParseXMLToConfig, xenXMAttachInterface):
+       * src/xml.c (virDomainParseXMLDiskDesc, virDomainParseXMLIfDesc):
+       (virDomainXMLDevID):
+
 Tue Apr 29 11:36:00 EST 2008 Daniel P. Berrange <berrange@redhat.com>
 
        * src/Makefile.am: Explicitly tell gcov to look in .libs for

diff --git a/src/qemu_conf.c b/src/qemu_conf.c
index 07dfe47d8bd838a6aece626af5660f8e531922c4..a0c4a8ff7b4c30461866735034a3493f0fb279e0 100644 (file)
--- a/src/qemu_conf.c
+++ b/src/qemu_conf.c
@@ -680,14 +680,10 @@ static int qemudParseDiskXML(virConnectPtr conn,
     return 0;
 
  error:
-    if (type)
-        xmlFree(type);
-    if (target)
-        xmlFree(target);
-    if (source)
-        xmlFree(source);
-    if (device)
-        xmlFree(device);
+    xmlFree(type);
+    xmlFree(target);
+    xmlFree(source);
+    xmlFree(device);
     return -1;
 }
 
@@ -941,18 +937,12 @@ static int qemudParseInterfaceXML(virConnectPtr conn,
     return 0;
 
  error:
-    if (network)
-        xmlFree(network);
-    if (address)
-        xmlFree(address);
-    if (port)
-        xmlFree(port);
-    if (ifname)
-        xmlFree(ifname);
-    if (script)
-        xmlFree(script);
-    if (bridge)
-        xmlFree(bridge);
+    xmlFree(network);
+    xmlFree(address);
+    xmlFree(port);
+    xmlFree(ifname);
+    xmlFree(script);
+    xmlFree(bridge);
     return -1;
 }
 
@@ -1334,18 +1324,14 @@ static int qemudParseInputXML(virConnectPtr conn,
             input->bus = QEMU_INPUT_BUS_USB;
     }
 
-    if (type)
-        xmlFree(type);
-    if (bus)
-        xmlFree(bus);
+    xmlFree(type);
+    xmlFree(bus);
 
     return 0;
 
  error:
-    if (type)
-        xmlFree(type);
-    if (bus)
-        xmlFree(bus);
+    xmlFree(type);
+    xmlFree(bus);
 
     return -1;
 }
@@ -2860,10 +2846,8 @@ static int qemudParseDhcpRangesXML(virConnectPtr conn,
             free(range);
         }
 
-        if (start)
-            xmlFree(start);
-        if (end)
-            xmlFree(end);
+        xmlFree(start);
+        xmlFree(end);
 
         cur = cur->next;
     }

diff --git a/src/remote_internal.c b/src/remote_internal.c
index ef34a3a09d4bbf8eb4f7b623f3b0234a47936bbf..70aa5e9065b8561f3fa846fdc5013932a6b42ea3 100644 (file)
--- a/src/remote_internal.c
+++ b/src/remote_internal.c
@@ -434,9 +434,9 @@ doRemoteOpen (virConnectPtr conn,
     }
 
 #ifdef HAVE_XMLURI_QUERY_RAW
-    if (uri->query_raw) xmlFree (uri->query_raw);
+    xmlFree (uri->query_raw);
 #else
-    if (uri->query) xmlFree (uri->query);
+    xmlFree (uri->query);
 #endif
 
     if ((
@@ -464,10 +464,10 @@ doRemoteOpen (virConnectPtr conn,
             transport_str[-1] = '\0';
         }
         /* Remove the username, server name and port number. */
-        if (uri->user) xmlFree (uri->user);
+        xmlFree (uri->user);
         uri->user = 0;
 
-        if (uri->server) xmlFree (uri->server);
+        xmlFree (uri->server);
         uri->server = 0;
 
         uri->port = 0;

diff --git a/src/storage_conf.c b/src/storage_conf.c
index 4499ae28ec66ad2b9b828ca494316a154705902b..be21d3bf5b3f6d54c3b10b7354c6650b97a30c05 100644 (file)
--- a/src/storage_conf.c
+++ b/src/storage_conf.c
@@ -357,8 +357,7 @@ virStoragePoolDefParseDoc(virConnectPtr conn,
 
  cleanup:
     free(uuid);
-    if (type)
-        xmlFree(type);
+    xmlFree(type);
     virStoragePoolDefFree(ret);
     return NULL;
 }

diff --git a/src/xm_internal.c b/src/xm_internal.c
index a70436d271de514321b337eb9d65f78971afd05f..08e3e8e09e9836fe89b2058b2ff7da527e6d0b9f 100644 (file)
--- a/src/xm_internal.c
+++ b/src/xm_internal.c
@@ -1657,10 +1657,8 @@ static int xenXMParseXMLDisk(xmlNodePtr node, int hvm, int xendConfigVersion, ch
     }
 
     if (target == NULL) {
-        if (source != NULL)
-            xmlFree(source);
-        if (device != NULL)
-            xmlFree(device);
+        xmlFree(source);
+        xmlFree(device);
         return (-1);
     }
 
@@ -1687,10 +1685,8 @@ static int xenXMParseXMLDisk(xmlNodePtr node, int hvm, int xendConfigVersion, ch
     }
 
     if (source == NULL && !cdrom) {
-        if (target != NULL)
-            xmlFree(target);
-        if (device != NULL)
-            xmlFree(device);
+        xmlFree(target);
+        xmlFree(device);
         return (-1);
     }
 
@@ -1765,8 +1761,7 @@ static int xenXMParseXMLDisk(xmlNodePtr node, int hvm, int xendConfigVersion, ch
     xmlFree(drvName);
     xmlFree(device);
     xmlFree(target);
-    if(source)
-        xmlFree(source);
+    xmlFree(source);
     *disk = buf;
 
     return (ret);
@@ -1877,14 +1872,10 @@ static char *xenXMParseXMLVif(virConnectPtr conn, xmlNodePtr node, int hvm) {
 
  cleanup:
     free(bridge);
-    if (mac != NULL)
-        xmlFree(mac);
-    if (source != NULL)
-        xmlFree(source);
-    if (script != NULL)
-        xmlFree(script);
-    if (ip != NULL)
-        xmlFree(ip);
+    xmlFree(mac);
+    xmlFree(source);
+    xmlFree(script);
+    xmlFree(ip);
 
     return buf;
 }
@@ -2164,8 +2155,7 @@ virConfPtr xenXMParseXMLToConfig(virConnectPtr conn, const char *xml) {
                             strcat(val, ",vncdisplay=");
                             strcat(val, portstr);
                         }
-                        if (vncport)
-                            xmlFree(vncport);
+                        xmlFree(vncport);
                         if (vnclisten) {
                             strcat(val, ",vnclisten=");
                             strcat(val, (const char*)vnclisten);
@@ -2310,8 +2300,7 @@ virConfPtr xenXMParseXMLToConfig(virConnectPtr conn, const char *xml) {
  error:
     if (conf)
         virConfFree(conf);
-    if (prop != NULL)
-        xmlFree(prop);
+    xmlFree(prop);
     xmlXPathFreeObject(obj);
     xmlXPathFreeContext(ctxt);
     if (doc != NULL)
@@ -2960,12 +2949,9 @@ xenXMAttachInterface(virDomainPtr domain, xmlXPathContextPtr ctxt, int hvm,
     goto cleanup;
 
  node_cleanup:
-    if (node_tmp)
-        xmlFree(node_tmp);
-    if (attr_node)
-        xmlFree(attr_node);
-    if (text_node)
-        xmlFree(text_node);
+    xmlFree(node_tmp);
+    xmlFree(attr_node);
+    xmlFree(text_node);
  cleanup:
     free(type);
     free(source);

diff --git a/src/xml.c b/src/xml.c
index e889cdd8a6eda0d91e890a0dbfb6db806f5b0e27..25eba3d1b593c2234931276acce21b539fe5dcaf 100644 (file)
--- a/src/xml.c
+++ b/src/xml.c
@@ -1341,16 +1341,11 @@ virDomainParseXMLDiskDesc(virConnectPtr conn, xmlNodePtr node,
     virBufferAddLit(buf, ")");
 
   cleanup:
-    if (drvType)
-        xmlFree(drvType);
-    if (drvName)
-        xmlFree(drvName);
-    if (device)
-        xmlFree(device);
-    if (target)
-        xmlFree(target);
-    if (source)
-        xmlFree(source);
+    xmlFree(drvType);
+    xmlFree(drvName);
+    xmlFree(device);
+    xmlFree(target);
+    xmlFree(source);
     return (ret);
 }
 
@@ -1466,14 +1461,10 @@ virDomainParseXMLIfDesc(virConnectPtr conn ATTRIBUTE_UNUSED,
     virBufferAddLit(buf, ")");
     ret = 0;
   error:
-    if (mac != NULL)
-        xmlFree(mac);
-    if (source != NULL)
-        xmlFree(source);
-    if (script != NULL)
-        xmlFree(script);
-    if (ip != NULL)
-        xmlFree(ip);
+    xmlFree(mac);
+    xmlFree(source);
+    xmlFree(script);
+    xmlFree(ip);
     return (ret);
 }
 
@@ -1953,8 +1944,7 @@ virDomainXMLDevID(virDomainPtr domain, const char *xmldesc, char *class,
   cleanup:
     if (xml != NULL)
         xmlFreeDoc(xml);
-    if (attr != NULL)
-        xmlFree(attr);
+    xmlFree(attr);
     return ret;
 }
 #endif /* WITH_XEN */