pidgin.yaz: src/protocols/msn/switchboard.c comparison

comparison src/protocols/msn/switchboard.c @ 10403:e5455f1dc9b6

[gaim-migrate @ 11648] This is "Yet another MSN fix" from Felipe, it actually turned into "Several MSN fixes", see bug 1088651 for the details. committer: Tailor Script <tailor@pidgin.im>

author	Stu Tomlinson <stu@nosnilmot.com>
date	Thu, 23 Dec 2004 20:13:54 +0000
parents	38256d3e8324
children	6be6da69d603

comparison

equal deleted inserted replaced

-:3d2dd0ed368d
+:e5455f1dc9b6
 		{
 			/* The messages could not be sent due to a switchboard error */
 			msg_error_helper(swboard->servconn->cmdproc, msg,
 							 MSN_MSG_ERROR_SB);
 		}
-		msn_message_destroy(msg);
+		msn_message_unref(msg);
 	}
 	g_queue_free(swboard->im_queue);
+	for (l = swboard->ack_list; l != NULL; l = l->next)
+		msn_message_unref(l->data);
 	if (swboard->im_user != NULL)
 		g_free(swboard->im_user);
 	if (swboard->auth_key != NULL)
 }
 static void
 msg_error_helper(MsnCmdProc *cmdproc, MsnMessage *msg, MsnMsgErrorType error)
 {
+	MsnSwitchBoard *swboard;
 	g_return_if_fail(cmdproc != NULL);
 	g_return_if_fail(msg     != NULL);
-	if (msg->nak_cb != NULL)
+	if ((error != MSN_MSG_ERROR_SB) && (msg->nak_cb != NULL))
 		msg->nak_cb(msg, msg->ack_data);
+	swboard = cmdproc->servconn->data;
 	if (msg->type == MSN_MSG_TEXT)
 	{
-		MsnSwitchBoard *swboard;
 		const char *format;
 		char *body_str, *body_enc, *pre, *post;
 		char *str_reason;
-		swboard = cmdproc->servconn->data;
 #if 0
 		if (swboard->conv == NULL)
 		{
 			if (msg->ack_ref)
-				 msn_message_unref(msg);
+				msn_message_unref(msg);
 			return;
 		}
 #endif
 		body_str = g_strdup_printf("%s%s%s", pre, body_enc, post);
 		g_free(body_enc);
 		g_free(pre);
 		g_free(post);
-		msn_switchboard_report_user(cmdproc->servconn->data,
+		msn_switchboard_report_user(swboard, GAIM_MESSAGE_ERROR,
-									GAIM_MESSAGE_ERROR, str_reason);
+									str_reason);
-		msn_switchboard_report_user(cmdproc->servconn->data,
+		msn_switchboard_report_user(swboard, GAIM_MESSAGE_RAW,
-									GAIM_MESSAGE_RAW, body_str);
+									body_str);
 		g_free(body_str);
 	}
-	if (msg->ack_ref)
+	/* If a timeout occures we will want the msg around just in case we
+	 * receive the ACK after the timeout. */
+	if (msg->ack_ref && error != MSN_MSG_ERROR_TIMEOUT)
+	{
+		swboard->ack_list = g_list_remove(swboard->ack_list, msg);
 		msn_message_unref(msg);
+	}
 }
 /**************************************************************************
 * Switchboard Commands
 **************************************************************************/
 	msn_switchboard_add_user(swboard, passport);
 	msn_switchboard_process_queue(swboard);
 	send_clientcaps(swboard);
+	if (swboard->closed)
+		msn_switchboard_close(swboard);
 }
 static void
 msg_cmd_post(MsnCmdProc *cmdproc, MsnCommand *cmd, char *payload, size_t len)
 {
 }
 static void
 ack_cmd(MsnCmdProc *cmdproc, MsnCommand *cmd)
 {
+	MsnSwitchBoard *swboard;
 	MsnMessage *msg;
 	msg = cmd->trans->data;
 	if (msg->ack_cb != NULL)
 		msg->ack_cb(msg, msg->ack_data);
+	swboard = cmdproc->servconn->data;
+	swboard->ack_list = g_list_remove(swboard->ack_list, msg);
 	msn_message_unref(msg);
 }
 static void
 out_cmd(MsnCmdProc *cmdproc, MsnCommand *cmd)
 	if (msg->type == MSN_MSG_TEXT)
 	{
 		msg->ack_ref = TRUE;
 		msn_message_ref(msg);
+		swboard->ack_list = g_list_append(swboard->ack_list, msg);
 		msn_transaction_set_timeout_cb(trans, msg_timeout);
 	}
 	else if (msg->type == MSN_MSG_SLP)
 	{
 		msg->ack_ref = TRUE;
 		msn_message_ref(msg);
+		swboard->ack_list = g_list_append(swboard->ack_list, msg);
 		msn_transaction_set_timeout_cb(trans, msg_timeout);
 #if 0
 		if (msg->ack_cb != NULL)
 		{
 			msn_transaction_add_cb(trans, "ACK", msg_ack);
 }
 /**************************************************************************
 * Call stuff
 **************************************************************************/
+static void
+got_cal(MsnCmdProc *cmdproc, MsnCommand *cmd)
+{
 #if 0
-static void
-got_cal(MsnCmdProc *cmdproc, MsnCommand *cmd)
-{
 	MsnSwitchBoard *swboard;
 	const char *user;
 	swboard = cmdproc->servconn->data;
 	user = cmd->params[0];
 	msn_switchboard_add_user(swboard, user);
-}
 #endif
+}
 static void
 cal_timeout(MsnCmdProc *cmdproc, MsnTransaction *trans)
 {
 	cal_error_helper(trans, MSN_SB_ERROR_UNKNOWN);
 	g_return_if_fail(swboard != NULL);
 	cmdproc = swboard->servconn->cmdproc;
 	trans = msn_transaction_new(cmdproc, "CAL", "%s", user);
-	/* msn_transaction_add_cb(trans, "CAL", got_cal); */
+	/* this doesn't do anything, but users seem to think that
+	 * 'Unhandled command' is some kind of error, so we don't report it */
+	msn_transaction_add_cb(trans, "CAL", got_cal);
 	msn_transaction_set_data(trans, swboard);
 	msn_transaction_set_timeout_cb(trans, cal_timeout);
 	if (swboard->ready)
 {
 	MsnSwitchBoard *swboard;
 	char *host;
 	int port;
 	swboard = cmd->trans->data;
+	if (g_list_find(cmdproc->session->switches, swboard) == NULL)
+		/* The conversation window was closed. */
+		return;
 	msn_switchboard_set_auth_key(swboard, cmd->params[4]);
 	msn_parse_socket(cmd->params[2], &host, &port);
 	cmdproc = swboard->session->notification->cmdproc;
 	trans = msn_transaction_new(cmdproc, "XFR", "%s", "SB");
 	msn_transaction_add_cb(trans, "XFR", got_swboard);
 	msn_transaction_set_data(trans, swboard);
 	msn_transaction_set_error_cb(trans, xfr_error);
 	msn_cmdproc_send_trans(cmdproc, trans);
+}
+void
+msn_switchboard_close(MsnSwitchBoard *swboard)
+{
+	g_return_if_fail(swboard != NULL);
+	if (g_queue_is_empty(swboard->im_queue))
+	{
+		MsnCmdProc *cmdproc;
+		cmdproc = swboard->servconn->cmdproc;
+		msn_cmdproc_send_quick(cmdproc, "OUT", NULL, NULL);
+		msn_switchboard_destroy(swboard);
+	}
+	else
+		swboard->closed = TRUE;
 }
 /**************************************************************************
 * Init stuff
 **************************************************************************/

Mercurial > pidgin.yaz

comparison src/protocols/msn/switchboard.c @ 10403:e5455f1dc9b6