pidgin: 48c01b8f: I think errdesc was meant to be used her...

markdoliner at pidgin.im markdoliner at pidgin.im
Sun Nov 22 13:25:53 EST 2009


-----------------------------------------------------------------
Revision: 48c01b8f3e78e30b7a04a09f6b92beecd2953c00
Ancestor: ddaadfba202c412c985805db34ae4f690d433df5
Author: markdoliner at pidgin.im
Date: 2009-11-22T18:22:18
Branch: im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/48c01b8f3e78e30b7a04a09f6b92beecd2953c00

Modified files:
        libpurple/protocols/mxit/protocol.c

ChangeLog: 

I think errdesc was meant to be used here, not errmsg.  errmsg isn't
initialized and errdesc is, and errdesc makes the happy little error
box show up at the bottom of the buddy list.  Fixes a crash when
attempting to register for a new account (and failing because my
phone number is invalid--maybe I have to be in South Africa?)

-------------- next part --------------
============================================================
--- libpurple/protocols/mxit/protocol.c	de8d2063f9faa43a12faab67561b877d4737d0e1
+++ libpurple/protocols/mxit/protocol.c	31ebea27f7e865e35eb1a9dbfa2e4dfb8b681846
@@ -1944,7 +1944,7 @@ static int process_error_response( struc
 
 	if ( packet->errcode == MXIT_ERRCODE_LOGGEDOUT ) {
 		/* we are not currently logged in, so we need to reconnect */
-		purple_connection_error( session->con, _( errmsg ) );
+		purple_connection_error( session->con, _( errdesc ) );
 	}
 
 	/* packet command */


More information about the Commits mailing list