[PATCH 5/5] ddraw/tests: Test invalid clipper objects.
Stefan Dösinger
stefan at codeweavers.com
Mon Mar 4 13:59:04 CST 2019
Signed-off-by: Stefan Dösinger <stefan at codeweavers.com>
---
dlls/ddraw/tests/ddraw1.c | 195 ++++++++++++++++++++++++++++++++++++
dlls/ddraw/tests/ddraw2.c | 201 ++++++++++++++++++++++++++++++++++++++
dlls/ddraw/tests/ddraw4.c | 201 ++++++++++++++++++++++++++++++++++++++
dlls/ddraw/tests/ddraw7.c | 201 ++++++++++++++++++++++++++++++++++++++
4 files changed, 798 insertions(+)
diff --git a/dlls/ddraw/tests/ddraw1.c b/dlls/ddraw/tests/ddraw1.c
index d0ed33dbc41..cf25bcbe653 100644
--- a/dlls/ddraw/tests/ddraw1.c
+++ b/dlls/ddraw/tests/ddraw1.c
@@ -12293,6 +12293,200 @@ static void test_alphatest(void)
DestroyWindow(window);
}
+static void test_clipper_refcount(void)
+{
+ IDirectDrawSurface *surface;
+ IDirectDrawClipper *clipper, *clipper2;
+ DDSURFACEDESC surface_desc;
+ IDirectDraw *ddraw;
+ ULONG refcount;
+ HWND window;
+ HRESULT hr;
+ BOOL changed;
+ const IDirectDrawClipperVtbl *orig_vtbl;
+
+ window = create_window();
+ ddraw = create_ddraw();
+ ok(!!ddraw, "Failed to create a ddraw object.\n");
+ hr = IDirectDraw_SetCooperativeLevel(ddraw, window, DDSCL_NORMAL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+
+ memset(&surface_desc, 0, sizeof(surface_desc));
+ surface_desc.dwSize = sizeof(surface_desc);
+ surface_desc.dwFlags = DDSD_CAPS;
+ surface_desc.ddsCaps.dwCaps = DDSCAPS_PRIMARYSURFACE;
+ hr = IDirectDraw_CreateSurface(ddraw, &surface_desc, &surface, NULL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+
+ hr = IDirectDraw_CreateClipper(ddraw, 0, &clipper, NULL);
+ ok(SUCCEEDED(hr), "Failed to create clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 1, "Got unexpected refcount %u.\n", refcount);
+
+ /* Show that clipper validation doesn't somehow happen through vtable pointers. */
+ hr = IDirectDraw2_CreateClipper(ddraw, 0, &clipper2, NULL);
+ ok(SUCCEEDED(hr), "Failed to create clipper, hr %#x.\n", hr);
+ ok(clipper->lpVtbl == clipper2->lpVtbl, "Got different clipper vtables %p and %p.\n",
+ clipper->lpVtbl, clipper2->lpVtbl);
+ IDirectDrawClipper_Release(clipper2);
+
+ /* Surfaces hold a reference to clippers. No surprises there. */
+ hr = IDirectDrawSurface_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+ refcount = IDirectDrawClipper_Release(clipper2);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface_SetClipper(surface, NULL);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 1, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ refcount = IDirectDrawSurface_Release(surface);
+ ok(!refcount, "%u references left.\n", refcount);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 1, "Got unexpected refcount %u.\n", refcount);
+
+ /* Break the clipper. Clipper-related ops don't crash. AddRef works, Release doesn't. */
+ hr = IDirectDraw_CreateSurface(ddraw, &surface_desc, &surface, NULL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+ hr = IDirectDrawSurface_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ orig_vtbl = clipper->lpVtbl;
+ clipper->lpVtbl = (void *)0xdeadbeef;
+
+ refcount = orig_vtbl->AddRef(clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+ refcount = orig_vtbl->Release(clipper);
+ ok(!refcount, "Got unexpected refcount %u.\n", refcount);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = orig_vtbl->Release(clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ /* GetClipper works just fine. */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ hr = IDirectDrawSurface_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+ IDirectDrawClipper_Release(clipper2);
+
+ /* So does SetClipper, except for the Release being a no-op. */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ hr = IDirectDrawSurface_SetClipper(surface, NULL);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface_GetClipper(surface, &clipper2);
+ ok(hr == DDERR_NOCLIPPERATTACHED, "Got unexpected hr %#x.\n", hr);
+ ok(!clipper2, "Got clipper %p, expected NULL.\n", clipper2);
+
+ /* Set a broken clipper. This updates the reference and assigns the clipper */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ hr = IDirectDrawSurface_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+
+ /* Refcounting works again with a proper vtbl - GetClipper adds, our release releases. */
+ hr = IDirectDrawSurface_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 4, "Got unexpected refcount %u.\n", refcount);
+ refcount = IDirectDrawClipper_Release(clipper2);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+
+ /* Surface destruction with a broken clipper - Refcount is not changed. */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ refcount = IDirectDrawSurface_Release(surface);
+ ok(!refcount, "%u references left.\n", refcount);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+
+ IDirectDrawClipper_Release(clipper); /* For the non-released surface ref from unset. */
+ IDirectDrawClipper_Release(clipper); /* For the non-released surface ref from release. */
+ refcount = IDirectDrawClipper_Release(clipper); /* Our ref. */
+ ok(!refcount, "%u references left.\n", refcount);
+
+ hr = IDirectDraw_CreateSurface(ddraw, &surface_desc, &surface, NULL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+ hr = IDirectDraw_CreateClipper(ddraw, 0, &clipper, NULL);
+ ok(SUCCEEDED(hr), "Failed to create clipper, hr %#x.\n", hr);
+ hr = IDirectDrawSurface_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ /* Steal the reference and see what happens - releasing the surface works fine.
+ * The clipper is destroyed and not kept alive by a hidden refcount - trying to
+ * release it after the GetClipper call is likely to crash, and certain to crash
+ * if we allocate and zero as much heap memory as we can get. */
+ IDirectDrawClipper_Release(clipper);
+ IDirectDrawClipper_Release(clipper);
+
+ hr = IDirectDrawSurface_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+ /* Don't attempt to release clipper2, it is not a valid interface pointer. */
+
+ refcount = IDirectDrawSurface_Release(surface);
+ ok(!refcount, "%u references left.\n", refcount);
+
+ /* Trying to set a pointer that is valid read-only memory will crash though,
+ * e.g. surface->SetClipper((IDirectDrawClipper *)test_clipper_refcount). */
+
+ /* It looks like the protection against invalid thispointers is part of
+ * the IDirectDrawClipper method implementation, not IDirectDrawSurface. */
+ clipper = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, 0x1000);
+ ok(!!clipper, "failed to allocate memory\n");
+ clipper->lpVtbl = orig_vtbl;
+
+ refcount = orig_vtbl->AddRef(clipper);
+ ok(!refcount, "Got refcount %u.\n", refcount);
+ refcount = orig_vtbl->AddRef((IDirectDrawClipper *)0xdeadbeef);
+ ok(!refcount, "Got refcount %u.\n", refcount);
+
+ changed = 0x1234;
+ hr = orig_vtbl->IsClipListChanged(clipper, &changed);
+ ok(hr == DDERR_INVALIDPARAMS, "Got unexpected hr %#x.\n", hr);
+ ok(changed == 0x1234, "'changed' changed: %x.\n", changed);
+
+ hr = orig_vtbl->IsClipListChanged((IDirectDrawClipper *)0xdeadbeef, &changed);
+ ok(hr == DDERR_INVALIDPARAMS, "Got unexpected hr %#x.\n", hr);
+ ok(changed == 0x1234, "'changed' changed: %x.\n", changed);
+
+ /* Nope, we can't initialize our fake clipper. */
+ hr = orig_vtbl->Initialize(clipper, ddraw, 0);
+ ok(hr == DDERR_INVALIDPARAMS, "Got unexpected hr %#x.\n", hr);
+
+ HeapFree(GetProcessHeap(), 0, clipper);
+
+ refcount = IDirectDraw2_Release(ddraw);
+ ok(!refcount, "%u references left.\n", refcount);
+ DestroyWindow(window);
+}
+
START_TEST(ddraw1)
{
DDDEVICEIDENTIFIER identifier;
@@ -12401,4 +12595,5 @@ START_TEST(ddraw1)
test_killfocus();
test_gdi_surface();
test_alphatest();
+ test_clipper_refcount();
}
diff --git a/dlls/ddraw/tests/ddraw2.c b/dlls/ddraw/tests/ddraw2.c
index 37238e162fb..86205db8aa8 100644
--- a/dlls/ddraw/tests/ddraw2.c
+++ b/dlls/ddraw/tests/ddraw2.c
@@ -13552,6 +13552,206 @@ static void test_alphatest(void)
DestroyWindow(window);
}
+static void test_clipper_refcount(void)
+{
+ IDirectDrawSurface *surface;
+ IDirectDrawClipper *clipper, *clipper2;
+ DDSURFACEDESC surface_desc;
+ IDirectDraw2 *ddraw;
+ IDirectDraw *ddraw1;
+ ULONG refcount;
+ HWND window;
+ HRESULT hr;
+ BOOL changed;
+ const IDirectDrawClipperVtbl *orig_vtbl;
+
+ window = create_window();
+ ddraw = create_ddraw();
+ ok(!!ddraw, "Failed to create a ddraw object.\n");
+ hr = IDirectDraw2_SetCooperativeLevel(ddraw, window, DDSCL_NORMAL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+
+ memset(&surface_desc, 0, sizeof(surface_desc));
+ surface_desc.dwSize = sizeof(surface_desc);
+ surface_desc.dwFlags = DDSD_CAPS;
+ surface_desc.ddsCaps.dwCaps = DDSCAPS_PRIMARYSURFACE;
+ hr = IDirectDraw2_CreateSurface(ddraw, &surface_desc, &surface, NULL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+
+ hr = IDirectDraw2_CreateClipper(ddraw, 0, &clipper, NULL);
+ ok(SUCCEEDED(hr), "Failed to create clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 1, "Got unexpected refcount %u.\n", refcount);
+
+ /* Show that clipper validation doesn't somehow happen through vtable pointers. */
+ hr = IDirectDraw2_CreateClipper(ddraw, 0, &clipper2, NULL);
+ ok(SUCCEEDED(hr), "Failed to create clipper, hr %#x.\n", hr);
+ ok(clipper->lpVtbl == clipper2->lpVtbl, "Got different clipper vtables %p and %p.\n",
+ clipper->lpVtbl, clipper2->lpVtbl);
+ IDirectDrawClipper_Release(clipper2);
+
+ /* Surfaces hold a reference to clippers. No surprises there. */
+ hr = IDirectDrawSurface_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+ refcount = IDirectDrawClipper_Release(clipper2);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface_SetClipper(surface, NULL);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 1, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ refcount = IDirectDrawSurface_Release(surface);
+ ok(!refcount, "%u references left.\n", refcount);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 1, "Got unexpected refcount %u.\n", refcount);
+
+ /* Break the clipper. Clipper-related ops don't crash. AddRef works, Release doesn't. */
+ hr = IDirectDraw2_CreateSurface(ddraw, &surface_desc, &surface, NULL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+ hr = IDirectDrawSurface_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ orig_vtbl = clipper->lpVtbl;
+ clipper->lpVtbl = (void *)0xdeadbeef;
+
+ refcount = orig_vtbl->AddRef(clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+ refcount = orig_vtbl->Release(clipper);
+ ok(!refcount, "Got unexpected refcount %u.\n", refcount);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = orig_vtbl->Release(clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ /* GetClipper works just fine. */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ hr = IDirectDrawSurface_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+ IDirectDrawClipper_Release(clipper2);
+
+ /* So does SetClipper, except for the Release being a no-op. */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ hr = IDirectDrawSurface_SetClipper(surface, NULL);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface_GetClipper(surface, &clipper2);
+ ok(hr == DDERR_NOCLIPPERATTACHED, "Got unexpected hr %#x.\n", hr);
+ ok(!clipper2, "Got clipper %p, expected NULL.\n", clipper2);
+
+ /* Set a broken clipper. This updates the reference and assigns the clipper */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ hr = IDirectDrawSurface_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+
+ /* Refcounting works again with a proper vtbl - GetClipper adds, our release releases. */
+ hr = IDirectDrawSurface_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 4, "Got unexpected refcount %u.\n", refcount);
+ refcount = IDirectDrawClipper_Release(clipper2);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+
+ /* Surface destruction with a broken clipper - Refcount is not changed. */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ refcount = IDirectDrawSurface_Release(surface);
+ ok(!refcount, "%u references left.\n", refcount);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+
+ IDirectDrawClipper_Release(clipper); /* For the non-released surface ref from unset. */
+ IDirectDrawClipper_Release(clipper); /* For the non-released surface ref from release. */
+ refcount = IDirectDrawClipper_Release(clipper); /* Our ref. */
+ ok(!refcount, "%u references left.\n", refcount);
+
+ hr = IDirectDraw2_CreateSurface(ddraw, &surface_desc, &surface, NULL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+ hr = IDirectDraw2_CreateClipper(ddraw, 0, &clipper, NULL);
+ ok(SUCCEEDED(hr), "Failed to create clipper, hr %#x.\n", hr);
+ hr = IDirectDrawSurface_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ /* Steal the reference and see what happens - releasing the surface works fine.
+ * The clipper is destroyed and not kept alive by a hidden refcount - trying to
+ * release it after the GetClipper call is likely to crash, and certain to crash
+ * if we allocate and zero as much heap memory as we can get. */
+ IDirectDrawClipper_Release(clipper);
+ IDirectDrawClipper_Release(clipper);
+
+ hr = IDirectDrawSurface_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+ /* Don't attempt to release clipper2, it is not a valid interface pointer. */
+
+ refcount = IDirectDrawSurface_Release(surface);
+ ok(!refcount, "%u references left.\n", refcount);
+
+ /* Trying to set a pointer that is valid read-only memory will crash though,
+ * e.g. surface->SetClipper((IDirectDrawClipper *)test_clipper_refcount). */
+
+ /* It looks like the protection against invalid thispointers is part of
+ * the IDirectDrawClipper method implementation, not IDirectDrawSurface. */
+ clipper = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, 0x1000);
+ ok(!!clipper, "failed to allocate memory\n");
+ clipper->lpVtbl = orig_vtbl;
+
+ refcount = orig_vtbl->AddRef(clipper);
+ ok(!refcount, "Got refcount %u.\n", refcount);
+ refcount = orig_vtbl->AddRef((IDirectDrawClipper *)0xdeadbeef);
+ ok(!refcount, "Got refcount %u.\n", refcount);
+
+ changed = 0x1234;
+ hr = orig_vtbl->IsClipListChanged(clipper, &changed);
+ ok(hr == DDERR_INVALIDPARAMS, "Got unexpected hr %#x.\n", hr);
+ ok(changed == 0x1234, "'changed' changed: %x.\n", changed);
+
+ hr = orig_vtbl->IsClipListChanged((IDirectDrawClipper *)0xdeadbeef, &changed);
+ ok(hr == DDERR_INVALIDPARAMS, "Got unexpected hr %#x.\n", hr);
+ ok(changed == 0x1234, "'changed' changed: %x.\n", changed);
+
+ /* Nope, we can't initialize our fake clipper. */
+ hr = IDirectDraw2_QueryInterface(ddraw, &IID_IDirectDraw, (void **)&ddraw1);
+ ok(SUCCEEDED(hr), "Failed to get ddraw1 interface, hr %#x.\n", hr);
+
+ hr = orig_vtbl->Initialize(clipper, ddraw1, 0);
+ ok(hr == DDERR_INVALIDPARAMS, "Got unexpected hr %#x.\n", hr);
+
+ IDirectDraw_Release(ddraw1);
+
+ HeapFree(GetProcessHeap(), 0, clipper);
+
+ refcount = IDirectDraw2_Release(ddraw);
+ ok(!refcount, "%u references left.\n", refcount);
+ DestroyWindow(window);
+}
+
START_TEST(ddraw2)
{
DDDEVICEIDENTIFIER identifier;
@@ -13668,4 +13868,5 @@ START_TEST(ddraw2)
test_killfocus();
test_gdi_surface();
test_alphatest();
+ test_clipper_refcount();
}
diff --git a/dlls/ddraw/tests/ddraw4.c b/dlls/ddraw/tests/ddraw4.c
index b60606fca3a..f477599f974 100644
--- a/dlls/ddraw/tests/ddraw4.c
+++ b/dlls/ddraw/tests/ddraw4.c
@@ -15798,6 +15798,206 @@ static void test_alphatest(void)
DestroyWindow(window);
}
+static void test_clipper_refcount(void)
+{
+ IDirectDrawSurface4 *surface;
+ IDirectDrawClipper *clipper, *clipper2;
+ DDSURFACEDESC2 surface_desc;
+ IDirectDraw4 *ddraw;
+ IDirectDraw *ddraw1;
+ ULONG refcount;
+ HWND window;
+ HRESULT hr;
+ BOOL changed;
+ const IDirectDrawClipperVtbl *orig_vtbl;
+
+ window = create_window();
+ ddraw = create_ddraw();
+ ok(!!ddraw, "Failed to create a ddraw object.\n");
+ hr = IDirectDraw4_SetCooperativeLevel(ddraw, window, DDSCL_NORMAL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+
+ memset(&surface_desc, 0, sizeof(surface_desc));
+ surface_desc.dwSize = sizeof(surface_desc);
+ surface_desc.dwFlags = DDSD_CAPS;
+ surface_desc.ddsCaps.dwCaps = DDSCAPS_PRIMARYSURFACE;
+ hr = IDirectDraw4_CreateSurface(ddraw, &surface_desc, &surface, NULL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+
+ hr = IDirectDraw4_CreateClipper(ddraw, 0, &clipper, NULL);
+ ok(SUCCEEDED(hr), "Failed to create clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 1, "Got unexpected refcount %u.\n", refcount);
+
+ /* Show that clipper validation doesn't somehow happen through vtable pointers. */
+ hr = IDirectDraw4_CreateClipper(ddraw, 0, &clipper2, NULL);
+ ok(SUCCEEDED(hr), "Failed to create clipper, hr %#x.\n", hr);
+ ok(clipper->lpVtbl == clipper2->lpVtbl, "Got different clipper vtables %p and %p.\n",
+ clipper->lpVtbl, clipper2->lpVtbl);
+ IDirectDrawClipper_Release(clipper2);
+
+ /* Surfaces hold a reference to clippers. No surprises there. */
+ hr = IDirectDrawSurface4_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface4_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+ refcount = IDirectDrawClipper_Release(clipper2);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface4_SetClipper(surface, NULL);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 1, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface4_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ refcount = IDirectDrawSurface4_Release(surface);
+ ok(!refcount, "%u references left.\n", refcount);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 1, "Got unexpected refcount %u.\n", refcount);
+
+ /* Break the clipper. Clipper-related ops don't crash. AddRef works, Release doesn't. */
+ hr = IDirectDraw4_CreateSurface(ddraw, &surface_desc, &surface, NULL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+ hr = IDirectDrawSurface4_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ orig_vtbl = clipper->lpVtbl;
+ clipper->lpVtbl = (void *)0xdeadbeef;
+
+ refcount = orig_vtbl->AddRef(clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+ refcount = orig_vtbl->Release(clipper);
+ ok(!refcount, "Got unexpected refcount %u.\n", refcount);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = orig_vtbl->Release(clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ /* GetClipper works just fine. */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ hr = IDirectDrawSurface4_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+ IDirectDrawClipper_Release(clipper2);
+
+ /* So does SetClipper, except for the Release being a no-op. */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ hr = IDirectDrawSurface4_SetClipper(surface, NULL);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface4_GetClipper(surface, &clipper2);
+ ok(hr == DDERR_NOCLIPPERATTACHED, "Got unexpected hr %#x.\n", hr);
+ ok(!clipper2, "Got clipper %p, expected NULL.\n", clipper2);
+
+ /* Set a broken clipper. This updates the reference and assigns the clipper */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ hr = IDirectDrawSurface4_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+
+ /* Refcounting works again with a proper vtbl - GetClipper adds, our release releases. */
+ hr = IDirectDrawSurface4_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 4, "Got unexpected refcount %u.\n", refcount);
+ refcount = IDirectDrawClipper_Release(clipper2);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+
+ /* Surface destruction with a broken clipper - Refcount is not changed. */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ refcount = IDirectDrawSurface4_Release(surface);
+ ok(!refcount, "%u references left.\n", refcount);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+
+ IDirectDrawClipper_Release(clipper); /* For the non-released surface ref from unset. */
+ IDirectDrawClipper_Release(clipper); /* For the non-released surface ref from release. */
+ refcount = IDirectDrawClipper_Release(clipper); /* Our ref. */
+ ok(!refcount, "%u references left.\n", refcount);
+
+ hr = IDirectDraw4_CreateSurface(ddraw, &surface_desc, &surface, NULL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+ hr = IDirectDraw4_CreateClipper(ddraw, 0, &clipper, NULL);
+ ok(SUCCEEDED(hr), "Failed to create clipper, hr %#x.\n", hr);
+ hr = IDirectDrawSurface4_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ /* Steal the reference and see what happens - releasing the surface works fine.
+ * The clipper is destroyed and not kept alive by a hidden refcount - trying to
+ * release it after the GetClipper call is likely to crash, and certain to crash
+ * if we allocate and zero as much heap memory as we can get. */
+ IDirectDrawClipper_Release(clipper);
+ IDirectDrawClipper_Release(clipper);
+
+ hr = IDirectDrawSurface4_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+ /* Don't attempt to release clipper2, it is not a valid interface pointer. */
+
+ refcount = IDirectDrawSurface4_Release(surface);
+ ok(!refcount, "%u references left.\n", refcount);
+
+ /* Trying to set a pointer that is valid read-only memory will crash though,
+ * e.g. surface->SetClipper((IDirectDrawClipper *)test_clipper_refcount). */
+
+ /* It looks like the protection against invalid thispointers is part of
+ * the IDirectDrawClipper method implementation, not IDirectDrawSurface. */
+ clipper = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, 0x1000);
+ ok(!!clipper, "failed to allocate memory\n");
+ clipper->lpVtbl = orig_vtbl;
+
+ refcount = orig_vtbl->AddRef(clipper);
+ ok(!refcount, "Got refcount %u.\n", refcount);
+ refcount = orig_vtbl->AddRef((IDirectDrawClipper *)0xdeadbeef);
+ ok(!refcount, "Got refcount %u.\n", refcount);
+
+ changed = 0x1234;
+ hr = orig_vtbl->IsClipListChanged(clipper, &changed);
+ ok(hr == DDERR_INVALIDPARAMS, "Got unexpected hr %#x.\n", hr);
+ ok(changed == 0x1234, "'changed' changed: %x.\n", changed);
+
+ hr = orig_vtbl->IsClipListChanged((IDirectDrawClipper *)0xdeadbeef, &changed);
+ ok(hr == DDERR_INVALIDPARAMS, "Got unexpected hr %#x.\n", hr);
+ ok(changed == 0x1234, "'changed' changed: %x.\n", changed);
+
+ /* Nope, we can't initialize our fake clipper. */
+ hr = IDirectDraw4_QueryInterface(ddraw, &IID_IDirectDraw, (void **)&ddraw1);
+ ok(SUCCEEDED(hr), "Failed to get ddraw1 interface, hr %#x.\n", hr);
+
+ hr = orig_vtbl->Initialize(clipper, ddraw1, 0);
+ ok(hr == DDERR_INVALIDPARAMS, "Got unexpected hr %#x.\n", hr);
+
+ IDirectDraw_Release(ddraw1);
+
+ HeapFree(GetProcessHeap(), 0, clipper);
+
+ refcount = IDirectDraw4_Release(ddraw);
+ ok(!refcount, "%u references left.\n", refcount);
+ DestroyWindow(window);
+}
+
START_TEST(ddraw4)
{
DDDEVICEIDENTIFIER identifier;
@@ -15929,4 +16129,5 @@ START_TEST(ddraw4)
test_sysmem_draw();
test_gdi_surface();
test_alphatest();
+ test_clipper_refcount();
}
diff --git a/dlls/ddraw/tests/ddraw7.c b/dlls/ddraw/tests/ddraw7.c
index 3339b32d200..aaf7f651780 100644
--- a/dlls/ddraw/tests/ddraw7.c
+++ b/dlls/ddraw/tests/ddraw7.c
@@ -15599,6 +15599,206 @@ static void test_alphatest(void)
DestroyWindow(window);
}
+static void test_clipper_refcount(void)
+{
+ IDirectDrawSurface7 *surface;
+ IDirectDrawClipper *clipper, *clipper2;
+ DDSURFACEDESC2 surface_desc;
+ IDirectDraw7 *ddraw;
+ IDirectDraw *ddraw1;
+ ULONG refcount;
+ HWND window;
+ HRESULT hr;
+ BOOL changed;
+ const IDirectDrawClipperVtbl *orig_vtbl;
+
+ window = create_window();
+ ddraw = create_ddraw();
+ ok(!!ddraw, "Failed to create a ddraw object.\n");
+ hr = IDirectDraw7_SetCooperativeLevel(ddraw, window, DDSCL_NORMAL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+
+ memset(&surface_desc, 0, sizeof(surface_desc));
+ surface_desc.dwSize = sizeof(surface_desc);
+ surface_desc.dwFlags = DDSD_CAPS;
+ surface_desc.ddsCaps.dwCaps = DDSCAPS_PRIMARYSURFACE;
+ hr = IDirectDraw7_CreateSurface(ddraw, &surface_desc, &surface, NULL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+
+ hr = IDirectDraw7_CreateClipper(ddraw, 0, &clipper, NULL);
+ ok(SUCCEEDED(hr), "Failed to create clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 1, "Got unexpected refcount %u.\n", refcount);
+
+ /* Show that clipper validation doesn't somehow happen through vtable pointers. */
+ hr = IDirectDraw7_CreateClipper(ddraw, 0, &clipper2, NULL);
+ ok(SUCCEEDED(hr), "Failed to create clipper, hr %#x.\n", hr);
+ ok(clipper->lpVtbl == clipper2->lpVtbl, "Got different clipper vtables %p and %p.\n",
+ clipper->lpVtbl, clipper2->lpVtbl);
+ IDirectDrawClipper_Release(clipper2);
+
+ /* Surfaces hold a reference to clippers. No surprises there. */
+ hr = IDirectDrawSurface7_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface7_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+ refcount = IDirectDrawClipper_Release(clipper2);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface7_SetClipper(surface, NULL);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 1, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface7_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ refcount = IDirectDrawSurface7_Release(surface);
+ ok(!refcount, "%u references left.\n", refcount);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 1, "Got unexpected refcount %u.\n", refcount);
+
+ /* Break the clipper. Clipper-related ops don't crash. AddRef works, Release doesn't. */
+ hr = IDirectDraw7_CreateSurface(ddraw, &surface_desc, &surface, NULL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+ hr = IDirectDrawSurface7_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ orig_vtbl = clipper->lpVtbl;
+ clipper->lpVtbl = (void *)0xdeadbeef;
+
+ refcount = orig_vtbl->AddRef(clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+ refcount = orig_vtbl->Release(clipper);
+ ok(!refcount, "Got unexpected refcount %u.\n", refcount);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = orig_vtbl->Release(clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ /* GetClipper works just fine. */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ hr = IDirectDrawSurface7_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+ IDirectDrawClipper_Release(clipper2);
+
+ /* So does SetClipper, except for the Release being a no-op. */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ hr = IDirectDrawSurface7_SetClipper(surface, NULL);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 2, "Got unexpected refcount %u.\n", refcount);
+
+ hr = IDirectDrawSurface7_GetClipper(surface, &clipper2);
+ ok(hr == DDERR_NOCLIPPERATTACHED, "Got unexpected hr %#x.\n", hr);
+ ok(!clipper2, "Got clipper %p, expected NULL.\n", clipper2);
+
+ /* Set a broken clipper. This updates the reference and assigns the clipper */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ hr = IDirectDrawSurface7_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+
+ /* Refcounting works again with a proper vtbl - GetClipper adds, our release releases. */
+ hr = IDirectDrawSurface7_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 4, "Got unexpected refcount %u.\n", refcount);
+ refcount = IDirectDrawClipper_Release(clipper2);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+
+ /* Surface destruction with a broken clipper - Refcount is not changed. */
+ clipper->lpVtbl = (void *)0xdeadbeef;
+ refcount = IDirectDrawSurface7_Release(surface);
+ ok(!refcount, "%u references left.\n", refcount);
+
+ clipper->lpVtbl = orig_vtbl;
+ refcount = get_refcount((IUnknown *)clipper);
+ ok(refcount == 3, "Got unexpected refcount %u.\n", refcount);
+
+ IDirectDrawClipper_Release(clipper); /* For the non-released surface ref from unset. */
+ IDirectDrawClipper_Release(clipper); /* For the non-released surface ref from release. */
+ refcount = IDirectDrawClipper_Release(clipper); /* Our ref. */
+ ok(!refcount, "%u references left.\n", refcount);
+
+ hr = IDirectDraw7_CreateSurface(ddraw, &surface_desc, &surface, NULL);
+ ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr);
+ hr = IDirectDraw7_CreateClipper(ddraw, 0, &clipper, NULL);
+ ok(SUCCEEDED(hr), "Failed to create clipper, hr %#x.\n", hr);
+ hr = IDirectDrawSurface7_SetClipper(surface, clipper);
+ ok(SUCCEEDED(hr), "Failed to set clipper, hr %#x.\n", hr);
+
+ /* Steal the reference and see what happens - releasing the surface works fine.
+ * The clipper is destroyed and not kept alive by a hidden refcount - trying to
+ * release it after the GetClipper call is likely to crash, and certain to crash
+ * if we allocate and zero as much heap memory as we can get. */
+ IDirectDrawClipper_Release(clipper);
+ IDirectDrawClipper_Release(clipper);
+
+ hr = IDirectDrawSurface7_GetClipper(surface, &clipper2);
+ ok(SUCCEEDED(hr), "Failed to get clipper, hr %#x.\n", hr);
+ ok(clipper == clipper2, "Got clipper %p, expected %p.\n", clipper2, clipper);
+ /* Don't attempt to release clipper2, it is not a valid interface pointer. */
+
+ refcount = IDirectDrawSurface7_Release(surface);
+ ok(!refcount, "%u references left.\n", refcount);
+
+ /* Trying to set a pointer that is valid read-only memory will crash though,
+ * e.g. surface->SetClipper((IDirectDrawClipper *)test_clipper_refcount). */
+
+ /* It looks like the protection against invalid thispointers is part of
+ * the IDirectDrawClipper method implementation, not IDirectDrawSurface. */
+ clipper = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, 0x1000);
+ ok(!!clipper, "failed to allocate memory\n");
+ clipper->lpVtbl = orig_vtbl;
+
+ refcount = orig_vtbl->AddRef(clipper);
+ ok(!refcount, "Got refcount %u.\n", refcount);
+ refcount = orig_vtbl->AddRef((IDirectDrawClipper *)0xdeadbeef);
+ ok(!refcount, "Got refcount %u.\n", refcount);
+
+ changed = 0x1234;
+ hr = orig_vtbl->IsClipListChanged(clipper, &changed);
+ ok(hr == DDERR_INVALIDPARAMS, "Got unexpected hr %#x.\n", hr);
+ ok(changed == 0x1234, "'changed' changed: %x.\n", changed);
+
+ hr = orig_vtbl->IsClipListChanged((IDirectDrawClipper *)0xdeadbeef, &changed);
+ ok(hr == DDERR_INVALIDPARAMS, "Got unexpected hr %#x.\n", hr);
+ ok(changed == 0x1234, "'changed' changed: %x.\n", changed);
+
+ /* Nope, we can't initialize our fake clipper. */
+ hr = IDirectDraw7_QueryInterface(ddraw, &IID_IDirectDraw, (void **)&ddraw1);
+ ok(SUCCEEDED(hr), "Failed to get ddraw1 interface, hr %#x.\n", hr);
+
+ hr = orig_vtbl->Initialize(clipper, ddraw1, 0);
+ ok(hr == DDERR_INVALIDPARAMS, "Got unexpected hr %#x.\n", hr);
+
+ IDirectDraw_Release(ddraw1);
+
+ HeapFree(GetProcessHeap(), 0, clipper);
+
+ refcount = IDirectDraw7_Release(ddraw);
+ ok(!refcount, "%u references left.\n", refcount);
+ DestroyWindow(window);
+}
+
START_TEST(ddraw7)
{
DDDEVICEIDENTIFIER2 identifier;
@@ -15742,4 +15942,5 @@ START_TEST(ddraw7)
test_gdi_surface();
test_multiply_transform();
test_alphatest();
+ test_clipper_refcount();
}
--
2.19.2
More information about the wine-devel
mailing list