From e491f8cd485f5602c056b8b6c5e0c8c395e4a633 Mon Sep 17 00:00:00 2001
From: ShadowNinja <shadowninja@minetest.net>
Date: Sun, 1 Jun 2014 13:11:37 -0400
Subject: Only try to load from possible player files

---
 src/environment.cpp | 41 +++++++++++++++--------------------------
 1 file changed, 15 insertions(+), 26 deletions(-)

(limited to 'src/environment.cpp')

diff --git a/src/environment.cpp b/src/environment.cpp
index 91f5ea2b6..d068aa788 100644
--- a/src/environment.cpp
+++ b/src/environment.cpp
@@ -443,46 +443,35 @@ void ServerEnvironment::savePlayer(const std::string &playername)
 
 Player *ServerEnvironment::loadPlayer(const std::string &playername)
 {
-	std::string players_path = m_path_world + DIR_DELIM "players";
+	std::string players_path = m_path_world + DIR_DELIM "players" DIR_DELIM;
 
 	RemotePlayer *player = static_cast<RemotePlayer*>(getPlayer(playername.c_str()));
 	bool newplayer = false;
-	bool foundplayer = false;
+	bool found = false;
 	if (!player) {
 		player = new RemotePlayer(m_gamedef);
 		newplayer = true;
 	}
 
-	std::vector<fs::DirListNode> player_files = fs::GetDirListing(players_path);
-	for (u32 i = 0; i < player_files.size(); i++) {
-		if (player_files[i].dir)
-			continue;
-		
-		// Full path to this file
-		std::string path = players_path + "/" + player_files[i].name;
-
-		// Load player to see what is its name
+	RemotePlayer testplayer(m_gamedef);
+	std::string path = players_path + playername;
+	for (u32 i = 0; i < 1000; i++) {
+		// Open file and deserialize
 		std::ifstream is(path.c_str(), std::ios_base::binary);
 		if (!is.good()) {
-			infostream << "Failed to read " << path << std::endl;
-			continue;
+			return NULL;
 		}
-		player->deSerialize(is, player_files[i].name);
-
-		if (!string_allowed(player->getName(), PLAYERNAME_ALLOWED_CHARS)) {
-			infostream << "Not loading player with invalid name: "
-					<< player->getName() << std::endl;
-			continue;
-		}
-
-		if (player->getName() == playername) {
-			// We found our player
-			foundplayer = true;
+		testplayer.deSerialize(is, path);
+		if (testplayer.getName() == playername) {
+			*player = testplayer;
+			found = true;
 			break;
 		}
-
+		path = players_path + playername + itos(i);
 	}
-	if (!foundplayer) {
+	if (!found) {
+		infostream << "Player file for player " << playername
+				<< " not found" << std::endl;
 		return NULL;
 	}
 	if (newplayer) {
-- 
cgit v1.2.3