Log and advert crash in CCore::OnPostCreateDevice

Author: Dutchman101

Client/core/DXHook/CProxyDirect3D9.cpp

@@ -929,6 +929,30 @@ HRESULT CCore::OnPostCreateDevice(HRESULT hResult, IDirect3D9* pDirect3D, UINT A
     if (!UsingAltD3DSetup())
         return D3D_OK;
 
+    if (!ppReturnedDeviceInterface)
+    {
+        AddReportLog(8744, SString("CCore::OnPostCreateDevice - missing ppReturnedDeviceInterface pointer"));
+        return hResult;
+    }
+
+    if (!SharedUtil::IsReadablePointer(ppReturnedDeviceInterface, sizeof(*ppReturnedDeviceInterface)))
+    {
+        AddReportLog(8745, SString("CCore::OnPostCreateDevice - invalid ppReturnedDeviceInterface pointer %p", ppReturnedDeviceInterface));
+        return hResult;
+    }
+
+    if (!*ppReturnedDeviceInterface)
+    {
+        AddReportLog(8746, SString("CCore::OnPostCreateDevice - ppReturnedDeviceInterface dereferenced to nullptr"));
+        return hResult;
+    }
+
+    if (!SharedUtil::IsReadablePointer(*ppReturnedDeviceInterface, sizeof(void*)))
+    {
+        AddReportLog(8747, SString("CCore::OnPostCreateDevice - invalid IDirect3DDevice9 pointer %p (via %p)", *ppReturnedDeviceInterface, ppReturnedDeviceInterface));
+        return hResult;
+    }
+
     //
     // - Allow create device with no changes
     // - Check caps and report diff with GTA caps

Client/multiplayer_sa/CMultiplayerSA_Direct3D.cpp

@@ -20,6 +20,7 @@ namespace
     DWORD                  ms_BehaviorFlags = 0;
     D3DPRESENT_PARAMETERS* ms_pPresentationParameters = NULL;
     IDirect3DDevice9**     ms_ppReturnedDeviceInterface = NULL;
+    bool                   ms_hasDeviceArgs = false;
 }            // namespace
 
 DWORD RESTORE_Addr_PreCreateDevice;
@@ -45,7 +46,7 @@ void _cdecl OnPreCreateDevice(IDirect3D9* pDirect3D, UINT Adapter, D3DDEVTYPE De
     }
 
     // Validate critical parameters before dereferencing
-    if (BehaviorFlags && pPresentationParameters)
+    if (BehaviorFlags && pPresentationParameters && ppReturnedDeviceInterface)
     {
         ms_pDirect3D = pDirect3D;
         ms_Adapter = Adapter;
@@ -54,6 +55,12 @@ void _cdecl OnPreCreateDevice(IDirect3D9* pDirect3D, UINT Adapter, D3DDEVTYPE De
         ms_BehaviorFlags = *BehaviorFlags;
         ms_pPresentationParameters = pPresentationParameters;
         ms_ppReturnedDeviceInterface = ppReturnedDeviceInterface;
+        ms_hasDeviceArgs = true;
+    }
+    else
+    {
+        ms_hasDeviceArgs = false;
+        AddReportLog(8740, SString("OnPreCreateDevice: missing device arguments for alt startup path"));
     }
 }
 
@@ -114,12 +121,36 @@ static void __declspec(naked) HOOK_PreCreateDevice()
 ////////////////////////////////////////////////////////////////
 HRESULT _cdecl OnPostCreateDevice(HRESULT hResult)
 {
-    if (g_pCore)
+    if (!g_pCore)
+    {
+        ms_hasDeviceArgs = false;
+        return hResult;
+    }
+
+    if (!ms_hasDeviceArgs)
+    {
+        AddReportLog(8741, SString("OnPostCreateDevice: device arguments were not captured; skipping alt startup logic"));
+        return hResult;
+    }
+
+    if (!SharedUtil::IsReadablePointer(ms_ppReturnedDeviceInterface, sizeof(*ms_ppReturnedDeviceInterface)))
+    {
+        AddReportLog(8742, SString("OnPostCreateDevice: invalid device pointer reference %p", ms_ppReturnedDeviceInterface));
+        ms_hasDeviceArgs = false;
+        return hResult;
+    }
+
+    if (!SharedUtil::IsReadablePointer(ms_pPresentationParameters, sizeof(*ms_pPresentationParameters)))
     {
-        return g_pCore->OnPostCreateDevice(hResult, ms_pDirect3D, ms_Adapter, ms_DeviceType, ms_hFocusWindow, ms_BehaviorFlags, ms_pPresentationParameters,
-                                           ms_ppReturnedDeviceInterface);
+        AddReportLog(8743, SString("OnPostCreateDevice: invalid presentation parameters pointer %p", ms_pPresentationParameters));
+        ms_hasDeviceArgs = false;
+        return hResult;
     }
-    return hResult;
+
+    HRESULT result = g_pCore->OnPostCreateDevice(hResult, ms_pDirect3D, ms_Adapter, ms_DeviceType, ms_hFocusWindow, ms_BehaviorFlags, ms_pPresentationParameters,
+                                                 ms_ppReturnedDeviceInterface);
+    ms_hasDeviceArgs = false;
+    return result;
 }
 
 // Hook info