From c0ef2ba55c1f9e2c5c49403dfb80260339fca0d3 Mon Sep 17 00:00:00 2001
From: ariel- <ariel-@users.noreply.github.com>
Date: Thu, 27 Oct 2016 13:29:56 -0300
Subject: [PATCH] Core/Unit: fix logic error in DamageInfo::ModifyDamage.

Previous code did not protect against m_damage underflow, rather only allow up to duplicate damage.
It now should work as intended.

Closes #18154
---
 src/server/game/Entities/Unit/Unit.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/server/game/Entities/Unit/Unit.cpp b/src/server/game/Entities/Unit/Unit.cpp
index cdc3ad324f3..3de7e965a98 100644
--- a/src/server/game/Entities/Unit/Unit.cpp
+++ b/src/server/game/Entities/Unit/Unit.cpp
@@ -122,7 +122,8 @@ m_damageType(DIRECT_DAMAGE), m_attackType(dmgInfo.attackType)
 
 void DamageInfo::ModifyDamage(int32 amount)
 {
-    amount = std::min(amount, int32(GetDamage()));
+    if (amount < 0)
+        amount = std::max(amount, static_cast<int32>(-GetDamage()));
     m_damage += amount;
 }