[GEN][ZH] Fix drawable assignment in StealthUpdate::changeVisualDisguise()

Mauller · helmutbuhler · Mauller · commit 98af79cbb1f5 · 2025-04-26T08:11:28.000+01:00
Co-authored-by: Helmut Buhler &lt;buhler@8gadgetpack.net&gt;
diff --git a/Generals/Code/GameEngine/Source/GameLogic/Object/Update/StealthUpdate.cpp b/Generals/Code/GameEngine/Source/GameLogic/Object/Update/StealthUpdate.cpp
@@ -755,7 +755,9 @@ void StealthUpdate::changeVisualDisguise()
 
 		const ThingTemplate *tTemplate = self->getTemplate();
 
-		TheThingFactory->newDrawable( tTemplate );
+		// TheSuperHackers @bugfix helmutbuhler 13/04/2025
+		// draw was originally not assigned here and potentially caused memory corruption
+		draw = TheThingFactory->newDrawable( tTemplate );
 		if( draw )
 		{
 			TheGameLogic->bindObjectAndDrawable(self, draw);
diff --git a/GeneralsMD/Code/GameEngine/Source/GameLogic/Object/Update/StealthUpdate.cpp b/GeneralsMD/Code/GameEngine/Source/GameLogic/Object/Update/StealthUpdate.cpp
@@ -1055,7 +1055,9 @@ void StealthUpdate::changeVisualDisguise()
 
 		const ThingTemplate *tTemplate = self->getTemplate();
 
-		TheThingFactory->newDrawable( tTemplate );
+		// TheSuperHackers @bugfix helmutbuhler 13/04/2025
+		// draw was originally not assigned here and potentially caused memory corruption
+		draw = TheThingFactory->newDrawable( tTemplate );
 		if( draw )
 		{
 			TheGameLogic->bindObjectAndDrawable(self, draw);

Original file line number	Diff line number	Diff line change
`@@ -755,7 +755,9 @@ void StealthUpdate::changeVisualDisguise()`
`755`	`755`
`756`	`756`	`const ThingTemplate *tTemplate = self->getTemplate();`
`757`	`757`
`758`		`- TheThingFactory->newDrawable( tTemplate );`
	`758`	`+ // TheSuperHackers @bugfix helmutbuhler 13/04/2025`
	`759`	`+ // draw was originally not assigned here and potentially caused memory corruption`
	`760`	`+ draw = TheThingFactory->newDrawable( tTemplate );`
`759`	`761`	`if( draw )`
`760`	`762`	`{`
`761`	`763`	`TheGameLogic->bindObjectAndDrawable(self, draw);`
Original file line number	Diff line number	Diff line change
`@@ -1055,7 +1055,9 @@ void StealthUpdate::changeVisualDisguise()`
`1055`	`1055`
`1056`	`1056`	`const ThingTemplate *tTemplate = self->getTemplate();`
`1057`	`1057`
`1058`		`- TheThingFactory->newDrawable( tTemplate );`
	`1058`	`+ // TheSuperHackers @bugfix helmutbuhler 13/04/2025`
	`1059`	`+ // draw was originally not assigned here and potentially caused memory corruption`
	`1060`	`+ draw = TheThingFactory->newDrawable( tTemplate );`
`1059`	`1061`	`if( draw )`
`1060`	`1062`	`{`
`1061`	`1063`	`TheGameLogic->bindObjectAndDrawable(self, draw);`