July 2022 - Wine-Commits - list.winehq.org

Rémi Bernon : widl: Fix missing support for struct / union parameterized type replacement.
by Alexandre Julliard July 7, 2022

July 7, 2022

Module: wine Branch: master Commit: 25e1fb45db5c31faff0dab4d26737d9800623742 URL: https://source.winehq.org/git/wine.git/?a=commit;h=25e1fb45db5c31faff0dab4d… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Sun Feb 13 10:52:44 2022 +0100 widl: Fix missing support for struct / union parameterized type replacement. And we can just pass them through, as with other class types. This fixes widl not accepting EventRegistrationToken method argument type in a parameterized type. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> --- tools/widl/typetree.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tools/widl/typetree.c b/tools/widl/typetree.c index 800ddf77219..3ad140d520c 100644 --- a/tools/widl/typetree.c +++ b/tools/widl/typetree.c @@ -1157,6 +1157,9 @@ static type_t *replace_type_parameters_in_type(type_t *type, typeref_list_t *ori case TYPE_INTERFACE: case TYPE_RUNTIMECLASS: case TYPE_DELEGATE: + case TYPE_STRUCT: + case TYPE_ENCAPSULATED_UNION: + case TYPE_UNION: return type; case TYPE_PARAMETER: if (!orig || !repl) return NULL; @@ -1194,9 +1197,6 @@ static type_t *replace_type_parameters_in_type(type_t *type, typeref_list_t *ori if (t->type_type != TYPE_PARAMETERIZED_TYPE) return find_parameterized_type(type, repl); repl = replace_type_parameters_in_type_list(type->details.parameterized.params, orig, repl); return replace_type_parameters_in_type(t, t->details.parameterized.params, repl); - case TYPE_STRUCT: - case TYPE_ENCAPSULATED_UNION: - case TYPE_UNION: case TYPE_MODULE: case TYPE_COCLASS: case TYPE_APICONTRACT:

1 0

Dmitry Timoshkov : include: Fix a typo in _InterlockedXor64() intrinsic declaration.
by Alexandre Julliard July 7, 2022

July 7, 2022

Module: wine Branch: master Commit: 82c30480c98cf57b66601c9f9b0b24561ad112ce URL: https://source.winehq.org/git/wine.git/?a=commit;h=82c30480c98cf57b66601c9f… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Thu Jul 7 12:44:05 2022 +0300 include: Fix a typo in _InterlockedXor64() intrinsic declaration. Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/winnt.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/winnt.h b/include/winnt.h index 50f6452b145..5de6884e7de 100644 --- a/include/winnt.h +++ b/include/winnt.h @@ -6421,7 +6421,7 @@ DECLSPEC_NORETURN void __fastfail(unsigned int); #pragma intrinsic(_InterlockedExchangePointer) #pragma intrinsic(_InterlockedIncrement64) #pragma intrinsic(_InterlockedOr64) -#pragma intrinsic(_InterlockedXord64) +#pragma intrinsic(_InterlockedXor64) __int64 _InterlockedAnd64(__int64 volatile *, __int64); void * _InterlockedCompareExchangePointer(void *volatile*,void*,void*);

1 0

Roman Pišl : wpcap: Implement pcap_dispatch.
by Alexandre Julliard July 7, 2022

July 7, 2022

Module: wine Branch: master Commit: 556077e674ce4a44bf1ae2ea2e4dd5b2501ce175 URL: https://source.winehq.org/git/wine.git/?a=commit;h=556077e674ce4a44bf1ae2ea… Author: Roman Pišl <rpisl(a)seznam.cz> Date: Tue Jul 5 12:11:03 2022 +0200 wpcap: Implement pcap_dispatch. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53104 Signed-off-by: Roman Pišl <rpisl(a)seznam.cz> Signed-off-by: André Zwing <nerv(a)dawncrow.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wpcap/wpcap.c | 41 ++++++++++++++++++++++++++++++++--------- 1 file changed, 32 insertions(+), 9 deletions(-) diff --git a/dlls/wpcap/wpcap.c b/dlls/wpcap/wpcap.c index a295819f7fe..df35954bfa7 100644 --- a/dlls/wpcap/wpcap.c +++ b/dlls/wpcap/wpcap.c @@ -99,15 +99,6 @@ const char * CDECL pcap_datalink_val_to_name( int link ) return ret; } -int CDECL pcap_dispatch( struct pcap *pcap, int count, - void (CALLBACK *callback)(unsigned char *, const struct pcap_pkthdr_win32 *, const unsigned char *), - unsigned char *user ) -{ - /* FIXME: reimplement on top of pcap_next_ex */ - FIXME( "%p, %d, %p, %p: not implemented\n", pcap, count, callback, user ); - return -1; -} - void CDECL pcap_dump( unsigned char *user, const struct pcap_pkthdr_win32 *hdr, const unsigned char *packet ) { struct dump_params params = { user, hdr, packet }; @@ -543,6 +534,38 @@ const unsigned char * CDECL pcap_next( struct pcap *pcap, struct pcap_pkthdr_win return data; } +int CDECL pcap_dispatch( struct pcap *pcap, int count, + void (CALLBACK *callback)(unsigned char *, const struct pcap_pkthdr_win32 *, const unsigned char *), + unsigned char *user ) +{ + int processed = 0; + TRACE( "%p, %d, %p, %p\n", pcap, count, callback, user ); + + while (processed < count) + { + struct pcap_pkthdr_win32 *hdr = NULL; + const unsigned char *data = NULL; + + int ret = pcap_next_ex( pcap, &hdr, &data ); + + if (ret == 1) + processed++; + else if (ret == 0) + break; + else if (ret == -2) + { + if (processed == 0) return -2; + break; + } + else + return ret; + + callback( user, hdr, data ); + } + + return processed; +} + static char *strdupWA( const WCHAR *src ) { char *dst;

1 0

Zhiyi Zhang : atl: Avoid NULL pointer reference in AtlComModuleRegisterClassObjects().
by Alexandre Julliard July 7, 2022

July 7, 2022

Module: wine Branch: master Commit: f4890dac17adbba96375c320cc50293949cc0eaf URL: https://source.winehq.org/git/wine.git/?a=commit;h=f4890dac17adbba96375c320… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Fri Jul 1 10:41:16 2022 +0800 atl: Avoid NULL pointer reference in AtlComModuleRegisterClassObjects(). Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> --- dlls/atl/atl.c | 4 ++-- dlls/atl100/tests/atl.c | 28 ++++++++++++++++++++++++++++ dlls/atl110/tests/atl.c | 28 ++++++++++++++++++++++++++++ 3 files changed, 58 insertions(+), 2 deletions(-) diff --git a/dlls/atl/atl.c b/dlls/atl/atl.c index d501e7a6d76..5a302621d60 100644 --- a/dlls/atl/atl.c +++ b/dlls/atl/atl.c @@ -538,7 +538,7 @@ HRESULT WINAPI AtlComModuleRegisterClassObjects(_ATL_COM_MODULE *module, DWORD c return E_INVALIDARG; for(iter = module->m_ppAutoObjMapFirst; iter < module->m_ppAutoObjMapLast; iter++) { - if(!(*iter)->pfnGetClassObject) + if(!(*iter) || !(*iter)->pfnGetClassObject) continue; hres = (*iter)->pfnGetClassObject((*iter)->pfnCreateInstance, &IID_IUnknown, (void**)&unk); @@ -566,7 +566,7 @@ HRESULT WINAPI AtlComModuleRegisterClassObjects(_ATL_COM_MODULE *module, DWORD c return E_INVALIDARG; for(iter = module->m_ppAutoObjMapFirst; iter < module->m_ppAutoObjMapLast; iter++) { - if(!(*iter)->pfnGetClassObject) + if(!(*iter) || !(*iter)->pfnGetClassObject) continue; hres = (*iter)->pfnGetClassObject((*iter)->pfnCreateInstance, &IID_IUnknown, (void**)&unk); diff --git a/dlls/atl100/tests/atl.c b/dlls/atl100/tests/atl.c index e002af0d24c..7da6e5a9cd6 100644 --- a/dlls/atl100/tests/atl.c +++ b/dlls/atl100/tests/atl.c @@ -1088,6 +1088,33 @@ static void test_AtlComModuleGetClassObject(void) ok(hr == CLASS_E_CLASSNOTAVAILABLE, "Unexpected hr %#lx.\n", hr); } +static void test_AtlComModuleRegisterClassObjects(void) +{ + _ATL_OBJMAP_ENTRY *null_entry = NULL; + _ATL_COM_MODULE module; + HRESULT hr; + + /* Test NULL module */ + hr = AtlComModuleRegisterClassObjects(NULL, CLSCTX_INPROC_SERVER, REGCLS_MULTIPLEUSE); + ok(hr == E_INVALIDARG, "Unexpected hr %#lx.\n", hr); + + /* Test NULL m_ppAutoObjMapFirst and m_ppAutoObjMapLast */ + module.cbSize = sizeof(module); + module.m_ppAutoObjMapFirst = NULL; + module.m_ppAutoObjMapLast = NULL; + hr = AtlComModuleRegisterClassObjects(&module, CLSCTX_INPROC_SERVER, REGCLS_MULTIPLEUSE); + todo_wine_if(hr == S_OK) + ok(hr == S_FALSE, "Unexpected hr %#lx.\n", hr); + + /* Test m_ppAutoObjMapFirst and m_ppAutoObjMapLast both pointing to a NULL entry */ + module.cbSize = sizeof(module); + module.m_ppAutoObjMapFirst = &null_entry; + module.m_ppAutoObjMapLast = &null_entry; + hr = AtlComModuleRegisterClassObjects(&module, CLSCTX_INPROC_SERVER, REGCLS_MULTIPLEUSE); + todo_wine_if(hr == S_OK) + ok(hr == S_FALSE, "Unexpected hr %#lx.\n", hr); +} + START_TEST(atl) { if (!register_class()) @@ -1104,6 +1131,7 @@ START_TEST(atl) test_AtlAxAttachControl(); test_AtlAxCreateControl(); test_AtlComModuleGetClassObject(); + test_AtlComModuleRegisterClassObjects(); CoUninitialize(); } diff --git a/dlls/atl110/tests/atl.c b/dlls/atl110/tests/atl.c index cb8f667596b..9e4bad87e97 100644 --- a/dlls/atl110/tests/atl.c +++ b/dlls/atl110/tests/atl.c @@ -55,11 +55,39 @@ static void test_AtlComModuleGetClassObject(void) ok(hr == CLASS_E_CLASSNOTAVAILABLE, "Unexpected hr %#lx.\n", hr); } +static void test_AtlComModuleRegisterClassObjects(void) +{ + _ATL_OBJMAP_ENTRY_EX *null_entry = NULL; + _ATL_COM_MODULE module; + HRESULT hr; + + /* Test NULL module */ + hr = AtlComModuleRegisterClassObjects(NULL, CLSCTX_INPROC_SERVER, REGCLS_MULTIPLEUSE); + ok(hr == E_INVALIDARG, "Unexpected hr %#lx.\n", hr); + + /* Test NULL m_ppAutoObjMapFirst and m_ppAutoObjMapLast */ + module.cbSize = sizeof(module); + module.m_ppAutoObjMapFirst = NULL; + module.m_ppAutoObjMapLast = NULL; + hr = AtlComModuleRegisterClassObjects(&module, CLSCTX_INPROC_SERVER, REGCLS_MULTIPLEUSE); + todo_wine_if(hr == S_OK) + ok(hr == S_FALSE, "Unexpected hr %#lx.\n", hr); + + /* Test m_ppAutoObjMapFirst and m_ppAutoObjMapLast both pointing to a NULL entry */ + module.cbSize = sizeof(module); + module.m_ppAutoObjMapFirst = &null_entry; + module.m_ppAutoObjMapLast = &null_entry; + hr = AtlComModuleRegisterClassObjects(&module, CLSCTX_INPROC_SERVER, REGCLS_MULTIPLEUSE); + todo_wine_if(hr == S_OK) + ok(hr == S_FALSE, "Unexpected hr %#lx.\n", hr); +} + START_TEST(atl) { CoInitialize(NULL); test_AtlComModuleGetClassObject(); + test_AtlComModuleRegisterClassObjects(); CoUninitialize(); }

1 0

Alexandre Julliard : server: Use standard named object functions for registry keys.
by Alexandre Julliard July 7, 2022

July 7, 2022

Module: wine Branch: master Commit: 569c27945e9479e9b27a5ffaa9b0587e6f1f2ac7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=569c27945e9479e9b27a5ffa… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jul 7 12:32:22 2022 +0200 server: Use standard named object functions for registry keys. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/tests/om.c | 4 - dlls/ntdll/tests/reg.c | 32 +--- server/registry.c | 487 +++++++++++++++---------------------------------- 3 files changed, 152 insertions(+), 371 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=569c27945e9479e9b27a…

1 0

Zhiyi Zhang : atl: Avoid NULL pointer reference in AtlComModuleGetClassObject().
by Alexandre Julliard July 7, 2022

July 7, 2022

Module: wine Branch: master Commit: 5739db59d4c9c379776257d99a3e447f078e43bf URL: https://source.winehq.org/git/wine.git/?a=commit;h=5739db59d4c9c379776257d9… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Tue Jun 28 11:57:28 2022 +0800 atl: Avoid NULL pointer reference in AtlComModuleGetClassObject(). Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> --- configure | 1 + configure.ac | 1 + dlls/atl/atl.c | 4 +-- dlls/atl100/tests/atl.c | 27 ++++++++++++++++++ dlls/atl110/tests/Makefile.in | 6 ++++ dlls/atl110/tests/atl.c | 65 +++++++++++++++++++++++++++++++++++++++++++ 6 files changed, 102 insertions(+), 2 deletions(-) diff --git a/configure b/configure index e563a2f9502..d60418ff60a 100755 --- a/configure +++ b/configure @@ -21227,6 +21227,7 @@ wine_fn_config_makefile dlls/atl/tests enable_tests wine_fn_config_makefile dlls/atl100 enable_atl100 wine_fn_config_makefile dlls/atl100/tests enable_tests wine_fn_config_makefile dlls/atl110 enable_atl110 +wine_fn_config_makefile dlls/atl110/tests enable_tests wine_fn_config_makefile dlls/atl80 enable_atl80 wine_fn_config_makefile dlls/atl80/tests enable_tests wine_fn_config_makefile dlls/atl90 enable_atl90 diff --git a/configure.ac b/configure.ac index d5791bab7b1..5ea0d9120f0 100644 --- a/configure.ac +++ b/configure.ac @@ -2394,6 +2394,7 @@ WINE_CONFIG_MAKEFILE(dlls/atl/tests) WINE_CONFIG_MAKEFILE(dlls/atl100) WINE_CONFIG_MAKEFILE(dlls/atl100/tests) WINE_CONFIG_MAKEFILE(dlls/atl110) +WINE_CONFIG_MAKEFILE(dlls/atl110/tests) WINE_CONFIG_MAKEFILE(dlls/atl80) WINE_CONFIG_MAKEFILE(dlls/atl80/tests) WINE_CONFIG_MAKEFILE(dlls/atl90) diff --git a/dlls/atl/atl.c b/dlls/atl/atl.c index 87a24e33ed7..d501e7a6d76 100644 --- a/dlls/atl/atl.c +++ b/dlls/atl/atl.c @@ -482,7 +482,7 @@ HRESULT WINAPI AtlComModuleGetClassObject(_ATL_COM_MODULE *pm, REFCLSID rclsid, return E_INVALIDARG; for(iter = pm->m_ppAutoObjMapFirst; iter < pm->m_ppAutoObjMapLast; iter++) { - if(IsEqualCLSID((*iter)->pclsid, rclsid) && (*iter)->pfnGetClassObject) { + if(*iter && IsEqualCLSID((*iter)->pclsid, rclsid) && (*iter)->pfnGetClassObject) { if(!(*iter)->pCF) hres = (*iter)->pfnGetClassObject((*iter)->pfnCreateInstance, &IID_IUnknown, (void**)&(*iter)->pCF); if((*iter)->pCF) @@ -507,7 +507,7 @@ HRESULT WINAPI AtlComModuleGetClassObject(_ATL_COM_MODULE *pm, REFCLSID rclsid, return E_INVALIDARG; for(iter = pm->m_ppAutoObjMapFirst; iter < pm->m_ppAutoObjMapLast; iter++) { - if(IsEqualCLSID((*iter)->pclsid, rclsid) && (*iter)->pfnGetClassObject) { + if(*iter && IsEqualCLSID((*iter)->pclsid, rclsid) && (*iter)->pfnGetClassObject) { if(!(*iter)->pCache->pCF) hres = (*iter)->pfnGetClassObject((*iter)->pfnCreateInstance, &IID_IUnknown, (void**)&(*iter)->pCache->pCF); if((*iter)->pCache->pCF) diff --git a/dlls/atl100/tests/atl.c b/dlls/atl100/tests/atl.c index e161878f8ea..e002af0d24c 100644 --- a/dlls/atl100/tests/atl.c +++ b/dlls/atl100/tests/atl.c @@ -1062,6 +1062,32 @@ static void test_AtlAxCreateControl(void) DestroyWindow(hwnd); } +static void test_AtlComModuleGetClassObject(void) +{ + _ATL_OBJMAP_ENTRY *null_entry = NULL; + _ATL_COM_MODULE module; + HRESULT hr; + void *ret; + + /* Test NULL module */ + hr = AtlComModuleGetClassObject(NULL, &GUID_NULL, &IID_NULL, &ret); + ok(hr == E_INVALIDARG, "Unexpected hr %#lx.\n", hr); + + /* Test NULL m_ppAutoObjMapFirst and m_ppAutoObjMapLast */ + module.cbSize = sizeof(module); + module.m_ppAutoObjMapFirst = NULL; + module.m_ppAutoObjMapLast = NULL; + hr = AtlComModuleGetClassObject(&module, &GUID_NULL, &IID_NULL, &ret); + ok(hr == CLASS_E_CLASSNOTAVAILABLE, "Unexpected hr %#lx.\n", hr); + + /* Test m_ppAutoObjMapFirst and m_ppAutoObjMapLast both pointing to a NULL entry */ + module.cbSize = sizeof(module); + module.m_ppAutoObjMapFirst = &null_entry; + module.m_ppAutoObjMapLast = &null_entry; + hr = AtlComModuleGetClassObject(&module, &GUID_NULL, &IID_NULL, &ret); + ok(hr == CLASS_E_CLASSNOTAVAILABLE, "Unexpected hr %#lx.\n", hr); +} + START_TEST(atl) { if (!register_class()) @@ -1077,6 +1103,7 @@ START_TEST(atl) test_ax_win(); test_AtlAxAttachControl(); test_AtlAxCreateControl(); + test_AtlComModuleGetClassObject(); CoUninitialize(); } diff --git a/dlls/atl110/tests/Makefile.in b/dlls/atl110/tests/Makefile.in new file mode 100644 index 00000000000..6374117e1be --- /dev/null +++ b/dlls/atl110/tests/Makefile.in @@ -0,0 +1,6 @@ +TESTDLL = atl110.dll +IMPORTS = uuid ole32 atl110 +EXTRADEFS = -D_ATL_VER=_ATL_VER_110 + +C_SRCS = \ + atl.c diff --git a/dlls/atl110/tests/atl.c b/dlls/atl110/tests/atl.c new file mode 100644 index 00000000000..cb8f667596b --- /dev/null +++ b/dlls/atl110/tests/atl.c @@ -0,0 +1,65 @@ +/* + * Copyright 2022 Zhiyi Zhang for CodeWeavers + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include <stdarg.h> +#include <stdio.h> + +#define COBJMACROS +#define CONST_VTABLE + +#include <windef.h> +#include <winbase.h> +#include <winuser.h> +#include <atlbase.h> + +#include <wine/test.h> + +static void test_AtlComModuleGetClassObject(void) +{ + _ATL_OBJMAP_ENTRY_EX *null_entry = NULL; + _ATL_COM_MODULE module; + HRESULT hr; + void *ret; + + /* Test NULL module */ + hr = AtlComModuleGetClassObject(NULL, &GUID_NULL, &IID_NULL, &ret); + ok(hr == E_INVALIDARG, "Unexpected hr %#lx.\n", hr); + + /* Test NULL m_ppAutoObjMapFirst and m_ppAutoObjMapLast */ + module.cbSize = sizeof(module); + module.m_ppAutoObjMapFirst = NULL; + module.m_ppAutoObjMapLast = NULL; + hr = AtlComModuleGetClassObject(&module, &GUID_NULL, &IID_NULL, &ret); + ok(hr == CLASS_E_CLASSNOTAVAILABLE, "Unexpected hr %#lx.\n", hr); + + /* Test m_ppAutoObjMapFirst and m_ppAutoObjMapLast both pointing to a NULL entry */ + module.cbSize = sizeof(module); + module.m_ppAutoObjMapFirst = &null_entry; + module.m_ppAutoObjMapLast = &null_entry; + hr = AtlComModuleGetClassObject(&module, &GUID_NULL, &IID_NULL, &ret); + ok(hr == CLASS_E_CLASSNOTAVAILABLE, "Unexpected hr %#lx.\n", hr); +} + +START_TEST(atl) +{ + CoInitialize(NULL); + + test_AtlComModuleGetClassObject(); + + CoUninitialize(); +}

1 0

Zebediah Figura : d3d8/tests: Make getPixelColor() checks into actual ok() calls.
by Alexandre Julliard July 7, 2022

July 7, 2022

Module: wine Branch: master Commit: dae1c93f5cc8c2f071fc22deaf2a95b1c7768195 URL: https://source.winehq.org/git/wine.git/?a=commit;h=dae1c93f5cc8c2f071fc22de… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Thu Jul 7 03:46:04 2022 +0000 d3d8/tests: Make getPixelColor() checks into actual ok() calls. These should never fail. --- dlls/d3d8/tests/visual.c | 92 +++++++++++++++++++++++++----------------------- 1 file changed, 48 insertions(+), 44 deletions(-) diff --git a/dlls/d3d8/tests/visual.c b/dlls/d3d8/tests/visual.c index fa0030b2187..bd600b44c2f 100644 --- a/dlls/d3d8/tests/visual.c +++ b/dlls/d3d8/tests/visual.c @@ -114,56 +114,28 @@ static void get_rt_readback(IDirect3DSurface8 *surface, struct surface_readback hr = IDirect3DSurface8_GetDesc(surface, &desc); ok(SUCCEEDED(hr), "Failed to get surface desc, hr %#x.\n", hr); hr = IDirect3DDevice8_CreateTexture(device, desc.Width, desc.Height, 1, 0, desc.Format, D3DPOOL_SYSTEMMEM, &tex); - if (FAILED(hr) || !tex) - { - trace("Can't create an offscreen plain surface to read the render target data, hr %#x.\n", hr); - goto error; - } + ok(hr == S_OK, "Got hr %#x.\n", hr); hr = IDirect3DTexture8_GetSurfaceLevel(tex, 0, &rb->surface); - if (FAILED(hr)) - { - trace("Can't get surface from texture, hr %#x.\n", hr); - goto error; - } + ok(hr == S_OK, "Got hr %#x.\n", hr); hr = IDirect3DDevice8_CopyRects(device, surface, NULL, 0, rb->surface, NULL); - if (FAILED(hr)) - { - trace("Can't read the render target, hr %#x.\n", hr); - goto error; - } + ok(hr == S_OK, "Got hr %#x.\n", hr); hr = IDirect3DSurface8_LockRect(rb->surface, &rb->locked_rect, NULL, D3DLOCK_READONLY); - if (FAILED(hr)) - { - trace("Can't lock the offscreen surface, hr %#x.\n", hr); - goto error; - } + ok(hr == S_OK, "Got hr %#x.\n", hr); IDirect3DTexture8_Release(tex); IDirect3DDevice8_Release(device); - return; - -error: - if (rb->surface) - IDirect3DSurface8_Release(rb->surface); - rb->surface = NULL; - if (tex) - IDirect3DTexture8_Release(tex); - IDirect3DDevice8_Release(device); } static DWORD get_readback_color(struct surface_readback *rb, unsigned int x, unsigned int y) { - return rb->locked_rect.pBits - ? ((DWORD *)rb->locked_rect.pBits)[y * rb->locked_rect.Pitch / sizeof(DWORD) + x] : 0xdeadbeef; + return ((DWORD *)rb->locked_rect.pBits)[y * rb->locked_rect.Pitch / sizeof(DWORD) + x]; } static void release_surface_readback(struct surface_readback *rb) { HRESULT hr; - if (!rb->surface) - return; - if (rb->locked_rect.pBits && FAILED(hr = IDirect3DSurface8_UnlockRect(rb->surface))) - trace("Can't unlock the offscreen surface, hr %#x.\n", hr); + hr = IDirect3DSurface8_UnlockRect(rb->surface); + ok(hr == S_OK, "Got hr %#x.\n", hr); IDirect3DSurface8_Release(rb->surface); } @@ -175,11 +147,7 @@ static DWORD getPixelColor(IDirect3DDevice8 *device, UINT x, UINT y) HRESULT hr; hr = IDirect3DDevice8_GetRenderTarget(device, &rt); - if (FAILED(hr)) - { - trace("Can't get the render target, hr %#x.\n", hr); - return 0xdeadbeef; - } + ok(hr == S_OK, "Got hr %#x.\n", hr); get_rt_readback(rt, &rb); /* Remove the X channel for now. DirectX and OpenGL have different ideas how to treat it apparently, and it isn't @@ -6812,6 +6780,7 @@ done: static void test_updatetexture(void) { + D3DADAPTER_IDENTIFIER8 identifier; IDirect3DDevice8 *device; IDirect3D8 *d3d; HWND window; @@ -6922,6 +6891,8 @@ static void test_updatetexture(void) return; } + hr = IDirect3D8_GetAdapterIdentifier(d3d, D3DADAPTER_DEFAULT, 0, &identifier); + ok(hr == S_OK, "Got hr %#x.\n", hr); hr = IDirect3DDevice8_GetDeviceCaps(device, &caps); ok(SUCCEEDED(hr), "Failed to get caps, hr %#x.\n", hr); @@ -7110,6 +7081,11 @@ static void test_updatetexture(void) if (do_visual_test) { + IDirect3DTexture8 *rb_texture; + struct surface_readback rb; + IDirect3DSurface8 *rt; + D3DSURFACE_DESC desc; + hr = IDirect3DDevice8_SetTexture(device, 0, dst); ok(SUCCEEDED(hr), "Failed to set texture, hr %#x.\n", hr); @@ -7124,10 +7100,38 @@ static void test_updatetexture(void) hr = IDirect3DDevice8_EndScene(device); ok(SUCCEEDED(hr), "Failed to end scene, hr %#x.\n", hr); - color = getPixelColor(device, 320, 240); - ok (color_match(color, 0x007f7f00, 3) || broken(tests[i].broken_result) - || broken(color == 0x00adbeef), /* WARP device often just breaks down. */ - "Got unexpected color 0x%08x, case %u, %u.\n", color, t, i); + /* Read back manually. WARP often completely breaks down with + * these draws and fails to copy to the staging texture. */ + + hr = IDirect3DDevice8_GetRenderTarget(device, &rt); + ok(hr == S_OK, "Got hr %#x.\n", hr); + + hr = IDirect3DSurface8_GetDesc(rt, &desc); + ok(hr == S_OK, "Got hr %#x.\n", hr); + hr = IDirect3DDevice8_CreateTexture(device, desc.Width, desc.Height, + 1, 0, desc.Format, D3DPOOL_SYSTEMMEM, &rb_texture); + ok(hr == S_OK, "Got hr %#x.\n", hr); + hr = IDirect3DTexture8_GetSurfaceLevel(rb_texture, 0, &rb.surface); + ok(hr == S_OK, "Got hr %#x.\n", hr); + + hr = IDirect3DDevice8_CopyRects(device, rt, NULL, 0, rb.surface, NULL); + ok(hr == S_OK || broken(adapter_is_warp(&identifier)), "Got hr %#x.\n", hr); + if (SUCCEEDED(hr)) + { + hr = IDirect3DSurface8_LockRect(rb.surface, &rb.locked_rect, NULL, D3DLOCK_READONLY); + ok(hr == S_OK, "Got hr %#x.\n", hr); + + color = get_readback_color(&rb, 320, 240) & 0x00ffffff; + ok(color_match(color, 0x007f7f00, 3) || broken(tests[i].broken_result), + "Got unexpected color 0x%08x, case %u, %u.\n", color, t, i); + + hr = IDirect3DSurface8_UnlockRect(rb.surface); + ok(hr == S_OK, "Got hr %#x.\n", hr); + } + + IDirect3DSurface8_Release(rb.surface); + IDirect3DTexture8_Release(rb_texture); + IDirect3DSurface8_Release(rt); } IDirect3DBaseTexture8_Release(src);

1 0

Zebediah Figura : d3d8/tests: Build without -DWINE_NO_LONG_TYPES.
by Alexandre Julliard July 7, 2022

July 7, 2022

Module: wine Branch: master Commit: 43f23d3e63f3943b286400b151a1ff4adc9250b4 URL: https://source.winehq.org/git/wine.git/?a=commit;h=43f23d3e63f3943b286400b1… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Thu Jul 7 03:46:05 2022 +0000 d3d8/tests: Build without -DWINE_NO_LONG_TYPES. --- dlls/d3d8/tests/Makefile.in | 1 - dlls/d3d8/tests/device.c | 2538 ++++++++++++++++---------------- dlls/d3d8/tests/stateblock.c | 118 +- dlls/d3d8/tests/visual.c | 3278 +++++++++++++++++++++--------------------- dlls/d3d9/tests/d3d9ex.c | 3 +- 5 files changed, 2942 insertions(+), 2996 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=43f23d3e63f3943b2864…

1 0

Zhiyi Zhang : atl110: Add importlib.
by Alexandre Julliard July 7, 2022

July 7, 2022

Module: wine Branch: master Commit: b6225478e5b255777e9df7c6aee5941c22673d42 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b6225478e5b255777e9df7c6… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Wed Jul 6 21:26:18 2022 +0800 atl110: Add importlib. Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> --- dlls/atl110/Makefile.in | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/atl110/Makefile.in b/dlls/atl110/Makefile.in index 49ba9338125..d33d30d2820 100644 --- a/dlls/atl110/Makefile.in +++ b/dlls/atl110/Makefile.in @@ -1,4 +1,5 @@ MODULE = atl110.dll +IMPORTLIB = atl110 IMPORTS = oleaut32 ole32 user32 gdi32 advapi32 uuid shlwapi EXTRADEFS = -D_ATL_VER=_ATL_VER_110 PARENTSRC = ../atl

1 0

Alexandre Julliard : server: Implement lookup_name() for registry keys.
by Alexandre Julliard July 7, 2022

July 7, 2022

Module: wine Branch: master Commit: b79ff648ef6b18ac49c87571a3f78f731af78d25 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b79ff648ef6b18ac49c87571… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jul 7 11:52:59 2022 +0200 server: Implement lookup_name() for registry keys. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- server/registry.c | 87 ++++++++++++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 86 insertions(+), 1 deletion(-) diff --git a/server/registry.c b/server/registry.c index 2c52ec85d29..f800bcd842c 100644 --- a/server/registry.c +++ b/server/registry.c @@ -164,6 +164,8 @@ static void key_dump( struct object *obj, int verbose ); static unsigned int key_map_access( struct object *obj, unsigned int access ); static struct security_descriptor *key_get_sd( struct object *obj ); static WCHAR *key_get_full_name( struct object *obj, data_size_t *len ); +static struct object *key_lookup_name( struct object *obj, struct unicode_str *name, + unsigned int attr, struct object *root ); static int key_link_name( struct object *obj, struct object_name *name, struct object *parent ); static void key_unlink_name( struct object *obj, struct object_name *name ); static int key_close_handle( struct object *obj, struct process *process, obj_handle_t handle ); @@ -184,7 +186,7 @@ static const struct object_ops key_ops = key_get_sd, /* get_sd */ default_set_sd, /* set_sd */ key_get_full_name, /* get_full_name */ - no_lookup_name, /* lookup_name */ + key_lookup_name, /* lookup_name */ key_link_name, /* link_name */ key_unlink_name, /* unlink_name */ no_open_file, /* open_file */ @@ -485,6 +487,89 @@ static WCHAR *key_get_full_name( struct object *obj, data_size_t *ret_len ) return (WCHAR *)ret; } +static struct object *key_lookup_name( struct object *obj, struct unicode_str *name, + unsigned int attr, struct object *root ) +{ + struct key *found, *key = (struct key *)obj; + struct unicode_str tmp; + data_size_t next; + int index; + + assert( obj->ops == &key_ops ); + + if (!name) return NULL; /* open the key itself */ + + if (key->flags & KEY_DELETED) + { + set_error( STATUS_KEY_DELETED ); + return NULL; + } + + if (key->flags & KEY_SYMLINK) + { + struct unicode_str name_left; + struct key_value *value; + + if (!name->len && (attr & OBJ_OPENLINK)) return NULL; + + if (!(value = find_value( key, &symlink_str, &index )) || + value->len < sizeof(WCHAR) || *(WCHAR *)value->data != '\\') + { + set_error( STATUS_OBJECT_NAME_NOT_FOUND ); + return NULL; + } + tmp.str = value->data; + tmp.len = (value->len / sizeof(WCHAR)) * sizeof(WCHAR); + if ((obj = lookup_named_object( NULL, &tmp, OBJ_CASE_INSENSITIVE, &name_left ))) + { + if (!name->len) *name = name_left; /* symlink destination can be created if missing */ + else if (name_left.len) /* symlink must have been resolved completely */ + { + release_object( obj ); + obj = NULL; + set_error( STATUS_OBJECT_NAME_NOT_FOUND ); + } + } + return obj; + } + + if (!name->str) return NULL; + + tmp.str = name->str; + tmp.len = get_path_element( name->str, name->len ); + + if (tmp.len > MAX_NAME_LEN * sizeof(WCHAR)) + { + set_error( STATUS_INVALID_PARAMETER ); + return 0; + } + + /* skip trailing backslashes */ + for (next = tmp.len; next < name->len; next += sizeof(WCHAR)) + if (name->str[next / sizeof(WCHAR)] != '\\') break; + + if (!(found = find_subkey( key, &tmp, &index ))) + { + if (next < name->len) /* path still has elements */ + set_error( STATUS_OBJECT_NAME_NOT_FOUND ); + else /* only trailing backslashes */ + name->len = tmp.len; + return NULL; + } + + if (next < name->len) /* move to the next element */ + { + name->str += next / sizeof(WCHAR); + name->len -= next; + } + else + { + name->str = NULL; + name->len = 0; + } + return grab_object( found ); +} + static int key_link_name( struct object *obj, struct object_name *name, struct object *parent ) { struct key *key = (struct key *)obj;

1 0