[libvirt] [PATCH] storage: gluster: Don't leak private data when storage file init fails
Peter Krempa
pkrempa at redhat.com
Fri Feb 14 12:16:10 UTC 2014
In a44b7b87bcc6681e2939f65a3552fc96f68bc7b6 I've introduced a function
that initializes a storage file wrapper object on gluster based volumes.
The initialization function leaks the private data pointer in case of
failure. This patch fixes it.
Reported by John Ferlan.
---
Notes:
Pushed under the trivial rule.
src/storage/storage_backend_gluster.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/storage/storage_backend_gluster.c b/src/storage/storage_backend_gluster.c
index a9c33c2..202a441 100644
--- a/src/storage/storage_backend_gluster.c
+++ b/src/storage/storage_backend_gluster.c
@@ -570,6 +570,7 @@ virStorageFileBackendGlusterInit(virStorageFilePtr file)
error:
VIR_FREE(priv->volname);
glfs_fini(priv->vol);
+ VIR_FREE(priv);
return -1;
}
--
1.8.5.3
More information about the libvir-list
mailing list