[libvirt] [PATCH 1/2] virCgroupNewPartition: Don't leak @newpath

Michal Privoznik mprivozn at redhat.com
Wed Jul 3 07:47:25 UTC 2013


The @newpath variable is allocated in virCgroupSetPartitionSuffix(). But
it's newer freed.
---
 src/util/vircgroup.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/util/vircgroup.c b/src/util/vircgroup.c
index ae71859..e0b25ed 100644
--- a/src/util/vircgroup.c
+++ b/src/util/vircgroup.c
@@ -1249,7 +1249,7 @@ int virCgroupNewPartition(const char *path,
     int rc;
     char *parentPath = NULL;
     virCgroupPtr parent = NULL;
-    char *newpath;
+    char *newpath = NULL;
     VIR_DEBUG("path=%s create=%d controllers=%x",
               path, create, controllers);
 
@@ -1295,6 +1295,7 @@ cleanup:
         virCgroupFree(group);
     virCgroupFree(&parent);
     VIR_FREE(parentPath);
+    VIR_FREE(newpath);
     return rc;
 }
 #else
-- 
1.8.1.5




More information about the libvir-list mailing list