=?UTF-8?Q?J=C3=B3zef=20Kucia=20?=: d3d10: Validate SDK version.
Alexandre Julliard
julliard at winehq.org
Fri May 5 15:05:58 CDT 2017
Module: wine
Branch: master
Commit: 1a86bc9d758c6d0c7c14ad8b152ae8ab4aded38f
URL: http://source.winehq.org/git/wine.git/?a=commit;h=1a86bc9d758c6d0c7c14ad8b152ae8ab4aded38f
Author: Józef Kucia <jkucia at codeweavers.com>
Date: Fri May 5 11:59:23 2017 +0200
d3d10: Validate SDK version.
Signed-off-by: Józef Kucia <jkucia at codeweavers.com>
Signed-off-by: Henri Verbeet <hverbeet at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
dlls/d3d10/d3d10_main.c | 30 ++++++++++++++++++------------
dlls/d3d10/tests/device.c | 23 +++++++++++++++++++++++
2 files changed, 41 insertions(+), 12 deletions(-)
diff --git a/dlls/d3d10/d3d10_main.c b/dlls/d3d10/d3d10_main.c
index c9d5531..52585f0 100644
--- a/dlls/d3d10/d3d10_main.c
+++ b/dlls/d3d10/d3d10_main.c
@@ -33,16 +33,22 @@ HRESULT WINAPI D3D10CreateDevice(IDXGIAdapter *adapter, D3D10_DRIVER_TYPE driver
IDXGIFactory *factory;
HRESULT hr;
- TRACE("adapter %p, driver_type %s, swrast %p, flags %#x, sdk_version %d, device %p\n",
+ TRACE("adapter %p, driver_type %s, swrast %p, flags %#x, sdk_version %#x, device %p.\n",
adapter, debug_d3d10_driver_type(driver_type), swrast, flags, sdk_version, device);
+ if (sdk_version != D3D10_SDK_VERSION)
+ {
+ WARN("Invalid SDK version %#x.\n", sdk_version);
+ return E_INVALIDARG;
+ }
+
if (adapter)
{
IDXGIAdapter_AddRef(adapter);
hr = IDXGIAdapter_GetParent(adapter, &IID_IDXGIFactory, (void **)&factory);
if (FAILED(hr))
{
- WARN("Failed to get dxgi factory, returning %#x\n", hr);
+ WARN("Failed to get dxgi factory, returning %#x.\n", hr);
return hr;
}
}
@@ -51,18 +57,18 @@ HRESULT WINAPI D3D10CreateDevice(IDXGIAdapter *adapter, D3D10_DRIVER_TYPE driver
hr = CreateDXGIFactory(&IID_IDXGIFactory, (void **)&factory);
if (FAILED(hr))
{
- WARN("Failed to create dxgi factory, returning %#x\n", hr);
+ WARN("Failed to create dxgi factory, returning %#x.\n", hr);
return hr;
}
- switch(driver_type)
+ switch (driver_type)
{
case D3D10_DRIVER_TYPE_HARDWARE:
{
hr = IDXGIFactory_EnumAdapters(factory, 0, &adapter);
if (FAILED(hr))
{
- WARN("No adapters found, returning %#x\n", hr);
+ WARN("No adapters found, returning %#x.\n", hr);
IDXGIFactory_Release(factory);
return hr;
}
@@ -70,14 +76,14 @@ HRESULT WINAPI D3D10CreateDevice(IDXGIAdapter *adapter, D3D10_DRIVER_TYPE driver
}
case D3D10_DRIVER_TYPE_NULL:
- FIXME("NULL device not implemented, falling back to refrast\n");
+ FIXME("NULL device not implemented, falling back to refrast.\n");
/* fall through, for now */
case D3D10_DRIVER_TYPE_REFERENCE:
{
HMODULE refrast = LoadLibraryA("d3d10ref.dll");
if (!refrast)
{
- WARN("Failed to load refrast, returning E_FAIL\n");
+ WARN("Failed to load refrast, returning E_FAIL.\n");
IDXGIFactory_Release(factory);
return E_FAIL;
}
@@ -85,7 +91,7 @@ HRESULT WINAPI D3D10CreateDevice(IDXGIAdapter *adapter, D3D10_DRIVER_TYPE driver
FreeLibrary(refrast);
if (FAILED(hr))
{
- WARN("Failed to create a software adapter, returning %#x\n", hr);
+ WARN("Failed to create a software adapter, returning %#x.\n", hr);
IDXGIFactory_Release(factory);
return hr;
}
@@ -96,14 +102,14 @@ HRESULT WINAPI D3D10CreateDevice(IDXGIAdapter *adapter, D3D10_DRIVER_TYPE driver
{
if (!swrast)
{
- WARN("Software device requested, but NULL swrast passed, returning E_FAIL\n");
+ WARN("Software device requested, but NULL swrast passed, returning E_FAIL.\n");
IDXGIFactory_Release(factory);
return E_FAIL;
}
hr = IDXGIFactory_CreateSoftwareAdapter(factory, swrast, &adapter);
if (FAILED(hr))
{
- WARN("Failed to create a software adapter, returning %#x\n", hr);
+ WARN("Failed to create a software adapter, returning %#x.\n", hr);
IDXGIFactory_Release(factory);
return hr;
}
@@ -122,11 +128,11 @@ HRESULT WINAPI D3D10CreateDevice(IDXGIAdapter *adapter, D3D10_DRIVER_TYPE driver
IDXGIFactory_Release(factory);
if (FAILED(hr))
{
- WARN("Failed to create a device, returning %#x\n", hr);
+ WARN("Failed to create a device, returning %#x.\n", hr);
return hr;
}
- TRACE("Created ID3D10Device %p\n", *device);
+ TRACE("Created ID3D10Device %p.\n", *device);
return hr;
}
diff --git a/dlls/d3d10/tests/device.c b/dlls/d3d10/tests/device.c
index fa8f708..3e2731d 100644
--- a/dlls/d3d10/tests/device.c
+++ b/dlls/d3d10/tests/device.c
@@ -21,6 +21,28 @@
#include "d3d10.h"
#include "wine/test.h"
+static void test_create_device(void)
+{
+ ID3D10Device *device;
+ unsigned int i;
+ HRESULT hr;
+
+ if (FAILED(hr = D3D10CreateDevice(NULL, D3D10_DRIVER_TYPE_HARDWARE, NULL, 0, D3D10_SDK_VERSION, &device)))
+ {
+ skip("Failed to create HAL device.\n");
+ return;
+ }
+ ID3D10Device_Release(device);
+
+ for (i = 0; i < 100; ++i)
+ {
+ if (i == D3D10_SDK_VERSION)
+ continue;
+ hr = D3D10CreateDevice(NULL, D3D10_DRIVER_TYPE_HARDWARE, NULL, 0, i, &device);
+ ok(hr == E_INVALIDARG, "Got unexpected hr %#x for SDK version %#x.\n", hr, i);
+ }
+}
+
static void test_stateblock_mask(void)
{
static const struct
@@ -181,5 +203,6 @@ static void test_stateblock_mask(void)
START_TEST(device)
{
+ test_create_device();
test_stateblock_mask();
}
More information about the wine-cvs
mailing list