Mercurial > pidgin.yaz
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 |