[patch/dev] crash in HandleSetActiveMoverOpcode

[Insider42]

Вот недавно словил такой краш http://paste2.org/p/860715 . Я помню ещё как-то давно он был одним из самых частых крашей, видимо попытки hununza вправить его уменьшили его частоту, однако я его таки словил на ревизии 10011

В общем мои первые потуги его исправить окончательно. Т.к. мы незнаем откуда корень краша, пока предлагаю затык.
Написал два варианта.
1

diff --git a/src/game/MovementHandler.cpp b/src/game/MovementHandler.cpp
index d81bb8a..765cd2f 100644
--- a/src/game/MovementHandler.cpp
+++ b/src/game/MovementHandler.cpp
@@ -778,6 +778,12 @@ void WorldSession::HandleSetActiveMoverOpcode(WorldPacket &recv_data)
     uint64 guid;
     recv_data >> guid;
 
+    if(!_player || !guid)
+    {
+        sLog.outError("Prevented crash with HandleSetActiveMoverOpcode");
+        return;
+    }
+
     if(_player->m_mover->GetGUID() != guid)
     {
         sLog.outError("HandleSetActiveMoverOpcode: incorrect mover guid: mover is " I64FMT " and should be " I64FMT, _player->m_mover->GetGUID(), guid);

2

diff --git a/src/game/MovementHandler.cpp b/src/game/MovementHandler.cpp
index d81bb8a..765cd2f 100644
--- a/src/game/MovementHandler.cpp
+++ b/src/game/MovementHandler.cpp
@@ -778,6 +778,12 @@ void WorldSession::HandleSetActiveMoverOpcode(WorldPacket &recv_data)
     uint64 guid;
     recv_data >> guid;
 
+    if(!_player->m_mover->GetGUID() || !guid)
+    {
+        sLog.outError("Prevented crash with HandleSetActiveMoverOpcode");
+        return;
+    }
+
     if(_player->m_mover->GetGUID() != guid)
     {
         sLog.outError("HandleSetActiveMoverOpcode: incorrect mover guid: mover is " I64FMT " and should be " I64FMT, _player->m_mover->GetGUID(), guid);

Жду конструктивной критики