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

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



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


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