diff options
Diffstat (limited to 'calendar')
| -rw-r--r-- | calendar/ChangeLog | 7 | ||||
| -rw-r--r-- | calendar/gui/dialogs/comp-editor.c | 10 |
2 files changed, 9 insertions, 8 deletions
diff --git a/calendar/ChangeLog b/calendar/ChangeLog index 9c004bf36b..e57cd09a73 100644 --- a/calendar/ChangeLog +++ b/calendar/ChangeLog @@ -1,3 +1,10 @@ +2008-02-20 Milan Crha <mcrha@redhat.com> + + ** Part of fix for bug #515744 + + * gui/dialogs/comp-editor.c: (drop_action): + Fixing previous fix - do not double free. + 2008-02-18 Milan Crha <mcrha@redhat.com> ** Part of fix for bug #515744 diff --git a/calendar/gui/dialogs/comp-editor.c b/calendar/gui/dialogs/comp-editor.c index 5a8be391ee..17b24091d8 100644 --- a/calendar/gui/dialogs/comp-editor.c +++ b/calendar/gui/dialogs/comp-editor.c @@ -279,21 +279,16 @@ drop_action(CompEditor *editor, GdkDragContext *context, guint32 action, GtkSele for (i = 0; urls[i] != NULL; i++) { str = g_strstrip (urls[i]); - if (urls[i][0] == '#') { - g_free(str); + if (urls[i][0] == '#') continue; - } if (!g_ascii_strncasecmp (str, "mailto:", 7)) { /* TODO does not handle mailto now */ - g_free (str); } else { url = camel_url_new (str, NULL); - if (url == NULL) { - g_free (str); + if (url == NULL) continue; - } if (!g_ascii_strcasecmp (url->protocol, "file")) e_attachment_bar_attach @@ -306,7 +301,6 @@ drop_action(CompEditor *editor, GdkDragContext *context, guint32 action, GtkSele str, "attachment"); camel_url_free (url); - g_free (str); } } |