[crash fix] Aura::PeriodicTick

[Insider42]

Оригинальная тема тут http://getmangos.com/community/showthread.php?13636

Тут не проверяется есть ли цель в игре, по-моему более правильно ничего не делать с целью которая находится вне игры. По крайней мере ниразу не слышал от игроков с оффа чтобы что-то происходило с их персонажами когда они вышли из игры+их персонаж выгрузился из мира (как уже было обнаружено, на оффе персонаж ещё некоторое время стоит в игре после того как игрок уже вышел из игры. Возможно это происходит только если он был в бою).

diff --git a/src/game/SpellAuras.cpp b/src/game/SpellAuras.cpp
index 59b037f..d9af2ae 100644
--- a/src/game/SpellAuras.cpp
+++ b/src/game/SpellAuras.cpp
@@ -7284,13 +7284,32 @@ void Aura::PeriodicTick()
 {
     switch(m_modifier.m_auraname)
     {
+        // don't do anything to target that isn't in world or dead, possible death persistent effects
         case SPELL_AURA_PERIODIC_DAMAGE:
         case SPELL_AURA_PERIODIC_DAMAGE_PERCENT:
-        {
-            // don't damage target if not alive, possible death persistent effects
-            if (!m_target->isAlive())
+        case SPELL_AURA_PERIODIC_LEECH:
+        case SPELL_AURA_PERIODIC_HEALTH_FUNNEL:
+        case SPELL_AURA_PERIODIC_HEAL:
+        case SPELL_AURA_OBS_MOD_HEALTH:
+        case SPELL_AURA_PERIODIC_MANA_LEECH:
+        case SPELL_AURA_POWER_BURN_MANA:
+        case SPELL_AURA_PERIODIC_ENERGIZE:
+        case SPELL_AURA_OBS_MOD_MANA:
+        case SPELL_AURA_MOD_REGEN:
+        case SPELL_AURA_MOD_POWER_REGEN:
+            if (!m_target || !m_target->IsInWorld() || !m_target->isAlive())
                 return;
+            break;
+        default:
+            break;
+    }
 
+    switch(m_modifier.m_auraname)
+    {
+        case SPELL_AURA_PERIODIC_DAMAGE:
+        case SPELL_AURA_PERIODIC_DAMAGE_PERCENT:
+        {
             Unit *pCaster = GetCaster();
             if(!pCaster)
                 return;
@@ -7427,10 +7446,6 @@ void Aura::PeriodicTick()
         case SPELL_AURA_PERIODIC_LEECH:
         case SPELL_AURA_PERIODIC_HEALTH_FUNNEL:
         {
-            // don't damage target if not alive, possible death persistent effects
-            if (!m_target->isAlive())
-                return;
-
             Unit *pCaster = GetCaster();
             if(!pCaster)
                 return;
@@ -7516,10 +7531,6 @@ void Aura::PeriodicTick()
         case SPELL_AURA_PERIODIC_HEAL:
         case SPELL_AURA_OBS_MOD_HEALTH:
         {
-            // don't heal target if not alive, mostly death persistent effects from items
-            if (!m_target->isAlive())
-                return;
-
             Unit *pCaster = GetCaster();
             if(!pCaster)
                 return;
@@ -7618,10 +7629,6 @@ void Aura::PeriodicTick()
         }
         case SPELL_AURA_PERIODIC_MANA_LEECH:
         {
-            // don't damage target if not alive, possible death persistent effects
-            if (!m_target->isAlive())
-                return;
-
             if(m_modifier.m_miscvalue < 0 || m_modifier.m_miscvalue >= MAX_POWERS)
                 return;
 
@@ -7695,10 +7702,6 @@ void Aura::PeriodicTick()
         }
         case SPELL_AURA_PERIODIC_ENERGIZE:
         {
-            // don't energize target if not alive, possible death persistent effects
-            if (!m_target->isAlive())
-                return;
-
             // ignore non positive values (can be result apply spellmods to aura damage
             uint32 pdamage = m_modifier.m_amount > 0 ? m_modifier.m_amount : 0;
 
@@ -7724,10 +7727,6 @@ void Aura::PeriodicTick()
         }
         case SPELL_AURA_OBS_MOD_MANA:
         {
-            // don't energize target if not alive, possible death persistent effects
-            if (!m_target->isAlive())
-                return;
-
             // ignore non positive values (can be result apply spellmods to aura damage
             uint32 amount = m_modifier.m_amount > 0 ? m_modifier.m_amount : 0;
 
@@ -7750,10 +7749,6 @@ void Aura::PeriodicTick()
         }
         case SPELL_AURA_POWER_BURN_MANA:
         {
-            // don't mana burn target if not alive, possible death persistent effects
-            if (!m_target->isAlive())
-                return;
-
             Unit *pCaster = GetCaster();
             if(!pCaster)
                 return;
@@ -7800,10 +7795,6 @@ void Aura::PeriodicTick()
         }
         case SPELL_AURA_MOD_REGEN:
         {
-            // don't heal target if not alive, possible death persistent effects
-            if (!m_target->isAlive())
-                return;
-
             int32 gain = m_target->ModifyHealth(m_modifier.m_amount);
             if (Unit *caster = GetCaster())
                 m_target->getHostileRefManager().threatAssist(caster, float(gain) * 0.5f, GetSpellProto());
@@ -7811,10 +7802,6 @@ void Aura::PeriodicTick()
         }
         case SPELL_AURA_MOD_POWER_REGEN:
         {
-            // don't energize target if not alive, possible death persistent effects
-            if (!m_target->isAlive())
-                return;
-
             Powers pt = m_target->getPowerType();
             if(int32(pt) != m_modifier.m_miscvalue)
                 return;

Вынес в отдельный switch для обобщения условия

[Insider42]

добавил проверку "есть ли вообще" цель. Патч обновлен

[Vladimir]

if (!m_target

Чистой воды хак: m_target не может быть NULL без бага - m_target - это наком наложена аура и аура удаляется раньше чем сам носитель, _всегда_

!m_target->IsInWorld()

Хак. Обновление ауры не вызывается для юнитов вне мира, так как обновление вызывается для карты включая играков.

Конечно это все в чистом мангосе.

[Insider42]

Но ведь этот краш происходит и на чистом мангосе ... где-то явно бага закопалась

Как можно отследить?

[Vladimir]

ну как всегда - поймать в отладчике и отследить причину если получится...

[Mr.Grom]

Может одна проверка на 11 аур меньше грузит ядро чем 11 проверок на 11 аур?
Так же можно попробовать и с проверками

             Unit *pCaster = GetCaster();
             if(!pCaster)
                 return;

В патче 1 строка повтор.

+        case SPELL_AURA_POWER_BURN_MANA:

[Insider42]

Сообщение от Mr.Grom

Может одна проверка на 11 аур меньше грузит ядро чем 11 проверок на 11 аур?

обрабатываться будет точно также, просто меньше хлама.

[Vladimir]

у вас 2 раза select - что не одно тоже что один - что медленнее (хотя этот и не сильная разница), но что более неприятно - можно забыть добавить проверку когда будет добавляться новый вариант в отсновной выбор.