Re: upnp: plugging leaks



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






[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]