November 2022 - wine-gitlab

Re: [PATCH v8 0/116] MR955: Release 7.0.1. - approved
by Alexandre Julliard (＠julliard) 08 Nov '22

08 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/955

1 0

[PATCH v6 0/6] MR1001: winebuild: Reimplement dlltool for import lib and delay import lib generation.
by Rémi Bernon (＠rbernon) 08 Nov '22

08 Nov '22

This is slightly different from dlltool but I think it should be compatible. The transition is done by first replacing dlltool with its bugs, and fixing them in separate changes. I based the ARM / AARCH64 implementation on the existing code around, but I have no idea if it is correct, and dlltool also doesn't include any delay load implementation for ARM. -- v6: winebuild: Put the delay import descriptor in data section. winebuild: Fix import hint value for symbols imported by name. winebuild: Implement delay import lib generation without dlltool. winebuild: Enable unwind tables by default in PE files. winebuild: Implement import lib generation without dlltool. winebuild: Introduce a new --without-dlltool winebuild flag. https://gitlab.winehq.org/wine/wine/-/merge_requests/1001

3 10

[PATCH v2 0/4] MR1286: secur32: Implement graceful secure connection shutdown.
by Paul Gofman (＠gofman) 08 Nov '22

08 Nov '22

... and fix output buffer allocation on the way. I am not aware of any app which strictly depends on the ApplyControlToken / connection shutdown to be implemented. Yet I saw a few games calling ApplyToken to perform the connection shutdown (similar to what is described in [1]). I once implemented that while debugging something and probably it makes some sense support secure connection shutdown once the app tries to do so (which as I understand is now recommended to do before closing socket). 1. https://learn.microsoft.com/en-us/windows/win32/secauthn/shutting-down-an-s… -- v2: secur32: Implement ApplyControlToken for SCHANNEL_SHUTDOWN. secur32: Add semi-stub for ApplyControlToken (schannel). secur32: Allocate buffer for either ISC_REQ_ALLOCATE_MEMORY or NULL output in schan_InitializeSecurityContextW(). secure32/tests: Add tests for ApplyControlToken (schannel). https://gitlab.winehq.org/wine/wine/-/merge_requests/1286

3 5

[PATCH v6 0/3] MR574: mf: Improve session pause / restart handling.
by Paul Gofman (＠gofman) 08 Nov '22

08 Nov '22

Patch 1. The tests on Windows show that when the timer callback is queued behind the clock time it is queued and executed quite soon with 0-15ms delay which corresponds to default timer granularity. The timeout tolerance in the test is quite bigger to avoid a flaky test. Wine currently may execute the callback right away or delay it indefinitely depending on the time difference and current system time. Patch 2. It turns out when the session is paused and then restarted and all the samples were delivered to sample grabber before restart, nothing is going to request the samples again when the session is restarted. I am attaching a test in a patch which hopefully confirms how that should work. I am not including that in the patches as the test involves delays (to make sure all the async events settle at certain points) and inherently flaky. Also, _ IMFStreamSink_GetEvent(stream, ...) in the test crashes on Win7 for some unknown reason. [0001-mf-tests-Add-test-for-pausing-and-restarting-sample-.patch](/uploads/58c1738d48b3392e7ec430a0757210d0/0001-mf-tests-Add-test-for-pausing-and-restarting-sample-.patch) -- v6: mf/samplegrabber: Handle samples receieved in paused state. mf/samplegrabber: Process samples in paused state in ignore_clock mode. mf: Handle timer time behind clock time in present_clock_schedule_timer(). https://gitlab.winehq.org/wine/wine/-/merge_requests/574

4 6

[PATCH v3 0/5] MR1262: mf: Set source and output nodes media types in the media session.
by Bernhard Kölbl (＠besentv) 08 Nov '22

08 Nov '22

Supersedes: https://gitlab.winehq.org/wine/wine/-/merge_requests/789 This MR, with patches from Rémi should fix the no sound issue in media engine and also the black screen issue with MfPlayer.exe. -- v3: mf: Set media types for output nodes in the media session. mf: Assume same up and downstream media type for copier creation. mf/tests: Test for copier node in topology using evr. mf: Add some topology source node checks in IMFMediaSession_SetTopology. mf: Always enumerate branch source types for transform nodes. https://gitlab.winehq.org/wine/wine/-/merge_requests/1262

6 9

[PATCH 0/1] MR1294: netprofm: Store a list manager pointer in the network object instead of...
by Hans Leidekker (＠hans) 08 Nov '22

08 Nov '22

netprofm: Store a list manager pointer in the network object instead of creating a new list manager. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1294

2 1

[PATCH 0/1] MR1293: shell32: Improve timeout detection in shlexec tests.
by eric pouech (＠epo) 08 Nov '22

08 Nov '22

Some shlexec tests timeout on Windows10 (and generate errors as no action has been executed). Improve detection of timeout in child program (so that existing workaround in shell_execute_ex can be triggered). Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1293

2 1

[PATCH 0/2] MR1166: ntdll: don't claim to have not found a library that is in place
by Konstantin Kharlamov (＠Hi-Angel) 08 Nov '22

08 Nov '22

When `import_dll` function gets `STATUS_DLL_NOT_FOUND` from `load_dll()`, it has no way of distinguishing whether it's a dependency for the dll wasn't found, or the dll itself is missing. That results in confusing ERR messages, where a dll is found, but wine claims it isn't. Fix this by moving the print to the actual place that is aware of whether it's the parent or the child dll haven't been found. Log before: 0024:err:module:import_dll Library lib2.dll (which is needed by L"Z:\\tmp\\lib1.dll") not found 0024:err:module:import_dll Library lib1.dll (which is needed by L"Z:\\tmp\\a.exe") not found Log after: 0024:err:module:load_dll Library L"lib2.dll" (which is needed by L"Z:\\tmp\\foo\\lib1.dll") not found 0024:err:module:import_dll Loading library lib2.dll (which is needed by L"Z:\\tmp\\foo\\lib1.dll") failed (error c0000135). 0024:err:module:import_dll Loading library lib1.dll (which is needed by L"Z:\\tmp\\foo\\a.exe") failed (error c0000135). -------- While it's just about prints, but it costed me hours of wandering *(because if I know where to find the dependency dll, but I see that the main dll is also "not found" despite being "right there", I have to research the latter problem that looks more complicated. But given enough resources applied, it then turns out to be a faux)*. As a fun fact, I am apparently not the only one to have stumbled upon. Here's a screenshot of a `#english` chat where I asked about a commit title correctness. It's the last place where you'd expect to find someone familiar with WINE, and yet here's a person that had to deal with that exact error as well: ![Screenshot_20221026_023411](/uploads/55552d7c335ef5ce0b507f61adacbb00/Screenshot_20221026_023411.png) So, I think it's kinda serious problem. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1166

4 13

[PATCH 0/3] MR1291: dwrite: fix get fontface frome cache is error.
by Jiajin Cui (＠jin-king1) 08 Nov '22

08 Nov '22

Signed-off-by: Jiajin Cui <cuijiajin(a)uniontech.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1291

4 9

Re: [PATCH v2 0/1] MR1232: ntdll: Set the QoS class of the main Wine thread on macOS.
by Huw Davies (＠huw) 08 Nov '22

08 Nov '22

@bshanks this is producing a compiler warning here: ``` gcc -m32 -c -o dlls/ntdll/unix/loader.o dlls/ntdll/unix/loader.c -Idlls/ntdll -Iinclude -D__WINESRC__ -DWINE_NO_LONG_TYPES \ -D_NTSYSTEM_ -D_ACRTIMP= -DWINBASEAPI= -D_MSVCR_VER=0 -DBINDIR=\"/usr/local/bin\" \ -DSYSTEMDLLPATH=\"\" -DDLL_TO_BINDIR=\"`tools/makedep -R /usr/local/lib/wine /usr/local/bin`\" \ -DBIN_TO_DATADIR=\"`tools/makedep -R /usr/local/bin /usr/local/share/wine`\" -DWINE_UNIX_LIB -Wall -pipe \ -fno-stack-protector -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body \ -Wignored-qualifiers -Winit-self -Wno-pragma-pack -Wstrict-prototypes -Wtype-limits -Wvla \ -Wwrite-strings -Wpointer-arith -gdwarf-4 -fno-omit-frame-pointer -fPIC \ -fasynchronous-unwind-tables -O2 -g -Wno-deprecated-declarations -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=0 dlls/ntdll/unix/loader.c:2339:9: warning: address of function 'pthread_attr_set_qos_class_np' will always evaluate to 'true' [-Wpointer-bool-conversion] if (pthread_attr_set_qos_class_np) ~~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~ dlls/ntdll/unix/loader.c:2339:9: note: prefix with the address-of operator to silence this warning if (pthread_attr_set_qos_class_np) ^ & 1 warning generated. ``` Why do we need the test? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1232#note_13594

1 0

2025

2024

2023

2022

wine-gitlab November 2022