Mercurial > pidgin.yaz
annotate finch/gntft.h @ 22401:d9105ead88dc
When purple_buddy_icons_set_account_icon() is called, it unrefs the old PurpleStoredImage and refs the new one. Previously, it notified the prpl of the change in the buddy icon before updating pointer_icon_cache, which meant that if the prpl then called purple_buddy_icons_find_account_icon() it would get the old PurpleStoredImage (which is at this point not only old but also a pointer to invalid memory if unref'ing it caused it to be destroyed). This happens in jabber_set_info() as of 2.4.0, causing a crash when setting no-buddy-icon for an account after it has previously had an icon. I think this also means that XMPP accounts in 2.4.0 will also always set serverside the *last* icon set, not the current one, when changing icons, but I didn't test that.
The solution is simple: Update pointer_icon_cache earlier, setting the new img if there is one or removing the account entirely from the hash table if there isn't one now. This fixes both problems described above, making purple_buddy_icons_find_account_icon() return the new, current icon (or NULL) at all times.
author | Evan Schoenberg <evan.s@dreskin.net> |
---|---|
date | Tue, 04 Mar 2008 00:11:22 +0000 |
parents | c38d72677c8a |
children | 1cdae196aac8 |
rev | line source |
---|---|
15818 | 1 /** |
2 * @file gntft.h GNT File Transfer UI | |
16194
0f0832c13fcb
Rename the Doxygen group from gntui to finch and define the finch group
Richard Laager <rlaager@wiktel.com>
parents:
15871
diff
changeset
|
3 * @ingroup finch |
20074
6bf32c9e15a7
remove gpl boilerplate from doxygen docs
Sean Egan <seanegan@gmail.com>
parents:
19680
diff
changeset
|
4 */ |
6bf32c9e15a7
remove gpl boilerplate from doxygen docs
Sean Egan <seanegan@gmail.com>
parents:
19680
diff
changeset
|
5 |
6bf32c9e15a7
remove gpl boilerplate from doxygen docs
Sean Egan <seanegan@gmail.com>
parents:
19680
diff
changeset
|
6 /* finch |
15818 | 7 * |
15871
66dff3dfdea6
Re-sed the copyright notices so they don't all talk about Purple.
Richard Laager <rlaager@wiktel.com>
parents:
15823
diff
changeset
|
8 * Finch is the legal property of its developers, whose names are too numerous |
15818 | 9 * to list here. Please refer to the COPYRIGHT file distributed with this |
10 * source distribution. | |
11 * | |
12 * This program is free software; you can redistribute it and/or modify | |
13 * it under the terms of the GNU General Public License as published by | |
14 * the Free Software Foundation; either version 2 of the License, or | |
15 * (at your option) any later version. | |
16 * | |
17 * This program is distributed in the hope that it will be useful, | |
18 * but WITHOUT ANY WARRANTY; without even the implied warranty of | |
19 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
20 * GNU General Public License for more details. | |
21 * | |
22 * You should have received a copy of the GNU General Public License | |
23 * along with this program; if not, write to the Free Software | |
19680
44b4e8bd759b
The FSF changed its address a while ago; our files were out of date.
John Bailey <rekkanoryo@rekkanoryo.org>
parents:
16201
diff
changeset
|
24 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02111-1301 USA |
15818 | 25 */ |
15823 | 26 #ifndef _FINCHFT_H_ |
27 #define _FINCHFT_H_ | |
15818 | 28 |
29 #include "ft.h" | |
30 | |
31 | |
32 /**************************************************************************/ | |
33 /** @name GNT File Transfer Dialog API */ | |
34 /**************************************************************************/ | |
35 /*@{*/ | |
36 | |
37 /** | |
38 * Creates a new file transfer dialog. | |
39 * | |
40 * @return The new dialog. | |
41 */ | |
42 void finch_xfer_dialog_new(void); | |
43 | |
44 /** | |
45 * Destroys a file transfer dialog. | |
46 */ | |
47 void finch_xfer_dialog_destroy(void); | |
48 | |
49 /** | |
50 * Displays the file transfer dialog given. | |
51 * If dialog is @c NULL, displays the default dialog, creating one if necessary | |
52 */ | |
53 void finch_xfer_dialog_show(void); | |
54 | |
55 /** | |
56 * Hides the file transfer dialog. | |
57 */ | |
22007
c38d72677c8a
Probe for -Wstrict-prototypes to get some more warnings. I then cleaned up
Richard Laager <rlaager@wiktel.com>
parents:
20074
diff
changeset
|
58 void finch_xfer_dialog_hide(void); |
15818 | 59 |
60 /** | |
61 * Adds a file transfer to the dialog. | |
62 * | |
63 * @param xfer The file transfer. | |
64 */ | |
15823 | 65 void finch_xfer_dialog_add_xfer(PurpleXfer *xfer); |
15818 | 66 |
67 /** | |
68 * Removes a file transfer from the dialog. | |
69 * | |
70 * @param xfer The file transfer. | |
71 */ | |
15823 | 72 void finch_xfer_dialog_remove_xfer(PurpleXfer *xfer); |
15818 | 73 |
74 /** | |
75 * Indicate in a file transfer dialog that a transfer was canceled. | |
76 * | |
77 * @param xfer The file transfer that was canceled. | |
78 */ | |
15823 | 79 void finch_xfer_dialog_cancel_xfer(PurpleXfer *xfer); |
15818 | 80 |
81 /** | |
82 * Updates the information for a transfer in the dialog. | |
83 * | |
84 * @param xfer The file transfer. | |
85 */ | |
15823 | 86 void finch_xfer_dialog_update_xfer(PurpleXfer *xfer); |
15818 | 87 |
88 /*@}*/ | |
89 | |
90 /**************************************************************************/ | |
91 /** @name GNT File Transfer API */ | |
92 /**************************************************************************/ | |
93 /*@{*/ | |
94 | |
95 /** | |
96 * Initializes the GNT file transfer system. | |
97 */ | |
98 void finch_xfers_init(void); | |
99 | |
100 /** | |
101 * Uninitializes the GNT file transfer system. | |
102 */ | |
103 void finch_xfers_uninit(void); | |
104 | |
105 /** | |
106 * Returns the UI operations structure for the GNT file transfer UI. | |
107 * | |
108 * @return The GNT file transfer UI operations structure. | |
109 */ | |
15823 | 110 PurpleXferUiOps *finch_xfers_get_ui_ops(void); |
15818 | 111 |
112 /*@}*/ | |
113 | |
15823 | 114 #endif /* _FINCHFT_H_ */ |