aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGiacomo Pozzoni <giacomopoz@gmail.com>2019-06-14 10:13:46 +0200
committerGiacomo Pozzoni <giacomopoz@gmail.com>2019-06-14 10:13:46 +0200
commitf177790618e2ba9f88e2ffcef6d9acfaff88d48a (patch)
treea9a97262138c9058b7bb92a973e5b4240bb93f5b
parent1dedd5552d31734750c5873b87f594556a7942a4 (diff)
Core/Misc: Fix static analysis issues reported by Coverity Scan
-rw-r--r--src/server/scripts/EasternKingdoms/ZulGurub/boss_thekal.cpp4
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;