November 2022 - wine-gitlab

[PATCH v4 0/2] MR1263: kernelbase: Fix VerQueryValueW with no data
by Mark Jansen (＠learn-more) 04 Nov '22

04 Nov '22

xul.dll from Mypal 27.9.4 has a resource entry without any data. Windows returns this as null-terminated string, instead of pointing past the data. The first commit adds a test, the second commit fixes the bugs (and removes the todo_wine). Original ticket for reference: https://jira.reactos.org/browse/CORE-15781 -- v4: kernelbase: Fix VerQueryValueW with no data https://gitlab.winehq.org/wine/wine/-/merge_requests/1263

2 2

[PATCH v2 0/2] MR1162: ntdll: Use local unwinding reimplementations for Unix libraries on ARM
by Martin Storsjö (＠mstorsjo) 04 Nov '22

04 Nov '22

Currently, unwinding the unix libraries (or full ELF builds) requires having libunwind available. When starting up a wine environment, an exception of type `RPC_S_SERVER_UNAVAILABLE` gets thrown - and if libunwind isn't available, this breaks the startup. Thus, currently on ARM/ARM64, libunwind is essentially mandatory. Additionally, at least on ARM, libunwind seems brittle (commits in latest git master breaks the unwinding use cases in Wine, see e.g. https://github.com/libunwind/libunwind/pull/203#issuecomment-984126066. For ARM, libunwind has currently relied on DWARF debug info in the `.debug_frame` section, since Linux on ARM doesn't use DWARF for runtime unwinding. This MR adds ARM EHABI unwind opcodes where necessary, and adds a local reimplementation of an ARM EHABI/EXIDX/EXTBL unwinder, similar to the DWARF one. With these changes, unwinding on ARM seems to work fine, even without libunwind. See the individual commit messages for more commentary on the decisions taken so far (I've tried to write up all relevant considerations there - it's a fair amount of commentery). A couple open questions: - I wrote the ARM EHABI unwind code mostly with `stdint.h` types like `uint32_t`(as it's in the ntdll/unix directory anyway), but I can rewrite it with `DWORD` or similar if that's preferred. - The ARM EHABI opcodes are enabled for `defined(__arm__) && defined(__ELF__) && defined(__GNUC__) && !defined(__SEH__) && !defined(__ARM_DWARF_EH__)` - there's no define to check for for knowing it's used and enabled, but it has to be implicitly assumed if no other other unwind mechanism is signaled. - The `dl_iterate_phdr` function, used for finding the EXIDX section for locating ARM EHABI unwind info, is used within `#ifdef linux`. It might be available on other OSes too (like FreeBSD etc). Or should I go ahead and add a configure check for it? -- v2: ntdll: Implement ARM EHABI unwinding ntdll: Add ARM EHABI unwind instructions in assembly functions https://gitlab.winehq.org/wine/wine/-/merge_requests/1162

2 3

[PATCH v3 0/2] MR1263: kernelbase: Fix VerQueryValueW with no data
by Mark Jansen (＠learn-more) 04 Nov '22

04 Nov '22

xul.dll from Mypal 27.9.4 has a resource entry without any data. Windows returns this as null-terminated string, instead of pointing past the data. The first commit adds a test, the second commit fixes the bugs (and removes the todo_wine). Original ticket for reference: https://jira.reactos.org/browse/CORE-15781 -- v3: kernelbase: Fix VerQueryValueW with no data https://gitlab.winehq.org/wine/wine/-/merge_requests/1263

2 2

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

04 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. -- v3: 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. makedep: Pass DLLFLAGS to winebuild --implib invocations. 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 11

[PATCH 0/1] MR1264: winebuild: Avoid calling asm_name twice in function arguments.
by Rémi Bernon (＠rbernon) 04 Nov '22

04 Nov '22

It returns a static buffer pointer and will free the first returned pointer on the second call. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1264

2 1

[PATCH 0/2] MR1263: kernelbase: Fix VerQueryValueW with no data
by Mark Jansen (＠learn-more) 04 Nov '22

04 Nov '22

xul.dll from Mypal 27.9.4 has a resource entry without any data. Windows returns this as null-terminated string, instead of pointing past the data. The first commit adds a test, the second commit fixes the bugs (and removes the todo_wine). Original ticket for reference: https://jira.reactos.org/browse/CORE-15781 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1263

3 3

[PATCH v2 0/2] MR1263: kernelbase: Fix VerQueryValueW with no data
by Mark Jansen (＠learn-more) 04 Nov '22

04 Nov '22

xul.dll from Mypal 27.9.4 has a resource entry without any data. Windows returns this as null-terminated string, instead of pointing past the data. The first commit adds a test, the second commit fixes the bugs (and removes the todo_wine). Original ticket for reference: https://jira.reactos.org/browse/CORE-15781 -- v2: kernelbase: Fix VerQueryValueW with no data version/tests: Add a test for an empty value in VerQueryValueW https://gitlab.winehq.org/wine/wine/-/merge_requests/1263

2 2

[PATCH v4 0/8] MR1206: mshtml: More misc improvements.
by Gabriel Ivăncescu (＠insn) 04 Nov '22

04 Nov '22

The storage event url prop fix is tested after the location.hash tests are added, implicitly. I didn't find a need to add extra tests for no reason there. The super_navigate changes for fragments is also necessary for the location.hash tests (and its behavior, apparently)… -- v4: mshtml: Implement document.importNode. mshtml: Implement HTMLLocation_put_hash. mshtml: Always use super_navigate when navigating fragments. mshtml: Don't include fragment in storage event's url prop. mshtml: Return E_ABORT if wine-gecko's OnDataAvailable aborts the binding. mshtml: Improve locale stub for KeyboardEvent. mshtml: Implement isContentEditable for HTML elements. mshtml/tests: Handle broken localStorage on native. https://gitlab.winehq.org/wine/wine/-/merge_requests/1206

4 12

[PATCH 0/1] MR1259: gitlab: Run some tests with nulldrv driver.
by Rémi Bernon (＠rbernon) 04 Nov '22

04 Nov '22

Allowing the job to fail to avoid failing the pipeline. Some new tests are failing with it, and some are succeeding todo_wine. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1259

3 4

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

04 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. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1262

4 7

2025

2024

2023

2022

wine-gitlab November 2022