[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] PATCH: 0/12: Modular build & node devices integration



"Daniel P. Berrange" <berrange redhat com> wrote:
> The following series of patches are updated version of patches 7-11 of
> this series
>
>  http://www.redhat.com/archives/libvir-list/2008-October/msg00718.html
>
> And integrating David Lively's node device patches ontop

Hi Dan,

I built that and ran "make check" under valgrind.
There were two leaks in the result, though I'm not sure
the leaks are new with this patch series.  Here's one of them
(the other was similar):

249,136 (1,840 direct, 247,296 indirect) bytes in 115 blocks are definitely lost
in loss record 6 of 7
   at 0x4A05174: calloc (vg_replace_malloc.c:397)
   by 0x423C4A: virAlloc (memory.c:100)
   by 0x426E93: virHashCreate (hash.c:96)
   by 0x43724C: virGetConnect (datatypes.c:131)
   by 0x409B64: testCompareFormatXML (xmconfigtest.c:114)
   by 0x409D7B: testCompareHelper (xmconfigtest.c:172)
   by 0x40AD66: virtTestRun (testutils.c:92)
   by 0x409EC9: mymain (xmconfigtest.c:208)
   by 0x40B426: virtTestMain (testutils.c:443)

Here's the fix:

>From df72657ae1a6d16f1722d1517ec1a1f4ab1e302e Mon Sep 17 00:00:00 2001
From: Jim Meyering <meyering redhat com>
Date: Fri, 14 Nov 2008 18:43:48 +0100
Subject: [PATCH] tests: don't leak connection references

* tests/xmconfigtest.c (testCompareFormatXML): Use virUnrefConnect(conn),
not VIR_FREE(conn).
(testCompareParseXML): Likewise.
---
 tests/xmconfigtest.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/xmconfigtest.c b/tests/xmconfigtest.c
index 276a2e4..b88637f 100644
--- a/tests/xmconfigtest.c
+++ b/tests/xmconfigtest.c
@@ -93,7 +93,7 @@ static int testCompareParseXML(const char *xmcfg, const char *xml,
     if (conf)
         virConfFree(conf);
     virDomainDefFree(def);
-    VIR_FREE(conn);
+    virUnrefConnect(conn);

     return ret;
 }
@@ -146,7 +146,7 @@ static int testCompareFormatXML(const char *xmcfg, const char *xml,
         virConfFree(conf);
     VIR_FREE(gotxml);
     virDomainDefFree(def);
-    VIR_FREE(conn);
+    virUnrefConnect(conn);

     return ret;
 }
--
1.6.0.4.911.gc990


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]