TINSEL: Fixed deleting saved games from the launcher (bug #3387551)

NEWS

@@ -17,6 +17,10 @@ For a more comprehensive changelog of the latest experimental code, see:
  SDL ports:
    - Added support for OpenGL (GSoC Task).
 
+ TINSEL:
+   - Fixed deleting saved games from the list of saved games (from the launcher
+     and the in-game ScummVM menu)
+
 1.3.1 (2011-07-12)
  General:
    - Improved audio device detection and fallback.

engines/tinsel/detection.cpp

@@ -322,9 +322,21 @@ int TinselMetaEngine::getMaximumSaveSlot() const { return 99; }
 
 void TinselMetaEngine::removeSaveState(const char *target, int slot) const {
 	Tinsel::setNeedLoad();
-	Tinsel::getList(g_system->getSavefileManager(), target);
+	// Same issue here as with loadGameState(): we need the physical savegame
+	// slot. Refer to bug #3387551.
+	int listSlot = -1;
+	const int numStates = Tinsel::getList(g_system->getSavefileManager(), target);
+	for (int i = 0; i < numStates; ++i) {
+		const char *fileName = Tinsel::ListEntry(i, Tinsel::LE_NAME);
+		const int saveSlot = atoi(fileName + strlen(fileName) - 3);
 
-	g_system->getSavefileManager()->removeSavefile(Tinsel::ListEntry(slot, Tinsel::LE_NAME));
+		if (saveSlot == slot) {
+			listSlot = i;
+			break;
+		}
+	}
+
+	g_system->getSavefileManager()->removeSavefile(Tinsel::ListEntry(listSlot, Tinsel::LE_NAME));
 	Tinsel::setNeedLoad();
 	Tinsel::getList(g_system->getSavefileManager(), target);
 }

engines/tinsel/saveload.cpp

@@ -155,7 +155,15 @@ static bool syncSaveGameHeader(Common::Serializer &s, SaveGameHeader &hdr) {
 
 	int tmp = hdr.size - s.bytesSynced();
 	// Perform sanity check
-	if (tmp < 0 || hdr.id != SAVEGAME_ID || hdr.ver > CURRENT_VER || hdr.size > 1024)
+	if (tmp < 0 ||
+		// NOTE: We can't use SAVEGAME_ID here, as this function is called by the launcher
+		// when deleting saved games. SAVEGAME_ID calls TinselEngine::getVersion(), and
+		// TinselEngine isn't initialized then. Therefore, we use the two DW savegame
+		// IDs instead, which means that this sanity check won't detect badly named DW1/DW2
+		// saved games (i.e. a DW2 saved game named "dw.xxx"). Refer to bug #3387551.
+		/*hdr.id != SAVEGAME_ID ||*/ 
+		(hdr.id != DW1_SAVEGAME_ID && hdr.id != DW2_SAVEGAME_ID) ||
+		hdr.ver > CURRENT_VER || hdr.size > 1024)
 		return false;
 	// Skip over any extra bytes
 	s.skip(tmp);