[libvirt] [PATCH] Fix memory leak introduced by commit 501bfad1

Jim Fehlig jfehlig at suse.com
Tue Dec 4 18:10:24 UTC 2012


Daniel P. Berrange wrote:
> On Tue, Dec 04, 2012 at 10:42:43AM -0700, Jim Fehlig wrote:
>   
>> 501bfad1 missed freeing priv->saveDir when opening the Xen unified
>> driver failed.
>> ---
>>  src/xen/xen_driver.c |    1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/src/xen/xen_driver.c b/src/xen/xen_driver.c
>> index 3786176..d2de141 100644
>> --- a/src/xen/xen_driver.c
>> +++ b/src/xen/xen_driver.c
>> @@ -432,6 +432,7 @@ clean:
>>          if (priv->opened[i])
>>              drivers[i]->xenClose(conn);
>>      virMutexDestroy(&priv->lock);
>> +    VIR_FREE(priv->saveDir);
>>      VIR_FREE(priv);
>>      conn->privateData = NULL;
>>      return ret;
>>     
>
> ACK
>   

Thanks, pushed.

Jim




More information about the libvir-list mailing list