Re: [libvirt] [PATCH] conf: Fix one memory leak

On 06/21/2011 03:34 PM, Daniel Veillard wrote:

On Tue, Jun 21, 2011 at 02:45:54PM +0800, Osier Yang wrote:

...

Free def->numatune.memory.nodemask in virDomainDefFree. --- src/conf/domain_conf.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 5360863..38a8587 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -1092,6 +1092,8 @@ void virDomainDefFree(virDomainDefPtr def)

virDomainVcpupinDefFree(def->cputune.vcpupin, def->cputune.nvcpupin);

+ VIR_FREE(def->numatune.memory.nodemask); + virSysinfoDefFree(def->sysinfo);

if (def->namespaceData&& def->ns.free)

ACK,

Daniel

Applied, thanks! Regards Osier