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

Mauller · helmutbuhler · Mauller · commit 095f6a352a27 · 2025-04-27T07:44:00.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,8 @@ void StealthUpdate::changeVisualDisguise()
 
 		const ThingTemplate *tTemplate = self->getTemplate();
 
-		TheThingFactory->newDrawable( tTemplate );
+		// TheSuperHackers @bugfix helmutbuhler 13/04/2025 Draw was not being assigned, potentially leaking memory and causing invalid behaviour
+		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,8 @@ void StealthUpdate::changeVisualDisguise()
 
 		const ThingTemplate *tTemplate = self->getTemplate();
 
-		TheThingFactory->newDrawable( tTemplate );
+		// TheSuperHackers @bugfix helmutbuhler 13/04/2025 Draw was not being assigned, potentially leaking memory and causing invalid behaviour
+		draw = TheThingFactory->newDrawable( tTemplate );
 		if( draw )
 		{
 			TheGameLogic->bindObjectAndDrawable(self, draw);

Original file line number	Diff line number	Diff line change
`@@ -755,7 +755,8 @@ 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 Draw was not being assigned, potentially leaking memory and causing invalid behaviour`
	`759`	`+ draw = TheThingFactory->newDrawable( tTemplate );`
`759`	`760`	`if( draw )`
`760`	`761`	`{`
`761`	`762`	`TheGameLogic->bindObjectAndDrawable(self, draw);`
Original file line number	Diff line number	Diff line change
`@@ -1055,7 +1055,8 @@ 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 Draw was not being assigned, potentially leaking memory and causing invalid behaviour`
	`1059`	`+ draw = TheThingFactory->newDrawable( tTemplate );`
`1059`	`1060`	`if( draw )`
`1060`	`1061`	`{`
`1061`	`1062`	`TheGameLogic->bindObjectAndDrawable(self, draw);`