Connor McAdams : d3dx9/tests: Add a test for negative values in the source rectangle passed to D3DXLoadSurfaceFromMemory.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: c04d5622accc3f0f30e9255e297a95a743c49e18
URL:    https://gitlab.winehq.org/wine/wine/-/commit/c04d5622accc3f0f30e9255e297a95a...

Author: Connor McAdams <cmcadams(a)codeweavers.com>
Date:   Thu Apr 25 11:36:10 2024 -0400

d3dx9/tests: Add a test for negative values in the source rectangle passed to D3DXLoadSurfaceFromMemory.

This behavior is mentioned in the wine code documentation comment for
this function as being allowed, but the behavior is not tested.

Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com>

---

 dlls/d3dx9_36/tests/surface.c | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/dlls/d3dx9_36/tests/surface.c b/dlls/d3dx9_36/tests/surface.c
index 036326c56cf..feabd0f465b 100644
--- a/dlls/d3dx9_36/tests/surface.c
+++ b/dlls/d3dx9_36/tests/surface.c
@@ -1173,6 +1173,24 @@ static void test_D3DXLoadSurface(IDirect3DDevice9 *device)
     IDirect3DSurface9_LockRect(surf, &lockrect, NULL, D3DLOCK_READONLY);
     check_pixel_4bpp(&lockrect, 0, 0, 0x8dc32bf6);
     IDirect3DSurface9_UnlockRect(surf);
+
+    /*
+     * Test negative offsets in the source rectangle. Causes an access
+     * violation when run on 64-bit Windows.
+     */
+    if (sizeof(void *) != 8)
+    {
+        SetRect(&rect, 0, -1, 1, 0);
+        hr = D3DXLoadSurfaceFromMemory(surf, NULL, NULL, &pixdata_a8b8g8r8[2],
+                D3DFMT_A8R8G8B8, 8, NULL, &rect, D3DX_FILTER_NONE, 0);
+        ok(hr == D3D_OK, "Got unexpected hr %#lx.\n", hr);
+        IDirect3DSurface9_LockRect(surf, &lockrect, NULL, D3DLOCK_READONLY);
+        check_pixel_4bpp(&lockrect, 0, 0, pixdata_a8b8g8r8[0]);
+        IDirect3DSurface9_UnlockRect(surf);
+    }
+    else
+        skip("Skipping test for negative source rectangle values on 64-bit.\n");
+
     check_release((IUnknown *)surf, 0);
 
     /* test color conversion */