Mercurial > pidgin
changeset 20917:aa420b502a6b
nosnilmot points out that this is a better way to ensure the size is valid.
| author | Daniel Atallah <daniel.atallah@gmail.com> |
|---|---|
| date | Sat, 13 Oct 2007 17:23:40 +0000 |
| parents | e1f1f3c1c19c |
| children | 8fa6ff212db7 |
| files | libpurple/util.c |
| diffstat | 1 files changed, 3 insertions(+), 1 deletions(-) [+] |
line wrap: on
line diff
--- a/libpurple/util.c Sat Oct 13 17:20:19 2007 +0000 +++ b/libpurple/util.c Sat Oct 13 17:23:40 2007 +0000 @@ -2565,6 +2565,8 @@ purple_debug_info("util", "Writing file %s\n", filename_full); + g_return_val_if_fail((size >= -1), FALSE); + filename_temp = g_strdup_printf("%s.save", filename_full); /* Remove an old temporary file, if one exists */ @@ -2590,7 +2592,7 @@ } /* Write to file */ - real_size = (size == -1) ? strlen(data) : size; + real_size = (size == -1) ? strlen(data) : (size_t) size; byteswritten = fwrite(data, 1, real_size, file); /* Close file */