May 2024 - Wine-GitLab - list.winehq.org

Re: [PATCH v4 0/2] MR5392: win32u: Avoid writing past allocated memory in peek_message().
by Jacek Caban (＠jacek) May 28, 2024

May 28, 2024

Jacek Caban (@jacek) commented about dlls/win32u/message.c: > { > if (*prev_size < size) > { > - *buffer = malloc( size ); > + *buffer = realloc( *buffer, size ); This leaks on error, we could leave the buffer unchanged in that case. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5392#note_66128

1 0

Re: [PATCH v2 0/1] MR5402: wmic.exe: Support interactive mode and piped commands
by Hans Leidekker (＠hans) May 28, 2024

May 28, 2024

Creating a new process is inefficient and also visible to applications. You don't need to duplicate the code, it can be factored out in a couple of handlers used in both command line and interactive mode. In interactive mode you'd run a loop that accepts commands and executes them, until you hit 'quit' or 'exit'. wmic not handling spaces in the property list (unless quoted) is a bug, but you should be able to reuse process_property_list(). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5402#note_66122

1 0

[PATCH 0/1] MR5402: Draft: wmic.exe: Support interactive mode and piped commands
by Louis Lenders (＠xe) May 28, 2024

May 28, 2024

See also https://bugs.winehq.org/show_bug.cgi?id=56361 This patch makes piped commands work like for example "echo os get version|wmic" or "type file.txt | wmic" where file.txt contains some commands. (probably used by program in aforementioned bugreport) Also support interactive mode (wine wmic.exe) Marked for now as draft. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5402

4 10

[PATCH v5 0/1] MR5385: winebus: ignore udev events of devices which do not have devnodes
by Tuomas Räsänen May 28, 2024

May 28, 2024

Udev monitor monitors the whole input subsystem, but not all devices in the input subsystem have devnodes associated to them. This MR makes the event processing ignore such devices. All device handling assumes devices have devnodes, so here we just simply ignore all devices which do not have one. They are irrelevant. Previously, udev bus thread aborted when an event for a device without a devnode was processed: ``` 10111.330:0068:0084:trace:hid:process_monitor_event Received action "remove" for udev device (null) 10111.330:0068:0084:warn:hid:bus_main_thread L"UDEV" bus wait returned status 0xc0000005 ``` Just plugging in and out a normal mouse was enough cause this. This was because root input devices (which do not have devnodes) were handled too and `find_device_from_devnode()` choked on NULL argument. -- v5: winebus: ignore udev events of devices which do not have devnodes https://gitlab.winehq.org/wine/wine/-/merge_requests/5385

2 4

[PATCH 0/2] MR5292: Fixes to evr_copy_sample_buffer().
by eric pouech (＠epo) May 28, 2024

May 28, 2024

Two fixes for evr_copy_sample_buffer() where: - we are doing several copies of same area - we are copying from/to beyond image limit (wrong bpp) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5292

4 11

[PATCH v4 0/5] MR5385: winebus: ignore udev events of devices which do not have devnodes
by Tuomas Räsänen May 28, 2024

May 28, 2024

Udev monitor monitors the whole input subsystem, but not all devices in the input subsystem have devnodes associated to them. This MR makes the event processing ignore such devices. All device handling assumes devices have devnodes, so here we just simply ignore all devices which do not have one. They are irrelevant. Previously, udev bus thread aborted when an event for a device without a devnode was processed: ``` 10111.330:0068:0084:trace:hid:process_monitor_event Received action "remove" for udev device (null) 10111.330:0068:0084:warn:hid:bus_main_thread L"UDEV" bus wait returned status 0xc0000005 ``` Just plugging in and out a normal mouse was enough cause this. This was because root input devices (which do not have devnodes) were handled too and `find_device_from_devnode()` choked on NULL argument. -- v4: winebus: change debugging class of an error case from WARN to ERR. winebus: change debugging class of an error case from FIXME to ERR winebus: group local variable declarations https://gitlab.winehq.org/wine/wine/-/merge_requests/5385

2 8

[PATCH 0/3] MR5328: wined3d: Fixes for ARB_bindless_texture support.
by Zebediah Figura (＠zfigura) May 28, 2024

May 28, 2024

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

4 5

[PATCH v8 0/3] MR5060: win32u, explorer, winex11.drv: Restore display settings on process exit.
by Anton Baskanov (＠baskanov) May 28, 2024

May 28, 2024

-- v8: explorer: Restore display settings on process exit. winex11.drv: Process RRNotify events in xrandr14_get_id. user32/tests: Test that display settings are restored on process exit. https://gitlab.winehq.org/wine/wine/-/merge_requests/5060

4 10

[PATCH 0/1] MR5404: windows.perception.stub: Add stub IHolographicSpaceInterop interface.
by Paul Gofman (＠gofman) May 28, 2024

May 28, 2024

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

2 2

[PATCH v4 0/5] MR5306: win32u: Move D3DKMT vulkan implementation out of winex11.
by Rémi Bernon May 28, 2024

May 28, 2024

With some intermediate refactoring to make the code simpler. I believe the failures that happened previously were coming from how NtGdiDdDDIOpenAdapterFromLuid initialized desc->hAdapter in win32u while winex11 was then relying on it. I missed this detail before and it should be working fine now that the vulkan path is only used to retrieve the physical device. -- v4: win32u: Move D3DKMT vulkan implementation out of winex11. winex11: Introduce a new find_adapter_from_handle helper. winex11: Introduce a new get_vulkan_physical_device helper. winex11: Initialize D3DKMT vulkan instance only once. win32u: Open adapters in NtGdiDdDDIEnumAdapters2 outside of the display devices lock. https://gitlab.winehq.org/wine/wine/-/merge_requests/5306

2 7