From 080002f8ed1af6d34cdc6f5abff0f51586ca831c Mon Sep 17 00:00:00 2001
From: Perttu Ahola <celeron55@gmail.com>
Date: Fri, 14 Oct 2011 12:39:25 +0300
Subject: [PATCH] Lock mutex when accessing m_env in Server::~Server

---
 src/server.cpp | 30 ++++++++++++++++++------------
 1 file changed, 18 insertions(+), 12 deletions(-)

diff --git a/src/server.cpp b/src/server.cpp
index 73df81dde..5b9903f5a 100644
--- a/src/server.cpp
+++ b/src/server.cpp
@@ -1139,19 +1139,23 @@ Server::~Server()
 			{}
 		}
 	}
-
-	/*
-		Save players
-	*/
-	dstream<<"Server: Saving players"<<std::endl;
-	m_env.serializePlayers(m_mapsavedir);
-
-	/*
-		Save environment metadata
-	*/
-	dstream<<"Server: Saving environment metadata"<<std::endl;
-	m_env.saveMeta(m_mapsavedir);
 	
+	{
+		JMutexAutoLock envlock(m_env_mutex);
+
+		/*
+			Save players
+		*/
+		dstream<<"Server: Saving players"<<std::endl;
+		m_env.serializePlayers(m_mapsavedir);
+
+		/*
+			Save environment metadata
+		*/
+		dstream<<"Server: Saving environment metadata"<<std::endl;
+		m_env.saveMeta(m_mapsavedir);
+	}
+		
 	/*
 		Stop threads
 	*/
@@ -1201,6 +1205,8 @@ void Server::start(unsigned short port)
 void Server::stop()
 {
 	DSTACK(__FUNCTION_NAME);
+	
+	dout_server<<"Server: Stopping and waiting threads"<<std::endl;
 
 	// Stop threads (set run=false first so both start stopping)
 	m_thread.setRun(false);