Anonymous | Login | Signup for a new account | 2025-07-27 04:20 UTC | ![]() |
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0001841 | Zandronum | [All Projects] Bug | public | 2014-06-16 17:09 | 2018-09-30 23:08 | ||||
Reporter | Zalewa | ||||||||
Assigned To | Torr Samaho | ||||||||
Priority | normal | Severity | major | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | Microsoft | OS | Windows | OS Version | XP/Vista/7 | ||||
Product Version | 2.0-beta | ||||||||
Target Version | 2.0 | Fixed in Version | 2.0 | ||||||
Summary | 0001841: Zandronum 2.0 r140112-1124: Memory leak and crash in offline Survival on MAPINFO/monsters | ||||||||
Description | This is a memory leak in a something that looks like an infinite loop and it eventually leads to a crash. The provided WAD has two key elements: MAPINFO and 2 monsters. Although I didn't pinpoint the exact reason for the crash, I know of two alternative ways that this WAD can be fixed so that it won't cause the crash anymore: a) Remove one of the monsters from the map so that only one remains. b) Remove MAPINFO lump from the WAD. | ||||||||
Steps To Reproduce | 1. Load attached WAD in Zandronum 2.0 r140112-1124 (offline mode). 2. Type "survival 1" in console. 3. Type "map map03" in console. 4. Wait until countdown finishes, don't do anything. 5. Game should freeze now. | ||||||||
Additional Information | This was originally discovered in a WAD called "ph_jstc.wad" by James "Phobus" Cresswell. I'm unable to find the original WAD on the web at the moment, although I managed to strip my version to the minimum. Crash also happens when hosted on a server in which case it kicks the client and then kills the server. | ||||||||
Attached Files | ![]() ![]() | ||||||||
![]() |
||||||
|
![]() |
|
Dusk (developer) 2014-06-16 17:41 |
Quote |
Dusk (developer) 2014-06-16 17:54 edited on: 2014-06-16 17:59 |
The crash happens during the map reset while trying to spawn one of the pinkies.
|
Edward-san (developer) 2014-06-16 18:45 |
Yeah, the game is stuck inside the actor reloading loop inside GAME_ResetMap, as if there is an infinitely number of actors to iterate via ActorIterator. |
Torr Samaho (administrator) 2014-11-23 14:33 |
Interesting. In 2.0, TThinkerIterator<AActor> doesn't seem to like if we create new actors while iterating over all actors in the example wad. This should fix the issue. |
Arco (updater) 2014-11-24 14:12 edited on: 2014-11-24 19:18 |
Issue fixed in r141117-2018. EDIT:
|
Dusk (developer) 2014-11-24 19:16 |
It seems this crash is back.
|
Torr Samaho (administrator) 2014-11-24 19:21 |
I didn't commit the fix anywhere since it's still experimental. The only binary that contains the fix is the Windows binary I posted above, so it's not surprising if a build that you compiled yourself crashes. |
Torr Samaho (administrator) 2014-11-30 20:50 |
I attached the diff of the patch so that this can be tested under Linux. |
Torr Samaho (administrator) 2014-12-31 11:15 |
Since apparently nobody is going to test the diff I posted a month ago, I pushed the fix without further testing. |
Edward-san (developer) 2014-12-31 14:27 |
Whoops, sorry for the missing feedback... it works with that patch in linux, too. |
Dusk (developer) 2015-01-12 06:39 |
Works fine for me too. |
This issue is already marked as resolved. If you feel that is not the case, please reopen it and explain why. |
|
Supporters: | WaTaKiD |
Opponents: | No one explicitly opposes this issue yet. |
![]() |
|||
Date Modified | Username | Field | Change |
2014-06-16 17:09 | Zalewa | New Issue | |
2014-06-16 17:09 | Zalewa | File Added: memory_leak_crash_survival_zan_2.0.wad | |
2014-06-16 17:10 | Zalewa | Additional Information Updated | View Revisions |
2014-06-16 17:16 | Zalewa | Description Updated | View Revisions |
2014-06-16 17:16 | Zalewa | Additional Information Updated | View Revisions |
2014-06-16 17:41 | Dusk | Note Added: 0009523 | |
2014-06-16 17:41 | Dusk | Status | new => confirmed |
2014-06-16 17:54 | Dusk | Note Added: 0009525 | |
2014-06-16 17:59 | Dusk | Note Edited: 0009525 | View Revisions |
2014-06-16 18:45 | Edward-san | Note Added: 0009526 | |
2014-11-03 21:27 | Dusk | Product Version | 2.0 => 2.0-beta |
2014-11-03 21:30 | Dusk | Relationship added | has duplicate 0001979 |
2014-11-23 14:33 | Torr Samaho | Note Added: 0010942 | |
2014-11-23 14:33 | Torr Samaho | Assigned To | => Torr Samaho |
2014-11-23 14:33 | Torr Samaho | Status | confirmed => needs testing |
2014-11-24 14:12 | Arco | Note Added: 0010952 | |
2014-11-24 14:12 | Arco | Status | needs testing => resolved |
2014-11-24 14:12 | Arco | Resolution | open => fixed |
2014-11-24 14:12 | Arco | Fixed in Version | => 2.0 |
2014-11-24 14:12 | Arco | Target Version | => 2.0 |
2014-11-24 19:16 | Dusk | Note Added: 0010955 | |
2014-11-24 19:16 | Dusk | Status | resolved => feedback |
2014-11-24 19:16 | Dusk | Resolution | fixed => reopened |
2014-11-24 19:18 | Arco | Note Edited: 0010952 | View Revisions |
2014-11-24 19:21 | Torr Samaho | Note Added: 0010958 | |
2014-11-24 22:46 | Dusk | Status | feedback => assigned |
2014-11-24 22:46 | Dusk | Fixed in Version | 2.0 => |
2014-11-30 20:49 | Torr Samaho | File Added: ResetFreezeFix | |
2014-11-30 20:50 | Torr Samaho | Note Added: 0011014 | |
2014-12-31 11:15 | Torr Samaho | Note Added: 0011147 | |
2014-12-31 11:16 | Torr Samaho | Status | assigned => needs testing |
2014-12-31 14:27 | Edward-san | Note Added: 0011154 | |
2015-01-12 06:39 | Dusk | Note Added: 0011367 | |
2015-01-12 06:39 | Dusk | Status | needs testing => resolved |
2015-01-12 06:39 | Dusk | Fixed in Version | => 2.0 |
2015-01-12 06:39 | Dusk | Resolution | reopened => fixed |
2018-09-30 23:08 | Blzut3 | Status | resolved => closed |
Copyright © 2000 - 2025 MantisBT Team |