From 5561e9125fbe0cfb9c361ae880432ed3c6c6f219 Mon Sep 17 00:00:00 2001
From: Jim Meyering <meyering@redhat.com>
Date: Wed, 30 Jan 2008 19:50:14 +0000
Subject: [PATCH] Plug test-related leaks.

* src/test.c (testLoadNetwork): Free forwardDev.
(testLoadDomain): Free ctxt.
(testLoadNetwork): Likewise.
(testOpenFromFile): Likewise.
---
 ChangeLog  | 8 ++++++++
 src/test.c | 8 ++++++++
 2 files changed, 16 insertions(+)

diff --git a/ChangeLog b/ChangeLog
index a0eee38ad1..fad8577f11 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+Wed Jan 30 20:49:34 CET 2008  Jim Meyering  <meyering@redhat.com>
+
+	Plug test-related leaks.
+	* src/test.c (testLoadNetwork): Free forwardDev.
+	(testLoadDomain): Free ctxt.
+	(testLoadNetwork): Likewise.
+	(testOpenFromFile): Likewise.
+
 Wed Jan 30 13:18:25 EST 2008 Daniel P. Berrange <berrange@redhat.com>
 
 	* src/hash.c: Remove double mutex lock() flaw
diff --git a/src/test.c b/src/test.c
index 2afd86fd71..35e41a3dce 100644
--- a/src/test.c
+++ b/src/test.c
@@ -351,9 +351,11 @@ static int testLoadDomain(virConnectPtr conn,
     privconn->domains[handle].onPoweroff = onPoweroff;
     privconn->domains[handle].onCrash = onCrash;
 
+    xmlXPathFreeContext(ctxt);
     return (handle);
 
  error:
+    xmlXPathFreeContext(ctxt);
     free(name);
     return (-1);
 }
@@ -507,6 +509,7 @@ static int testLoadNetwork(virConnectPtr conn,
     if (forwardDev) {
         strncpy(privconn->networks[handle].forwardDev, forwardDev, sizeof(privconn->networks[handle].forwardDev)-1);
         privconn->networks[handle].forwardDev[sizeof(privconn->networks[handle].forwardDev)-1] = '\0';
+        free(forwardDev);
     }
 
     strncpy(privconn->networks[handle].ipAddress, ipaddress, sizeof(privconn->networks[handle].ipAddress)-1);
@@ -521,9 +524,12 @@ static int testLoadNetwork(virConnectPtr conn,
     strncpy(privconn->networks[handle].dhcpEnd, dhcpend, sizeof(privconn->networks[handle].dhcpEnd)-1);
     privconn->networks[handle].dhcpEnd[sizeof(privconn->networks[handle].dhcpEnd)-1] = '\0';
     free(dhcpend);
+    xmlXPathFreeContext(ctxt);
     return (handle);
 
  error:
+    xmlXPathFreeContext(ctxt);
+    free (forwardDev);
     free(ipaddress);
     free(ipnetmask);
     free(dhcpstart);
@@ -827,11 +833,13 @@ static int testOpenFromFile(virConnectPtr conn,
         }
     }
 
+    xmlXPathFreeContext(ctxt);
     xmlFreeDoc(xml);
 
     return (0);
 
  error:
+    xmlXPathFreeContext(ctxt);
     free(domains);
     free(networks);
     if (xml)