BRN_Systems/tnt
1
0
Fork 0
forked from Mirrorlandia_minetest/tnt
Code Issues Pull Requests Projects Releases Wiki Activity

TNT: Fix up nil derefs

Browse Source 
I spotted two places where under stress (many explosions) luajit would
end up passing nil to these functions. I'm not entirely sure how,
but it seems good form to guard against it, which does make it
more robust. After this patch, I'm not able to crash the server. With
many explosions, it may still lag significantly, but always returns
in the end.
This commit is contained in:
Auke Kok 2016-03-28 18:20:14 -07:00 committed by paramat
parent 802bc1edca
commit 52dd11fd83
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
init.lua
View File

@ -51,9 +51,9 @@ local function eject_drops(drops, pos, radius)
local count = item:get_count()
local take_est = math.log(count * count) + math.random(0,4) - 2
while count > 0 do
local take = math.min(take_est,
local take = math.max(1,math.min(take_est,
item:get_count(),
item:get_stack_max())
item:get_stack_max()))
rand_pos(pos, drop_pos, radius)
local obj = minetest.add_item(drop_pos, item:get_name() .. " " .. take)
if obj then
@ -165,7 +165,10 @@ local function add_effects(pos, radius, drops)
local count = stack:get_count()
if count > most then
most = count
texture = minetest.registered_nodes[name].tiles[1]
local def = minetest.registered_nodes[name]
if def and def.tiles and def.tiles[1] then
texture = def.tiles[1]
end
end
end