February 2023 - wine-gitlab

[PATCH v2 0/2] MR2136: Msdasql call sql driver connect w
by Alistair Leslie-Hughes (＠alesliehughes) 08 Feb '23

08 Feb '23

-- v2: msdasql: Use SQLDriverConnectW to connect to the ODBC driver https://gitlab.winehq.org/wine/wine/-/merge_requests/2136

2 2

[PATCH 0/1] MR2135: ole32: Avoid calling RtlInitUnicodeString on a static constant.
by Alex Henrie (＠alexhenrie) 08 Feb '23

08 Feb '23

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/2135

3 2

[PATCH 0/2] MR2136: Msdasql call sql driver connect w
by Alistair Leslie-Hughes (＠alesliehughes) 08 Feb '23

08 Feb '23

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/2136

3 3

[PATCH 0/1] MR1974: rpcrt4/tests: Remove a failing test.
by Huw Davies (＠huw) 08 Feb '23

08 Feb '23

A follow-up to !1807. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1974

4 3

[PATCH 0/2] MR2130: Fix missing *out=NULL in (WMReader|unknown_inner)_QueryInterface
by Ivan Chikish (＠nekotekina) 08 Feb '23

08 Feb '23

QueryInterface should set *out to NULL on failure, as it seems to do in many different places. This seems to be a regression from !888. One old game started to crash on startup with this merge request. I bisected it and it pointed to the commit 131ada052a2dbec66df695ce536ca048a2bd9174. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2130

3 3

Re: [PATCH v2 0/2] MR2114: kernelbase: Fix performance of beatmap search in osu! Stable.
by Torge Matthies (＠tmatthies) 08 Feb '23

08 Feb '23

On Wed Feb 8 01:17:39 2023 +0000, Torge Matthies wrote: > I can remove the second commit if desired, the first commit is enough to > make performance acceptable again. Also note that there are some more details about the commits in the commit message bodies. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2114#note_23391

1 0

Re: [PATCH v2 0/2] MR2114: kernelbase: Fix performance of beatmap search in osu! Stable.
by Torge Matthies (＠tmatthies) 08 Feb '23

08 Feb '23

I can remove the second commit if desired, the first commit is enough to make performance acceptable again. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2114#note_23389

1 0

[PATCH v2 0/4] MR2127: ntdll: Load known dlls from system directory
by Paul Gofman (＠gofman) 07 Feb '23

07 Feb '23

The practical issue I am trying to solve is the following scenario (encountered with a game using some Uplay plugin) which has regressed with apisets implementation in Wine. The game imports ucrtbase.dll. Then, a plugin calls GetModuleHandle("api-ms-win-crt-runtime-l1-1-0.dll") and expects GetProcAddress("\_crt_atexit") to succeed on the returned handle. The problem is that the game has ucrtbase.dll in its .exe directory. That one currently gets loaded. Then, find_dll_file() resolves api-ms-win-crt-runtime-l1-1-0.dll to the ucrtbase with a full path in system32 and tries to find that one (this part matches Windows behaviour as far as tests show). Since we have ucrtbase loaded with another path this results in NULL module handle. Windows maintains the list of "Known DLLs" (probably aimed for speeding up processes startup by using pre-mapped dll images for commonly used system dlls). On Windows, known dlls have specifics in load path resolution when dll name doesn't contain path (relative or full): - they are loaded from system directory even if there is another dll with the same name in priority search path; - they are found even if library search paths do not contain system directory; Those "Known DLLs" are basically listed in HKLM\\System\\CurrentControlSet\\Control\\Session Manager\KnownDLLs registry key. But the actual list of "known dlls" is bigger (probably includes all dependencies of those listed under registry key). The full actual list of "known dlls" is in '\\KnownDlls\' directory object which contains image sections under dll names. Most notably, ucrtbase.dll (which is of the concern in the regression) is not listed in registry but is present in \\KnownDlls. Technically nothing prevents us from implementing these mechanics and creating directory object with mapped load dlls, also resolving the explicitly listed dlls' dependencies. But it is not apparent to me if that is going to improve load times noticably in Wine, and also that is worth the inherent complications until anything actually depends on that \\KnownDlls sections. So instead of implementing all of that throughout the loader I added a few missing dlls to registry. The current list in this pat chset corresponds to the contents of \\KnownDlls directory on an up to date Windows 10. -- v2: ntdll: Load known dlls from system directory. https://gitlab.winehq.org/wine/wine/-/merge_requests/2127

3 6

[PATCH v3 0/4] MR2127: ntdll: Load known dlls from system directory
by Paul Gofman (＠gofman) 07 Feb '23

07 Feb '23

The practical issue I am trying to solve is the following scenario (encountered with a game using some Uplay plugin) which has regressed with apisets implementation in Wine. The game imports ucrtbase.dll. Then, a plugin calls GetModuleHandle("api-ms-win-crt-runtime-l1-1-0.dll") and expects GetProcAddress("\_crt_atexit") to succeed on the returned handle. The problem is that the game has ucrtbase.dll in its .exe directory. That one currently gets loaded. Then, find_dll_file() resolves api-ms-win-crt-runtime-l1-1-0.dll to the ucrtbase with a full path in system32 and tries to find that one (this part matches Windows behaviour as far as tests show). Since we have ucrtbase loaded with another path this results in NULL module handle. Windows maintains the list of "Known DLLs" (probably aimed for speeding up processes startup by using pre-mapped dll images for commonly used system dlls). On Windows, known dlls have specifics in load path resolution when dll name doesn't contain path (relative or full): - they are loaded from system directory even if there is another dll with the same name in priority search path; - they are found even if library search paths do not contain system directory; Those "Known DLLs" are basically listed in HKLM\\System\\CurrentControlSet\\Control\\Session Manager\KnownDLLs registry key. But the actual list of "known dlls" is bigger (probably includes all dependencies of those listed under registry key). The full actual list of "known dlls" is in '\\KnownDlls\' directory object which contains image sections under dll names. Most notably, ucrtbase.dll (which is of the concern in the regression) is not listed in registry but is present in \\KnownDlls. Technically nothing prevents us from implementing these mechanics and creating directory object with mapped load dlls, also resolving the explicitly listed dlls' dependencies. But it is not apparent to me if that is going to improve load times noticably in Wine, and also that is worth the inherent complications until anything actually depends on that \\KnownDlls sections. So instead of implementing all of that throughout the loader I added a few missing dlls to registry. The current list in this pat chset corresponds to the contents of \\KnownDlls directory on an up to date Windows 10. -- v3: ntdll: Load known dlls from system directory. https://gitlab.winehq.org/wine/wine/-/merge_requests/2127

2 5

[PATCH 0/1] MR2129: loader: In macOS preloader, fix an environment variable sometimes being lost.
by Brendan Shanks (＠bshanks) 07 Feb '23

07 Feb '23

On Ventura (and likely also on Monterey), `dyld`/`libc` seems to store the `environ` pointer from when the preloader launches, ignores any change to `environ` done in the preloader, and then uses that original `environ` in the launched wine process. The preloader has to move the start of the environment down by one, and this was causing the first variable to be lost on Monterey/Ventura. Instead of moving the entire environment, add a dummy variable at the beginning so nothing important is lost. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2129

2 1

2025

2024

2023

2022

wine-gitlab February 2023