Fix potential crash when few loot items are registered

minetest · Jul 16, 2019 · 1f8b489ac2c0802a07b690a61a145ad550e27030 · 1f8b489
1 parent c340393
commit 1f8b489ac2c0802a07b690a61a145ad550e27030
Unified Split

Showing with 2 additions and 2 deletions.

+2 −2 mods/dungeon_loot/mapgen.lua
diff --git a/mods/dungeon_loot/mapgen.lua b/mods/dungeon_loot/mapgen.lua
@@ -76,8 +76,8 @@ local function populate_chest(pos, rand, dungeontype)
 	local item_list = dungeon_loot._internal_get_loot(pos.y, dungeontype)
 	-- take random (partial) sample of all possible items
-  
-	assert(#item_list >= dungeon_loot.STACKS_PER_CHEST_MAX)
-  
-	item_list = random_sample(rand, item_list, dungeon_loot.STACKS_PER_CHEST_MAX)
+  
+	local n = math.min(#item_list, dungeon_loot.STACKS_PER_CHEST_MAX)
+  
+	item_list = random_sample(rand, item_list, n)
 	-- apply chances / randomized amounts and collect resulting items
 	local items = {}