We need to free the string returned by gvir_config_object_to_xml() after using it. Signed-off-by: Christophe Fergeau <cfergeau(a)redhat.com&gt; --- 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 c48d51b9..15b47093 100644 --- a/libvirt-gobject/libvirt-gobject-connection.c +++ b/libvirt-gobject/libvirt-gobject-connection.c @@ -1999,7 +1999,7 @@ GVirStoragePool *gvir_connection_create_storage_pool GVirConfigStoragePool *conf, guint flags, GError **err) { - const gchar *xml; + gchar *xml; virStoragePoolPtr handle; GVirConnectionPrivate *priv; @@ -2012,7 +2012,9 @@ GVirStoragePool *gvir_connection_create_storage_pool g_return_val_if_fail(xml != NULL, NULL); priv = conn->priv; - if (!(handle = virStoragePoolDefineXML(priv->conn, xml, flags))) { + handle = virStoragePoolDefineXML(priv->conn, xml, flags); + g_free(xml); + if (!handle) { gvir_set_error_literal(err, GVIR_CONNECTION_ERROR, flags, _("Failed to create storage pool")); -- 2.20.1

It's not used outside of the libvirt-glib-event.c file, so there is no good reason for not having it static. As it was not listed in libvirt-glib.sym, this will make no changes to the publicly exported symbols (ie this is not an ABI change). Signed-off-by: Christophe Fergeau <cfergeau(a)redhat.com&gt; --- libvirt-glib/libvirt-glib-event.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libvirt-glib/libvirt-glib-event.c b/libvirt-glib/libvirt-glib-event.c index f8227d61..4548aa65 100644 --- a/libvirt-glib/libvirt-glib-event.c +++ b/libvirt-glib/libvirt-glib-event.c @@ -110,7 +110,7 @@ struct gvir_event_timeout virFreeCallback ff; }; -GMutex *eventlock = NULL; +static GMutex *eventlock = NULL; static int nextwatch = 1; static GPtrArray *handles; -- 2.20.1