comparison 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
comparison
equal deleted inserted replaced
18908:6541b0edee26 18909:c32fcdef2809
423 if (!msg) { 423 if (!msg) {
424 /* already free as can be */ 424 /* already free as can be */
425 return; 425 return;
426 } 426 }
427 427
428 #ifdef MSIM_MSG_DEBUG_FREE
428 msim_msg_dump("msim_msg_free: freeing %s", msg); 429 msim_msg_dump("msim_msg_free: freeing %s", msg);
430 #endif
429 431
430 g_list_foreach(msg, msim_msg_free_element, NULL); 432 g_list_foreach(msg, msim_msg_free_element, NULL);
431 g_list_free(msg); 433 g_list_free(msg);
432 } 434 }
433 435