From 6976ec94624d74cc983c4f6f85a41ab6d6e31ce0 Mon Sep 17 00:00:00 2001 From: Xavier Claessens Date: Mon, 14 Jul 2008 11:01:21 +0000 Subject: [PATCH] Really fix the leak. Address is created using g_value_dup_boxed so it should be freed using g_boxed_free. svn path=/trunk/; revision=1229 --- libempathy/empathy-tp-tube.c | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/libempathy/empathy-tp-tube.c b/libempathy/empathy-tp-tube.c index 4bd08a71..5f421ac7 100644 --- a/libempathy/empathy-tp-tube.c +++ b/libempathy/empathy-tp-tube.c @@ -477,9 +477,12 @@ empathy_tp_tube_get_socket (EmpathyTpTube *tube, EmpathyTpTubePriv *priv = GET_PRIV (tube); GValue *address; guint address_type; + gchar *ret_hostname = NULL; + guint ret_port; GError *error = NULL; g_return_if_fail (EMPATHY_IS_TP_TUBE (tube)); + g_return_if_fail (hostname != NULL || port != NULL); DEBUG ("Getting stream tube socket address"); @@ -495,14 +498,20 @@ empathy_tp_tube_get_socket (EmpathyTpTube *tube, { case TP_SOCKET_ADDRESS_TYPE_UNIX: case TP_SOCKET_ADDRESS_TYPE_ABSTRACT_UNIX: - dbus_g_type_struct_get (address, 0, hostname, G_MAXUINT); + dbus_g_type_struct_get (address, 0, &ret_hostname, G_MAXUINT); break; case TP_SOCKET_ADDRESS_TYPE_IPV4: case TP_SOCKET_ADDRESS_TYPE_IPV6: - dbus_g_type_struct_get (address, 0, hostname, 1, port, G_MAXUINT); + dbus_g_type_struct_get (address, 0, &ret_hostname, 1, &ret_port, G_MAXUINT); break; } - g_free (address); + if (hostname) { + *hostname = g_strdup (ret_hostname); + } + if (port) { + *port = ret_port; + } + g_boxed_free (G_TYPE_VALUE, address); } -- 2.39.2