On 04/23/2010 02:08 PM, Matthias Bolte wrote:
> 2010/4/23 Chris Lalancette <[email protected]>:
>> We were freeing the virDomainSnapshotDefPtr, but not
>> the virDomainSnapshotObjPtr in virDomainSnapshotObjFree.
>>
>> Signed-off-by: Chris Lalancette <[email protected]>
>> ---
>>  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

--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to