/pidgin/main: f5bdb19e56b8: Fix some crashes

Gary Kramlich <[email protected]>
Newsgroups gmane.comp.gnome.gaim.cvs
Message-ID <[email protected]>
Changeset: f5bdb19e56b8c362d7b52b58ddbcc0530a5711d7
Author:	 Gary Kramlich <[email protected]>
Date:	 2015-12-18 22:29 -0600
Branch:	 use-gresolver
URL: https://hg.pidgin.im/pidgin/main/rev/f5bdb19e56b8

Description:

Fix some crashes

diffstat:

 libpurple/proxy.c |  19 +++++++++----------
 1 files changed, 9 insertions(+), 10 deletions(-)

diffs (73 lines):

diff --git a/libpurple/proxy.c b/libpurple/proxy.c
--- a/libpurple/proxy.c
+++ b/libpurple/proxy.c
@@ -1,4 +1,4 @@
-/* purple
+ /* purple
  *
  * Purple is the legal property of its developers, whose names are too numerous
  * to list here.  Please refer to the COPYRIGHT file distributed with this
@@ -2170,7 +2170,7 @@ static void try_connect(PurpleProxyConne
 	GError *error = NULL;
 	char *ipaddr;
 
-	common_sockaddr_t *addr;
+	common_sockaddr_t addr;
 	socklen_t addrlen = 0;
 
 	address = G_INET_ADDRESS(connect_data->hosts->data);
@@ -2182,7 +2182,7 @@ static void try_connect(PurpleProxyConne
 	socket_address = g_inet_socket_address_new(address, connect_data->port);
 	addrlen = g_socket_address_get_native_size(socket_address);
 
-	g_socket_address_to_native(socket_address, addr, addrlen, &error);
+	g_socket_address_to_native(socket_address, &addr, addrlen, &error);
 	if(error != NULL) {
 		purple_debug_info("proxy", "failed connnection : %s\n", error->message);
 
@@ -2192,31 +2192,31 @@ static void try_connect(PurpleProxyConne
 	}
 
 	if (connect_data->socket_type == SOCK_DGRAM) {
-		proxy_connect_udp_none(connect_data, addr, addrlen);
+		proxy_connect_udp_none(connect_data, &addr, addrlen);
 
 		return;
 	}
 
 	switch (purple_proxy_info_get_proxy_type(connect_data->gpi)) {
 		case PURPLE_PROXY_NONE:
-			proxy_connect_none(connect_data, addr, addrlen);
+			proxy_connect_none(connect_data, &addr, addrlen);
 			break;
 
 		case PURPLE_PROXY_HTTP:
-			proxy_connect_http(connect_data, addr, addrlen);
+			proxy_connect_http(connect_data, &addr, addrlen);
 			break;
 
 		case PURPLE_PROXY_SOCKS4:
-			proxy_connect_socks4(connect_data, addr, addrlen);
+			proxy_connect_socks4(connect_data, &addr, addrlen);
 			break;
 
 		case PURPLE_PROXY_SOCKS5:
 		case PURPLE_PROXY_TOR:
-			proxy_connect_socks5(connect_data, addr, addrlen);
+			proxy_connect_socks5(connect_data, &addr, addrlen);
 			break;
 
 		case PURPLE_PROXY_USE_ENVVAR:
-			proxy_connect_http(connect_data, addr, addrlen);
+			proxy_connect_http(connect_data, &addr, addrlen);
 			break;
 
 		default:
@@ -2224,7 +2224,6 @@ static void try_connect(PurpleProxyConne
 	}
 
 	g_object_unref(G_OBJECT(socket_address));
-	g_free(addr);
 }
 
 static void

_______________________________________________
Commits mailing list
[email protected]
https://pidgin.im/cgi-bin/mailman/listinfo/commits
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.