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

Re: [libvirt] [PATCH 1/2] conf: Fix cpumask leak in virDomainDefFree



On 25.01.2013 16:36, Ján Tomko wrote:
> def->cpumask is a bitmap and needs to be freed by virBitmapFree.
> ---
>  src/conf/domain_conf.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
> index 7273790..c29d2ef 100644
> --- a/src/conf/domain_conf.c
> +++ b/src/conf/domain_conf.c
> @@ -1738,7 +1738,7 @@ void virDomainDefFree(virDomainDefPtr def)
>      virDomainClockDefClear(&def->clock);
>  
>      VIR_FREE(def->name);
> -    VIR_FREE(def->cpumask);
> +    virBitmapFree(def->cpumask);
>      VIR_FREE(def->emulator);
>      VIR_FREE(def->description);
>      VIR_FREE(def->title);
> 

ACK

Michal


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