Please be a bit more verbose in the commit log as to what the leak is.
(a virConnect reference is leaked in error cases, the unref is moved
after the label we jump to on errors to avoid the leak)
Looks good otherwise,
Christophe
On Wed, Jul 01, 2015 at 09:40:46PM +0100, Zeeshan Ali (Khattak) wrote:
---
libvirt-gobject/libvirt-gobject-connection.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/libvirt-gobject/libvirt-gobject-connection.c
b/libvirt-gobject/libvirt-gobject-connection.c
index 1576906..dddbd3a 100644
--- a/libvirt-gobject/libvirt-gobject-connection.c
+++ b/libvirt-gobject/libvirt-gobject-connection.c
@@ -748,7 +748,6 @@ gboolean gvir_connection_fetch_domains(GVirConnection *conn,
if (priv->domains)
g_hash_table_unref(priv->domains);
priv->domains = doms;
- virConnectClose(vconn);
g_mutex_unlock(priv->lock);
ret = TRUE;
@@ -759,6 +758,8 @@ cleanup:
virDomainFree(domains[i]);
free(domains);
}
+ if (vconn != NULL)
+ virConnectClose(vconn);
return ret;
}
@@ -835,7 +836,6 @@ gboolean gvir_connection_fetch_storage_pools(GVirConnection *conn,
if (priv->pools)
g_hash_table_unref(priv->pools);
priv->pools = pools;
- virConnectClose(vconn);
g_mutex_unlock(priv->lock);
ret = TRUE;
@@ -846,6 +846,8 @@ cleanup:
virStoragePoolFree(vpools[i]);
free(vpools);
}
+ if (vconn != NULL)
+ virConnectClose(vconn);
return ret;
}
--
2.4.3
--
libvir-list mailing list
libvir-list(a)redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list