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

Zhiyi Zhang : uxtheme/tests: Use the GetDpiForSystem() function pointer.
by Alexandre Julliard 13 Jul '22

13 Jul '22

Module: wine Branch: master Commit: c1c3c68e9fcd7b9d8981a314f02761c4061abfe3 URL: https://gitlab.winehq.org/wine/wine/-/commit/c1c3c68e9fcd7b9d8981a314f02761… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Wed Jul 13 17:35:48 2022 +0800 uxtheme/tests: Use the GetDpiForSystem() function pointer. GetDpiForSystem() requires Windows 10 1607+. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53341 Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> --- dlls/uxtheme/tests/system.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/dlls/uxtheme/tests/system.c b/dlls/uxtheme/tests/system.c index 22762ecd4a9..debc712a73c 100644 --- a/dlls/uxtheme/tests/system.c +++ b/dlls/uxtheme/tests/system.c @@ -52,6 +52,7 @@ static HRESULT (WINAPI *pGetThemeTransitionDuration)(HTHEME, int, int, int, int, static LONG (WINAPI *pDisplayConfigGetDeviceInfo)(DISPLAYCONFIG_DEVICE_INFO_HEADER *); static LONG (WINAPI *pDisplayConfigSetDeviceInfo)(DISPLAYCONFIG_DEVICE_INFO_HEADER *); static BOOL (WINAPI *pGetDpiForMonitorInternal)(HMONITOR, UINT, UINT *, UINT *); +static UINT (WINAPI *pGetDpiForSystem)(void); static DPI_AWARENESS_CONTEXT (WINAPI *pSetThreadDpiAwarenessContext)(DPI_AWARENESS_CONTEXT); static NTSTATUS (WINAPI *pD3DKMTCloseAdapter)(const D3DKMT_CLOSEADAPTER *); @@ -94,6 +95,7 @@ static void init_funcs(void) GET_PROC(user32, DisplayConfigGetDeviceInfo) GET_PROC(user32, DisplayConfigSetDeviceInfo) GET_PROC(user32, GetDpiForMonitorInternal) + GET_PROC(user32, GetDpiForSystem) GET_PROC(user32, SetThreadDpiAwarenessContext) GET_PROC(gdi32, D3DKMTCloseAdapter) @@ -1073,9 +1075,9 @@ static void test_GetThemePartSize(void) HRESULT hr; SIZE size; - if (!pSetThreadDpiAwarenessContext) + if (!pSetThreadDpiAwarenessContext || !pGetDpiForSystem) { - win_skip("SetThreadDpiAwarenessContext is unavailable.\n"); + win_skip("SetThreadDpiAwarenessContext() or GetDpiForSystem() is unavailable.\n"); return; } @@ -1088,7 +1090,7 @@ static void test_GetThemePartSize(void) old_context = pSetThreadDpiAwarenessContext(DPI_AWARENESS_CONTEXT_PER_MONITOR_AWARE_V2); current_dpi = get_primary_monitor_effective_dpi(); old_dpi = current_dpi; - system_dpi = GetDpiForSystem(); + system_dpi = pGetDpiForSystem(); target_dpi = system_dpi; hwnd = CreateWindowA("Button", "Test", WS_POPUP, 100, 100, 100, 100, NULL, NULL, NULL, NULL);

1 0

Zebediah Figura : ddraw/tests: Build ddraw4.c without -DWINE_NO_LONG_TYPES.
by Alexandre Julliard 13 Jul '22

13 Jul '22

Module: wine Branch: master Commit: b176b9dab1f019b99126da40aff71d218f8079f0 URL: https://gitlab.winehq.org/wine/wine/-/commit/b176b9dab1f019b99126da40aff71d… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Sun Jul 10 22:20:41 2022 -0500 ddraw/tests: Build ddraw4.c without -DWINE_NO_LONG_TYPES. --- dlls/ddraw/tests/ddraw4.c | 5409 ++++++++++++++++++++++----------------------- 1 file changed, 2689 insertions(+), 2720 deletions(-)

1 0

Zebediah Figura : ddraw/tests: Build ddraw7.c without -DWINE_NO_LONG_TYPES.
by Alexandre Julliard 13 Jul '22

13 Jul '22

Module: wine Branch: master Commit: 7375e8a9fcbaaf0c4016f973c5f926a5a76ad772 URL: https://gitlab.winehq.org/wine/wine/-/commit/7375e8a9fcbaaf0c4016f973c5f926… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Sun Jul 10 22:20:47 2022 -0500 ddraw/tests: Build ddraw7.c without -DWINE_NO_LONG_TYPES. --- dlls/ddraw/tests/ddraw7.c | 5232 ++++++++++++++++++++++----------------------- 1 file changed, 2599 insertions(+), 2633 deletions(-)

1 0

Zebediah Figura : ntoskrnl/tests: Fix a test failure.
by Alexandre Julliard 13 Jul '22

13 Jul '22

Module: wine Branch: master Commit: 03034f2c0ef23964891d60e5995d167a113f2d3e URL: https://gitlab.winehq.org/wine/wine/-/commit/03034f2c0ef23964891d60e5995d16… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Tue Jul 12 21:26:07 2022 -0500 ntoskrnl/tests: Fix a test failure. --- dlls/ntoskrnl.exe/tests/driver_pnp.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dlls/ntoskrnl.exe/tests/driver_pnp.c b/dlls/ntoskrnl.exe/tests/driver_pnp.c index 9727292d65d..1f22c4b9997 100644 --- a/dlls/ntoskrnl.exe/tests/driver_pnp.c +++ b/dlls/ntoskrnl.exe/tests/driver_pnp.c @@ -700,7 +700,8 @@ static NTSTATUS fdo_ioctl(IRP *irp, IO_STACK_LOCATION *stack, ULONG code) * handles to the device are closed (and the user-space thread is * currently blocked in this ioctl and won't close its handle * yet.) */ - ok(!remove_device_count, "Got %u remove events.\n", remove_device_count); + todo_wine_if (remove_device_count) + ok(!remove_device_count, "Got %u remove events.\n", remove_device_count); return STATUS_SUCCESS; }

1 0

Zebediah Figura : wineusb.sys: Move the libusb_init() and libusb_exit() calls to the Unix library.
by Alexandre Julliard 13 Jul '22

13 Jul '22

Module: wine Branch: master Commit: 05fe2f3f82df8713a9fc69db17c6ec71dd663c98 URL: https://gitlab.winehq.org/wine/wine/-/commit/05fe2f3f82df8713a9fc69db17c6ec… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Tue Jul 5 23:55:48 2022 -0500 wineusb.sys: Move the libusb_init() and libusb_exit() calls to the Unix library. --- dlls/wineusb.sys/unixlib.c | 9 +++++++++ dlls/wineusb.sys/wineusb.c | 8 -------- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/dlls/wineusb.sys/unixlib.c b/dlls/wineusb.sys/unixlib.c index 091d85d1ef1..cfced53d9bf 100644 --- a/dlls/wineusb.sys/unixlib.c +++ b/dlls/wineusb.sys/unixlib.c @@ -173,12 +173,19 @@ static NTSTATUS usb_main_loop(void *args) TRACE("Starting libusb event thread.\n"); + if ((ret = libusb_init(NULL))) + { + ERR("Failed to initialize libusb: %s\n", libusb_strerror(ret)); + return STATUS_UNSUCCESSFUL; + } + if ((ret = libusb_hotplug_register_callback(NULL, LIBUSB_HOTPLUG_EVENT_DEVICE_ARRIVED | LIBUSB_HOTPLUG_EVENT_DEVICE_LEFT, LIBUSB_HOTPLUG_ENUMERATE, LIBUSB_HOTPLUG_MATCH_ANY, LIBUSB_HOTPLUG_MATCH_ANY, LIBUSB_HOTPLUG_MATCH_ANY, hotplug_cb, NULL, &hotplug_cb_handle))) { ERR("Failed to register callback: %s\n", libusb_strerror(ret)); + libusb_exit(NULL); return STATUS_UNSUCCESSFUL; } @@ -188,6 +195,8 @@ static NTSTATUS usb_main_loop(void *args) ERR("Error handling events: %s\n", libusb_strerror(ret)); } + libusb_exit(NULL); + queue_event(&shutdown_event); TRACE("Shutting down libusb event thread.\n"); diff --git a/dlls/wineusb.sys/wineusb.c b/dlls/wineusb.sys/wineusb.c index 49033c76213..474ee6e96a6 100644 --- a/dlls/wineusb.sys/wineusb.c +++ b/dlls/wineusb.sys/wineusb.c @@ -731,14 +731,12 @@ static NTSTATUS WINAPI driver_add_device(DRIVER_OBJECT *driver, DEVICE_OBJECT *p static void WINAPI driver_unload(DRIVER_OBJECT *driver) { - libusb_exit(NULL); } NTSTATUS WINAPI DriverEntry(DRIVER_OBJECT *driver, UNICODE_STRING *path) { NTSTATUS status; void *instance; - int err; TRACE("driver %p, path %s.\n", driver, debugstr_w(path->Buffer)); @@ -752,12 +750,6 @@ NTSTATUS WINAPI DriverEntry(DRIVER_OBJECT *driver, UNICODE_STRING *path) driver_obj = driver; - if ((err = libusb_init(NULL))) - { - ERR("Failed to initialize libusb: %s\n", libusb_strerror(err)); - return STATUS_UNSUCCESSFUL; - } - driver->DriverExtension->AddDevice = driver_add_device; driver->DriverUnload = driver_unload; driver->MajorFunction[IRP_MJ_PNP] = driver_pnp;

1 0

Zebediah Figura : wineusb.sys: Pass the device descriptor as part of the event data.
by Alexandre Julliard 13 Jul '22

13 Jul '22

Module: wine Branch: master Commit: c698dc6e3634f94da24905ba56d9315133ccc13d URL: https://gitlab.winehq.org/wine/wine/-/commit/c698dc6e3634f94da24905ba56d931… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed May 25 17:37:37 2022 -0500 wineusb.sys: Pass the device descriptor as part of the event data. --- dlls/wineusb.sys/unixlib.c | 8 +++++++- dlls/wineusb.sys/unixlib.h | 7 ++++++- dlls/wineusb.sys/wineusb.c | 27 ++++++++++++--------------- 3 files changed, 25 insertions(+), 17 deletions(-) diff --git a/dlls/wineusb.sys/unixlib.c b/dlls/wineusb.sys/unixlib.c index cfced53d9bf..90847bc6a7d 100644 --- a/dlls/wineusb.sys/unixlib.c +++ b/dlls/wineusb.sys/unixlib.c @@ -133,7 +133,13 @@ static void add_usb_device(libusb_device *libusb_device) pthread_mutex_unlock(&device_mutex); usb_event.type = USB_EVENT_ADD_DEVICE; - usb_event.u.added_device = unix_device; + usb_event.u.added_device.device = unix_device; + usb_event.u.added_device.vendor = device_desc.idVendor; + usb_event.u.added_device.product = device_desc.idProduct; + usb_event.u.added_device.revision = device_desc.bcdDevice; + usb_event.u.added_device.class = device_desc.bDeviceClass; + usb_event.u.added_device.subclass = device_desc.bDeviceSubClass; + usb_event.u.added_device.protocol = device_desc.bDeviceProtocol; queue_event(&usb_event); } diff --git a/dlls/wineusb.sys/unixlib.h b/dlls/wineusb.sys/unixlib.h index e8e2850c57a..d76818cc907 100644 --- a/dlls/wineusb.sys/unixlib.h +++ b/dlls/wineusb.sys/unixlib.h @@ -47,7 +47,12 @@ struct usb_event union { - struct unix_device *added_device; + struct usb_add_device_event + { + struct unix_device *device; + UINT16 vendor, product, revision; + UINT8 class, subclass, protocol; + } added_device; struct unix_device *removed_device; IRP *completed_irp; } u; diff --git a/dlls/wineusb.sys/wineusb.c b/dlls/wineusb.sys/wineusb.c index 474ee6e96a6..b833dbc9824 100644 --- a/dlls/wineusb.sys/wineusb.c +++ b/dlls/wineusb.sys/wineusb.c @@ -137,12 +137,11 @@ static void add_usb_interface(struct usb_device *parent, const struct libusb_int LeaveCriticalSection(&wineusb_cs); } -static void add_unix_device(struct unix_device *unix_device) +static void add_unix_device(const struct usb_add_device_event *event) { static const WCHAR formatW[] = {'\\','D','e','v','i','c','e','\\','U','S','B','P','D','O','-','%','u',0}; - libusb_device *libusb_device = libusb_get_device(unix_device->handle); + libusb_device *libusb_device = libusb_get_device(event->device->handle); struct libusb_config_descriptor *config_desc; - struct libusb_device_descriptor device_desc; static unsigned int name_index; struct usb_device *device; DEVICE_OBJECT *device_obj; @@ -151,10 +150,8 @@ static void add_unix_device(struct unix_device *unix_device) WCHAR name[26]; int ret; - libusb_get_device_descriptor(libusb_device, &device_desc); - - TRACE("Adding new device %p, vendor %04x, product %04x.\n", unix_device, - device_desc.idVendor, device_desc.idProduct); + TRACE("Adding new device %p, vendor %04x, product %04x.\n", event->device, + event->vendor, event->product); sprintfW(name, formatW, name_index++); RtlInitUnicodeString(&string, name); @@ -167,16 +164,16 @@ static void add_unix_device(struct unix_device *unix_device) device = device_obj->DeviceExtension; device->device_obj = device_obj; - device->unix_device = unix_device; + device->unix_device = event->device; InitializeListHead(&device->irp_list); device->removed = FALSE; - device->class = device_desc.bDeviceClass; - device->subclass = device_desc.bDeviceSubClass; - device->protocol = device_desc.bDeviceProtocol; - device->vendor = device_desc.idVendor; - device->product = device_desc.idProduct; - device->revision = device_desc.bcdDevice; + device->class = event->class; + device->subclass = event->subclass; + device->protocol = event->protocol; + device->vendor = event->vendor; + device->product = event->product; + device->revision = event->revision; EnterCriticalSection(&wineusb_cs); list_add_tail(&device_list, &device->entry); @@ -279,7 +276,7 @@ static DWORD CALLBACK event_thread_proc(void *arg) switch (event.type) { case USB_EVENT_ADD_DEVICE: - add_unix_device(event.u.added_device); + add_unix_device(&event.u.added_device); break; case USB_EVENT_REMOVE_DEVICE:

1 0

Zebediah Figura : wineusb.sys: Create separate unix devices for each interface.
by Alexandre Julliard 13 Jul '22

13 Jul '22

Module: wine Branch: master Commit: b41fe10fd487ca18183a65ff33090801ff80dfb1 URL: https://gitlab.winehq.org/wine/wine/-/commit/b41fe10fd487ca18183a65ff330908… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed May 25 18:03:11 2022 -0500 wineusb.sys: Create separate unix devices for each interface. --- dlls/wineusb.sys/unixlib.c | 65 +++++++++++++++++++++++++++++++++++++ dlls/wineusb.sys/unixlib.h | 9 ++---- dlls/wineusb.sys/wineusb.c | 81 +++------------------------------------------- 3 files changed, 72 insertions(+), 83 deletions(-)

1 0

Zebediah Figura : wineusb.sys: Build with msvcrt.
by Alexandre Julliard 13 Jul '22

13 Jul '22

Module: wine Branch: master Commit: c63294fd89c87e080e9dc28e09d556b6e6649703 URL: https://gitlab.winehq.org/wine/wine/-/commit/c63294fd89c87e080e9dc28e09d556… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed May 25 18:08:54 2022 -0500 wineusb.sys: Build with msvcrt. --- dlls/wineusb.sys/Makefile.in | 2 +- dlls/wineusb.sys/wineusb.c | 52 +++++++++++++++----------------------------- 2 files changed, 18 insertions(+), 36 deletions(-) diff --git a/dlls/wineusb.sys/Makefile.in b/dlls/wineusb.sys/Makefile.in index 5547200596e..66a9f56b213 100644 --- a/dlls/wineusb.sys/Makefile.in +++ b/dlls/wineusb.sys/Makefile.in @@ -5,7 +5,7 @@ IMPORTS = ntoskrnl UNIX_LIBS = $(USB_LIBS) $(PTHREAD_LIBS) UNIX_CFLAGS = $(USB_CFLAGS) -EXTRADLLFLAGS = -Wl,--subsystem,native -mcygwin +EXTRADLLFLAGS = -Wl,--subsystem,native C_SRCS = \ unixlib.c \ diff --git a/dlls/wineusb.sys/wineusb.c b/dlls/wineusb.sys/wineusb.c index d432bdc5666..2f918cdeeb7 100644 --- a/dlls/wineusb.sys/wineusb.c +++ b/dlls/wineusb.sys/wineusb.c @@ -21,9 +21,8 @@ #include <assert.h> #include <stdarg.h> #include <stdbool.h> +#include <stdint.h> #include <stdlib.h> -#include <pthread.h> -#include <libusb.h> #include "ntstatus.h" #define WIN32_NO_STATUS @@ -36,7 +35,6 @@ #include "wine/asm.h" #include "wine/debug.h" #include "wine/list.h" -#include "wine/unicode.h" #include "unixlib.h" @@ -106,7 +104,6 @@ static void destroy_unix_device(struct unix_device *unix_device) static void add_unix_device(const struct usb_add_device_event *event) { - static const WCHAR formatW[] = {'\\','D','e','v','i','c','e','\\','U','S','B','P','D','O','-','%','u',0}; static unsigned int name_index; struct usb_device *device; DEVICE_OBJECT *device_obj; @@ -117,7 +114,7 @@ static void add_unix_device(const struct usb_add_device_event *event) TRACE("Adding new device %p, vendor %04x, product %04x.\n", event->device, event->vendor, event->product); - sprintfW(name, formatW, name_index++); + swprintf(name, ARRAY_SIZE(name), L"\\Device\\USBPDO-%u", name_index++); RtlInitUnicodeString(&string, name); if ((status = IoCreateDevice(driver_obj, sizeof(*device), &string, FILE_DEVICE_USB, 0, FALSE, &device_obj))) @@ -366,54 +363,39 @@ static void WINAPIV append_id(struct string_buffer *buffer, const WCHAR *format, __ms_va_end(args); } -static const WCHAR emptyW[] = {0}; - static void get_device_id(const struct usb_device *device, struct string_buffer *buffer) { - static const WCHAR interface_formatW[] = {'U','S','B','\\','V','I','D','_','%','0','4','X', - '&','P','I','D','_','%','0','4','X','&','M','I','_','%','0','2','X',0}; - static const WCHAR formatW[] = {'U','S','B','\\','V','I','D','_','%','0','4','X', - '&','P','I','D','_','%','0','4','X',0}; - if (device->interface) - append_id(buffer, interface_formatW, device->vendor, device->product, device->interface_index); + append_id(buffer, L"USB\\VID_%04X&PID_%04X&MI_%02X", + device->vendor, device->product, device->interface_index); else - append_id(buffer, formatW, device->vendor, device->product); + append_id(buffer, L"USB\\VID_%04X&PID_%04X", device->vendor, device->product); } static void get_hardware_ids(const struct usb_device *device, struct string_buffer *buffer) { - static const WCHAR interface_formatW[] = {'U','S','B','\\','V','I','D','_','%','0','4','X', - '&','P','I','D','_','%','0','4','X','&','R','E','V','_','%','0','4','X','&','M','I','_','%','0','2','X',0}; - static const WCHAR formatW[] = {'U','S','B','\\','V','I','D','_','%','0','4','X', - '&','P','I','D','_','%','0','4','X','&','R','E','V','_','%','0','4','X',0}; - if (device->interface) - append_id(buffer, interface_formatW, device->vendor, device->product, device->revision, device->interface_index); + append_id(buffer, L"USB\\VID_%04X&PID_%04X&REV_%04X&MI_%02X", + device->vendor, device->product, device->revision, device->interface_index); else - append_id(buffer, formatW, device->vendor, device->product, device->revision); + append_id(buffer, L"USB\\VID_%04X&PID_%04X&REV_%04X", + device->vendor, device->product, device->revision); + get_device_id(device, buffer); - append_id(buffer, emptyW); + append_id(buffer, L""); } static void get_compatible_ids(const struct usb_device *device, struct string_buffer *buffer) { - static const WCHAR prot_format[] = {'U','S','B','\\','C','l','a','s','s','_','%','0','2','x', - '&','S','u','b','C','l','a','s','s','_','%','0','2','x', - '&','P','r','o','t','_','%','0','2','x',0}; - static const WCHAR subclass_format[] = {'U','S','B','\\','C','l','a','s','s','_','%','0','2','x', - '&','S','u','b','C','l','a','s','s','_','%','0','2','x',0}; - static const WCHAR class_format[] = {'U','S','B','\\','C','l','a','s','s','_','%','0','2','x',0}; - - append_id(buffer, prot_format, device->class, device->subclass, device->protocol); - append_id(buffer, subclass_format, device->class, device->subclass); - append_id(buffer, class_format, device->class); - append_id(buffer, emptyW); + append_id(buffer, L"USB\\Class_%02x&SubClass_%02x&Prot_%02x", + device->class, device->subclass, device->protocol); + append_id(buffer, L"USB\\Class_%02x&SubClass_%02x", device->class, device->subclass); + append_id(buffer, L"USB\\Class_%02x", device->class); + append_id(buffer, L""); } static NTSTATUS query_id(struct usb_device *device, IRP *irp, BUS_QUERY_ID_TYPE type) { - static const WCHAR instance_idW[] = {'0',0}; struct string_buffer buffer = {0}; TRACE("type %#x.\n", type); @@ -425,7 +407,7 @@ static NTSTATUS query_id(struct usb_device *device, IRP *irp, BUS_QUERY_ID_TYPE break; case BusQueryInstanceID: - append_id(&buffer, instance_idW); + append_id(&buffer, L"0"); break; case BusQueryHardwareIDs:

1 0

Alexandre Julliard : ntdll/tests: Add some more tests for shared Software\Classes key.
by Alexandre Julliard 13 Jul '22

13 Jul '22

Module: wine Branch: master Commit: adb0ff570c44b51b20f8a3d1a2da405536f7d15b URL: https://gitlab.winehq.org/wine/wine/-/commit/adb0ff570c44b51b20f8a3d1a2da40… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Jul 13 11:26:22 2022 +0200 ntdll/tests: Add some more tests for shared Software\Classes key. --- dlls/ntdll/tests/reg.c | 236 +++++++++++++++++++++++++++++++++++++------------ 1 file changed, 180 insertions(+), 56 deletions(-)

1 0

Alexandre Julliard : server: Create new keys in the parent of a shared key.
by Alexandre Julliard 13 Jul '22

13 Jul '22

Module: wine Branch: master Commit: bd5edbe1eb1d1e57ded07e291fdc0fd0000a40dc URL: https://gitlab.winehq.org/wine/wine/-/commit/bd5edbe1eb1d1e57ded07e291fdc0f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Jul 13 12:29:01 2022 +0200 server: Create new keys in the parent of a shared key. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53343 --- dlls/ntdll/tests/reg.c | 8 -------- server/registry.c | 7 +++++-- 2 files changed, 5 insertions(+), 10 deletions(-) diff --git a/dlls/ntdll/tests/reg.c b/dlls/ntdll/tests/reg.c index 10e69b73e6d..f3c4eb15da0 100644 --- a/dlls/ntdll/tests/reg.c +++ b/dlls/ntdll/tests/reg.c @@ -1829,30 +1829,22 @@ static void test_redirection(void) attr.RootDirectory = root64; status = pNtOpenKey( &key, KEY_ALL_ACCESS, &attr ); - todo_wine_if( ptr_size == 32) ok( status == STATUS_SUCCESS, "NtOpenKey failed: 0x%08lx\n", status ); - if (!status) - { len = sizeof(buffer); status = pNtQueryValueKey( key, &value_str, KeyValuePartialInformation, info, len, &len ); ok( status == STATUS_SUCCESS, "NtQueryValueKey failed: 0x%08lx\n", status ); dw = *(DWORD *)info->Data; ok( dw == 32, "wrong value %lu\n", dw ); pNtClose( key ); - } attr.RootDirectory = root64; status = pNtOpenKey( &key64, KEY_WOW64_64KEY | KEY_ALL_ACCESS, &attr ); - todo_wine_if( ptr_size == 32) ok( status == STATUS_SUCCESS, "NtOpenKey failed: 0x%08lx\n", status ); - if (!status) - { len = sizeof(buffer); status = pNtQueryValueKey( key64, &value_str, KeyValuePartialInformation, info, len, &len ); ok( status == STATUS_SUCCESS, "NtQueryValueKey failed: 0x%08lx\n", status ); dw = *(DWORD *)info->Data; ok( dw == 32, "wrong value %lu\n", dw ); - } pNtDeleteKey( key32 ); pNtClose( key32 ); diff --git a/server/registry.c b/server/registry.c index f9a9f75a974..96ba18a0a5a 100644 --- a/server/registry.c +++ b/server/registry.c @@ -536,9 +536,12 @@ static struct object *key_lookup_name( struct object *obj, struct unicode_str *n if (!(found = find_subkey( key, &tmp, &index ))) { - /* try in the 64-bit parent */ if ((key->flags & KEY_WOWSHARE) && (attr & OBJ_KEY_WOW64)) - found = find_subkey( get_parent( key ), &tmp, &index ); + { + /* try in the 64-bit parent */ + key = get_parent( key ); + if (!(found = find_subkey( key, &tmp, &index ))) return grab_object( key ); + } } if (!found)

1 0