Chests: Check 'def' of node above chest to avoid crash

In 'chest_lid_obstructed(pos)' check for nil 'def' to avoid a crash caused by an unknown node above the chest.
minetest · Jun 16, 2017 · 120e969 · 120e969
1 parent 502720b
commit 120e969
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/mods/default/nodes.lua b/mods/default/nodes.lua
@@ -1783,13 +1783,14 @@ local function get_chest_formspec(pos)
 end
 
 local function chest_lid_obstructed(pos)
-	local above = { x = pos.x, y = pos.y + 1, z = pos.z }
+	local above = {x = pos.x, y = pos.y + 1, z = pos.z}
 	local def = minetest.registered_nodes[minetest.get_node(above).name]
 	-- allow ladders, signs, wallmounted things and torches to not obstruct
-	if def.drawtype == "airlike" or
+	if def and
+			(def.drawtype == "airlike" or
 			def.drawtype == "signlike" or
 			def.drawtype == "torchlike" or
-			(def.drawtype == "nodebox" and def.paramtype2 == "wallmounted") then
+			(def.drawtype == "nodebox" and def.paramtype2 == "wallmounted")) then
 		return false
 	end
 	return true