std::string(char*) crashes if it's given a nullptr.

5 years ago · f0ecc0e2eb
parent b6622a386b
commit f0ecc0e2eb
1 changed files with 5 additions and 2 deletions
--- a/gnucash/gnome-utils/dialog-options.cpp
+++ b/gnucash/gnome-utils/dialog-options.cpp
@ -2320,8 +2320,11 @@ public:
    {
        auto string = gtk_file_chooser_get_filename(GTK_FILE_CHOOSER(get_widget()));
        DEBUG("filename %s", string ? string : "(null)");
-        option.set_value(std::string{string});
-        g_free(string);
+        if (string)
+        {
+            option.set_value(std::string{string});
+            g_free(string);
+        }
    }
 };