Fix for #37

DJPlaya · Apr 13, 2020 · cb1bd15 · cb1bd15
1 parent 1b5fe35
commit cb1bd15
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 2 deletions.
diff --git a/plugins/kigen-ac_redux.smx b/plugins/kigen-ac_redux.smx
diff --git a/plugins/kigen-ac_redux_legacy.smx b/plugins/kigen-ac_redux_legacy.smx
diff --git a/scripting/1.10/kigen-ac_redux/eyetest.sp b/scripting/1.10/kigen-ac_redux/eyetest.sp
@@ -270,7 +270,7 @@ public Action Eyetest_Equip(client, weapon) // The Player Picked up a Weapon? Le
 
 public Action Eyetest_Drop(client, weapon)
 {
-	if (g_iWeaponOwner[weapon] >= 1) // Is a Player linked to the Entity? Else we do not Care // TODO: Array index out-of-bounds (index -1, limit 4096) - Issue #13
+	if (weapon > 0 && g_iWeaponOwner[weapon] > 0) // Is a Player linked to the Entity? Else we do not Care // Check if the Weapon exists first, it may got deleted in the same Frame, Bugfix for #37
 	{
 		g_iWeaponOwner[weapon] = 0;
 		SDKUnhook(weapon, SDKHook_SetTransmit, Eyetest_WeaponTransmit);

diff --git a/scripting/1.8/kigen-ac_redux/eyetest.sp b/scripting/1.8/kigen-ac_redux/eyetest.sp
@@ -270,7 +270,7 @@ public Action Eyetest_Equip(client, weapon) // The Player Picked up a Weapon? Le
 
 public Action Eyetest_Drop(client, weapon)
 {
-	if (g_iWeaponOwner[weapon] >= 1) // Is a Player linked to the Entity? Else we do not Care // TODO: Array index out-of-bounds (index -1, limit 4096) - Issue #13
+	if (weapon > 0 && g_iWeaponOwner[weapon] > 0) // Is a Player linked to the Entity? Else we do not Care // Check if the Weapon exists first, it may got deleted in the same Frame, Bugfix for #37
 	{
 		g_iWeaponOwner[weapon] = 0;
 		SDKUnhook(weapon, SDKHook_SetTransmit, Eyetest_WeaponTransmit);