MantisBT - Zandronum
View Issue Details
0001247Zandronum[All Projects] Bugpublic2013-01-12 13:342018-09-30 23:07
Zalewa 
Torr Samaho 
normalcrashsometimes
closedfixed 
Windows7
1.0 
2.02.0 
0001247: DEHACKED with no trailing new line in a WAD causes game to crash on start
If DEHACKED lump is present in a WAD, and this lump ends with a "Text" patch, and this "Text" patch doesn't end with a trailing new line then the game is almost certain to crash. The crash doesn't occur all the time, but it occurs on most attempts to start the game.

I'm uploading the offending WAD and a patch that fixes the problem. I'm not sure if the change made by the patch is the correct way to fix the issue but the game doesn't crash anymore.

This problem occurs on Zandronum 1.0 and on changeset "3612 (a0f56d2944ca)" from the bitbucket repo (https://bitbucket.org/Torr_Samaho/zandronum/commits/a0f56d2944ca49657ca9e7526fd1d3218f38019c)
I have AMD CPU.
dehacked
child of 0001490closed Torr Samaho Backport ZDoom 2.5.0 
? dehacked-offender.wad (5,102) 2013-01-12 13:34
/tracker/file_download.php?file_id=915&type=bug
txt dehacked-lump-patch.txt (403) 2013-01-12 13:34
/tracker/file_download.php?file_id=916&type=bug
Issue History
2013-01-12 13:34ZalewaNew Issue
2013-01-12 13:34ZalewaFile Added: dehacked-offender.wad
2013-01-12 13:34ZalewaFile Added: dehacked-lump-patch.txt
2013-01-12 13:39ZalewaDescription Updatedbug_revision_view_page.php?rev_id=3154#r3154
2013-01-12 13:39ZalewaDescription Updatedbug_revision_view_page.php?rev_id=3155#r3155
2013-01-12 13:53ZalewaDescription Updatedbug_revision_view_page.php?rev_id=3156#r3156
2013-01-12 14:01ZalewaOS => Windows
2013-01-12 14:01ZalewaOS Version => 7
2013-01-12 14:01ZalewaAdditional Information Updatedbug_revision_view_page.php?rev_id=3158#r3158
2013-01-12 19:47ZalewaProduct Version => 1.0
2013-01-12 20:17Torr SamahoNote Added: 0005738
2013-01-12 20:18Torr SamahoStatusnew => feedback
2013-01-12 20:27ZalewaNote Added: 0005741
2013-01-12 20:27ZalewaStatusfeedback => new
2013-01-12 20:34ZalewaNote Edited: 0005741bug_revision_view_page.php?bugnote_id=5741#r3160
2013-01-13 09:53Torr SamahoNote Added: 0005749
2013-01-13 09:55Torr SamahoStatusnew => feedback
2015-01-15 17:56DuskNote Added: 0011385
2015-01-15 17:56DuskAssigned To => Torr Samaho
2015-01-15 17:56DuskStatusfeedback => needs testing
2015-01-15 17:56DuskRelationship addedchild of 0001490
2015-01-15 17:57DuskTarget Version => 2.0
2015-01-15 17:57Edward-sanNote Added: 0011386
2015-01-15 17:57Edward-sanStatusneeds testing => needs review
2015-01-15 17:57Edward-sanStatusneeds review => needs testing
2015-01-15 17:58Edward-sanNote Deleted: 0011386
2015-01-22 20:09WaTaKiDNote Added: 0011504
2015-01-22 22:35ArcoStatusneeds testing => resolved
2015-01-22 22:35ArcoFixed in Version => 2.0
2015-01-22 22:35ArcoResolutionopen => fixed
2018-09-30 23:07Blzut3Statusresolved => closed

Notes
(0005738)
Torr Samaho   
2013-01-12 20:17   
Is GZDoom 323 having the same problem? If so, is the latest ZDoom version also affected?
(0005741)
Zalewa   
2013-01-12 20:27   
(edited on: 2013-01-12 20:34)
I managed to crash GZDoom once in several attempts. ZDoom 2.6.1 doesn't want to crash but I may just be lucky. After all, Zandronum doesn't crash always either, but it crashes usually. Running ZDoom in debugger would probably tell us for sure but I don't have that project set up to check it easily.

(0005749)
Torr Samaho   
2013-01-13 09:53   
After skimming through ZDoom SVN history I think ZDoom fixed this in revision 2035. Unfortunately their fix is intertwined with other changes to the parser made in earlier revisions, so I can't backport it as is.
(0011385)
Dusk   
2015-01-15 17:56   
r2035 should be ported into 2.0 by now.
(0011504)
WaTaKiD   
2015-01-22 20:09   
after testing with 2.0-r150119-2001, issue seems fixed