From ad208810f0313f111529e1605f72adf32b68b57f Mon Sep 17 00:00:00 2001 From: John Ralls Date: Sun, 11 Dec 2016 09:58:54 -0800 Subject: [PATCH] Bug 773808 - Export Report Crash It's possible for the GtkFileChooserDialog to return a NULL filename, so defend against that. --- src/gnome-utils/gnc-file.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/gnome-utils/gnc-file.c b/src/gnome-utils/gnc-file.c index ed01fa982a..2e61c9af7e 100644 --- a/src/gnome-utils/gnc-file.c +++ b/src/gnome-utils/gnc-file.c @@ -174,12 +174,15 @@ gnc_file_dialog (const char * title, { /* look for constructs like postgres://foo */ internal_name = gtk_file_chooser_get_uri(GTK_FILE_CHOOSER (file_box)); - if (strstr (internal_name, "file://") == internal_name) + if (internal_name != NULL) { - /* nope, a local file name */ - internal_name = gtk_file_chooser_get_filename(GTK_FILE_CHOOSER (file_box)); + if (strstr (internal_name, "file://") == internal_name) + { + /* nope, a local file name */ + internal_name = gtk_file_chooser_get_filename(GTK_FILE_CHOOSER (file_box)); + } + file_name = g_strdup(internal_name); } - file_name = g_strdup(internal_name); } gtk_widget_destroy(GTK_WIDGET(file_box)); LEAVE("%s", file_name ? file_name : "(null)");