ClientInterface: add a function to verify (correctly) if user limit was reached ...
authorLoïc Blot <nerzhul@users.noreply.github.com>
Wed, 16 Aug 2017 21:48:29 +0000 (23:48 +0200)
committerGitHub <noreply@github.com>
Wed, 16 Aug 2017 21:48:29 +0000 (23:48 +0200)
commit1d8d01074fdb52946f81110bebf1d001185b394b
tree64cbf6f73bc0f18cbf8333247cd71ab1856ed4c2
parent85511a642f851100d0d856f4ecbe7fea7a7bb049
ClientInterface: add a function to verify (correctly) if user limit was reached (#6258)

* ClientInterface: add a function to verify (correctly) if user limit was reached

CS_HelloSent is a better indicator of active slots than CS_Created, which are session objects created after init packet reception

Switch existing checks to ClientInterface::isUserLimitReached()

Use range-based for loop for getClientIds() used function too

This will fix #6254 (not the memory overhead if init is flooded)
src/clientiface.cpp
src/clientiface.h
src/network/serverpackethandler.cpp