libpurple/protocols/jabber/useravatar.c

Fri, 24 Mar 2017 02:23:46 +0300

author
qarkai <qarkai@gmail.com>
date
Fri, 24 Mar 2017 02:23:46 +0300
branch
string-comparison-r2
changeset 38259
c593fc9f5438
parent 38258
9a6551eba09c
child 38260
dcada91c0912
permissions
-rw-r--r--

Replace strcmp() with purple_strequal()
Didn't touch finch/libgnt files

25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
1 /*
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
2 * purple - Jabber Protocol Plugin
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
3 *
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
4 * Purple is the legal property of its developers, whose names are too numerous
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
5 * to list here. Please refer to the COPYRIGHT file distributed with this
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
6 * source distribution.
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
7 *
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
8 * This program is free software; you can redistribute it and/or modify
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
9 * it under the terms of the GNU General Public License as published by
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
10 * the Free Software Foundation; either version 2 of the License, or
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
11 * (at your option) any later version.
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
12 *
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
13 * This program is distributed in the hope that it will be useful,
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
14 * but WITHOUT ANY WARRANTY; without even the implied warranty of
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
15 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
16 * GNU General Public License for more details.
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
17 *
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
18 * You should have received a copy of the GNU General Public License
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
19 * along with this program; if not, write to the Free Software
28321
c8d617c408ab Update various header copyrights thanks to licensecheck.
Paul Aurich <darkrain42@pidgin.im>
parents: 28048
diff changeset
20 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02111-1301 USA
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
21 *
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
22 */
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
23
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
24 #include "internal.h"
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
25
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
26 #include "useravatar.h"
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
27 #include "pep.h"
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
28 #include "debug.h"
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
29
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
30 #define MAX_HTTP_BUDDYICON_BYTES (200 * 1024)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
31
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
32 static void update_buddy_metadata(JabberStream *js, const char *from, xmlnode *items);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
33
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
34 void jabber_avatar_init(void)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
35 {
26959
8482365bb2ec Fix merging.
Paul Aurich <darkrain42@pidgin.im>
parents: 26950
diff changeset
36 jabber_add_feature(NS_AVATAR_1_1_METADATA,
25479
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
37 jabber_pep_namespace_only_when_pep_enabled_cb);
26959
8482365bb2ec Fix merging.
Paul Aurich <darkrain42@pidgin.im>
parents: 26950
diff changeset
38 jabber_add_feature(NS_AVATAR_1_1_DATA,
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
39 jabber_pep_namespace_only_when_pep_enabled_cb);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
40
26959
8482365bb2ec Fix merging.
Paul Aurich <darkrain42@pidgin.im>
parents: 26950
diff changeset
41 jabber_pep_register_handler(NS_AVATAR_1_1_METADATA,
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
42 update_buddy_metadata);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
43 }
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
44
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
45 static void
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
46 remove_avatar_0_12_nodes(JabberStream *js)
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
47 {
27025
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
48 #if 0
27035
2246f8a9d680 The correct fix here is to place non-code in a comment (yes, even if it's
Elliott Sales de Andrade <qulogic@pidgin.im>
parents: 27034
diff changeset
49 /* See note below for why this is #if 0'd */
27025
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
50
27017
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
51 /* Publish an empty avatar according to the XEP-0084 v0.12 semantics */
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
52 xmlnode *publish, *item, *metadata;
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
53 /* publish the metadata */
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
54 publish = xmlnode_new("publish");
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
55 xmlnode_set_attrib(publish, "node", NS_AVATAR_0_12_METADATA);
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
56
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
57 item = xmlnode_new_child(publish, "item");
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
58 xmlnode_set_attrib(item, "id", "stop");
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
59
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
60 metadata = xmlnode_new_child(item, "metadata");
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
61 xmlnode_set_namespace(metadata, NS_AVATAR_0_12_METADATA);
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
62
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
63 xmlnode_new_child(metadata, "stop");
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
64
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
65 /* publish */
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
66 jabber_pep_publish(js, publish);
27025
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
67 #endif
27017
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
68
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
69 /*
27025
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
70 * This causes ejabberd 2.0.0 to kill the connection unceremoniously.
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
71 * See https://support.process-one.net/browse/EJAB-623. When adiumx.com
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
72 * was upgraded, the issue went away.
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
73 *
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
74 * I think it makes a lot of sense to not have an avatar at the old
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
75 * node instead of having something interpreted as "no avatar". When
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
76 * a contact with an older client logs in, in the latter situation,
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
77 * there's a race between interpreting the <presence/> vcard-temp:x:update
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
78 * avatar (non-empty) and the XEP-0084 v0.12 avatar (empty, so show no
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
79 * avatar for the buddy) which leads to unhappy and confused users.
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
80 *
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
81 * A deluge of frustrating "Read error" bug reports may change my mind
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
82 * about this.
cda390e09aaf Revert the partial hack for ejabberd 2.0.0. See comment in code.
Paul Aurich <darkrain42@pidgin.im>
parents: 27017
diff changeset
83 * --darkrain42
27017
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
84 */
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
85 jabber_pep_delete_node(js, NS_AVATAR_0_12_METADATA);
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
86 jabber_pep_delete_node(js, NS_AVATAR_0_12_DATA);
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
87 }
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
88
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
89 void jabber_avatar_set(JabberStream *js, PurpleStoredImage *img)
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
90 {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
91 xmlnode *publish, *metadata, *item;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
92
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
93 if (!js->pep)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
94 return;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
95
27017
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
96 /* Hmmm, not sure if this is worth the traffic, but meh */
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
97 remove_avatar_0_12_nodes(js);
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
98
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
99 if (!img) {
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
100 publish = xmlnode_new("publish");
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
101 xmlnode_set_attrib(publish, "node", NS_AVATAR_1_1_METADATA);
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
102
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
103 item = xmlnode_new_child(publish, "item");
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
104 metadata = xmlnode_new_child(item, "metadata");
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
105 xmlnode_set_namespace(metadata, NS_AVATAR_1_1_METADATA);
25479
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
106
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
107 /* publish */
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
108 jabber_pep_publish(js, publish);
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
109 } else {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
110 /*
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
111 * TODO: This is pretty gross. The Jabber PRPL really shouldn't
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
112 * do voodoo to try to determine the image type, height
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
113 * and width.
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
114 */
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
115 /* A PNG header, including the IHDR, but nothing else */
33766
49eb0136f2ec jabber: avoid possible heap overflow when a very small file has been chosen for
Daniel Atallah <datallah@pidgin.im>
parents: 29910
diff changeset
116 /* ATTN: this is in network byte order! */
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
117 const struct {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
118 guchar signature[8]; /* must be hex 89 50 4E 47 0D 0A 1A 0A */
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
119 struct {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
120 guint32 length; /* must be 0x0d */
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
121 guchar type[4]; /* must be 'I' 'H' 'D' 'R' */
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
122 guint32 width;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
123 guint32 height;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
124 guchar bitdepth;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
125 guchar colortype;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
126 guchar compression;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
127 guchar filter;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
128 guchar interlace;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
129 } ihdr;
33766
49eb0136f2ec jabber: avoid possible heap overflow when a very small file has been chosen for
Daniel Atallah <datallah@pidgin.im>
parents: 29910
diff changeset
130 } *png = NULL;
49eb0136f2ec jabber: avoid possible heap overflow when a very small file has been chosen for
Daniel Atallah <datallah@pidgin.im>
parents: 29910
diff changeset
131
49eb0136f2ec jabber: avoid possible heap overflow when a very small file has been chosen for
Daniel Atallah <datallah@pidgin.im>
parents: 29910
diff changeset
132 if (purple_imgstore_get_size(img) > sizeof(*png))
49eb0136f2ec jabber: avoid possible heap overflow when a very small file has been chosen for
Daniel Atallah <datallah@pidgin.im>
parents: 29910
diff changeset
133 png = purple_imgstore_get_data(img);
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
134
26947
176ef2bfef9f Use purple_strequal and g_str_equal
Paul Aurich <darkrain42@pidgin.im>
parents: 25811
diff changeset
135 /* check if the data is a valid png file (well, at least to some extent) */
33766
49eb0136f2ec jabber: avoid possible heap overflow when a very small file has been chosen for
Daniel Atallah <datallah@pidgin.im>
parents: 29910
diff changeset
136 if(png && png->signature[0] == 0x89 &&
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
137 png->signature[1] == 0x50 &&
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
138 png->signature[2] == 0x4e &&
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
139 png->signature[3] == 0x47 &&
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
140 png->signature[4] == 0x0d &&
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
141 png->signature[5] == 0x0a &&
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
142 png->signature[6] == 0x1a &&
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
143 png->signature[7] == 0x0a &&
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
144 ntohl(png->ihdr.length) == 0x0d &&
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
145 png->ihdr.type[0] == 'I' &&
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
146 png->ihdr.type[1] == 'H' &&
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
147 png->ihdr.type[2] == 'D' &&
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
148 png->ihdr.type[3] == 'R') {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
149 /* parse PNG header to get the size of the image (yes, this is required) */
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
150 guint32 width = ntohl(png->ihdr.width);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
151 guint32 height = ntohl(png->ihdr.height);
25480
6e01ab2bc943 Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents: 25479
diff changeset
152 xmlnode *data, *info;
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
153 char *lengthstring, *widthstring, *heightstring;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
154
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
155 /* compute the sha1 hash */
29910
f1c6e586e5c1 jabber: Validate the hash on incoming BoB objects (in case the CID is on the
Marcus Lundblad <malu@pidgin.im>
parents: 29782
diff changeset
156 char *hash = jabber_calculate_data_hash(purple_imgstore_get_data(img),
f1c6e586e5c1 jabber: Validate the hash on incoming BoB objects (in case the CID is on the
Marcus Lundblad <malu@pidgin.im>
parents: 29782
diff changeset
157 purple_imgstore_get_size(img),
f1c6e586e5c1 jabber: Validate the hash on incoming BoB objects (in case the CID is on the
Marcus Lundblad <malu@pidgin.im>
parents: 29782
diff changeset
158 "sha1");
25480
6e01ab2bc943 Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents: 25479
diff changeset
159 char *base64avatar = purple_base64_encode(purple_imgstore_get_data(img),
6e01ab2bc943 Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents: 25479
diff changeset
160 purple_imgstore_get_size(img));
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
161
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
162 publish = xmlnode_new("publish");
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
163 xmlnode_set_attrib(publish, "node", NS_AVATAR_1_1_DATA);
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
164
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
165 item = xmlnode_new_child(publish, "item");
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
166 xmlnode_set_attrib(item, "id", hash);
25479
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
167
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
168 data = xmlnode_new_child(item, "data");
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
169 xmlnode_set_namespace(data, NS_AVATAR_1_1_DATA);
25480
6e01ab2bc943 Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents: 25479
diff changeset
170
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
171 xmlnode_insert_data(data, base64avatar, -1);
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
172 /* publish the avatar itself */
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
173 jabber_pep_publish(js, publish);
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
174
25480
6e01ab2bc943 Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents: 25479
diff changeset
175 g_free(base64avatar);
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
176
25479
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
177 lengthstring = g_strdup_printf("%" G_GSIZE_FORMAT,
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
178 purple_imgstore_get_size(img));
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
179 widthstring = g_strdup_printf("%u", width);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
180 heightstring = g_strdup_printf("%u", height);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
181
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
182 /* publish the metadata */
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
183 publish = xmlnode_new("publish");
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
184 xmlnode_set_attrib(publish, "node", NS_AVATAR_1_1_METADATA);
25480
6e01ab2bc943 Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents: 25479
diff changeset
185
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
186 item = xmlnode_new_child(publish, "item");
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
187 xmlnode_set_attrib(item, "id", hash);
25480
6e01ab2bc943 Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents: 25479
diff changeset
188
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
189 metadata = xmlnode_new_child(item, "metadata");
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
190 xmlnode_set_namespace(metadata, NS_AVATAR_1_1_METADATA);
25480
6e01ab2bc943 Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents: 25479
diff changeset
191
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
192 info = xmlnode_new_child(metadata, "info");
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
193 xmlnode_set_attrib(info, "id", hash);
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
194 xmlnode_set_attrib(info, "type", "image/png");
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
195 xmlnode_set_attrib(info, "bytes", lengthstring);
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
196 xmlnode_set_attrib(info, "width", widthstring);
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
197 xmlnode_set_attrib(info, "height", heightstring);
25480
6e01ab2bc943 Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents: 25479
diff changeset
198
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
199 jabber_pep_publish(js, publish);
25480
6e01ab2bc943 Support both XEP-0084 namespaces for fetching our own avatars at login
Paul Aurich <darkrain42@pidgin.im>
parents: 25479
diff changeset
200
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
201 g_free(lengthstring);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
202 g_free(widthstring);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
203 g_free(heightstring);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
204 g_free(hash);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
205 } else {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
206 purple_debug_error("jabber", "Cannot set PEP avatar to non-PNG data\n");
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
207 }
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
208 }
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
209 }
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
210
25477
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
211 static void
27017
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
212 do_got_own_avatar_0_12_cb(JabberStream *js, const char *from, xmlnode *items)
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
213 {
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
214 if (items)
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
215 /* It wasn't an error (i.e. 'item-not-found') */
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
216 remove_avatar_0_12_nodes(js);
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
217 }
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
218
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
219 static void
25477
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
220 do_got_own_avatar_cb(JabberStream *js, const char *from, xmlnode *items)
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
221 {
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
222 xmlnode *item = NULL, *metadata = NULL, *info = NULL;
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
223 PurpleAccount *account = purple_connection_get_account(js->gc);
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
224 const char *server_hash = NULL;
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
225
27017
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
226 if (items && (item = xmlnode_get_child(items, "item")) &&
28048
cdc92f9d634a jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents: 27358
diff changeset
227 (metadata = xmlnode_get_child(item, "metadata")) &&
cdc92f9d634a jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents: 27358
diff changeset
228 (info = xmlnode_get_child(metadata, "info"))) {
25477
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
229 server_hash = xmlnode_get_attrib(info, "id");
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
230 }
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
231
28048
cdc92f9d634a jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents: 27358
diff changeset
232 /*
cdc92f9d634a jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents: 27358
diff changeset
233 * If we have an avatar and the server returned an error/malformed data,
cdc92f9d634a jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents: 27358
diff changeset
234 * push our avatar. If the server avatar doesn't match the local one, push
cdc92f9d634a jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents: 27358
diff changeset
235 * our avatar.
cdc92f9d634a jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents: 27358
diff changeset
236 */
28881
244fb329e5df jabber: Go back to not overwriting avatars if there is not one locally set.
Paul Aurich <darkrain42@pidgin.im>
parents: 28828
diff changeset
237 if ((!items || !metadata) ||
28048
cdc92f9d634a jabber: Fix setting the PEP node the first time.
Paul Aurich <darkrain42@pidgin.im>
parents: 27358
diff changeset
238 !purple_strequal(server_hash, js->initial_avatar_hash)) {
25477
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
239 PurpleStoredImage *img = purple_buddy_icons_find_account_icon(account);
26950
842628304397 Publish only 'new' xmlns of avatars and delete old.
Paul Aurich <darkrain42@pidgin.im>
parents: 26949
diff changeset
240 jabber_avatar_set(js, img);
25477
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
241 purple_imgstore_unref(img);
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
242 }
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
243 }
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
244
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
245 void jabber_avatar_fetch_mine(JabberStream *js)
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
246 {
28881
244fb329e5df jabber: Go back to not overwriting avatars if there is not one locally set.
Paul Aurich <darkrain42@pidgin.im>
parents: 28828
diff changeset
247 if (js->initial_avatar_hash) {
29782
ca4fedd9e890 jabber: Allow jabber_pep_request_item() to take NULL for JID; server's routing takes care of the rest
Paul Aurich <darkrain42@pidgin.im>
parents: 28881
diff changeset
248 jabber_pep_request_item(js, NULL, NS_AVATAR_0_12_METADATA, NULL,
28881
244fb329e5df jabber: Go back to not overwriting avatars if there is not one locally set.
Paul Aurich <darkrain42@pidgin.im>
parents: 28828
diff changeset
249 do_got_own_avatar_0_12_cb);
29782
ca4fedd9e890 jabber: Allow jabber_pep_request_item() to take NULL for JID; server's routing takes care of the rest
Paul Aurich <darkrain42@pidgin.im>
parents: 28881
diff changeset
250 jabber_pep_request_item(js, NULL, NS_AVATAR_1_1_METADATA, NULL,
28881
244fb329e5df jabber: Go back to not overwriting avatars if there is not one locally set.
Paul Aurich <darkrain42@pidgin.im>
parents: 28828
diff changeset
251 do_got_own_avatar_cb);
244fb329e5df jabber: Go back to not overwriting avatars if there is not one locally set.
Paul Aurich <darkrain42@pidgin.im>
parents: 28828
diff changeset
252 }
25477
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
253 }
ada9e5c4a10a Only (re)publish XMPP avatars at login if the server's avatar differs
Paul Aurich <darkrain42@pidgin.im>
parents: 25476
diff changeset
254
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
255 typedef struct _JabberBuddyAvatarUpdateURLInfo {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
256 JabberStream *js;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
257 char *from;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
258 char *id;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
259 } JabberBuddyAvatarUpdateURLInfo;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
260
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
261 static void
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
262 do_buddy_avatar_update_fromurl(PurpleUtilFetchUrlData *url_data,
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
263 gpointer user_data, const gchar *url_text,
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
264 gsize len, const gchar *error_message)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
265 {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
266 JabberBuddyAvatarUpdateURLInfo *info = user_data;
27358
a1e3486558a0 Need to make a copy of this because purple_util_fetch_url frees its.
Paul Aurich <darkrain42@pidgin.im>
parents: 27035
diff changeset
267 gpointer icon_data;
a1e3486558a0 Need to make a copy of this because purple_util_fetch_url frees its.
Paul Aurich <darkrain42@pidgin.im>
parents: 27035
diff changeset
268
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
269 if(!url_text) {
28828
7c1855f037e7 jabber: Convert to purple_debug_* convenience functions.
Paul Aurich <darkrain42@pidgin.im>
parents: 28321
diff changeset
270 purple_debug_error("jabber",
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
271 "do_buddy_avatar_update_fromurl got error \"%s\"",
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
272 error_message);
25476
d31fcddd65a4 Don't leak a JabberBuddyAvatarUpdateURLInfo struct in the error case
Paul Aurich <darkrain42@pidgin.im>
parents: 25475
diff changeset
273 goto out;
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
274 }
25481
46c34a4f99df Whitespace changes
Paul Aurich <darkrain42@pidgin.im>
parents: 25480
diff changeset
275
27358
a1e3486558a0 Need to make a copy of this because purple_util_fetch_url frees its.
Paul Aurich <darkrain42@pidgin.im>
parents: 27035
diff changeset
276 icon_data = g_memdup(url_text, len);
a1e3486558a0 Need to make a copy of this because purple_util_fetch_url frees its.
Paul Aurich <darkrain42@pidgin.im>
parents: 27035
diff changeset
277 purple_buddy_icons_set_for_user(purple_connection_get_account(info->js->gc), info->from, icon_data, len, info->id);
25476
d31fcddd65a4 Don't leak a JabberBuddyAvatarUpdateURLInfo struct in the error case
Paul Aurich <darkrain42@pidgin.im>
parents: 25475
diff changeset
278
d31fcddd65a4 Don't leak a JabberBuddyAvatarUpdateURLInfo struct in the error case
Paul Aurich <darkrain42@pidgin.im>
parents: 25475
diff changeset
279 out:
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
280 g_free(info->from);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
281 g_free(info->id);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
282 g_free(info);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
283 }
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
284
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
285 static void
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
286 do_buddy_avatar_update_data(JabberStream *js, const char *from, xmlnode *items)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
287 {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
288 xmlnode *item, *data;
27017
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
289 const char *checksum;
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
290 char *b64data;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
291 void *img;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
292 size_t size;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
293 if(!items)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
294 return;
25481
46c34a4f99df Whitespace changes
Paul Aurich <darkrain42@pidgin.im>
parents: 25480
diff changeset
295
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
296 item = xmlnode_get_child(items, "item");
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
297 if(!item)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
298 return;
25479
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
299
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
300 data = xmlnode_get_child(item, "data");
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
301 if(!data)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
302 return;
25479
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
303
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
304 checksum = xmlnode_get_attrib(item,"id");
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
305 if(!checksum)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
306 return;
25481
46c34a4f99df Whitespace changes
Paul Aurich <darkrain42@pidgin.im>
parents: 25480
diff changeset
307
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
308 b64data = xmlnode_get_data(data);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
309 if(!b64data)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
310 return;
25481
46c34a4f99df Whitespace changes
Paul Aurich <darkrain42@pidgin.im>
parents: 25480
diff changeset
311
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
312 img = purple_base64_decode(b64data, &size);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
313 if(!img) {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
314 g_free(b64data);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
315 return;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
316 }
25481
46c34a4f99df Whitespace changes
Paul Aurich <darkrain42@pidgin.im>
parents: 25480
diff changeset
317
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
318 purple_buddy_icons_set_for_user(purple_connection_get_account(js->gc), from, img, size, checksum);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
319 g_free(b64data);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
320 }
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
321
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
322 static void
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
323 update_buddy_metadata(JabberStream *js, const char *from, xmlnode *items)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
324 {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
325 PurpleBuddy *buddy = purple_find_buddy(purple_connection_get_account(js->gc), from);
27017
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
326 const char *checksum;
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
327 xmlnode *item, *metadata;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
328 if(!buddy)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
329 return;
25481
46c34a4f99df Whitespace changes
Paul Aurich <darkrain42@pidgin.im>
parents: 25480
diff changeset
330
25809
f0d37a73663a I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents: 25543
diff changeset
331 if (!items)
f0d37a73663a I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents: 25543
diff changeset
332 return;
f0d37a73663a I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents: 25543
diff changeset
333
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
334 item = xmlnode_get_child(items,"item");
25809
f0d37a73663a I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents: 25543
diff changeset
335 if (!item)
f0d37a73663a I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents: 25543
diff changeset
336 return;
f0d37a73663a I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents: 25543
diff changeset
337
25479
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
338 metadata = xmlnode_get_child(item, "metadata");
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
339 if(!metadata)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
340 return;
25479
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
341
25809
f0d37a73663a I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents: 25543
diff changeset
342 checksum = purple_buddy_icons_get_checksum_for_user(buddy);
f0d37a73663a I had the distinct pleasure of throwing invalid garbage at this code; it
Paul Aurich <darkrain42@pidgin.im>
parents: 25543
diff changeset
343
27017
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
344 /* <stop/> was the pre-v1.1 method of publishing an empty avatar */
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
345 if(xmlnode_get_child(metadata, "stop")) {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
346 purple_buddy_icons_set_for_user(purple_connection_get_account(js->gc), from, NULL, 0, NULL);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
347 } else {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
348 xmlnode *info, *goodinfo = NULL;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
349 gboolean has_children = FALSE;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
350
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
351 /* iterate over all info nodes to get one we can use */
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
352 for(info = metadata->child; info; info = info->next) {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
353 if(info->type == XMLNODE_TYPE_TAG)
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
354 has_children = TRUE;
38258
9a6551eba09c Replace !strcmp() with purple_strequal()
qarkai <qarkai@gmail.com>
parents: 33766
diff changeset
355 if(info->type == XMLNODE_TYPE_TAG && purple_strequal(info->name,"info")) {
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
356 const char *type = xmlnode_get_attrib(info,"type");
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
357 const char *id = xmlnode_get_attrib(info,"id");
25481
46c34a4f99df Whitespace changes
Paul Aurich <darkrain42@pidgin.im>
parents: 25480
diff changeset
358
38258
9a6551eba09c Replace !strcmp() with purple_strequal()
qarkai <qarkai@gmail.com>
parents: 33766
diff changeset
359 if(checksum && id && purple_strequal(id, checksum)) {
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
360 /* we already have that avatar, so we don't have to do anything */
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
361 goodinfo = NULL;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
362 break;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
363 }
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
364 /* We'll only pick the png one for now. It's a very nice image format anyways. */
38258
9a6551eba09c Replace !strcmp() with purple_strequal()
qarkai <qarkai@gmail.com>
parents: 33766
diff changeset
365 if(type && id && !goodinfo && purple_strequal(type, "image/png"))
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
366 goodinfo = info;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
367 }
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
368 }
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
369 if(has_children == FALSE) {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
370 purple_buddy_icons_set_for_user(purple_connection_get_account(js->gc), from, NULL, 0, NULL);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
371 } else if(goodinfo) {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
372 const char *url = xmlnode_get_attrib(goodinfo, "url");
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
373 const char *id = xmlnode_get_attrib(goodinfo,"id");
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
374
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
375 /* the avatar might either be stored in a pep node, or on a HTTP(S) URL */
25479
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
376 if(!url) {
27017
02ddb8e8c3d2 Drop support for XEP-0084 v0.12 and publish a <stop/> instead of deleting the node.
Paul Aurich <darkrain42@pidgin.im>
parents: 26959
diff changeset
377 jabber_pep_request_item(js, from, NS_AVATAR_1_1_DATA, id,
25479
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
378 do_buddy_avatar_update_data);
416673fd83b4 Support XEP-0084 (User Avatar) v1.1 alongside v0.12
Paul Aurich <darkrain42@pidgin.im>
parents: 25477
diff changeset
379 } else {
25475
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
380 PurpleUtilFetchUrlData *url_data;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
381 JabberBuddyAvatarUpdateURLInfo *info = g_new0(JabberBuddyAvatarUpdateURLInfo, 1);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
382 info->js = js;
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
383
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
384 url_data = purple_util_fetch_url_len(url, TRUE, NULL, TRUE,
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
385 MAX_HTTP_BUDDYICON_BYTES,
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
386 do_buddy_avatar_update_fromurl, info);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
387 if (url_data) {
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
388 info->from = g_strdup(from);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
389 info->id = g_strdup(id);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
390 js->url_datas = g_slist_prepend(js->url_datas, url_data);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
391 } else
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
392 g_free(info);
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
393
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
394 }
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
395 }
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
396 }
ad7612a5d678 Migrate the XMPP User Avatar (XEP-0084) code to its own file
Paul Aurich <darkrain42@pidgin.im>
parents:
diff changeset
397 }

mercurial