wine-commits June 2020

wine-commits@winehq.org

1 participants
947 discussions

Henri Verbeet : wined3d: Get rid of the "d3d_initialized" check in device_resource_released().
by Alexandre Julliard 03 Jun '20

03 Jun '20

Module: wine Branch: master Commit: cc6d90b8236ce5ba73b09e6f5eeb164e506a07a7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=cc6d90b8236ce5ba73b09e6f… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Jun 3 15:58:05 2020 +0430 wined3d: Get rid of the "d3d_initialized" check in device_resource_released(). This used to protect against accessing the framebuffer state on the no3d adapter; that's no longer a concern. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/device.c | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 4544e28bc4..2ce3e346ef 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -5493,18 +5493,15 @@ void device_resource_released(struct wined3d_device *device, struct wined3d_reso TRACE("device %p, resource %p, type %s.\n", device, resource, debug_d3dresourcetype(type)); - if (device->d3d_initialized) + for (i = 0; i < ARRAY_SIZE(device->state.fb.render_targets); ++i) { - for (i = 0; i < ARRAY_SIZE(device->state.fb.render_targets); ++i) - { - if ((rtv = device->state.fb.render_targets[i]) && rtv->resource == resource) - ERR("Resource %p is still in use as render target %u.\n", resource, i); - } - - if ((rtv = device->state.fb.depth_stencil) && rtv->resource == resource) - ERR("Resource %p is still in use as depth/stencil buffer.\n", resource); + if ((rtv = device->state.fb.render_targets[i]) && rtv->resource == resource) + ERR("Resource %p is still in use as render target %u.\n", resource, i); } + if ((rtv = device->state.fb.depth_stencil) && rtv->resource == resource) + ERR("Resource %p is still in use as depth/stencil buffer.\n", resource); + switch (type) { case WINED3D_RTYPE_TEXTURE_1D:

1 0

Henri Verbeet : wined3d: Get rid of the "d3d_initialized" check in context_resource_released().
by Alexandre Julliard 03 Jun '20

03 Jun '20

Module: wine Branch: master Commit: c125c65846b843ab5f3369a4fa38956d5aca5a21 URL: https://source.winehq.org/git/wine.git/?a=commit;h=c125c65846b843ab5f3369a4… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Jun 3 15:58:04 2020 +0430 wined3d: Get rid of the "d3d_initialized" check in context_resource_released(). When this was introduced in commit 401e99b0c074af848379b2602df37fb1e6c7536b, it protected against calling the equivalent of context_acquire() on the equivalent of the no3d adapter. Commit 94d33d3e861d8e01b4edcb940e8ab1aa98359520 removed the call in question. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/context.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index 7959199e1f..3a263e1f6b 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -36,9 +36,6 @@ void context_resource_released(const struct wined3d_device *device, struct wined { unsigned int i; - if (!device->d3d_initialized) - return; - for (i = 0; i < device->context_count; ++i) { struct wined3d_context *context = device->contexts[i];

1 0

Henri Verbeet : wined3d: Get rid of the "d3d_initialized" check in wined3d_device_reset().
by Alexandre Julliard 03 Jun '20

03 Jun '20

Module: wine Branch: master Commit: 2200b841d566580eeb77655c1a1fc2a797855fe6 URL: https://source.winehq.org/git/wine.git/?a=commit;h=2200b841d566580eeb77655c… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Jun 3 15:58:03 2020 +0430 wined3d: Get rid of the "d3d_initialized" check in wined3d_device_reset(). We require an implicit swapchain, so "d3d_initialized" is always true. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/device.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 9b59371bb2..4544e28bc4 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -5385,8 +5385,7 @@ HRESULT CDECL wined3d_device_reset(struct wined3d_device *device, wined3d_cs_emit_unload_resource(device->cs, resource); } - if (device->d3d_initialized) - device->adapter->adapter_ops->adapter_uninit_3d(device); + device->adapter->adapter_ops->adapter_uninit_3d(device); memset(&device->state, 0, sizeof(device->state)); state_init(&device->state, &device->adapter->d3d_info, WINED3D_STATE_INIT_DEFAULT); @@ -5403,7 +5402,7 @@ HRESULT CDECL wined3d_device_reset(struct wined3d_device *device, wined3d_device_set_depth_stencil_view(device, view); } - if (device->d3d_initialized && reset_state) + if (reset_state) hr = device->adapter->adapter_ops->adapter_init_3d(device); /* All done. There is no need to reload resources or shaders, this will happen automatically on the

1 0

Rémi Bernon : sxs: Fix uninitialized variable warning.
by Alexandre Julliard 03 Jun '20

03 Jun '20

Module: wine Branch: master Commit: 2c9c20c0a6c55daf234318cb529a08646d410e02 URL: https://source.winehq.org/git/wine.git/?a=commit;h=2c9c20c0a6c55daf234318cb… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Wed Jun 3 11:50:36 2020 +0200 sxs: Fix uninitialized variable warning. Reported by MinGW GCC 10 with -Wmaybe-uninitialized, this seems to be a typo. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/sxs/name.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/dlls/sxs/name.c b/dlls/sxs/name.c index a76510a9fe..6b3ae5197a 100644 --- a/dlls/sxs/name.c +++ b/dlls/sxs/name.c @@ -135,7 +135,6 @@ static HRESULT WINAPI name_GetDisplayName( { static const WCHAR fmtW[] = {',','%','s','=','\"','%','s','\"',0}; struct name *name = impl_from_IAssemblyName( iface ); - WCHAR version[30]; unsigned int len; TRACE("%p, %p, %p, 0x%08x\n", iface, buffer, buflen, flags); @@ -146,7 +145,7 @@ static HRESULT WINAPI name_GetDisplayName( if (name->arch) len += lstrlenW( archW ) + lstrlenW( name->arch ) + 4; if (name->token) len += lstrlenW( tokenW ) + lstrlenW( name->token ) + 4; if (name->type) len += lstrlenW( typeW ) + lstrlenW( name->type ) + 4; - if (name->version) len += lstrlenW( versionW ) + lstrlenW( version ) + 4; + if (name->version) len += lstrlenW( versionW ) + lstrlenW( name->version ) + 4; if (len > *buflen) { *buflen = len;

1 0

Rémi Bernon : dxgi/tests: Fix signed integral cast warning.
by Alexandre Julliard 03 Jun '20

03 Jun '20

Module: wine Branch: master Commit: abced34e67587a4441b61e1f8d3b8ca0e9bec31e URL: https://source.winehq.org/git/wine.git/?a=commit;h=abced34e67587a4441b61e1f… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Wed Jun 3 11:50:33 2020 +0200 dxgi/tests: Fix signed integral cast warning. First cast to LONG, then to DWORD. Reported by MinGW GCC 10 with -Woverflow. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dxgi/tests/dxgi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/dxgi/tests/dxgi.c b/dlls/dxgi/tests/dxgi.c index e8dfa036e0..8f52e1d9e7 100644 --- a/dlls/dxgi/tests/dxgi.c +++ b/dlls/dxgi/tests/dxgi.c @@ -1103,7 +1103,7 @@ static void test_check_interface_support(void) hr = IDXGIAdapter_CheckInterfaceSupport(adapter, &IID_ID3D11Device, NULL); ok(hr == DXGI_ERROR_UNSUPPORTED, "Got unexpected hr %#x.\n", hr); - driver_version.HighPart = driver_version.LowPart = 0xdeadbeef; + driver_version.LowPart = driver_version.HighPart = 0xdeadbeef; hr = IDXGIAdapter_CheckInterfaceSupport(adapter, &IID_ID3D11Device, &driver_version); ok(hr == DXGI_ERROR_UNSUPPORTED, "Got unexpected hr %#x.\n", hr); ok(driver_version.HighPart == 0xdeadbeef, "Got unexpected driver version %#x.\n", driver_version.HighPart);

1 0

Rémi Bernon : crypt32/tests: Fix signed integral cast warning.
by Alexandre Julliard 03 Jun '20

03 Jun '20

Module: wine Branch: master Commit: 4e94e2106099a008d64fef0ed3566aab35361534 URL: https://source.winehq.org/git/wine.git/?a=commit;h=4e94e2106099a008d64fef0e… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Wed Jun 3 11:50:32 2020 +0200 crypt32/tests: Fix signed integral cast warning. First cast to int, then to DWORD. Reported by MinGW GCC 10 with -Woverflow. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/crypt32/tests/cert.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/crypt32/tests/cert.c b/dlls/crypt32/tests/cert.c index 880ae720a4..7d6d07fe02 100644 --- a/dlls/crypt32/tests/cert.c +++ b/dlls/crypt32/tests/cert.c @@ -2909,7 +2909,7 @@ static void testGetValidUsages(void) ret = pCertGetValidUsages(0, NULL, NULL, NULL, &size); */ contexts[0] = NULL; - numOIDs = size = 0xdeadbeef; + size = numOIDs = 0xdeadbeef; SetLastError(0xdeadbeef); ret = pCertGetValidUsages(1, &contexts[0], &numOIDs, NULL, &size); ok(ret, "CertGetValidUsages failed: %d\n", GetLastError()); @@ -2921,12 +2921,12 @@ static void testGetValidUsages(void) sizeof(certWithUsage)); contexts[2] = CertCreateCertificateContext(X509_ASN_ENCODING, cert2WithUsage, sizeof(cert2WithUsage)); - numOIDs = size = 0xdeadbeef; + size = numOIDs = 0xdeadbeef; ret = pCertGetValidUsages(0, NULL, &numOIDs, NULL, &size); ok(ret, "CertGetValidUsages failed: %08x\n", GetLastError()); ok(numOIDs == -1, "Expected -1, got %d\n", numOIDs); ok(size == 0, "Expected size 0, got %d\n", size); - numOIDs = size = 0xdeadbeef; + size = numOIDs = 0xdeadbeef; ret = pCertGetValidUsages(1, contexts, &numOIDs, NULL, &size); ok(ret, "CertGetValidUsages failed: %08x\n", GetLastError()); ok(numOIDs == -1, "Expected -1, got %d\n", numOIDs);

1 0

Daniel Lehman : vcruntime140_1: Unwind state 0.
by Alexandre Julliard 03 Jun '20

03 Jun '20

Module: wine Branch: master Commit: b33e1d4cb73cde216d00e15f041ffca3175162b7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b33e1d4cb73cde216d00e15f… Author: Daniel Lehman <dlehman25(a)gmail.com> Date: Wed Jun 3 11:09:40 2020 +0200 vcruntime140_1: Unwind state 0. Signed-off-by: Daniel Lehman <dlehman25(a)gmail.com> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vcruntime140_1/except_x86_64.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/vcruntime140_1/except_x86_64.c b/dlls/vcruntime140_1/except_x86_64.c index 627e9694ec..4203373294 100644 --- a/dlls/vcruntime140_1/except_x86_64.c +++ b/dlls/vcruntime140_1/except_x86_64.c @@ -435,7 +435,7 @@ static void cxx_local_unwind4(ULONG64 frame, DISPATCHER_CONTEXT *dispatch, if (trylevel <= last_level) return; unwind_data = rva_to_ptr(descr->unwind_map, dispatch->ImageBase); - last = unwind_data; + last = unwind_data - 1; for (i = 0; i < trylevel; i++) { BYTE *addr = unwind_data;

1 0

Daniel Lehman : vcruntime140_1: Don't terminate on state of -1.
by Alexandre Julliard 03 Jun '20

03 Jun '20

Module: wine Branch: master Commit: 7d2675d19972ce294d37dc145f85ee8c0fc8d88d URL: https://source.winehq.org/git/wine.git/?a=commit;h=7d2675d19972ce294d37dc14… Author: Daniel Lehman <dlehman25(a)gmail.com> Date: Wed Jun 3 11:09:31 2020 +0200 vcruntime140_1: Don't terminate on state of -1. Signed-off-by: Daniel Lehman <dlehman25(a)gmail.com> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vcruntime140_1/except_x86_64.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/vcruntime140_1/except_x86_64.c b/dlls/vcruntime140_1/except_x86_64.c index 42eab3b046..627e9694ec 100644 --- a/dlls/vcruntime140_1/except_x86_64.c +++ b/dlls/vcruntime140_1/except_x86_64.c @@ -426,7 +426,7 @@ static void cxx_local_unwind4(ULONG64 frame, DISPATCHER_CONTEXT *dispatch, TRACE("current level: %d, last level: %d\n", trylevel, last_level); - if (trylevel<0 || trylevel>=descr->unwind_count) + if (trylevel<-1 || trylevel>=descr->unwind_count) { ERR("invalid trylevel %d\n", trylevel); terminate();

1 0

Daniel Lehman : vcruntime140_1: Skip unwind if already at last level.
by Alexandre Julliard 03 Jun '20

03 Jun '20

Module: wine Branch: master Commit: 94b25a47ba1c92eeb987a1ed6d251a12ce9a30e8 URL: https://source.winehq.org/git/wine.git/?a=commit;h=94b25a47ba1c92eeb987a1ed… Author: Daniel Lehman <dlehman25(a)gmail.com> Date: Wed Jun 3 11:09:19 2020 +0200 vcruntime140_1: Skip unwind if already at last level. Signed-off-by: Daniel Lehman <dlehman25(a)gmail.com> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vcruntime140_1/except_x86_64.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dlls/vcruntime140_1/except_x86_64.c b/dlls/vcruntime140_1/except_x86_64.c index 48f824ae3f..42eab3b046 100644 --- a/dlls/vcruntime140_1/except_x86_64.c +++ b/dlls/vcruntime140_1/except_x86_64.c @@ -432,6 +432,8 @@ static void cxx_local_unwind4(ULONG64 frame, DISPATCHER_CONTEXT *dispatch, terminate(); } + if (trylevel <= last_level) return; + unwind_data = rva_to_ptr(descr->unwind_map, dispatch->ImageBase); last = unwind_data; for (i = 0; i < trylevel; i++)

1 0

Daniel Lehman : vcruntime140_1: Change pointer math in decode_uint.
by Alexandre Julliard 03 Jun '20

03 Jun '20

Module: wine Branch: master Commit: 0f15bfee338d7926ad491969d23268f873c45cc5 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0f15bfee338d7926ad491969… Author: Daniel Lehman <dlehman25(a)gmail.com> Date: Wed Jun 3 11:07:58 2020 +0200 vcruntime140_1: Change pointer math in decode_uint. Signed-off-by: Daniel Lehman <dlehman25(a)gmail.com> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vcruntime140_1/except_x86_64.c | 28 +++++++++++++++------------- 1 file changed, 15 insertions(+), 13 deletions(-) diff --git a/dlls/vcruntime140_1/except_x86_64.c b/dlls/vcruntime140_1/except_x86_64.c index 2c1ed63152..48f824ae3f 100644 --- a/dlls/vcruntime140_1/except_x86_64.c +++ b/dlls/vcruntime140_1/except_x86_64.c @@ -117,34 +117,36 @@ typedef struct static UINT decode_uint(BYTE **b) { UINT ret; + BYTE *p = *b; - if ((**b & 1) == 0) + if ((*p & 1) == 0) { - ret = *b[0] >> 1; - *b += 1; + ret = p[0] >> 1; + p += 1; } - else if ((**b & 3) == 1) + else if ((*p & 3) == 1) { - ret = (*b[0] >> 2) + (*b[1] << 6); - *b += 2; + ret = (p[0] >> 2) + (p[1] << 6); + p += 2; } - else if ((**b & 7) == 3) + else if ((*p & 7) == 3) { - ret = (*b[0] >> 3) + (*b[1] << 5) + (*b[2] << 13); - *b += 3; + ret = (p[0] >> 3) + (p[1] << 5) + (p[2] << 13); + p += 3; } - else if ((**b & 15) == 7) + else if ((*p & 15) == 7) { - ret = (*b[0] >> 4) + (*b[1] << 4) + (*b[2] << 12) + (*b[3] << 20); - *b += 4; + ret = (p[0] >> 4) + (p[1] << 4) + (p[2] << 12) + (p[3] << 20); + p += 4; } else { FIXME("not implemented - expect crash\n"); ret = 0; - *b += 5; + p += 5; } + *b = p; return ret; }

1 0

← Newer
1
...
74
75
76
77
78
79
80
...
95
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits June 2020