pidgin: 4d36ca58: I guess I re-arranged a little off. We p...

qulogic at pidgin.im qulogic at pidgin.im
Sat Aug 9 19:30:44 EDT 2008


-----------------------------------------------------------------
Revision: 4d36ca589250afa99189df464a7700bc0b38b3c4
Ancestor: 0c9dafd42a9975139f059cd2112c8db91b58bae6
Author: qulogic at pidgin.im
Date: 2008-08-09T23:23:48
Branch: im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/4d36ca589250afa99189df464a7700bc0b38b3c4

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

ChangeLog: 

I guess I re-arranged a little off. We probably should disconnect 
*before* freeing things, right?

Fixes #6547.

-------------- next part --------------
============================================================
--- libpurple/protocols/msn/session.c	b3f34a2dd3ba29c2dc965f64c17b298bbcf0d7b6
+++ libpurple/protocols/msn/session.c	43e65308d6827f3ebad797cf2e9552b4bf78f6fb
@@ -57,6 +57,9 @@ msn_session_destroy(MsnSession *session)
 
 	session->destroying = TRUE;
 
+	if (session->connected)
+		msn_session_disconnect(session);
+
 	if (session->soap_cleanup_handle)
 		purple_timeout_remove(session->soap_cleanup_handle);
 
@@ -84,9 +87,6 @@ msn_session_destroy(MsnSession *session)
 	if (session->notification != NULL)
 		msn_notification_destroy(session->notification);
 
-	if (session->connected)
-		msn_session_disconnect(session);
-
 	msn_userlist_destroy(session->userlist);
 
 	g_free(session->psm);


More information about the Commits mailing list