On 04/23/2010 02:08 PM, Matthias Bolte wrote:
2010/4/23 Chris Lalancette <clalance(a)redhat.com>:
> We were freeing the virDomainSnapshotDefPtr, but not
> the virDomainSnapshotObjPtr in virDomainSnapshotObjFree.
>
> Signed-off-by: Chris Lalancette <clalance(a)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