I have committed a patch combining this and the handle clean up.

Fabio has tested the patch and it solves the memory leaks he was
seeing.

Commited as 2562.

Regards
Angus Salkeld

On Thu, 3 Dec 2009, angus salkeld wrote:

> Hi
>
> Fabio: this should fix your mem leak (can you test please?).
>
> The key object was not getting free'ed in _clear_object().
>
> -Angus
>
> Signed-off-by: Angus Salkeld <angus.salk...@alliedtelesis.co.nz>
> ---
> exec/objdb.c |    1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/exec/objdb.c b/exec/objdb.c
> index 588f4c0..91dcebe 100644
> --- a/exec/objdb.c
> +++ b/exec/objdb.c
> @@ -683,6 +683,7 @@ static int _clear_object(struct object_instance *instance)
>               list_del(&object_key->list);
>               free(object_key->key_name);
>               free(object_key->value);
> +             free(object_key);
>       }
>
>       for (list = instance->child_head.next;
> -- 
> 1.6.3.4
>
> _______________________________________________
> Openais mailing list
> Openais@lists.linux-foundation.org
> https://lists.linux-foundation.org/mailman/listinfo/openais
>
_______________________________________________
Openais mailing list
Openais@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to