From 80c720a5696e701bff16798510e7422316711658 Mon Sep 17 00:00:00 2001 From: Jeffrey Stedfast Date: Tue, 5 Dec 2000 23:44:15 +0000 Subject: Use a charset filter to make sure the data is written out in the charset 2000-12-05 Jeffrey Stedfast * mail-display.c (write_data_to_file): Use a charset filter to make sure the data is written out in the charset it was meant to be in instead of UTF-8. * mail-format.c (mail_format_raw_message): Don't use the raw message body as the format argument, use "%s" instead. If the raw message contains %'s then it will segfault otherwise. svn path=/trunk/; revision=6803 --- mail/ChangeLog | 4 ++++ mail/mail-display.c | 44 ++++++++++++++++++++++++++++++++++++-------- 2 files changed, 40 insertions(+), 8 deletions(-) (limited to 'mail') diff --git a/mail/ChangeLog b/mail/ChangeLog index 5aebe431af..8e309f3f87 100644 --- a/mail/ChangeLog +++ b/mail/ChangeLog @@ -1,5 +1,9 @@ 2000-12-05 Jeffrey Stedfast + * mail-display.c (write_data_to_file): Use a charset filter to + make sure the data is written out in the charset it was meant to + be in instead of UTF-8. + * mail-format.c (mail_format_raw_message): Don't use the raw message body as the format argument, use "%s" instead. If the raw message contains %'s then it will segfault otherwise. diff --git a/mail/mail-display.c b/mail/mail-display.c index 0d1f205559..323b62b6a8 100644 --- a/mail/mail-display.c +++ b/mail/mail-display.c @@ -53,13 +53,17 @@ struct _PixbufLoader { static gboolean write_data_to_file (CamelMimePart *part, const char *name, gboolean unique) { - CamelDataWrapper *data; + CamelMimeFilterCharset *charsetfilter; + GMimeContentField *content_type; + CamelStreamFilter *filtered_stream; CamelStream *stream_fs; - int fd; - + CamelDataWrapper *data; + const char *charset; + int fd, chid; + g_return_val_if_fail (CAMEL_IS_MIME_PART (part), FALSE); data = camel_medium_get_content_object (CAMEL_MEDIUM (part)); - + fd = open (name, O_WRONLY | O_CREAT | O_EXCL, S_IRUSR | S_IWUSR); if (fd == -1 && errno == EEXIST && !unique) { GtkWidget *dlg; @@ -89,20 +93,44 @@ write_data_to_file (CamelMimePart *part, const char *name, gboolean unique) g_free (msg); return FALSE; } - + + content_type = camel_mime_part_get_content_type (part); + charset = gmime_content_field_get_parameter (content_type, "charset"); + if (!charset) + charset = "us-ascii"; + + fprintf (stderr, "mime part charset = %s\n", charset); + + charsetfilter = camel_mime_filter_charset_new_convert ("utf-8", charset); + stream_fs = camel_stream_fs_new_with_fd (fd); - if (camel_data_wrapper_write_to_stream (data, stream_fs) == -1 - || camel_stream_flush (stream_fs) == -1) { + + filtered_stream = camel_stream_filter_new_with_stream (stream_fs); + chid = camel_stream_filter_add (filtered_stream, CAMEL_MIME_FILTER (charsetfilter)); + + if (camel_data_wrapper_write_to_stream (data, CAMEL_STREAM (filtered_stream)) == -1 + || camel_stream_flush (CAMEL_STREAM (filtered_stream)) == -1) { char *msg; - + msg = g_strdup_printf (_("Could not write data: %s"), strerror (errno)); gnome_error_dialog (msg); g_free (msg); + + camel_stream_filter_remove (filtered_stream, chid); + camel_object_unref (CAMEL_OBJECT (charsetfilter)); + camel_object_unref (CAMEL_OBJECT (filtered_stream)); camel_object_unref (CAMEL_OBJECT (stream_fs)); + return FALSE; } + + camel_stream_filter_remove (filtered_stream, chid); + camel_object_unref (CAMEL_OBJECT (charsetfilter)); + camel_stream_flush (CAMEL_STREAM (filtered_stream)); + camel_object_unref (CAMEL_OBJECT (filtered_stream)); camel_object_unref (CAMEL_OBJECT (stream_fs)); + return TRUE; } -- cgit v1.2.3