# HG changeset patch
# User Nathan Walp <nwalp@pidgin.im>
# Date 1051995526 0
# Node ID a6f37a893b497064efd93985620c82099e0adebf
# Parent  1f9e349b4e09ac07ccd692d47cae166e8c4004bf
[gaim-migrate @ 5663]
this will make things not crash, fix a bug I introduced in the last commit,
and handle multiple resources correctly.

committer: Tailor Script <tailor@pidgin.im>

diff -r 1f9e349b4e09 -r a6f37a893b49 src/protocols/jabber/jabber.c
--- a/src/protocols/jabber/jabber.c	Sat May 03 16:36:08 2003 +0000
+++ b/src/protocols/jabber/jabber.c	Sat May 03 20:58:46 2003 +0000
@@ -1088,6 +1088,21 @@
 	return jri;
 }
 
+static gboolean jabber_is_default_resource(struct gaim_connection *gc, const char *who)
+{
+	jab_res_info jri = jabber_find_resource(gc, who);
+	char *buddy = g_strdup(who);
+	char *resource = strrchr(buddy, '/');
+
+	if(!resource || !strcmp(resource+1, jri->name)) {
+		g_free(buddy);
+		return TRUE;
+	}
+
+	g_free(buddy);
+	return FALSE;
+}
+
 /*
  * if the resource doesn't exist, create it.  otherwise, just update the priority
  */
@@ -1340,8 +1355,6 @@
 						jri->has_composing = TRUE;
 					jri->has_xhtml = has_xhtml;
 				}
-				if (xmlnode_get_tag(p->x, "gaim"))
-					flags = IM_FLAG_GAIMUSER;
 				jabber_track_convo_thread(gjc, from, thread_id);
 				if (gaim_find_conversation(from))
 					serv_got_im(GJ_GC(gjc), from, m, flags,
@@ -1454,6 +1467,11 @@
 	if((buddy = get_realwho(gjc, from, FALSE, NULL)) == NULL)
 		return;
 
+	if(!jabber_is_default_resource(GJ_GC(gjc), buddy)) {
+		g_free(buddy);
+		return;
+	}
+
 	data = xmlnode_get_tag(querynode, "data");
 
 	if(!data) {
@@ -1559,6 +1577,9 @@
 		}
 	}
 
+	if ((y = xmlnode_get_tag(p->x, "priority")))
+		priority = atoi(xmlnode_get_data(y));
+
 	/* um. we're going to check if it's a chat. if it isn't, and there are pending
 	 * chats, create the chat. if there aren't pending chats and we don't have the
 	 * buddy on our list, simply bail out. */
@@ -1585,11 +1606,14 @@
 	}
 
 	for(y = xmlnode_get_firstchild(p->x); y; y = xmlnode_get_nextsibling(y)) {
-		if(!strcmp(xmlnode_get_name(y), "priority")) {
-			priority = atoi(xmlnode_get_data(y));
-		} else if(!strcmp(xmlnode_get_name(y), "x")) {
+		char *tagname = xmlnode_get_name(y);
+		if(!tagname) {
+			continue;
+		} else if(!strcmp(tagname, "x")) {
 			if(!strcmp(xmlnode_get_attrib(y, "xmlns"), "jabber:x:avatar")) {
 				xmlnode hash = xmlnode_get_tag(y, "hash");
+				if(!jabber_is_default_resource(GJ_GC(gjc), from))
+					continue;
 				if(hash) {
 					char *old_hash = gaim_buddy_get_setting(b, "icon_checksum");
 					char *new_hash = xmlnode_get_data(hash);
@@ -1606,8 +1630,6 @@
 		}
 	}
 
-
-
 	if (!cnv) {
 		/* this is where we handle presence information for "regular" buddies */
 		jab_res_info jri = jabber_find_resource(GJ_GC(gjc), buddy);
@@ -2501,7 +2523,6 @@
 
 	x = xmlnode_new_tag("message");
 	xmlnode_put_attrib(x, "to", realwho);
-	xmlnode_insert_tag(x, "gaim");
 
 	y = xmlnode_insert_tag(x, "x");
 	xmlnode_put_attrib(y, "xmlns", "jabber:x:event");
@@ -2565,7 +2586,6 @@
 		g_free(thread_id);
 	}
 
-	xmlnode_insert_tag(x, "gaim");
 	xmlnode_put_attrib(x, "type", "chat");
 
 	/* let other clients know we support typing notification */