On 04/23/2010 02:08 PM, Matthias Bolte wrote:
> 2010/4/23 Chris Lalancette <clalance@xxxxxxxxxx>:
>> We were freeing the virDomainSnapshotDefPtr, but not
>> the virDomainSnapshotObjPtr in virDomainSnapshotObjFree.
>>
>> Signed-off-by: Chris Lalancette <clalance@xxxxxxxxxx>
>> ---
>> 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
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list