From 16eb63dfcd487fb990eb4d151a1eed2316487fe6 Mon Sep 17 00:00:00 2001
From: "simon.kagstrom" <simon.kagstrom@feb19cf6-d755-11dd-8179-53c65a4d6d95>
Date: Sat, 30 Jan 2010 08:35:29 +0000
Subject: [PATCH] Fix saving of game info

---
 Src/gui/gui.cpp | 32 +++++++++++++++++++++-----------
 1 file changed, 21 insertions(+), 11 deletions(-)

diff --git a/Src/gui/gui.cpp b/Src/gui/gui.cpp
index e91bac7..d0babd9 100644
--- a/Src/gui/gui.cpp
+++ b/Src/gui/gui.cpp
@@ -374,7 +374,11 @@ Font *Gui::loadThemeFont(const char *dir, const char *what, int size)
 void Gui::updateGameInfo(GameInfo *gi)
 {
 	panic_if(!gi, "gi must be set\n");
+
+	/* Store the current game info */
+	this->saveGameInfo();
 	delete this->cur_gameInfo;
+
 	this->cur_gameInfo = gi;
 	this->gameInfoChanged = true;
 }
@@ -385,21 +389,27 @@ void Gui::saveGameInfo()
 
 	if (p)
 	{
-		char buf[255];
+		char *new_name = (char *)xmalloc(strlen(this->metadata_base_path) +
+				3 + strlen(this->cur_gameInfo->filename));
 		FILE *fp;
 
-		snprintf(buf, sizeof(buf), "%s/%s",
-				METADATA_ROOT_PATH, this->cur_gameInfo->filename);
-		fp = fopen(buf, "w");
-		if (!fp)
+		sprintf(new_name, "%s/%s", this->metadata_base_path,
+				this->cur_gameInfo->filename);
+		fp = fopen(new_name, "w");
+		if (fp)
 		{
-			warning("Could not open %s for writing\n", buf);
-			return;
+			int n = fwrite((const void*)p, 1, p->sz, fp);
+
+			if (ferror(fp))
+				warning("Write error on %s\n", new_name);
+			else if ((size_t)n != p->sz)
+				warning("Could only write %d bytes of %s\n", n, new_name);
+			fclose(fp);
 		}
-		int n = fwrite((const void*)p, 1, p->sz, fp);
-		if (n != (int)p->sz)
-			warning("Could only write %d bytes of %s\n", n, buf);
-		fclose(fp);
+		else
+			warning("Could not open %s for writing\n", new_name);
+
+		free(new_name);
 	}
 
 	this->gameInfoChanged = false;