[PATCH pushed] conf: Don't leak virDomainDiskDef's 'statistics' member

Peter Krempa via Devel posted 1 patch 2 weeks, 1 day ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/9b7ef8058b18c7e90448b33664b25392fc600fee.1762355115.git.pkrempa@redhat.com
src/conf/domain_conf.c | 1 +
1 file changed, 1 insertion(+)
[PATCH pushed] conf: Don't leak virDomainDiskDef's 'statistics' member
Posted by Peter Krempa via Devel 2 weeks, 1 day ago
From: Peter Krempa <pkrempa@redhat.com>

In commit 19fc614d531f I've added an option to configure statistics but
forgot to free it once the disk definition struct is freed.

Fixes: 19fc614d531fb75877abb59baaf33bc1add4d483
Signed-off-by: Peter Krempa <pkrempa@redhat.com>
---
Pushed as a build fix as this was found by CI ... since I didn't run all
of this via CI :(.

 src/conf/domain_conf.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index c749d505bf..396cd1c0db 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -2444,6 +2444,7 @@ virDomainDiskDefFree(virDomainDiskDef *def)
     virDomainDeviceInfoClear(&def->info);
     virObjectUnref(def->privateData);
     g_slist_free_full(def->iothreads, (GDestroyNotify) virDomainIothreadMappingDefFree);
+    g_free(def->statistics);

     if (def->throttlefilters) {
         size_t i;
-- 
2.51.1