# HG changeset patch
# User Ma Yuan <mayuan2006@gmail.com>
# Date 1161188931 0
# Node ID 21cb7a79ac7f09cd0e08626f3cf5c50fcbf9301d
# Parent  6fc33ac621d955781483f8a260bc5ca7ebc470a8
[gaim-migrate @ 17523]
accurate with the Offline Message send's Fault Code recognize and resend the OIM Get Message.

committed by MaYuan<mayuan2006@gmail.com>


committer: Ethan Blanton <elb@pidgin.im>

diff -r 6fc33ac621d9 -r 21cb7a79ac7f src/protocols/msn/msg.c
--- a/src/protocols/msn/msg.c	Sun Oct 15 13:26:34 2006 +0000
+++ b/src/protocols/msn/msg.c	Wed Oct 18 16:28:51 2006 +0000
@@ -215,7 +215,7 @@
 	char **elems, **cur, **tokens;
 
 	g_return_if_fail(payload != NULL);
-	gaim_debug_info("MaYuan","payload:{%s}\n",payload);
+//	gaim_debug_info("MaYuan","payload:{%s}\n",payload);
 	tmp_base = tmp = g_malloc0(payload_len + 1);
 	memcpy(tmp_base, payload, payload_len);
 
diff -r 6fc33ac621d9 -r 21cb7a79ac7f src/protocols/msn/notification.c
--- a/src/protocols/msn/notification.c	Sun Oct 15 13:26:34 2006 +0000
+++ b/src/protocols/msn/notification.c	Wed Oct 18 16:28:51 2006 +0000
@@ -1409,7 +1409,7 @@
 	const char *passport, *psm_str;
 
 	/*get the payload content*/
-	gaim_debug_info("MaYuan","UBX {%s} payload{%s}\n",cmd->params[0], cmd->payload);
+//	gaim_debug_info("MaYuan","UBX {%s} payload{%s}\n",cmd->params[0], cmd->payload);
 
 	session = cmdproc->session;
 	account = session->account;
diff -r 6fc33ac621d9 -r 21cb7a79ac7f src/protocols/msn/oim.c
--- a/src/protocols/msn/oim.c	Sun Oct 15 13:26:34 2006 +0000
+++ b/src/protocols/msn/oim.c	Wed Oct 18 16:28:51 2006 +0000
@@ -157,8 +157,10 @@
 void
 msn_oim_send_process(MsnOim *oim,char *body,int len)
 {
-	xmlnode *responseNode,*bodyNode,*faultNode;
+	xmlnode *responseNode,*bodyNode;
+	xmlnode	*faultNode,*faultCodeNode,*faultstringNode;
 	xmlnode *detailNode,*challengeNode;
+	char *faultCodeStr,*faultstring;
 	char *challenge;
 
 	responseNode = xmlnode_from_str(body,len);
@@ -178,18 +180,43 @@
 		return;
 	}
 	/*get the challenge,and repost it*/
+	faultCodeNode = xmlnode_get_child(faultNode,"faultcode");
+	if(faultCodeNode == NULL){
+		gaim_debug_info("MaYuan","faultcode Node is NULL\n");
+		goto oim_send_process_fail;
+	}
+	faultCodeStr = xmlnode_get_data(faultCodeNode);
+	gaim_debug_info("MaYuan","fault code:{%s}\n",faultCodeStr);
+
+	if(strcmp(faultCodeStr,"q0:AuthenticationFailed")){
+		/*other Fault Reason?*/
+		goto oim_send_process_fail;
+	}
+
+	faultstringNode = xmlnode_get_child(faultNode,"faultstring");
+	faultstring = xmlnode_get_data(faultstringNode);
+	gaim_debug_info("MaYuan","fault string :{%s}\n",faultstring);
+
+	/* lock key fault reason,
+	 * compute the challenge and resend it
+	 */
 	detailNode = xmlnode_get_child(faultNode, "detail");
+	if(detailNode == NULL){
+			goto oim_send_process_fail;
+	}
 	challengeNode = xmlnode_get_child(detailNode,"LockKeyChallenge");
 
 	g_free(oim->challenge);
 	oim->challenge = xmlnode_get_data(challengeNode);
 	gaim_debug_info("MaYuan","lockkey:{%s}\n",oim->challenge);
 
-	xmlnode_free(responseNode);
-
 	/*repost the send*/
 	gaim_debug_info("MaYuan","prepare to repost the send...\n");
 	msn_oim_send_msg(oim);
+
+oim_send_process_fail:
+	xmlnode_free(responseNode);
+	return ;
 }
 
 static void