From f81ae069e4ef8a49f826c458c7b80996cec2036a Mon Sep 17 00:00:00 2001 From: NotZed Date: Mon, 28 Feb 2000 23:26:13 +0000 Subject: Fix a bunch of serious small bugs. 2000-02-28 NotZed * camel-mime-part.c (_parse_header_pair): Dont free this either. * camel-medium.c (_remove_header): Ugh, dont free the header before we actually remove it. (_add_header): Ugh, dont free hashtable entries which may be duplicated (hash_insert _will_ reference that memory). * string-utils.c (string_trim): Trimming a 0-length string is not an error. * camel-mime-message.c (_parse_header_pair): Fixed very broken memory handling of header_name/value. * providers/mbox/camel-mbox-utils.c (camel_mbox_write_xev): Initialise end_of_last_message always. (camel_mbox_copy_file_chunk): Stop trying to read if we run out of data, rather than looping forever. * providers/mbox/camel-mbox-folder.c (_init): Set search cap on. (_open): Call parent class to perform open. Remove folder-open check to parent instead. (_create): open takes a creation mask, dont use umask to try and set the open mode. (_delete): Dont bother checking folder==NULL, its already been checked on the external interface (changed to an assertion, this would have to be a camel bug). (_delete_messages): Likewise. (_create): Ditto. (_init): Dont go and clear all the paths and shit that the parent open just setup for us. (_delete_messages): Get rid of more umask stuff. (_append_message): Make sure we pass file mode to open with create. (_append_message): Cleaned up some indenting to make it readable. svn path=/trunk/; revision=1985 --- camel/ChangeLog | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) (limited to 'camel/ChangeLog') diff --git a/camel/ChangeLog b/camel/ChangeLog index 37c2dbda96..7a1e304924 100644 --- a/camel/ChangeLog +++ b/camel/ChangeLog @@ -1,5 +1,23 @@ 2000-02-28 NotZed + * camel-mime-part.c (_parse_header_pair): Dont free this either. + + * camel-medium.c (_remove_header): Ugh, dont free the header + before we actually remove it. + (_add_header): Ugh, dont free hashtable entries which may be + duplicated (hash_insert _will_ reference that memory). + + * string-utils.c (string_trim): Trimming a 0-length string is not + an error. + + * camel-mime-message.c (_parse_header_pair): Fixed very broken + memory handling of header_name/value. + + * providers/mbox/camel-mbox-utils.c (camel_mbox_write_xev): + Initialise end_of_last_message always. + (camel_mbox_copy_file_chunk): Stop trying to read if we run out of + data, rather than looping forever. + * camel-folder.c (camel_folder_search_by_expression): No, its not a fatal error to search on a non-searchable folder, you just dont get any matches. @@ -9,6 +27,18 @@ * providers/mbox/camel-mbox-folder.c (_init): Set search cap on. (_open): Call parent class to perform open. Remove folder-open check to parent instead. + (_create): open takes a creation mask, dont use umask to try and + set the open mode. + (_delete): Dont bother checking folder==NULL, its already been + checked on the external interface (changed to an assertion, this + would have to be a camel bug). + (_delete_messages): Likewise. + (_create): Ditto. + (_init): Dont go and clear all the paths and shit that the parent + open just setup for us. + (_delete_messages): Get rid of more umask stuff. + (_append_message): Make sure we pass file mode to open with create. + (_append_message): Cleaned up some indenting to make it readable. * camel-stream-b64.c (my_read_encode): Fixed a typo. -- cgit v1.2.3