[libvirt] [PATCH] cpu: plug memory leak

Eric Blake eblake at redhat.com
Mon Jan 10 23:59:12 UTC 2011


* src/cpu/cpu_x86.c (x86ModelLoad): Free data before overwriting.
---

Yet another valgrind finding; 320 bytes per domain XML parse.

 src/cpu/cpu_x86.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/src/cpu/cpu_x86.c b/src/cpu/cpu_x86.c
index df1e431..ae8e5e7 100644
--- a/src/cpu/cpu_x86.c
+++ b/src/cpu/cpu_x86.c
@@ -1,7 +1,7 @@
 /*
  * cpu_x86.c: CPU driver for CPUs with x86 compatible CPUID instruction
  *
- * Copyright (C) 2009-2010 Red Hat, Inc.
+ * Copyright (C) 2009-2011 Red Hat, Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ -973,6 +973,7 @@ x86ModelLoad(xmlXPathContextPtr ctxt,
         VIR_FREE(name);

         model->vendor = ancestor->vendor;
+        x86DataFree(model->data);
         if (!(model->data = x86DataCopy(ancestor->data)))
             goto no_memory;
     }
-- 
1.7.3.4




More information about the libvir-list mailing list