Fix possible missing unlock of env_lock

minetest · Feb 12, 2014 · ded5f8b · ded5f8b
1 parent 89f7dc1
commit ded5f8b
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/server.cpp b/src/server.cpp
@@ -4222,9 +4222,10 @@ void Server::DeleteClient(u16 peer_id, ClientDeletionReason reason)
 						<<" List of players: "<<os.str()<<std::endl;
 			}
 		}
-		m_env_mutex.Lock();
-		m_clients.DeleteClient(peer_id);
-		m_env_mutex.Unlock();
+		{
+			JMutexAutoLock env_lock(m_env_mutex);
+			m_clients.DeleteClient(peer_id);
+		}
 	}
 
 	// Send leave chat message to all remaining clients