# HG changeset patch # User Daniel Atallah # Date 1192296220 0 # Node ID aa420b502a6b675e0ee155eae43d1e4d1bb13b13 # Parent e1f1f3c1c19cb2d31caedb2714d3de20d0604cb6 nosnilmot points out that this is a better way to ensure the size is valid. diff -r e1f1f3c1c19c -r aa420b502a6b libpurple/util.c --- 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 */