HTTP: Prevent use-after-free in HTTPRequest callback during shutdown

GeneralsMD/Code/GameEngine/Include/GameNetwork/GeneralsOnline/HTTP/HTTPRequest.h

@@ -59,6 +59,9 @@ class HTTPRequest
 
 	std::string GetURI() { return m_strURI; }
 
+	void SetSkipCallback(bool bSkip) { m_bSkipCallback = bSkip; }
+
+
 private:
 	void PlatformStartRequest();
 
@@ -91,10 +94,12 @@ class HTTPRequest
 	bool m_bNeedsProgressUpdate = false;
 	bool m_bIsStarted = false;
 	bool m_bIsComplete = false;
+	bool m_bSkipCallback = false;
+
 
 	struct curl_slist* headers = nullptr;
 
 	std::function<void(bool bSuccess, int statusCode, std::string strBody, HTTPRequest* pReq)> m_completionCallback = nullptr;
 
 	std::function<void(size_t bytesReceived)> m_progressCallback = nullptr;
-};
+};

GeneralsMD/Code/GameEngine/Source/GameNetwork/GeneralsOnline/HTTP/HTTPManager.cpp

@@ -91,6 +91,8 @@ void HTTPManager::Shutdown()
 						HTTPRequest* pRequest = *it;
 						if (pRequest != nullptr && pRequest->EasyHandleMatches(pCurlHandle))
 						{
+							// Skip callback invocation during shutdown to prevent use-after-free
+							pRequest->SetSkipCallback(true);
 							pRequest->Threaded_SetComplete(m->data.result);
 							delete pRequest;
 							m_vecRequestsInFlight.erase(it);

GeneralsMD/Code/GameEngine/Source/GameNetwork/GeneralsOnline/HTTP/HTTPRequest.cpp

@@ -92,6 +92,12 @@ void HTTPRequest::InvokeCallbackIfComplete()
 {
 	if (m_bIsComplete)
 	{
+		// Skip callback invocation if we're shutting down to avoid use-after-free
+		if (m_bSkipCallback)
+		{
+			return;
+		}
+
 		if (m_completionCallback != nullptr)
 		{
 			// Convert m_vecBuffer to std::string for m_strResponse