diff libpurple/protocols/myspace/message.c @ 18909:c32fcdef2809

Only say we are connected after the MsimSession structure is fully setup. (Also change a few debugging statements to help find this bug.) Previously, purple_connection_set_state(..., PURPLE_CONNECTED) was called too early, so that if a client called any msimprpl functions that operated on MsimSession (pretty much all of them) within the signed-on signal handler, the userid etc., would not yet be correct and the call would fail. Now, the call is called at the right time so it works as expected. Closes #2520.
author Jeffrey Connelly <jaconnel@calpoly.edu>
date Sun, 12 Aug 2007 18:48:03 +0000
parents 6541b0edee26
children 692b128fe74a
line wrap: on
line diff
--- a/libpurple/protocols/myspace/message.c	Sun Aug 12 18:20:24 2007 +0000
+++ b/libpurple/protocols/myspace/message.c	Sun Aug 12 18:48:03 2007 +0000
@@ -425,7 +425,9 @@
 		return;
 	}
 
+#ifdef MSIM_MSG_DEBUG_FREE
 	msim_msg_dump("msim_msg_free: freeing %s", msg);
+#endif
 
 	g_list_foreach(msg, msim_msg_free_element, NULL);
 	g_list_free(msg);