[gaim-migrate @ 10644]

Thu, 19 Aug 2004 01:13:58 +0000

author
Daniel Atallah <datallah@pidgin.im>
date
Thu, 19 Aug 2004 01:13:58 +0000
changeset 9776
18190cb3c009
parent 9775
e3a3555b0621
child 9777
82ae75055e22

[gaim-migrate @ 10644]
A patch from Daniel Atallah that should fix sf bug 1008489:
"Windows Messenger BOT Crashes Gaim"

Someone MSN-savvy should check this to make sure it's a valid fix.

committer: Mark Doliner <markdoliner@pidgin.im>

src/protocols/msn/object.c file | annotate | diff | comparison | revisions
src/protocols/msn/user.c file | annotate | diff | comparison | revisions
--- a/src/protocols/msn/object.c	Thu Aug 19 01:06:34 2004 +0000
+++ b/src/protocols/msn/object.c	Thu Aug 19 01:13:58 2004 +0000
@@ -76,6 +76,14 @@
 	GET_STRING_TAG(sha1d,    "SHA1D");
 	GET_STRING_TAG(sha1c,    "SHA1C");
 
+	/* If we are missing any of the required elements then discard the object */
+	if (obj->creator == NULL || obj->size == 0 || obj->type == 0
+			|| obj->location == NULL || obj->friendly == NULL
+			|| obj->sha1d == NULL || obj->sha1c == NULL) {
+		msn_object_destroy(obj);
+		obj = NULL;
+	}
+
 	return obj;
 }
 
--- a/src/protocols/msn/user.c	Thu Aug 19 01:06:34 2004 +0000
+++ b/src/protocols/msn/user.c	Thu Aug 19 01:13:58 2004 +0000
@@ -285,7 +285,7 @@
 
 	user->msnobj = obj;
 
-	if (user->list_op & MSN_LIST_FL_OP)
+	if ((obj != NULL) && (user->list_op & MSN_LIST_FL_OP))
 	{
 		/* TODO: I think we need better buddy icon core functions */
 		GaimAccount *account;

mercurial