cpw.qulogic.msn: 48f78161: Use purple_url_parse for splitting up th...

qulogic at pidgin.im qulogic at pidgin.im
Mon Jun 16 21:15:50 EDT 2008


-----------------------------------------------------------------
Revision: 48f78161c1bcbd6b11d6daecff7e833a63850f31
Ancestor: 4111770b80a7087ebe9a19f64e3b6856aebc1e4e
Author: qulogic at pidgin.im
Date: 2008-06-17T00:55:26
Branch: im.pidgin.cpw.qulogic.msn
URL: http://d.pidgin.im/viewmtn/revision/info/48f78161c1bcbd6b11d6daecff7e833a63850f31

Modified files:
        libpurple/protocols/msn/soap2.c

ChangeLog: 

Use purple_url_parse for splitting up the MSN SOAP redirect instead of 
trying to figure it out ourselves (and doing it wrongly, I might add). 
Thanks again, Dimmuxx for spotting that.

-------------- next part --------------
============================================================
--- libpurple/protocols/msn/soap2.c	a05ed086316fd6face9947b5a9545f4d23cc337b
+++ libpurple/protocols/msn/soap2.c	50a5a784aae0337cf5697dca6e6a993d44ecb181
@@ -183,18 +183,10 @@ msn_soap_handle_redirect(MsnSoapConnecti
 static gboolean
 msn_soap_handle_redirect(MsnSoapConnection *conn, const char *url)
 {
-	char *c;
+	char *host;
+	char *path;
 
-	/* Skip the http:// */
-	if ((c = strchr(url, '/')) != NULL)
-		url += 2;
-
-	if ((c = strchr(url, '/')) != NULL) {
-		char *host, *path;
-
-		host = g_strndup(url, c - url);
-		path = g_strdup(c);
-
+	if (purple_url_parse(url, &host, NULL, &path, NULL, NULL)) {
 		msn_soap_message_send_internal(conn->session,
 			conn->current_request->message,	host, path,
 			conn->current_request->cb, conn->current_request->cb_data, TRUE);


More information about the Commits mailing list