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
This commit is contained in:
Kjartan Maraas
2005-07-13 10:37:16 +00:00
committed by Kjartan Maraas
parent 4afc6e2f2a
commit bb714f46f7
3 changed files with 12 additions and 2 deletions

View File

@ -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

View File

@ -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;

View File

@ -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);