Mercurial > pidgin
diff libpurple/buddyicon.c @ 24074:2ecdd0403dc1
Don't try to read data from a NULL filename in
purple_buddy_icons_node_set_custom_icon_from_file. Also document that
you can use NULL to unset the icon.
Fixes #6998.
| author | Elliott Sales de Andrade <qulogic@pidgin.im> |
|---|---|
| date | Sun, 07 Sep 2008 19:58:04 +0000 |
| parents | 4044655e5615 |
| children | 8282911d5e17 |
line wrap: on
line diff
--- a/libpurple/buddyicon.c Sun Sep 07 19:08:04 2008 +0000 +++ b/libpurple/buddyicon.c Sun Sep 07 19:58:04 2008 +0000 @@ -934,8 +934,8 @@ purple_buddy_icons_node_set_custom_icon_from_file(PurpleBlistNode *node, const gchar *filename) { - size_t len; - guchar *data; + size_t len = 0; + guchar *data = NULL; g_return_val_if_fail(node != NULL, NULL); @@ -945,8 +945,10 @@ return NULL; } - if (!read_icon_file(filename, &data, &len)) { - return NULL; + if (filename != NULL) { + if (!read_icon_file(filename, &data, &len)) { + return NULL; + } } return purple_buddy_icons_node_set_custom_icon(node, data, len);
