[libvirt] [PATCH] cpu: plug memory leak
Stefan Berger
stefanb at us.ibm.com
Tue Jan 11 01:57:31 UTC 2011
libvir-list-bounces at redhat.com wrote on 01/10/2011 06:59:12 PM:
> From: Eric Blake <eblake at redhat.com>
> To: libvir-list at redhat.com
> Date: 01/10/2011 07:08 PM
> Subject: [libvirt] [PATCH] cpu: plug memory leak
> Sent by: libvir-list-bounces at redhat.com
>
> * 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;
> }
ACK
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110110/6ea8397a/attachment-0001.htm>
More information about the libvir-list
mailing list