aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKjartan Maraas <kmaraas@gnome.org>2005-07-13 18:37:16 +0800
committerKjartan Maraas <kmaraas@src.gnome.org>2005-07-13 18:37:16 +0800
commitbb714f46f72ea07579da4d988a08fa33eb8edb1e (patch)
treee3a966e00c8d09275a994a9b7b75a758f3cdc7af
parent4afc6e2f2a877441d36c670ef58fe31360a83e63 (diff)
downloadgsoc2013-evolution-bb714f46f72ea07579da4d988a08fa33eb8edb1e.tar.gz
gsoc2013-evolution-bb714f46f72ea07579da4d988a08fa33eb8edb1e.tar.zst
gsoc2013-evolution-bb714f46f72ea07579da4d988a08fa33eb8edb1e.zip
Don't leak the file name here.
2005-07-13 Kjartan Maraas <kmaraas@gnome.org> * e-msg-composer-select-file.c: (select_attach_response): Don't leak the file name here. svn path=/trunk/; revision=29757
-rw-r--r--composer/ChangeLog7
-rw-r--r--composer/e-msg-composer-select-file.c5
-rw-r--r--composer/e-msg-composer.c2
3 files changed, 12 insertions, 2 deletions
diff --git a/composer/ChangeLog b/composer/ChangeLog
index 432a9e25aa..dc9ee8f53d 100644
--- a/composer/ChangeLog
+++ b/composer/ChangeLog
@@ -1,3 +1,9 @@
+2005-07-13 Kjartan Maraas <kmaraas@gnome.org>
+
+ * e-msg-composer-select-file.c: (select_attach_response): Don't
+ leak the file name here.
+ * e-msg-composer.c: (build_message): Don't allocate charset twice.
+
2005-07-13 Tor Lillqvist <tml@novell.com>
* Makefile.am (INCLUDES): There are no camel or
@@ -15,6 +21,7 @@
Changed the code to use e_attachment_* from em_attachment*
* e-msg-composer.h: Added removed old header includes
+
2005-07-10 Shreyas Srinivasan <sshreyas@novell.com>
* e-msg-composer-hdrs.c: Take into account store
diff --git a/composer/e-msg-composer-select-file.c b/composer/e-msg-composer-select-file.c
index 7329d570c5..7a4b917f24 100644
--- a/composer/e-msg-composer-select-file.c
+++ b/composer/e-msg-composer-select-file.c
@@ -194,8 +194,11 @@ select_attach_response(GtkWidget *selector, guint response, struct _EMsgComposer
char *path;
#ifdef USE_GTKFILECHOOSER
+ char *filename;
names = gtk_file_chooser_get_filenames (GTK_FILE_CHOOSER (selector));
- path = g_path_get_dirname (gtk_file_chooser_get_filename (GTK_FILE_CHOOSER (selector)));
+ filename = gtk_file_chooser_get_filename (GTK_FILE_CHOOSER (selector));
+ path = g_path_get_dirname (filename);
+ g_free (filename);
#else
char **files;
int i;
diff --git a/composer/e-msg-composer.c b/composer/e-msg-composer.c
index 45bc84d4bc..e68384c3aa 100644
--- a/composer/e-msg-composer.c
+++ b/composer/e-msg-composer.c
@@ -492,7 +492,7 @@ build_message (EMsgComposer *composer, gboolean save_html_object_data)
}
/* FIXME: we may want to do better than this... */
- charset = best_charset (data, composer->charset, &plain_encoding);
+
type = camel_content_type_new ("text", "plain");
if ((charset = best_charset (data, composer->charset, &plain_encoding))) {
camel_content_type_set_param (type, "charset", charset);