games/vcmi
1
0
Fork 0
mirror of https://github.com/vcmi/vcmi.git synced 2025-08-13 19:54:17 +02:00
Code Issues Releases Activity

fix server crash on unexpected client disconnect

Browse Source
This commit is contained in:
Andrey Filipenkov
2021-03-20 17:04:47 +03:00
parent bc1d99431d
commit 7f3b06eb86
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
server/CGameHandler.cpp
View File

@@ -1195,11 +1195,11 @@ void CGameHandler::handleClientDisconnection(std::shared_ptr<CConnection> c)
{ {
for(auto playerConns : connections) for(auto playerConns : connections)
{ {
for(auto conn : playerConns.second) for(auto i = playerConns.second.begin(); i != playerConns.second.end(); )
{ {
if(lobby->state != EServerState::SHUTDOWN && conn == c) if(lobby->state != EServerState::SHUTDOWN && *i == c)
{ {
vstd::erase_if_present(playerConns.second, conn); i = playerConns.second.erase(i);
if(playerConns.second.size()) if(playerConns.second.size())
continue; continue;
PlayerCheated pc; PlayerCheated pc;
@@ -1208,6 +1208,8 @@ void CGameHandler::handleClientDisconnection(std::shared_ptr<CConnection> c)
sendAndApply(&pc); sendAndApply(&pc);
checkVictoryLossConditionsForPlayer(playerConns.first); checkVictoryLossConditionsForPlayer(playerConns.first);
} }
else
++i;
} }
} }
} }
@@ -5267,7 +5269,7 @@ void CGameHandler::checkVictoryLossConditionsForAll()
void CGameHandler::checkVictoryLossConditionsForPlayer(PlayerColor player) void CGameHandler::checkVictoryLossConditionsForPlayer(PlayerColor player)
{ {
const PlayerState * p = getPlayer(player); const PlayerState * p = getPlayer(player);
if (p->status != EPlayerStatus::INGAME) return; if (!p || p->status != EPlayerStatus::INGAME) return;
auto victoryLossCheckResult = gs->checkForVictoryAndLoss(player); auto victoryLossCheckResult = gs->checkForVictoryAndLoss(player);