EDIT: Bugfix PR.
Download the test.pk3 attached. Give SMM in console. A few seconds later, crash.
[Fixed] [Fix inside]Camera + Morph = Crash
Moderator: GZDoom Developers
23 posts
• Page 1 of 2 • 1, 2
[Fix inside]Camera + Morph = Crash
by Major Cooke » Sat Aug 13, 2016 4:23 pm
- Attachments
-
playermorphcrash.zip- Give SMM in console.
- (6.45 KiB) Downloaded 33 times
Last edited by Major Cooke on Thu Sep 01, 2016 1:08 pm, edited 8 times in total.
-
Major Cooke - QZDoom Maintenance Team
- Joined: 28 Jan 2007
Re: Odd unmorph crashing bug
by Edward-san » Sat Aug 13, 2016 4:37 pm
it would be interesting to know what are the values of 'type' and, if it's not null, 'i' and 'type->Slot[i]'..
- Edward-san
- Mathematics is the language with which God has written the universe. (Galilei)
- Joined: 17 Oct 2009
Re: Camera + Morph = Crash
by Major Cooke » Mon Aug 15, 2016 8:41 am
I have managed to reproduce it on a much simpler scale. Attached in the first post is a test example.
"Give SMM" in console. Don't mind the fact that you cannot move.
A couple seconds later, the game crashes from the morph expiring.
"Give SMM" in console. Don't mind the fact that you cannot move.
A couple seconds later, the game crashes from the morph expiring.
-
Major Cooke - QZDoom Maintenance Team
- Joined: 28 Jan 2007
Re: Camera + Morph = Crash
by Graf Zahl » Mon Aug 15, 2016 9:02 am
Still very much 'ugh'.
For some reason it encounters a player class object that isn't fully initialized and crashes on the garbage data.
The big problem is, that cause and effect are in completely separate parts of the code.
For some reason it encounters a player class object that isn't fully initialized and crashes on the garbage data.
The big problem is, that cause and effect are in completely separate parts of the code.
-
Graf Zahl - Lead GZDoom+Raze Developer
- Joined: 19 Jul 2003
- Location: Germany
Re: Camera + Morph = Crash
by Major Cooke » Mon Aug 15, 2016 12:31 pm
Youch. Yeah, it is weird when it happens especially because the player's view is just outside of the actor. Hmmm...
-
Major Cooke - QZDoom Maintenance Team
- Joined: 28 Jan 2007
Re: Camera + Morph = Crash
by Edward-san » Mon Aug 15, 2016 3:30 pm
Regarding the original problem, I got this asan error with the wad:
and this is gdb backtrace:
Incidentally, I got another asan error when I just got the morphing item and then directly removed the powerup (no camera switch):
and this is the backtrace from gdb:
with these messages from the log before the crash:
I wonder what's going on...
Spoiler:
and this is gdb backtrace:
Spoiler:
Incidentally, I got another asan error when I just got the morphing item and then directly removed the powerup (no camera switch):
Spoiler:
and this is the backtrace from gdb:
Spoiler:
with these messages from the log before the crash:
- Code: Select all • Expand view
user_lastHP is not a user variable in class DoomPlayer
user_camang is not a user variable in class DoomPlayer
user_dirget is not a user variable in class DoomPlayer
user_dirforward is not a user variable in class DoomPlayer
user_dirside is not a user variable in class DoomPlayer
I wonder what's going on...
- Edward-san
- Mathematics is the language with which God has written the universe. (Galilei)
- Joined: 17 Oct 2009
Re: Camera + Morph = Crash
by Major Cooke » Mon Aug 15, 2016 3:35 pm
Try the new test.pk3.
Now, if you're wondering, overlays are not cleared when unmorphing. Something I told Leonard about but he said they'll be fixed when Actor Overlays are implemented, he said. But as you can see from the new test.pk3, there's no overlays at all.
Now, if you're wondering, overlays are not cleared when unmorphing. Something I told Leonard about but he said they'll be fixed when Actor Overlays are implemented, he said. But as you can see from the new test.pk3, there's no overlays at all.
-
Major Cooke - QZDoom Maintenance Team
- Joined: 28 Jan 2007
Re: Camera + Morph = Crash
by Edward-san » Mon Aug 15, 2016 4:00 pm
urgh please use a different name to the wad instead of the repetitive 'test'. Anyways, I see no change either in gdb and asan output.
- Edward-san
- Mathematics is the language with which God has written the universe. (Galilei)
- Joined: 17 Oct 2009
Re: Camera + Morph = Crash
by Major Cooke » Sun Aug 21, 2016 7:21 am
Meh.
Anyway, anything else I can try to do for this? Or is this something randi has to look into?
Anyway, anything else I can try to do for this? Or is this something randi has to look into?
-
Major Cooke - QZDoom Maintenance Team
- Joined: 28 Jan 2007
Re: Camera + Morph = Crash
by Edward-san » Sun Aug 21, 2016 1:28 pm
Can you add a test wad which does just morph and unmorph? In a post above, I mentioned a problem with those steps and would like to check it again.
- Edward-san
- Mathematics is the language with which God has written the universe. (Galilei)
- Joined: 17 Oct 2009
Re: Camera + Morph = Crash
by Major Cooke » Thu Aug 25, 2016 12:34 pm
Okay, first post updated. User variables removed.
-
Major Cooke - QZDoom Maintenance Team
- Joined: 28 Jan 2007
Re: Camera + Morph = Crash
by Edward-san » Wed Aug 31, 2016 2:35 pm
It seems that this line in decorate.txt:
makes sure that the morphed actor's tracer, which points to the actor before morphing, is overridden with the camera:
from InitSpawnedItem, called by A_SpawnItemEx.
The old actor's information is lost, making the program crash when attempting to unmorph, because it's trying to get the player's info from invalid actor.
In order to fix this, I suspect the old actor pointer should be stored in a different place than the 'tracer'. Graf?
- Code: Select all • Expand view
A_SpawnItemEx("ChaseCam",-10,0,32,0,0,0,0,SXF_NOCHECKPOSITION|SXF_SETMASTER|SXF_ISTRACER,0,32700);
makes sure that the morphed actor's tracer, which points to the actor before morphing, is overridden with the camera:
- Code: Select all • Expand view
if (flags & SIXF_ISTRACER)
{
self->tracer = mo;
}
from InitSpawnedItem, called by A_SpawnItemEx.
The old actor's information is lost, making the program crash when attempting to unmorph, because it's trying to get the player's info from invalid actor.
In order to fix this, I suspect the old actor pointer should be stored in a different place than the 'tracer'. Graf?
- Edward-san
- Mathematics is the language with which God has written the universe. (Galilei)
- Joined: 17 Oct 2009
Re: Camera + Morph = Crash
by Major Cooke » Wed Aug 31, 2016 2:44 pm
I tested it again just to make sure:
This doesn't crash, backing up Edward's words possibly.
- Code: Select all • Expand view
Spawn:
PLAY A 95 NoDelay
{
SetPlayerProperty(0,1,PROP_TOTALLYFROZEN);
A_PrintBold("Camera On");
A_SpawnItemEx("ChaseCam",-10,0,32,0,0,0,0,SXF_NOCHECKPOSITION|SXF_SETMASTER|SXF_ISTRACER,0,32700);
ChangeCamera(32700,0,0);
}
PLAY A 0
{
A_RemoveChildren(true,RMVF_EVERYTHING,"ChaseCam");
}
This doesn't crash, backing up Edward's words possibly.
-
Major Cooke - QZDoom Maintenance Team
- Joined: 28 Jan 2007
Re: Camera + Morph = Crash
by Graf Zahl » Wed Aug 31, 2016 2:49 pm
Edward-san wrote:In order to fix this, I suspect the old actor pointer should be stored in a different place than the 'tracer'. Graf?
Correct. Since it's no longer safe to use tracer for this, a different variable is needed.
-
Graf Zahl - Lead GZDoom+Raze Developer
- Joined: 19 Jul 2003
- Location: Germany
Re: Camera + Morph = Crash
by Edward-san » Wed Aug 31, 2016 2:53 pm
I hope there's nothing which (ab)used the morphed tracer data...
- Edward-san
- Mathematics is the language with which God has written the universe. (Galilei)
- Joined: 17 Oct 2009
23 posts
• Page 1 of 2 • 1, 2
Who is online
Users browsing this forum: No registered users and 0 guests