cpw.darkrain42.oscar.ssl: 25873190: Use 'bos.oscar.aol.com' as the common na...

paul at darkrain42.org paul at darkrain42.org
Tue Jan 27 22:45:33 EST 2009


-----------------------------------------------------------------
Revision: 258731909ce7311479cee328638b52b0a33383b1
Ancestor: 95292cbdc267f26b5dc7bfbad187fc47ce870e9c
Author: paul at darkrain42.org
Date: 2009-01-28T03:40:55
Branch: im.pidgin.cpw.darkrain42.oscar.ssl
URL: http://d.pidgin.im/viewmtn/revision/info/258731909ce7311479cee328638b52b0a33383b1

Modified files:
        libpurple/protocols/oscar/oscar.c

ChangeLog: 

Use 'bos.oscar.aol.com' as the common name for the initial BOS connection.

The redirect from the login server to first BOS server doesn't include the
magic TLV with a name to use for comparison; I missed this because I cached
all those certificates before I fixed the code for the other SNAC family
servers.

-------------- next part --------------
============================================================
--- libpurple/protocols/oscar/oscar.c	af6614db2fc18b8f37b786e594b123142c1592fc
+++ libpurple/protocols/oscar/oscar.c	92c02b8386bbf8e68ebd89b1ce782ca1c14710d6
@@ -1691,9 +1691,13 @@ purple_parse_auth_resp(OscarData *od, Fl
 
 	if (od->use_ssl)
 	{
-		newconn->gsc = purple_ssl_connect(account, host, port,
-				ssl_connection_established_cb, ssl_connection_error_cb,
-				newconn);
+		/*
+		 * This shouldn't be hardcoded except that the server isn't sending
+		 * us a name to use for comparing the certificate common name.
+		 */
+		newconn->ssl_cert_cn = g_strdup("bos.oscar.aol.com");
+		newconn->connect_data = purple_proxy_connect(NULL, account, host, port,
+				ssl_proxy_conn_established_cb, newconn);
 	}
 	else
 	{


More information about the Commits mailing list