From d9501acb3f837fa3f206be1f9402cdf40d87eb03 Mon Sep 17 00:00:00 2001 From: jackpoz Date: Sat, 2 Dec 2017 16:08:48 +0100 Subject: [PATCH] Core/Spell: Validate spell_dbc DB data on startup Validate spell_dbc DB data on startup (max effect type, max aura type, max target type). Delete some invalid spells from spell_dbc table. Make sure to apply the SQL script or worldserver will assert on startup. --- sql/updates/world/3.3.5/2017_12_02_00_world.sql | 1 + src/server/game/Spells/SpellMgr.cpp | 12 ++++++------ 2 files changed, 7 insertions(+), 6 deletions(-) create mode 100644 sql/updates/world/3.3.5/2017_12_02_00_world.sql diff --git a/sql/updates/world/3.3.5/2017_12_02_00_world.sql b/sql/updates/world/3.3.5/2017_12_02_00_world.sql new file mode 100644 index 00000000000..cbcf0331531 --- /dev/null +++ b/sql/updates/world/3.3.5/2017_12_02_00_world.sql @@ -0,0 +1 @@ +DELETE FROM `spell_dbc` WHERE `Effect1` >= 165 OR `Effect2` >= 165 OR `Effect3` >= 165 OR `EffectApplyAuraName1` >= 317 OR `EffectApplyAuraName2` >= 317 OR `EffectApplyAuraName3` >= 317 OR `EffectImplicitTargetA1` >= 111 OR `EffectImplicitTargetA2` >= 111 OR `EffectImplicitTargetA3` >= 111 OR `EffectImplicitTargetB1` >= 111 OR `EffectImplicitTargetB2` >= 111 OR `EffectImplicitTargetB3` >= 111; diff --git a/src/server/game/Spells/SpellMgr.cpp b/src/server/game/Spells/SpellMgr.cpp index 8d4af0a5f5a..8887ba17e1b 100644 --- a/src/server/game/Spells/SpellMgr.cpp +++ b/src/server/game/Spells/SpellMgr.cpp @@ -2622,13 +2622,13 @@ void SpellMgr::LoadSpellInfoStore() if (!mSpellInfoMap[spellIndex]) continue; - for (uint32 effectIndex = 0; effectIndex < MAX_SPELL_EFFECTS; ++effectIndex) + for (auto const& effect : mSpellInfoMap[spellIndex]->Effects) { - if (mSpellInfoMap[spellIndex]->Effects[effectIndex].Effect >= TOTAL_SPELL_EFFECTS) - { - TC_LOG_ERROR("sql.sql", "Spell (Entry: %u) has `Effect` '%u' greater than max allowed value '%u', removing", spellIndex, mSpellInfoMap[spellIndex]->Effects[effectIndex].Effect, (TOTAL_SPELL_EFFECTS - 1)); - mSpellInfoMap[spellIndex]->Effects[effectIndex].Effect = 0; - } + //ASSERT(effect.EffectIndex < MAX_SPELL_EFFECTS, "MAX_SPELL_EFFECTS must be at least %u", effect.EffectIndex + 1); + ASSERT(effect.Effect < TOTAL_SPELL_EFFECTS, "TOTAL_SPELL_EFFECTS must be at least %u", effect.Effect + 1); + ASSERT(effect.ApplyAuraName < TOTAL_AURAS, "TOTAL_AURAS must be at least %u", effect.ApplyAuraName + 1); + ASSERT(effect.TargetA.GetTarget() < TOTAL_SPELL_TARGETS, "TOTAL_SPELL_TARGETS must be at least %u", effect.TargetA.GetTarget() + 1); + ASSERT(effect.TargetB.GetTarget() < TOTAL_SPELL_TARGETS, "TOTAL_SPELL_TARGETS must be at least %u", effect.TargetB.GetTarget() + 1); } }