Mirrorlandia_minetest/minetest
1
0
Fork 1
mirror of https://github.com/minetest/minetest.git synced 2024-12-04 05:23:44 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix race condition in registration leading to duplicate create_auth calls

Browse Source
This commit is contained in:
sfan5 2022-04-27 19:00:49 +02:00
parent 677dc2c155
commit d497c92684
1 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/network/serverpackethandler.cpp
View File

@ -1496,8 +1496,19 @@ void Server::handleCommand_FirstSrp(NetworkPacket* pkt)
} }
std::string initial_ver_key; std::string initial_ver_key;
initial_ver_key = encode_srp_verifier(verification_key, salt); initial_ver_key = encode_srp_verifier(verification_key, salt);
// It is possible for multiple connections to get this far with the same
// player name. In the end only one player with a given name will be emerged
// (see Server::StateTwoClientInit) but we still have to be careful here.
if (m_script->getAuth(playername, nullptr, nullptr)) {
// Another client beat us to it
actionstream << "Server: Client from " << addr_s
<< " tried to register " << playername << " a second time."
<< std::endl;
DenyAccess(peer_id, SERVER_ACCESSDENIED_ALREADY_CONNECTED);
return;
}
m_script->createAuth(playername, initial_ver_key); m_script->createAuth(playername, initial_ver_key);
m_script->on_authplayer(playername, addr_s, true); m_script->on_authplayer(playername, addr_s, true);