[GEN][ZH] Fix setting debugDisplay to NULL and releasing benchmarkDisplayString in W3DDisplay destructor

Author: Mauller

Generals/Code/GameEngineDevice/Source/W3DDevice/GameClient/W3DDisplay.cpp

@@ -376,11 +376,18 @@ W3DDisplay::~W3DDisplay()
 
 	// get rid of the debug display
 	delete m_debugDisplay;
+	m_debugDisplay = NULL;
+	m_nativeDebugDisplay = NULL;
 
 	// delete the display strings
 	for (int i = 0; i < DisplayStringCount; i++)
 		TheDisplayStringManager->freeDisplayString(m_displayStrings[i]);
 
+	// TheSuperHackers @fix Mauller/Tomsons26 28/04/2025 Delete benchmark display string to allow the clean destruction of the DisplayStringManager
+	if( m_benchmarkDisplayString ) {
+		TheDisplayStringManager->freeDisplayString(m_benchmarkDisplayString);
+	}
+
 	// delete 2D renderer
 	if( m_2DRender )
 	{

GeneralsMD/Code/GameEngineDevice/Source/W3DDevice/GameClient/W3DDisplay.cpp

@@ -426,11 +426,18 @@ W3DDisplay::~W3DDisplay()
 
 	// get rid of the debug display
 	delete m_debugDisplay;
+	m_debugDisplay = NULL;
+	m_nativeDebugDisplay = NULL;
 
 	// delete the display strings
 	for (int i = 0; i < DisplayStringCount; i++)
 		TheDisplayStringManager->freeDisplayString(m_displayStrings[i]);
 
+	// TheSuperHackers @fix Mauller/Tomsons26 28/04/2025 Delete benchmark display string to allow the clean destruction of the DisplayStringManager
+	if( m_benchmarkDisplayString ) {
+		TheDisplayStringManager->freeDisplayString(m_benchmarkDisplayString);
+	}
+
 	// delete 2D renderer
 	if( m_2DRender )
 	{