[libvirt] [PATCH 5/8] Fix virDomainSnapshotObjFree memory leak.

Chris Lalancette clalance at redhat.com
Fri Apr 23 18:21:44 UTC 2010


On 04/23/2010 02:08 PM, Matthias Bolte wrote:
> 2010/4/23 Chris Lalancette <clalance at redhat.com>:
>> We were freeing the virDomainSnapshotDefPtr, but not
>> the virDomainSnapshotObjPtr in virDomainSnapshotObjFree.
>>
>> Signed-off-by: Chris Lalancette <clalance at redhat.com>
>> ---
>>  src/conf/domain_conf.c |    1 +
>>  1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
>> index 139712a..3ad24cb 100644
>> --- a/src/conf/domain_conf.c
>> +++ b/src/conf/domain_conf.c
>> @@ -6754,6 +6754,7 @@ static void virDomainSnapshotObjFree(virDomainSnapshotObjPtr snapshot)
>>     VIR_DEBUG("obj=%p", snapshot);
>>
>>     virDomainSnapshotDefFree(snapshot->def);
>> +    VIR_FREE(snapshot);
>>  }
>>
>>  int virDomainSnapshotObjUnref(virDomainSnapshotObjPtr snapshot)
>> --
>> 1.6.6.1
>>
> 
> ACK.

Thanks, pushed.

-- 
Chris Lalancette




More information about the libvir-list mailing list