Mercurial > pidgin
comparison libpurple/buddyicon.c @ 16546:317f3ae0ef93
Fix a bug with the bugfix for empty icons. The logic was backwards here.
This caused all buddy icons to be lost on migration and as an added bonus,
it failed to solve the empty icon NULL extension bug!
author | Richard Laager <rlaager@wiktel.com> |
---|---|
date | Sun, 29 Apr 2007 01:08:11 +0000 |
parents | 175eadaead53 |
children | 8d8168fada21 |
comparison
equal
deleted
inserted
replaced
16543:175eadaead53 | 16546:317f3ae0ef93 |
---|---|
888 size_t icon_len; | 888 size_t icon_len; |
889 FILE *file; | 889 FILE *file; |
890 char *new_filename; | 890 char *new_filename; |
891 | 891 |
892 if (!read_icon_file(path, &icon_data, &icon_len) || | 892 if (!read_icon_file(path, &icon_data, &icon_len) || |
893 icon_data == NULL || icon_len > 0) | 893 icon_data == NULL || icon_len <= 0) |
894 { | 894 { |
895 delete_buddy_icon_settings(node, setting_name); | 895 delete_buddy_icon_settings(node, setting_name); |
896 g_free(path); | 896 g_free(path); |
897 return; | 897 return; |
898 } | 898 } |