From 02e6fcebcf3ab06076edcbae8f8da0777f5e0b9f Mon Sep 17 00:00:00 2001 From: Srinivasa Ragavan Date: Tue, 12 Jul 2005 05:53:08 +0000 Subject: Fixed some compiler warning and checked some negative cases 2005-07-12 Srinivasa Ragavan * gui/dialogs/comp-editor.c: (get_attachment_list) (attachment_guess_mime_type) (comp_editor_close) (comp_editor_get_mime_attach_list) Fixed some compiler warning and checked some negative cases svn path=/trunk/; revision=29733 --- calendar/ChangeLog | 7 +++++++ calendar/gui/dialogs/comp-editor.c | 16 +++++++++++----- 2 files changed, 18 insertions(+), 5 deletions(-) diff --git a/calendar/ChangeLog b/calendar/ChangeLog index e0a93ee5a9..c80ee17c60 100644 --- a/calendar/ChangeLog +++ b/calendar/ChangeLog @@ -1,4 +1,11 @@ +2005-07-12 Srinivasa Ragavan + + * gui/dialogs/comp-editor.c: (get_attachment_list) (attachment_guess_mime_type) + (comp_editor_close) (comp_editor_get_mime_attach_list) Fixed some compiler + warning and checked some negative cases + 2005-07-11 Srinivasa Ragavan + * gui/dialogs/comp-editor.c:(get_attachment_list) (attachment_bar_changed_cb) (attachment_bar_icon_clicked_cb) (cab_popups_free) (setup_widgets) (set_attachment_list) (comp_editor_get_mime_attach_list) diff --git a/calendar/gui/dialogs/comp-editor.c b/calendar/gui/dialogs/comp-editor.c index c5e0c1286c..1cbfb09257 100644 --- a/calendar/gui/dialogs/comp-editor.c +++ b/calendar/gui/dialogs/comp-editor.c @@ -628,12 +628,19 @@ get_attachment_list (CompEditor *editor) if (!stream) { /* TODO handle error conditions */ g_message ("DEBUG: could not open the file to write\n"); + g_free (attach_file_url); continue; } - camel_data_wrapper_decode_to_stream (wrapper, (CamelStream *) stream); + if (camel_data_wrapper_decode_to_stream (wrapper, (CamelStream *) stream) == -1) { + g_free (attach_file_url); + camel_stream_close (stream); + camel_object_unref (stream); + g_message ("DEBUG: could not write to file\n"); + } + camel_stream_close (stream); - g_object_unref (stream); + camel_object_unref (stream); list = g_slist_append (list, g_strdup (attach_file_url)); g_free (attach_file_url); @@ -1992,7 +1999,7 @@ attachment_guess_mime_type (const char *file_name) static void set_attachment_list (CompEditor *editor, GSList *attach_list) { - GSList *parts = NULL, *attachment_list = NULL, *p = NULL; + GSList *p = NULL; const char *comp_uid= NULL; const char *local_store = e_cal_get_local_attachment_store (editor->priv->client); @@ -2391,7 +2398,6 @@ comp_editor_close (CompEditor *editor) GSList * comp_editor_get_mime_attach_list (CompEditor *editor) { - GSList *mime_attach_list; struct CalMimeAttach *cal_mime_attach; GSList *attach_list = NULL, *l, *parts; @@ -2423,7 +2429,7 @@ comp_editor_get_mime_attach_list (CompEditor *editor) attach_list = g_slist_append (attach_list, cal_mime_attach); - g_object_unref (mstream); + camel_object_unref (mstream); } -- cgit v1.2.3