From 97c166f97eaa511698b74ff2456a5e2b5b44835f Mon Sep 17 00:00:00 2001 From: Srinivasa Ragavan Date: Mon, 9 Apr 2007 06:26:03 +0000 Subject: Fix for bug #356523 from Martin Olsson svn path=/trunk/; revision=33405 --- composer/ChangeLog | 7 +++++++ composer/e-msg-composer.c | 17 +++++++++++------ 2 files changed, 18 insertions(+), 6 deletions(-) diff --git a/composer/ChangeLog b/composer/ChangeLog index c835c61927..5fe718b2bf 100644 --- a/composer/ChangeLog +++ b/composer/ChangeLog @@ -1,3 +1,10 @@ +2007-04-09 Srinivasa Ragavan + + ** Fix for bug #356523 from Martin Olsson + + * e-msg-composer.c: (e_msg_composer_add_inline_image_from_file): Copies + the file uri and decodes it before trying to attach the image. + 2007-04-05 Sankar P * Patch committed on behalf of Paul Iadonisi diff --git a/composer/e-msg-composer.c b/composer/e-msg-composer.c index 6f06bd7e74..35120bdf05 100644 --- a/composer/e-msg-composer.c +++ b/composer/e-msg-composer.c @@ -5165,18 +5165,21 @@ CamelMimePart * e_msg_composer_add_inline_image_from_file (EMsgComposer *composer, const char *file_name) { - char *mime_type, *cid, *url, *name; + char *mime_type, *cid, *url, *name, *dec_file_name; CamelStream *stream; CamelDataWrapper *wrapper; CamelMimePart *part; struct stat statbuf; EMsgComposerPrivate *p = composer->priv; + + dec_file_name = g_strdup(file_name); + camel_url_decode(dec_file_name); /* check for regular file */ - if (g_stat (file_name, &statbuf) < 0 || !S_ISREG (statbuf.st_mode)) + if (g_stat (dec_file_name, &statbuf) < 0 || !S_ISREG (statbuf.st_mode)) return NULL; - stream = camel_stream_fs_new_with_name (file_name, O_RDONLY, 0); + stream = camel_stream_fs_new_with_name (dec_file_name, O_RDONLY, 0); if (!stream) return NULL; @@ -5184,7 +5187,7 @@ e_msg_composer_add_inline_image_from_file (EMsgComposer *composer, camel_data_wrapper_construct_from_stream (wrapper, stream); camel_object_unref (CAMEL_OBJECT (stream)); - mime_type = e_msg_composer_guess_mime_type (file_name); + mime_type = e_msg_composer_guess_mime_type (dec_file_name); camel_data_wrapper_set_mime_type (wrapper, mime_type ? mime_type : "application/octet-stream"); g_free (mime_type); @@ -5194,17 +5197,19 @@ e_msg_composer_add_inline_image_from_file (EMsgComposer *composer, cid = camel_header_msgid_generate (); camel_mime_part_set_content_id (part, cid); - name = g_path_get_basename(file_name); + name = g_path_get_basename(dec_file_name); camel_mime_part_set_filename (part, name); g_free(name); camel_mime_part_set_encoding (part, CAMEL_TRANSFER_ENCODING_BASE64); - url = g_strdup_printf ("file:%s", file_name); + url = g_strdup_printf ("file:%s", dec_file_name); g_hash_table_insert (p->inline_images_by_url, url, part); url = g_strdup_printf ("cid:%s", cid); g_hash_table_insert (p->inline_images, url, part); g_free (cid); + + g_free(dec_file_name); return part; } -- cgit v1.2.3