<div dir="ltr">this patch was already acked in February.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 16, 2015 at 3:36 PM, Christophe Fergeau <span dir="ltr"><<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">g_proxy_address_new() returns a new GProxyAddress, so we must unref it<br>
when no longer needed.<br>
<br>
This fixes:<br>
<br>
==6481== 234 (48 direct, 186 indirect) bytes in 1 blocks are definitely lost in loss record 10,062 of 10,<br>
==6481==    at 0x31FF230A58: g_type_create_instance (gtype.c:1849)<br>
==6481==    by 0x31FF21501A: g_object_new_internal (gobject.c:1774)<br>
==6481==    by 0x31FF216EB4: g_object_new_valist (gobject.c:2033)<br>
==6481==    by 0x31FF217220: g_object_new (gobject.c:1617)<br>
==6481==    by 0x3D4386F33A: g_proxy_address_new (gproxyaddress.c:325)<br>
==6481==    by 0x5717440: proxy_lookup_ready (spice-session.c:2011)<br>
==6481==    by 0x3D43885082: g_task_return_now (gtask.c:1088)<br>
==6481==    by 0x3D438850B8: complete_in_idle_cb (gtask.c:1102)<br>
==6481==    by 0x31FEE4A0B9: g_main_dispatch (gmain.c:3122)<br>
==6481==    by 0x31FEE4A0B9: g_main_context_dispatch (gmain.c:3737)<br>
==6481==    by 0x31FEE4A44F: g_main_context_iterate.isra.29 (gmain.c:3808)<br>
==6481==    by 0x31FEE4A771: g_main_loop_run (gmain.c:4002)<br>
==6481==    by 0x363AC06CC4: gtk_main (in /usr/lib64/libgtk-3.so.0.1600.1)<br>
---<br>
 gtk/spice-session.c | 1 +<br>
 1 file changed, 1 insertion(+)<br>
<br>
diff --git a/gtk/spice-session.c b/gtk/spice-session.c<br>
index 1a68d7d..020a70e 100644<br>
--- a/gtk/spice-session.c<br>
+++ b/gtk/spice-session.c<br>
@@ -2020,6 +2020,7 @@ static void proxy_lookup_ready(GObject *source_object, GAsyncResult *result,<br>
<br>
     open_host_connectable_connect(open_host, G_SOCKET_CONNECTABLE(address));<br>
     g_resolver_free_addresses(addresses);<br>
+    g_object_unref(address);<br>
 }<br>
<br>
 /* main context */<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.3.5<br>
<br>
_______________________________________________<br>
Spice-devel mailing list<br>
<a href="mailto:Spice-devel@lists.freedesktop.org">Spice-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/spice-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/spice-devel</a><br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">Marc-André Lureau</div>
</div>