- Wine-Devel - list.winehq.org

[PATCH v5] gdi32: Create the registry keys recursively, if needed.
by Gabriel Ivăncescu Oct. 5, 2021

Oct. 5, 2021

Fixes a regression introduced by 1cdc74b2d62d1c94005c46f9c8f4b566aa8bdcbd when creating new prefixes, as NtCreateKey does not recursively create keys. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- v5: Fix memcmp size... dlls/gdi32/font.c | 31 ++++++++++++++++++++++++++++++- 1 file changed, 30 insertions(+), 1 deletion(-) diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c index 98c70a6..4784c1d 100644 --- a/dlls/gdi32/font.c +++ b/dlls/gdi32/font.c @@ -29,6 +29,8 @@ #include <stdlib.h> #include <string.h> #include <assert.h> +#include "ntstatus.h" +#define WIN32_NO_STATUS #include "winerror.h" #include "windef.h" #include "winbase.h" @@ -569,11 +571,13 @@ static HKEY reg_open_key( HKEY root, const WCHAR *name, ULONG name_len ) return ret; } +/* wrapper for NtCreateKey that creates the key recursively if necessary */ static HKEY reg_create_key( HKEY root, const WCHAR *name, ULONG name_len, DWORD options, DWORD *disposition ) { UNICODE_STRING nameW = { name_len, name_len, (WCHAR *)name }; OBJECT_ATTRIBUTES attr; + NTSTATUS status; HANDLE ret; attr.Length = sizeof(attr); @@ -583,7 +587,32 @@ static HKEY reg_create_key( HKEY root, const WCHAR *name, ULONG name_len, attr.SecurityDescriptor = NULL; attr.SecurityQualityOfService = NULL; - if (NtCreateKey( &ret, MAXIMUM_ALLOWED, &attr, 0, NULL, options, disposition )) return 0; + status = NtCreateKey( &ret, MAXIMUM_ALLOWED, &attr, 0, NULL, options, disposition ); + if (status == STATUS_OBJECT_NAME_NOT_FOUND) + { + static const WCHAR registry_rootW[] = { '\\','R','e','g','i','s','t','r','y','\\' }; + DWORD pos = 0, i = 0, len = name_len / sizeof(WCHAR); + + /* don't try to create registry root */ + if (!root && len > 10 && !memcmp( name, registry_rootW, 10 * sizeof(WCHAR) )) i += 10; + + while (i < len && name[i] != '\\') i++; + if (i == len) return 0; + for (;;) + { + nameW.Buffer = (WCHAR *)name + pos; + nameW.Length = (i - pos) * sizeof(WCHAR); + status = NtCreateKey( &ret, MAXIMUM_ALLOWED, &attr, 0, NULL, options, disposition ); + + if (attr.RootDirectory != root) NtClose( attr.RootDirectory ); + if (!NT_SUCCESS(status)) return 0; + if (i == len) break; + attr.RootDirectory = ret; + while (i < len && name[i] == '\\') i++; + pos = i; + while (i < len && name[i] != '\\') i++; + } + } return ret; } -- 2.31.1

2 2

[PATCH v2 1/2] winevulkan: Support aliased constants.
by Georg Lehmann Oct. 5, 2021

Oct. 5, 2021

Signed-off-by: Georg Lehmann <dadschoorse(a)gmail.com> --- dlls/winevulkan/make_vulkan | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/dlls/winevulkan/make_vulkan b/dlls/winevulkan/make_vulkan index 3a6bc66d32b..9e55753ca38 100755 --- a/dlls/winevulkan/make_vulkan +++ b/dlls/winevulkan/make_vulkan @@ -3445,11 +3445,17 @@ class VkRegistry(object): enum.create_alias(enum_elem.attrib["name"], enum_elem.attrib["alias"]) elif "value" in enum_elem.keys(): - # Constants are not aliased, no need to add them here, they'll get added later on. + # Constant with an explicit value if only_aliased: return self.consts.append(VkConstant(enum_elem.attrib["name"], enum_elem.attrib["value"])) + elif "alias" in enum_elem.keys(): + # Aliased constant + if not only_aliased: + return + + self.consts.append(VkConstant(enum_elem.attrib["name"], enum_elem.attrib["alias"])) @staticmethod def _require_type(type_info): -- 2.33.0

1 1

[PATCH v2 6/6] gdi32: Use __wine_unix_call interface for Unix library.
by Jacek Caban Oct. 5, 2021

Oct. 5, 2021

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/gdi32/Makefile.in | 3 ++- dlls/gdi32/dibdrv/dc.c | 17 ++++++----------- dlls/gdi32/font.c | 35 ++++++++++++++++++----------------- dlls/gdi32/gdiobj.c | 14 +++++++++----- dlls/gdi32/wrappers.c | 14 ++++++++++++-- 5 files changed, 47 insertions(+), 36 deletions(-)

3 2

[PATCH] winevulkan: Update to VK spec version 1.2.195.
by Georg Lehmann Oct. 5, 2021

Oct. 5, 2021

Signed-off-by: Georg Lehmann <dadschoorse(a)gmail.com> --- dlls/winevulkan/make_vulkan | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/winevulkan/make_vulkan b/dlls/winevulkan/make_vulkan index 3a6bc66d32b..5bbc836941b 100755 --- a/dlls/winevulkan/make_vulkan +++ b/dlls/winevulkan/make_vulkan @@ -64,7 +64,7 @@ from enum import Enum LOGGER = logging.Logger("vulkan") LOGGER.addHandler(logging.StreamHandler()) -VK_XML_VERSION = "1.2.191" +VK_XML_VERSION = "1.2.195" WINE_VK_VERSION = (1, 2) # Filenames to create. -- 2.33.0

2 1

[PATCH] user32: Update exstyle when setting style in SetWindowLong.
by Piotr Caban Oct. 5, 2021

Oct. 5, 2021

Rockstar Games Launcher checks extended window style and exits if it is incorrect. Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> --- dlls/user32/tests/win.c | 18 +++++++++++++++++- dlls/user32/win.c | 15 ++++++++++++--- 2 files changed, 29 insertions(+), 4 deletions(-)

3 2

[PATCH v2 5/6] ntdll: Handle Unix lib exception outside Unix stack.
by Jacek Caban Oct. 5, 2021

Oct. 5, 2021

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- The next patch uses the new style Unix lib, but still bypasses syscall thunks for ntgdi functions. Once ntgdi is moved to win32u (and with this series it's ready to be moved; I plan to do that in the next series), it may start using proper syscall interface. While ntgdi internals are ready for this and we can change a number of functions right away, functions that call external drivers can't use syscalls interface until drivers are converted to Unix libs as well (and we can't convert drivers without useful win32u). The patch is not very invasive and my thinking is to break this circular dependency by allowing bypassing syscall thunks until win32u is ready (we may revert this patch by then). dlls/ntdll/unix/signal_arm.c | 2 +- dlls/ntdll/unix/signal_arm64.c | 2 +- dlls/ntdll/unix/signal_i386.c | 2 +- dlls/ntdll/unix/signal_x86_64.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-)

1 0

[PATCH v2 3/6] ntdll: Provide some Unicode helpers for Unix libs.
by Jacek Caban Oct. 5, 2021

Oct. 5, 2021

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- I think that those are useful enough to expose them from ntdll instead of copying to other libraries (at least gdi32/win32u will need those) and mapping casemap multiple times. I used Rtl* names instead of exposing ntdll_* as it seems nice to not use existing prototypes, but I don't feel strongly about any of above. dlls/ntdll/unix/env.c | 98 ++++++++++++++++++++++++++++++------------- 1 file changed, 69 insertions(+), 29 deletions(-)

1 0

[PATCH v2 2/6] ntdll: Introduce ntdll_get_build_dir and ntdll_get_data_dir.
by Jacek Caban Oct. 5, 2021

Oct. 5, 2021

To expose build and data dirs for Unix libs. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/ntdll/unix/env.c | 18 ++++++++++++++++++ include/wine/unixlib.h | 2 ++ 2 files changed, 20 insertions(+)

1 0

[PATCH v2 1/6] makedep: Always define _NTSYSTEM_ for native Unix libraries.
by Jacek Caban Oct. 5, 2021

Oct. 5, 2021

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- v2: Fixed ntdll_umbstowcs in patch 3. dlls/nsiproxy.sys/ip.c | 2 -- dlls/winepulse.drv/pulse.c | 2 -- tools/makedep.c | 2 ++ 3 files changed, 2 insertions(+), 4 deletions(-)

1 0

[PATCH 6/6] gdi32: Use __wine_unix_call interface for Unix library.
by Jacek Caban Oct. 5, 2021

Oct. 5, 2021

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/gdi32/Makefile.in | 3 ++- dlls/gdi32/dibdrv/dc.c | 17 ++++++----------- dlls/gdi32/font.c | 35 ++++++++++++++++++----------------- dlls/gdi32/gdiobj.c | 14 +++++++++----- dlls/gdi32/wrappers.c | 14 ++++++++++++-- 5 files changed, 47 insertions(+), 36 deletions(-)

2 1