Merge pull request #11734 from Menshin/sheets_inserting_safety_checks

Adds boundary checks for stack sheets inserting
2026-08-30 00:29:02 +01:00 · 2015-09-13 23:13:04 +10:00
parent 803b6dfd87 89ce4e112d
commit e718214a67
3 changed files with 21 additions and 9 deletions
@@ -60,9 +60,15 @@
 	return 0

 /datum/material_container/proc/insert_stack(obj/item/stack/S, amt = 0)
-	if(!amt)
+	if(amt <= 0)
+		return 0
+	if(amt > S.amount)
 		amt = S.amount
+
 	var/material_amt = get_item_material_amount(S)
+	if(!material_amt)
+		return 0
+
 	amt = min(amt, round(((max_amount - total_amount) / material_amt)))
 	if(!amt)
 		return 0