Core/Movement: Hotfix the crashes that can occur in the movement handlers (#26538)

2 files changed, 8 insertions, 1 deletions

diff --git a/src/server/game/Server/GameClient.cpp b/src/server/game/Server/GameClient.cpp
index f2a662d63dd..9dde28b3bfb 100644
--- a/src/server/game/Server/GameClient.cpp
+++ b/src/server/game/Server/GameClient.cpp
@@ -28,6 +28,9 @@ GameClient::GameClient(WorldSession* sessionToServer)
 
 void GameClient::AddAllowedMover(Unit* unit)
 {
+    if (GameClient* previousController = unit->GetGameClientMovingMe())
+        previousController->RemoveAllowedMover(unit);
+
     _allowedMovers.insert(unit->GetGUID());
     unit->SetGameClientMovingMe(this);
 }
@@ -36,7 +39,10 @@ void GameClient::RemoveAllowedMover(Unit* unit)
 {
     _allowedMovers.erase(unit->GetGUID());
     if (unit->GetGameClientMovingMe() == this)
+    {
         unit->SetGameClientMovingMe(nullptr);
+        SetActivelyMovedUnit(nullptr);
+    }
 }
 
 bool GameClient::IsAllowedToMove(Unit* unit) const
diff --git a/src/server/game/Server/WorldSession.cpp b/src/server/game/Server/WorldSession.cpp
index 2405631cd75..9cf46fba730 100644
--- a/src/server/game/Server/WorldSession.cpp
+++ b/src/server/game/Server/WorldSession.cpp
@@ -1665,9 +1665,10 @@ bool WorldSession::IsRightUnitBeingMoved(ObjectGuid guid)
     GameClient* client = GetGameClient();
 
     // the client is attempting to tamper movement data
+    // edit: this wouldn't happen in retail but it does in TC, even with a legitimate client.
     if (!client->GetActivelyMovedUnit() || client->GetActivelyMovedUnit()->GetGUID() != guid)
     {
-        TC_LOG_INFO("entities.unit", "Attempt at tampering movement data by Player %s", _player->GetName().c_str());
+        TC_LOG_DEBUG("entities.unit", "Attempt at tampering movement data by Player %s", _player->GetName().c_str());
         return false;
     }