diff libpurple/protocols/jabber/buddy.c @ 28682:d9eb51a3a3a2

jabber: Don't try to pull a photo out of the <PHOTO/> cdata. When the data from the server looks literally like: <PHOTO> <TYPE>image/jpeg</TYPE> <BINVAL></BINVAL> </PHOTO> xmlnode_get_data(photo_node) will contain (whitespace) data, but in no way is it a base64-encoded image. I can't find any reference to clients distributing avatars in that way in the XEP or the RFC.
author Paul Aurich <paul@darkrain42.org>
date Sat, 03 Oct 2009 02:50:48 +0000
parents c585572e80dd
children 51d507e6c8f3
line wrap: on
line diff
--- a/libpurple/protocols/jabber/buddy.c	Fri Oct 02 16:19:19 2009 +0000
+++ b/libpurple/protocols/jabber/buddy.c	Sat Oct 03 02:50:48 2009 +0000
@@ -1149,9 +1149,8 @@
 				char *bintext = NULL;
 				xmlnode *binval;
 
-				if( ((binval = xmlnode_get_child(child, "BINVAL")) &&
-						(bintext = xmlnode_get_data(binval))) ||
-						(bintext = xmlnode_get_data(child))) {
+				if ((binval = xmlnode_get_child(child, "BINVAL")) &&
+						(bintext = xmlnode_get_data(binval))) {
 					gsize size;
 					guchar *data;
 					gboolean photo = (strcmp(child->name, "PHOTO") == 0);