Re: upnp: plugging leaks

[Iago Toral <itoral igalia com>]

At first glance the patches look good to me, but since these are on top 
of the patch to add the query() support I'll let Juan and Guillaume to 
handle these after that one.

BTW, looks like we are leaking 'OperationSpec *os' also in 
upnp_source_browse, upnp_source_search and upnp_source_metadata when 
gupnp_service_proxy_begin_action fails, this also seems to be present in 
your patch to add grl_upnp_source_query.

Iago

On Thu, 13 Jan 2011 16:30:09 +0200, Jussi Kukkonen 
<jku linux intel com> wrote:
> Plugging some leaks in grl-upnp: the leaks are all in git master
> but the patches are on top of my earlier patches to keep
> things simple (as I managed to copy at least one of the leaks in 
> those
> patches). Again, let me know if I need to make them independent.
>
> _______________________________________________
> grilo-list mailing list
> grilo-list gnome org
> http://mail.gnome.org/mailman/listinfo/grilo-list