]> git.lizzy.rs Git - dragonfireclient.git/blobdiff - src/clientiface.cpp
Network: Remove large parts of deprecated legacy code (#6404)
[dragonfireclient.git] / src / clientiface.cpp
index b79d36ea2f1922e4e3a3e2ec3ee7ff3cac65101e..bb740d9f35469a0e6de7d2bfa8028e7799f94186 100644 (file)
@@ -18,18 +18,17 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 */
 
 #include <sstream>
-
 #include "clientiface.h"
+#include "network/connection.h"
+#include "network/serveropcodes.h"
 #include "remoteplayer.h"
 #include "settings.h"
 #include "mapblock.h"
-#include "network/connection.h"
 #include "serverenvironment.h"
 #include "map.h"
 #include "emerge.h"
 #include "content_sao.h"              // TODO this is used for cleanup of only
 #include "log.h"
-#include "network/serveropcodes.h"
 #include "util/srp.h"
 #include "face_position_cache.h"
 
@@ -67,9 +66,6 @@ void RemoteClient::GetNextBlocks (
                float dtime,
                std::vector<PrioritySortedBlockTransfer> &dest)
 {
-       DSTACK(FUNCTION_NAME);
-
-
        // Increment timers
        m_nothing_to_send_pause_timer -= dtime;
        m_nearest_unsent_reset_timer += dtime;
@@ -79,11 +75,11 @@ void RemoteClient::GetNextBlocks (
 
        RemotePlayer *player = env->getPlayer(peer_id);
        // This can happen sometimes; clients and players are not in perfect sync.
-       if (player == NULL)
+       if (!player)
                return;
 
        PlayerSAO *sao = player->getPlayerSAO();
-       if (sao == NULL)
+       if (!sao)
                return;
 
        // Won't send anything if already sending
@@ -95,7 +91,7 @@ void RemoteClient::GetNextBlocks (
        }
 
        v3f playerpos = sao->getBasePosition();
-       v3f playerspeed = player->getSpeed();
+       const v3f &playerspeed = player->getSpeed();
        v3f playerspeeddir(0,0,0);
        if(playerspeed.getLength() > 1.0*BS)
                playerspeeddir = playerspeed / playerspeed.getLength();
@@ -275,19 +271,17 @@ void RemoteClient::GetNextBlocks (
 
                        bool surely_not_found_on_disk = false;
                        bool block_is_invalid = false;
-                       if(block != NULL)
-                       {
+                       if (block) {
                                // Reset usage timer, this block will be of use in the future.
                                block->resetUsageTimer();
 
                                // Block is dummy if data doesn't exist.
                                // It means it has been not found from disk and not generated
-                               if(block->isDummy())
-                               {
+                               if (block->isDummy()) {
                                        surely_not_found_on_disk = true;
                                }
 
-                               if(block->isGenerated() == false)
+                               if (!block->isGenerated())
                                        block_is_invalid = true;
 
                                /*
@@ -297,9 +291,8 @@ void RemoteClient::GetNextBlocks (
                                        Block is near ground level if night-time mesh
                                        differs from day-time mesh.
                                */
-                               if(d >= d_opt)
-                               {
-                                       if(block->getDayNightDiff() == false)
+                               if (d >= d_opt) {
+                                       if (!block->getDayNightDiff())
                                                continue;
                                }
 
@@ -313,8 +306,7 @@ void RemoteClient::GetNextBlocks (
                                If block has been marked to not exist on disk (dummy)
                                and generating new ones is not wanted, skip block.
                        */
-                       if(generate == false && surely_not_found_on_disk == true)
-                       {
+                       if (!generate && surely_not_found_on_disk) {
                                // get next one.
                                continue;
                        }
@@ -415,11 +407,8 @@ void RemoteClient::SetBlocksNotSent(std::map<v3s16, MapBlock*> &blocks)
        m_nearest_unsent_d = 0;
        m_nothing_to_send_pause_timer = 0;
 
-       for(std::map<v3s16, MapBlock*>::iterator
-                       i = blocks.begin();
-                       i != blocks.end(); ++i)
-       {
-               v3s16 p = i->first;
+       for (auto &block : blocks) {
+               v3s16 p = block.first;
                m_blocks_modified.insert(p);
 
                if(m_blocks_sending.find(p) != m_blocks_sending.end())
@@ -465,8 +454,7 @@ void RemoteClient::notifyEvent(ClientStateEvent event)
                {
                case CSE_AuthAccept:
                        m_state = CS_AwaitingInit2;
-                       if ((chosen_mech == AUTH_MECHANISM_SRP)
-                                       || (chosen_mech == AUTH_MECHANISM_LEGACY_PASSWORD))
+                       if (chosen_mech == AUTH_MECHANISM_SRP)
                                srp_verifier_delete((SRPVerifier *) auth_data);
                        chosen_mech = AUTH_MECHANISM_NONE;
                        break;
@@ -475,8 +463,7 @@ void RemoteClient::notifyEvent(ClientStateEvent event)
                        break;
                case CSE_SetDenied:
                        m_state = CS_Denied;
-                       if ((chosen_mech == AUTH_MECHANISM_SRP)
-                                       || (chosen_mech == AUTH_MECHANISM_LEGACY_PASSWORD))
+                       if (chosen_mech == AUTH_MECHANISM_SRP)
                                srp_verifier_delete((SRPVerifier *) auth_data);
                        chosen_mech = AUTH_MECHANISM_NONE;
                        break;
@@ -554,8 +541,7 @@ void RemoteClient::notifyEvent(ClientStateEvent event)
                        break;
                case CSE_SudoSuccess:
                        m_state = CS_SudoMode;
-                       if ((chosen_mech == AUTH_MECHANISM_SRP)
-                                       || (chosen_mech == AUTH_MECHANISM_LEGACY_PASSWORD))
+                       if (chosen_mech == AUTH_MECHANISM_SRP)
                                srp_verifier_delete((SRPVerifier *) auth_data);
                        chosen_mech = AUTH_MECHANISM_NONE;
                        break;
@@ -595,7 +581,7 @@ u64 RemoteClient::uptime() const
        return porting::getTimeS() - m_connection_time;
 }
 
-ClientInterface::ClientInterface(con::Connection* con)
+ClientInterface::ClientInterface(const std::shared_ptr<con::Connection> & con)
 :
        m_con(con),
        m_env(NULL),
@@ -611,10 +597,9 @@ ClientInterface::~ClientInterface()
        {
                MutexAutoLock clientslock(m_clients_mutex);
 
-               for (RemoteClientMap::iterator i = m_clients.begin();
-                       i != m_clients.end(); ++i) {
+               for (auto &client_it : m_clients) {
                        // Delete client
-                       delete i->second;
+                       delete client_it.second;
                }
        }
 }
@@ -624,15 +609,24 @@ std::vector<u16> ClientInterface::getClientIDs(ClientState min_state)
        std::vector<u16> reply;
        MutexAutoLock clientslock(m_clients_mutex);
 
-       for (RemoteClientMap::iterator i = m_clients.begin();
-               i != m_clients.end(); ++i) {
-               if (i->second->getState() >= min_state)
-                       reply.push_back(i->second->peer_id);
+       for (const auto &m_client : m_clients) {
+               if (m_client.second->getState() >= min_state)
+                       reply.push_back(m_client.second->peer_id);
        }
 
        return reply;
 }
 
+/**
+ * Verify if user limit was reached.
+ * User limit count all clients from HelloSent state (MT protocol user) to Active state
+ * @return true if user limit was reached
+ */
+bool ClientInterface::isUserLimitReached()
+{
+       return getClientIDs(CS_HelloSent).size() >= g_settings->getU16("max_users");
+}
+
 void ClientInterface::step(float dtime)
 {
        m_print_info_timer += dtime;
@@ -645,7 +639,7 @@ void ClientInterface::step(float dtime)
 
 void ClientInterface::UpdatePlayerList()
 {
-       if (m_env != NULL) {
+       if (m_env) {
                std::vector<u16> clients = getClientIDs();
                m_clients_names.clear();
 
@@ -653,8 +647,8 @@ void ClientInterface::UpdatePlayerList()
                if(!clients.empty())
                        infostream<<"Players:"<<std::endl;
 
-               for (std::vector<u16>::iterator i = clients.begin(); i != clients.end(); ++i) {
-                       RemotePlayer *player = m_env->getPlayer(*i);
+               for (u16 i : clients) {
+                       RemotePlayer *player = m_env->getPlayer(i);
 
                        if (player == NULL)
                                continue;
@@ -663,12 +657,12 @@ void ClientInterface::UpdatePlayerList()
 
                        {
                                MutexAutoLock clientslock(m_clients_mutex);
-                               RemoteClient* client = lockedGetClientNoEx(*i);
-                               if(client != NULL)
+                               RemoteClient* client = lockedGetClientNoEx(i);
+                               if (client)
                                        client->PrintInfo(infostream);
                        }
 
-                       m_clients_names.push_back(player->getName());
+                       m_clients_names.emplace_back(player->getName());
                }
        }
 }
@@ -682,9 +676,8 @@ void ClientInterface::send(u16 peer_id, u8 channelnum,
 void ClientInterface::sendToAll(NetworkPacket *pkt)
 {
        MutexAutoLock clientslock(m_clients_mutex);
-       for (RemoteClientMap::iterator i = m_clients.begin();
-                       i != m_clients.end(); ++i) {
-               RemoteClient *client = i->second;
+       for (auto &client_it : m_clients) {
+               RemoteClient *client = client_it.second;
 
                if (client->net_proto_version != 0) {
                        m_con->Send(client->peer_id,
@@ -694,6 +687,31 @@ void ClientInterface::sendToAll(NetworkPacket *pkt)
        }
 }
 
+void ClientInterface::sendToAllCompat(NetworkPacket *pkt, NetworkPacket *legacypkt,
+               u16 min_proto_ver)
+{
+       MutexAutoLock clientslock(m_clients_mutex);
+       for (auto &client_it : m_clients) {
+               RemoteClient *client = client_it.second;
+               NetworkPacket *pkt_to_send = nullptr;
+
+               if (client->net_proto_version >= min_proto_ver) {
+                       pkt_to_send = pkt;
+               } else if (client->net_proto_version != 0) {
+                       pkt_to_send = legacypkt;
+               } else {
+                       warningstream << "Client with unhandled version to handle: '"
+                               << client->net_proto_version << "'";
+                       continue;
+               }
+
+               m_con->Send(client->peer_id,
+                       clientCommandFactoryTable[pkt_to_send->getCommand()].channel,
+                       pkt_to_send,
+                       clientCommandFactoryTable[pkt_to_send->getCommand()].reliable);
+       }
+}
+
 RemoteClient* ClientInterface::getClientNoEx(u16 peer_id, ClientState state_min)
 {
        MutexAutoLock clientslock(m_clients_mutex);
@@ -705,8 +723,8 @@ RemoteClient* ClientInterface::getClientNoEx(u16 peer_id, ClientState state_min)
 
        if (n->second->getState() >= state_min)
                return n->second;
-       else
-               return NULL;
+
+       return NULL;
 }
 
 RemoteClient* ClientInterface::lockedGetClientNoEx(u16 peer_id, ClientState state_min)
@@ -719,8 +737,8 @@ RemoteClient* ClientInterface::lockedGetClientNoEx(u16 peer_id, ClientState stat
 
        if (n->second->getState() >= state_min)
                return n->second;
-       else
-               return NULL;
+
+       return NULL;
 }
 
 ClientState ClientInterface::getClientState(u16 peer_id)
@@ -762,10 +780,8 @@ void ClientInterface::DeleteClient(u16 peer_id)
        //TODO this should be done by client destructor!!!
        RemoteClient *client = n->second;
        // Handle objects
-       for (std::set<u16>::iterator i = client->m_known_objects.begin();
-                       i != client->m_known_objects.end(); ++i) {
+       for (u16 id : client->m_known_objects) {
                // Get object
-               u16 id = *i;
                ServerActiveObject* obj = m_env->getActiveObject(id);
 
                if(obj && obj->m_known_by_count > 0)