Mercurial > pidgin.yaz
comparison pidgin/gtkaccount.c @ 16597:bc03699814a1
Part of the patch in ticket #383:
- Fix the argument type on set_dialog_icon's data
- Respect the return value of purple_imgstore_unref() to avoid a crash
committer: Richard Laager <rlaager@wiktel.com>
author | Sadrul Habib Chowdhury <imadil@gmail.com> |
---|---|
date | Sat, 28 Apr 2007 18:02:26 +0000 |
parents | 5e47b3427b28 |
children | fbf111436896 |
comparison
equal
deleted
inserted
replaced
16596:63a1de71e389 | 16597:bc03699814a1 |
---|---|
192 | 192 |
193 return hbox; | 193 return hbox; |
194 } | 194 } |
195 | 195 |
196 static void | 196 static void |
197 set_dialog_icon(AccountPrefsDialog *dialog, gpointer *data, size_t len, gchar *new_icon_path) | 197 set_dialog_icon(AccountPrefsDialog *dialog, gpointer data, size_t len, gchar *new_icon_path) |
198 { | 198 { |
199 GdkPixbuf *pixbuf = NULL; | 199 GdkPixbuf *pixbuf = NULL; |
200 | 200 |
201 purple_imgstore_unref(dialog->icon_img); | 201 dialog->icon_img = purple_imgstore_unref(dialog->icon_img); |
202 if (data != NULL) | 202 if (data != NULL) |
203 { | 203 { |
204 if (len > 0) | 204 if (len > 0) |
205 dialog->icon_img = purple_imgstore_add(data, len, new_icon_path); | 205 dialog->icon_img = purple_imgstore_add(data, len, new_icon_path); |
206 else | 206 else |