# HG changeset patch # User Elliott Sales de Andrade # Date 1246498241 0 # Node ID 47c0795eb4fd79b4508439401a7a6b96600f09ea # Parent 801e54fde3df439a2aefe9999b5646b1ac8eac0c We shouldn't de-reference variables before checking if they're not NULL. diff -r 801e54fde3df -r 47c0795eb4fd libpurple/protocols/msn/state.c --- a/libpurple/protocols/msn/state.c Wed Jul 01 03:13:50 2009 +0000 +++ b/libpurple/protocols/msn/state.c Thu Jul 02 01:30:41 2009 +0000 @@ -249,7 +249,7 @@ void msn_set_psm(MsnSession *session) { - PurpleAccount *account = session->account; + PurpleAccount *account; PurplePresence *presence; PurpleStatus *status; MsnCmdProc *cmdproc; @@ -261,6 +261,7 @@ g_return_if_fail(session != NULL); g_return_if_fail(session->notification != NULL); + account = session->account; cmdproc = session->notification->cmdproc; /* Get the PSM string from Purple's Status Line */