MantisBT - Zandronum
View Issue Details
0002483Zandronum[All Projects] Bugpublic2015-10-07 19:522018-09-30 22:17
Ru5tK1ng 
Torr Samaho 
highcrashalways
closedfixed 
MicrosoftWindowsXP/Vista/7
3.0-beta 
3.03.0 
0002483: Fatal Crash in LMS gamemode with DeHackED lump
Zandronum will completely crash when switching to LastManStanding using a wadfile that has a DeHackED lump present. Zandronum will crash offline and online the clients and the server will both crash. If the lump is removed, then the crash does not occur and everything runs normally.
1. Load either test wad provided below
2. Set 'lastmanstanding 1' in the console
3. Change the map
4. Zan will crash if offline. If performed online, a player or bot must join the map to invoke the crash
Links to test wads:
'http://www.best-ever.org/download?file=cybercrime3.wad [^]'
'http://www.best-ever.org/download?file=eondm1.wad [^]'
No tags attached.
Issue History
2015-10-07 19:52Ru5tK1ngNew Issue
2015-10-10 09:13Torr SamahoAssigned To => Torr Samaho
2015-10-10 09:13Torr SamahoStatusnew => assigned
2015-10-10 11:06Edward-sanNote Added: 0013642
2015-10-10 13:51cobaltStatusassigned => needs testing
2015-10-10 13:51cobaltSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=8121#r8121
2015-10-10 13:51cobaltAdditional Information Updatedbug_revision_view_page.php?rev_id=8123#r8123
2015-10-10 13:51cobaltNote Added: 0013643
2015-10-10 13:57Torr SamahoNote Added: 0013644
2015-11-09 23:08Ru5tK1ngNote Added: 0013769
2015-11-09 23:09Ru5tK1ngStatusneeds testing => resolved
2015-11-09 23:09Ru5tK1ngResolutionopen => fixed
2015-11-09 23:09Ru5tK1ngFixed in Version => 3.0
2015-11-09 23:09Ru5tK1ngSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=8207#r8207
2015-11-09 23:09Ru5tK1ngAdditional Information Updatedbug_revision_view_page.php?rev_id=8208#r8208
2018-09-30 22:17Blzut3Statusresolved => closed

Notes
(0013642)
Edward-san   
2015-10-10 11:06   
Backporting this zdoom commit fixes the crash, but it's still not clear why this happens.
(0013643)
cobalt   
2015-10-10 13:51   
Issue addressed by commit aabfd89a0e67: - added NULL pointer checks to AActor::RemoveInventory to prevent items that destroy their owner in their use state from crashing the engine.
Committed by Christoph Oelckers on Thursday 26 February 2015 12:13:17

Changes in files:

 src/p_mobj.cpp | 23 +++++++++++++----------
 1 files changed, 13 insertions(+), 10 deletions(-)

(0013644)
Torr Samaho   
2015-10-10 13:57   
Backported.
(0013769)
Ru5tK1ng   
2015-11-09 23:08   
I could not reproduce the crash online or offline with 3.0 151108-1238. Seems to be fixed.