Fix default filename handling errors

The default filename handling is broken in two different ways:

 (a) if we start subsurface with a non-existing file, we warn about
the inability to read that file, and then we exit without setting the
default filename.

This is broken because it means that if the user (perhaps by mistake,
by pressing ^S) now saves the file, he will overwrite the default
filename, even though that was *not* the file we read, and *not* the
file that subsurface was started with.

So just set the default filename even for a failed file open.

The exact same logic is true of a failed parse of an XML file that we
successfully opened. We do *not* want to leave the old default
filename in place just because the XML parsing failed, and possibly
then overwriting some file that was never involved with that failure
in the first place. So just get rid of all the logic to push the
filename saving into the XML parsing layer, it has zero relevance at
that point.

 (b) if we do replace the default filename with a NULL file, we need
to set that even if we cannot do a strdup() on the NULL.

This fixes both errors.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>

1 files changed, 15 insertions, 5 deletions

diff --git a/file.c b/file.c
index 93abb8b9e..db72b5f66 100644
--- a/file.c
+++ b/file.c
@@ -72,7 +72,7 @@ static void suunto_read(struct zip_file *file, GError **error)
 		size = read * 3 / 2;
 		mem = realloc(mem, size);
 	}
-	parse_xml_buffer(_("SDE file"), mem, read, error, FALSE);
+	parse_xml_buffer(_("SDE file"), mem, read, error);
 	free(mem);
 }
 #endif
@@ -240,14 +240,13 @@ static int open_by_filename(const char *filename, const char *fmt, struct memblo
 	return 0;
 }
 
-static void parse_file_buffer(const char *filename, struct memblock *mem, GError **error,
-	gboolean possible_default_filename)
+static void parse_file_buffer(const char *filename, struct memblock *mem, GError **error)
 {
 	char *fmt = strrchr(filename, '.');
 	if (fmt && open_by_filename(filename, fmt+1, mem, error))
 		return;
 
-	parse_xml_buffer(filename, mem->buffer, mem->size, error, possible_default_filename);
+	parse_xml_buffer(filename, mem->buffer, mem->size, error);
 }
 
 void parse_file(const char *filename, GError **error, gboolean possible_default_filename)
@@ -266,9 +265,20 @@ void parse_file(const char *filename, GError **error, gboolean possible_default_
 					     _("Failed to read '%s'"),
 					     filename);
 		}
+
+		/*
+		 * We do *not* want to leave the old default_filename
+		 * just because the open failed.
+		 */
+		if (possible_default_filename)
+			set_filename(filename, TRUE);
+
 		return;
 	}
 
-	parse_file_buffer(filename, &mem, error, possible_default_filename);
+	if (possible_default_filename)
+		set_filename(filename, TRUE);
+
+	parse_file_buffer(filename, &mem, error);
 	free(mem.buffer);
 }