Don't attempt to process zero-length DC messages. We should probably

Sun, 21 Nov 2010 00:53:26 +0000

author
Elliott Sales de Andrade <qulogic@pidgin.im>
date
Sun, 21 Nov 2010 00:53:26 +0000
changeset 31133
9383e6398f7d
parent 31132
bb0b787208f4
child 31134
cd372fe45f02

Don't attempt to process zero-length DC messages. We should probably
just use these for acking or something.

Fixes #12660.

libpurple/protocols/msn/directconn.c file | annotate | diff | comparison | revisions
--- a/libpurple/protocols/msn/directconn.c	Sat Nov 20 18:12:13 2010 +0000
+++ b/libpurple/protocols/msn/directconn.c	Sun Nov 21 00:53:26 2010 +0000
@@ -630,8 +630,10 @@
 
 	case DC_STATE_ESTABLISHED:
 
-		part = msn_slpmsgpart_new_from_data(dc->in_buffer + 4, dc->header.length);
-		msn_slplink_process_msg(dc->slplink, part);
+		if (dc->header.length) {
+			part = msn_slpmsgpart_new_from_data(dc->in_buffer + 4, dc->header.length);
+			msn_slplink_process_msg(dc->slplink, part);
+		}
 
 		/*
 		if (dc->num_calls == 0) {

mercurial