[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);

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

[rsa]

А оно точно не от античита? Мне наводку на правку похожего краша Wowka321 дал, оно из-за миндконтрола и дальновидения хантов вылетало. Там действительно проблемы в определении mover/m_mover были.

[zergtmn]

Эта функция совершенно бесполезна в текущей версии.
Чтобы не падал в HandleSetActiveMoverOpcode тело функции заменяем на:

recv_data.read_skip<uint64>();

Но все равно будет падать из-за неверного мувера в разных местах.

У меня такая версия:

PHP код:

void WorldSession::HandleSetActiveMoverOpcode(WorldPacket &recv_data)
{
    DEBUG_LOG("WORLD: Recvd CMSG_SET_ACTIVE_MOVER");
 
    uint64 guid;
    recv_data >> guid;

    if (Unit *mover = ObjectAccessor::GetUnit(*_player, guid))
        _player->SetMover(mover);
    else
        _player->SetMover(NULL);
} 


(подглядел в TC). На HandleSetActiveMoverOpcode и HandleMovementOpcodes больше не падает.

[Insider42]

Сообщение от zergtmn

подглядел в TC

Раз в TC то неплохо бы и автора указать