Aaryaman Vasishta : d3drm: GetDirect3DDevice2 returns D3DRM_OK when IDirect3DDevice2 is not available.
Alexandre Julliard
julliard at wine.codeweavers.com
Fri Aug 21 09:07:19 CDT 2015
Module: wine
Branch: master
Commit: c453cb7023fc111e0dc2bbf463695cdb31e41ba0
URL: http://source.winehq.org/git/wine.git/?a=commit;h=c453cb7023fc111e0dc2bbf463695cdb31e41ba0
Author: Aaryaman Vasishta <jem456.vasishta at gmail.com>
Date: Thu Aug 20 20:34:41 2015 +0530
d3drm: GetDirect3DDevice2 returns D3DRM_OK when IDirect3DDevice2 is not available.
---
dlls/d3drm/device.c | 4 +++-
dlls/d3drm/tests/d3drm.c | 18 ++++++++++++++++++
2 files changed, 21 insertions(+), 1 deletion(-)
diff --git a/dlls/d3drm/device.c b/dlls/d3drm/device.c
index 50d8b72..9c814fb 100644
--- a/dlls/d3drm/device.c
+++ b/dlls/d3drm/device.c
@@ -1351,7 +1351,9 @@ static HRESULT WINAPI d3drm_device3_GetDirect3DDevice2(IDirect3DRMDevice3 *iface
TRACE("iface %p, d3d_device %p.\n", iface, d3d_device);
- return IDirect3DDevice_QueryInterface(device->device, &IID_IDirect3DDevice2, (void**)d3d_device);
+ IDirect3DDevice_QueryInterface(device->device, &IID_IDirect3DDevice2, (void**)d3d_device);
+
+ return D3DRM_OK;
}
static HRESULT WINAPI d3drm_device3_FindPreferredTextureFormat(IDirect3DRMDevice3 *iface,
diff --git a/dlls/d3drm/tests/d3drm.c b/dlls/d3drm/tests/d3drm.c
index 4e5845c..50ba4c1 100644
--- a/dlls/d3drm/tests/d3drm.c
+++ b/dlls/d3drm/tests/d3drm.c
@@ -3118,7 +3118,10 @@ static void test_create_device_from_d3d1(void)
IDirect3D *d3d1 = NULL;
IDirect3DRM *d3drm1 = NULL;
IDirect3DRMDevice *device1 = (IDirect3DRMDevice *)0xdeadbeef;
+ IDirect3DRMDevice2 *device2;
+ IDirect3DRMDevice3 *device3;
IDirect3DDevice *d3ddevice1 = NULL, *d3drm_d3ddevice1 = NULL;
+ IDirect3DDevice2 *d3ddevice2 = (IDirect3DDevice2 *)0xdeadbeef;
IDirectDrawSurface *surface = NULL, *ds = NULL, *d3drm_ds = NULL;
DWORD expected_flags;
DDSCAPS caps = { DDSCAPS_ZBUFFER };
@@ -3167,6 +3170,21 @@ static void test_create_device_from_d3d1(void)
device_ref2 = get_refcount((IUnknown *)d3ddevice1);
ok(device_ref2 > device_ref1, "Expected device_ref2 > device_ref1, got device_ref1 = %u, device_ref2 = %u.\n", device_ref1, device_ref2);
+ hr = IDirect3DRMDevice_QueryInterface(device1, &IID_IDirect3DRMDevice2, (void **)&device2);
+ ok(SUCCEEDED(hr), "Cannot get IDirect3DRMDevice2 Interface (hr = %x).\n", hr);
+ hr = IDirect3DRMDevice2_GetDirect3DDevice2(device2, &d3ddevice2);
+ ok(SUCCEEDED(hr), "Expected hr == DD_OK, got %x).\n", hr);
+ ok(d3ddevice2 == NULL, "Expected d3ddevice2 == NULL, got %p.\n", d3ddevice2);
+ IDirect3DRMDevice2_Release(device2);
+
+ d3ddevice2 = (IDirect3DDevice2 *)0xdeadbeef;
+ hr = IDirect3DRMDevice_QueryInterface(device1, &IID_IDirect3DRMDevice2, (void **)&device3);
+ ok(hr == DD_OK, "Cannot get IDirect3DRMDevice2 Interface (hr = %x).\n", hr);
+ hr = IDirect3DRMDevice3_GetDirect3DDevice2(device3, &d3ddevice2);
+ ok(hr == DD_OK, "Expected hr == DD_OK, got %x).\n", hr);
+ ok(d3ddevice2 == NULL, "Expected d3ddevice2 == NULL, got %p.\n", d3ddevice2);
+ IDirect3DRMDevice3_Release(device3);
+
hr = IDirectDraw_EnumSurfaces(ddraw1, DDENUMSURFACES_ALL | DDENUMSURFACES_DOESEXIST,
NULL, &surface, surface_callback);
ok(hr == DD_OK, "Failed to enumerate surfaces (hr = %x).\n", hr);
More information about the wine-cvs
mailing list