diff options
author | Giacomo Pozzoni <giacomopoz@gmail.com> | 2019-06-14 10:13:46 +0200 |
---|---|---|
committer | Giacomo Pozzoni <giacomopoz@gmail.com> | 2019-06-14 10:13:46 +0200 |
commit | f177790618e2ba9f88e2ffcef6d9acfaff88d48a (patch) | |
tree | a9a97262138c9058b7bb92a973e5b4240bb93f5b | |
parent | 1dedd5552d31734750c5873b87f594556a7942a4 (diff) |
Core/Misc: Fix static analysis issues reported by Coverity Scan
-rw-r--r-- | src/server/scripts/EasternKingdoms/ZulGurub/boss_thekal.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/server/scripts/EasternKingdoms/ZulGurub/boss_thekal.cpp b/src/server/scripts/EasternKingdoms/ZulGurub/boss_thekal.cpp index dec999545f1..3d839e7bd0f 100644 --- a/src/server/scripts/EasternKingdoms/ZulGurub/boss_thekal.cpp +++ b/src/server/scripts/EasternKingdoms/ZulGurub/boss_thekal.cpp @@ -332,8 +332,8 @@ class npc_zealot_lorkhan : public CreatureScript bool roll = roll_chance_i(50); Creature* target = roll ? (thekal ? thekal : zath) : (zath ? zath : thekal); - if (!target) - return; + // As we check already above to ensure at least 1 out of Thekal/Zath is not null, target must be not null + ASSERT(target); if (!me->IsWithinMeleeRange(target)) target = roll ? zath : thekal; |