[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