OrdinaryMagician/vkdoom_m
OrdinaryMagician/vkdoom_m
1
Fork 0
Code Activity

Fixed a major bug

Browse source 
Fixed a potential loop crash caused by an undefined amount int in TakeInventory (and derivatives) now it will assume if by is less than 1 that the item is to be destroyed.
This commit is contained in:
XLightningStormL 2025-05-14 09:48:52 +10:00 • committed by Ricardo Luís Vaz Silva
commit 1f1d9dc1b5
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
wadsrc/static/zscript/actors/inventory/inventory.zs
View file

@ -932,7 +932,7 @@ class Inventory : Actor
//===========================================================================
virtual void DepleteBy(int by)
{
if (amount < 1 || by >= amount)
if (by < 1 || amount < 1 || by >= amount)
{
DepleteOrDestroy();
}