diff options
author | Loïc Blot <nerzhul@users.noreply.github.com> | 2017-08-16 23:48:29 +0200 |
---|---|---|
committer | SmallJoker <mk939@ymail.com> | 2018-06-03 17:31:59 +0200 |
commit | 151c19a6be591d19d83cdd3c69583e7bf0828440 (patch) | |
tree | 37470c00d6cb1472aff14cb86fe3bf53c13d7439 /src/clientiface.h | |
parent | 9d40d89d2721359b02c578366ebfd0ef1b91116d (diff) | |
download | minetest-151c19a6be591d19d83cdd3c69583e7bf0828440.tar.gz minetest-151c19a6be591d19d83cdd3c69583e7bf0828440.tar.bz2 minetest-151c19a6be591d19d83cdd3c69583e7bf0828440.zip |
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)
Diffstat (limited to 'src/clientiface.h')
-rw-r--r-- | src/clientiface.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/clientiface.h b/src/clientiface.h index d2299c879..6e7429309 100644 --- a/src/clientiface.h +++ b/src/clientiface.h @@ -449,6 +449,9 @@ public: /* get list of active client id's */ std::vector<u16> getClientIDs(ClientState min_state=CS_Active); + /* verify is server user limit was reached */ + bool isUserLimitReached(); + /* get list of client player names */ const std::vector<std::string> &getPlayerNames() const { return m_clients_names; } @@ -493,7 +496,6 @@ public: } static std::string state2Name(ClientState state); - protected: //TODO find way to avoid this functions void lock() { m_clients_mutex.lock(); } |