From ff387a952c15bb616408716d0c31dee54b3deb90 Mon Sep 17 00:00:00 2001 From: Randy Heit Date: Thu, 13 Mar 2008 00:45:35 +0000 Subject: [PATCH] - Fixed: D_dehacked.cpp/PatchThing() allocated an actor on the stack. SVN r800 (trunk) --- docs/rh-log.txt | 1 + src/d_dehacked.cpp | 9 +++++---- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/docs/rh-log.txt b/docs/rh-log.txt index 628663bf89..52cd6b4667 100644 --- a/docs/rh-log.txt +++ b/docs/rh-log.txt @@ -1,4 +1,5 @@ March 12, 2008 +- Fixed: D_dehacked.cpp/PatchThing() allocated an actor on the stack. - Changed the sentinels in the thinker lists into a proper thinker. The old way wasn't playing well with the write barriers. - Fixed: DFrameBuffer::WriteSavePic needs to fix the canvas in place while diff --git a/src/d_dehacked.cpp b/src/d_dehacked.cpp index 071e2a5ea1..8e0f4972f3 100644 --- a/src/d_dehacked.cpp +++ b/src/d_dehacked.cpp @@ -699,7 +699,8 @@ static int PatchThing (int thingy) }; int result; - AActor *info, dummy; + AActor *info; + BYTE dummy[sizeof(AActor)]; bool hadHeight = false; bool hadTranslucency = false; bool hadStyle = false; @@ -709,7 +710,7 @@ static int PatchThing (int thingy) SWORD *ednum, dummyed; type = NULL; - info = &dummy; + info = (AActor *)&dummy; ednum = &dummyed; if (thingy > NumInfos || thingy <= 0) { @@ -723,7 +724,7 @@ static int PatchThing (int thingy) type = PClass::FindClass (GetName (InfoNames[thingy - 1])); if (type == NULL) { - info = &dummy; + info = (AActor *)&dummy; ednum = &dummyed; Printf ("Could not find thing %s (index %d)\n", GetName (InfoNames[thingy - 1]), thingy); @@ -1024,7 +1025,7 @@ static int PatchThing (int thingy) else Printf (unknown_str, Line1, "Thing", thingy); } - if (info != &dummy) + if (info != (AActor *)&dummy) { // Reset heights for things hanging from the ceiling that // don't specify a new height.