Fixed crash in SignManager which seems to have been main cause of ins…

…tability.
sobta · Jan 30, 2022 · b94862b · b94862b
1 parent 20b1a47
commit b94862b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/Source/Server/Server/GameService/GameManagers/Signs/SignManager.cpp b/Source/Server/Server/GameService/GameManagers/Signs/SignManager.cpp
@@ -123,7 +123,7 @@ MessageHandleResult SignManager::Handle_RequestGetSignList(GameClient* Client, c
         std::unordered_set<uint32_t> ClientExistingSignId;
         for (int j = 0; j < Area.already_have_signs_size(); j++)
         {
-            ClientExistingSignId.insert(Area.already_have_signs(i).sign_id());
+            ClientExistingSignId.insert(Area.already_have_signs(j).sign_id());
         }
 
         OnlineAreaId AreaId = (OnlineAreaId)Area.online_area_id();

diff --git a/Source/Server/Server/Streams/Frpg2ReliableUdpPacketStream.cpp b/Source/Server/Server/Streams/Frpg2ReliableUdpPacketStream.cpp
@@ -485,7 +485,7 @@ void Frpg2ReliableUdpPacketStream::Handle_RACK(const Frpg2ReliableUdpPacket& Pac
     // I'm like 95% sure that RACK is "Reject ACK", its telling us the ACK recieved was invalid I think?
     // I think we can just ignore this ...
 
-    Log(Connection->GetName().c_str(), "Recieved RACK - Ignoring ...");
+    LogS(Connection->GetName().c_str(), "Recieved RACK - Ignoring ...");
 }
 
 void Frpg2ReliableUdpPacketStream::Handle_DAT(const Frpg2ReliableUdpPacket& Packet)

diff --git a/Source/Server/Server/WebUIService/Handlers/PlayersHandler.cpp b/Source/Server/Server/WebUIService/Handlers/PlayersHandler.cpp
@@ -91,7 +91,7 @@ bool PlayersHandler::handleGet(CivetServer* Server, struct mg_connection* Connec
                 {
                     if (playerStatus.net_mode() == Frpg2PlayerData::NetMode::NetMode_Host)
                     {
-                        playerJson["status"] = "Cooperating with another player";
+                        playerJson["status"] = "Hosting other players";
                     }
                     else
                     {