Mercurial > pidgin

diff libgaim/protocols/qq/im.c @ 14632:108152f29f37

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
[gaim-migrate @ 17378] There's no reason to generate our session_md5 more than once. This probably doesn't fix the bug from Adium crash 215722, but if it happens again, it'll be a bit easier to track down. Also, I was wrong about the login scheme. Nothing significant is different, so remove the comment. committer: Tailor Script <tailor@pidgin.im>
author Mark Huetsch <markhuetsch>
date Wed, 27 Sep 2006 00:31:39 +0000
parents 6b8bc59414f0
children d1a76ccb93c4
line wrap: on
line diff
--- a/libgaim/protocols/qq/im.c	Tue Sep 26 23:20:39 2006 +0000
+++ b/libgaim/protocols/qq/im.c	Wed Sep 27 00:31:39 2006 +0000
@@ -425,7 +425,7 @@
 void qq_send_packet_im(GaimConnection *gc, guint32 to_uid, gchar *msg, gint type)
 {
 	qq_data *qd;
-	guint8 *cursor, *raw_data, *send_im_tail, *md5;
+	guint8 *cursor, *raw_data, *send_im_tail;
 	guint16 client_tag, normal_im_type;
 	gint msg_len, raw_len, font_name_len, tail_len, bytes;
 	time_t now;
@@ -483,7 +483,6 @@
 	msg_filtered = gaim_markup_strip_html(msg);
 	msg_len = strlen(msg_filtered);
 	now = time(NULL);
-	md5 = _gen_session_md5(qd->uid, qd->session_key);
 
 	font_name_len = (font_name) ? strlen(font_name) : DEFAULT_FONT_NAME_LEN;
 	tail_len = font_name_len + QQ_SEND_IM_AFTER_MSG_HEADER_LEN + 1;
@@ -504,7 +503,7 @@
 	/* 014-017: sender uid */
 	bytes += create_packet_dw(raw_data, &cursor, to_uid);
 	/* 018-033: md5 of (uid+session_key) */
-	bytes += create_packet_data(raw_data, &cursor, md5, 16);
+	bytes += create_packet_data(raw_data, &cursor, qd->session_md5, 16);
 	/* 034-035: message type */
 	bytes += create_packet_w(raw_data, &cursor, normal_im_type);
 	/* 036-037: sequence number */