November 2025 - Wine-GitLab

Re: [PATCH v8 0/1] MR5373: shell32: Implement FCIDM_SHVIEW_CREATELINK
by Sven Baars (＠sbaars) Nov. 13, 2025

Nov. 13, 2025

Sven Baars (@sbaars) commented about dlls/shell32/shlview_cmenu.c: > + full_pidl = ILCombine(This->pidl, This->apidl[0]); > + if (!full_pidl) > + { > + ERR("Out of Memory\n"); > + return; > + } > + > + hr = IShellLinkW_SetIDList(shelllink, full_pidl); > + ILFree(full_pidl); > + if (FAILED(hr)) > + { > + ERR("SetIDList failed\n"); > + return; > + } > + _ILSimpleGetTextW(This->apidl[0], (LPVOID)filename, MAX_PATH); > + length = wcslen(filename) + wcslen(shortcutW); This looks like a duplicate check. There is another one in the loop below. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5373#note_121803

1 0

[PATCH 0/4] MR9429: Avoid a few compiler warnings.
by Bernhard Übelacker Nov. 13, 2025

Nov. 13, 2025

Gitlab CI uses `--enable-werror` for its builds, but to improve backtraces with my recent address sanitizer experiments I am using these flags `-g -O1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fno-inline-functions` with both, gcc at the unix side and llvm-mingw at the pe side. Therefore these warnings would turn into errors. This four patches are helping gcc at the unix side. Would these patches be acceptable? Should they be separate merge requests? <details> <summary>Details</summary> ``` ccache gcc -m64 -c -o tools/winebuild/utils.o .../wine/tools/winebuild/utils.c \ -Itools/winebuild -I.../wine/tools/winebuild -Iinclude \ -I.../wine/include -D__WINESRC__ -DWINE_UNIX_LIB -Wall -pipe \ -fcf-protection=none -fvisibility=hidden -fno-stack-protector -fno-strict-aliasing \ -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Winit-self -Wlogical-op \ -Wno-packed-not-aligned -Wpointer-arith -Wshift-overflow=2 -Wstrict-prototypes -Wtype-limits \ -Wunused-but-set-parameter -Wvla -Wwrite-strings -Wno-misleading-indentation -gdwarf-4 -Werror \ -g -O1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fno-inline-functions .../wine/tools/winebuild/utils.c: In function 'find_tool': .../wine/tools/winebuild/utils.c:230:8: error: 'file' may be used uninitialized [-Werror=maybe-uninitialized] 230 | if (!file && cc_command.count) file = find_clang_tool( cc_command, name ); | ^ .../wine/tools/winebuild/utils.c:214:17: note: 'file' was declared here 214 | const char *file; | ^~~~ cc1: all warnings being treated as errors make: *** [Makefile:610992: tools/winebuild/utils.o] Error 1 ccache gcc -m64 -c -o dlls/ntdll/unix/file.o .../wine/dlls/ntdll/unix/file.c -Idlls/ntdll \ -I.../wine/dlls/ntdll -Iinclude \ -I.../wine/include -D__WINESRC__ -DLTC_NO_PROTOTYPES \ -DLTC_SOURCE -D_NTSYSTEM_ -D_ACRTIMP= -DWINBASEAPI= -DWINE_UNIX_LIB \ -I/usr/include/x86_64-linux-gnu -Wall -pipe -fcf-protection=none -fvisibility=hidden \ -fno-stack-protector -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body \ -Wignored-qualifiers -Winit-self -Wlogical-op -Wno-packed-not-aligned -Wpointer-arith \ -Wshift-overflow=2 -Wstrict-prototypes -Wtype-limits -Wunused-but-set-parameter -Wvla \ -Wwrite-strings -Wno-misleading-indentation -gdwarf-4 -Werror -fPIC -fasynchronous-unwind-tables \ -g -O1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fno-inline-functions .../wine/dlls/ntdll/unix/file.c: In function 'NtReadFile': .../wine/dlls/ntdll/unix/file.c:5758:23: error: 'avail_mode' may be used uninitialized [-Werror=maybe-uninitialized] 5758 | if (total && avail_mode) | ~~~~~~^~~~~~~~~~~~~ .../wine/dlls/ntdll/unix/file.c:5755:18: note: 'avail_mode' was declared here 5755 | BOOL avail_mode; | ^~~~~~~~~~ cc1: all warnings being treated as errors make: *** [Makefile:318598: dlls/ntdll/unix/file.o] Error 1 ccache gcc -m64 -c -o dlls/sane.ds/unixlib.o .../wine/dlls/sane.ds/unixlib.c -Idlls/sane.ds \ -I.../wine/dlls/sane.ds -Iinclude \ -I.../wine/include -D__WINESRC__ -DWINE_UNIX_LIB -Wall -pipe \ -fcf-protection=none -fvisibility=hidden -fno-stack-protector -fno-strict-aliasing \ -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Winit-self -Wlogical-op \ -Wno-packed-not-aligned -Wpointer-arith -Wshift-overflow=2 -Wstrict-prototypes -Wtype-limits \ -Wunused-but-set-parameter -Wvla -Wwrite-strings -Wno-misleading-indentation -gdwarf-4 -Werror \ -fPIC -fasynchronous-unwind-tables -g -O1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fno-inline-functions .../wine/dlls/sane.ds/unixlib.c: In function 'read_data': .../wine/dlls/sane.ds/unixlib.c:332:8: error: 'status' may be used uninitialized [-Werror=maybe-uninitialized] 332 | if (status == SANE_STATUS_EOF) return TWCC_SUCCESS; | ^ .../wine/dlls/sane.ds/unixlib.c:321:17: note: 'status' was declared here 321 | SANE_Status status; | ^~~~~~ cc1: all warnings being treated as errors make: *** [Makefile:366624: dlls/sane.ds/unixlib.o] Error 1 ccache gcc -m64 -c -o server/directory.o .../wine/server/directory.c -Iserver \ -I.../wine/server -Iinclude \ -I.../wine/include -D__WINESRC__ -DWINE_UNIX_LIB -Wall -pipe \ -fcf-protection=none -fvisibility=hidden -fno-stack-protector -fno-strict-aliasing \ -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Winit-self -Wlogical-op \ -Wno-packed-not-aligned -Wpointer-arith -Wshift-overflow=2 -Wstrict-prototypes -Wtype-limits \ -Wunused-but-set-parameter -Wvla -Wwrite-strings -Wno-misleading-indentation -gdwarf-4 -Werror \ -g -O1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fno-inline-functions .../wine/server/directory.c: In function 'create_session': .../wine/server/directory.c:327:44: error: 'snprintf' output may be truncated before the last format character [-Werror=format-truncation=] 327 | snprintf( id_strA, sizeof(id_strA), "%u", id ); | ^ .../wine/server/directory.c:327:5: note: 'snprintf' output between 2 and 11 bytes into a destination of size 10 327 | snprintf( id_strA, sizeof(id_strA), "%u", id ); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ cc1: all warnings being treated as errors make: *** [Makefile:607457: server/directory.o] Error 1 ``` </details> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9429

2 7

[PATCH v3 0/1] MR9449: winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with...
by Zhiyi Zhang (＠zhiyi) Nov. 13, 2025

Nov. 13, 2025

winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with non-fullscreen rects. xinerama_get_fullscreen_monitors() was supposed to be called with fullscreen rects. Thus, we could assume the monitor indices are all zero when there is only one monitor as an optimization. However, after baf65867, xinerama_get_fullscreen_monitors() can be called with non-fullscreen rects. So we need to remove the optimization so that it works correctly for non-fullscreen rects as well. Fix the Guild Wars 2 game window having an offset in fullscreen mode. -- v3: winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with non-fullscreen rects. https://gitlab.winehq.org/wine/wine/-/merge_requests/9449

3 6

[PATCH 0/2] MR9446: dmsynth: Fix instrument selection
by Anton Baskanov (＠baskanov) Nov. 13, 2025

Nov. 13, 2025

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

3 4

[PATCH v2 0/1] MR9449: winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with...
by Zhiyi Zhang (＠zhiyi) Nov. 13, 2025

Nov. 13, 2025

winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with non-fullscreen rects. xinerama_get_fullscreen_monitors() was supposed to be called with fullscreen rects. Thus, we could assume the monitor indices are all zero when there is only one monitor as an optimization. However, after baf65867, xinerama_get_fullscreen_monitors() can be called with non-fullscreen rects. So we need to remove the optimization so that it works correctly for non-fullscreen rects as well. Fix the Guild Wars 2 game window having an offset in fullscreen mode. -- v2: winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with non-fullscreen rects. https://gitlab.winehq.org/wine/wine/-/merge_requests/9449

3 2

Re: [PATCH v4 0/1] MR9423: ntdll: Warn when environment block is too large, instead of removing it
by Alexandre Julliard (＠julliard) Nov. 13, 2025

Nov. 13, 2025

No, I don't think it addresses the issue at all. I'll try to come up with something. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9423#note_121789

1 0

[PATCH 0/1] MR9449: winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with...
by Zhiyi Zhang (＠zhiyi) Nov. 13, 2025

Nov. 13, 2025

winex11.drv: Fix xinerama_get_fullscreen_monitors() not working correctly with non-fullscreen rects. xinerama_get_fullscreen_monitors() was supposed to be called with fullscreen rects. Thus, we could assume the monitor indices are all zero when there is only one monitor as an optimization. However, after baf65867, xinerama_get_fullscreen_monitors() can be called with non-fullscreen rects. So we need to remove the optimization so that it works correctly for non-fullscreen rects as well. Fix the Guild Wars 2 game window having an offset in fullscreen mode. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9449

2 2

[PATCH 0/2] MR9447: bcrypt: Complete the remaining four RSA parameters when importing the private key.
by Haoyang Chen (＠chenhaoyang) Nov. 13, 2025

Nov. 13, 2025

Some versions of libgnutls require passing all parameters when importing private keys, otherwise they will crash. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9447

3 9

[PATCH 0/5] MR9448: opengl32: Match native GL_VENDOR and GL_RENDERER strings.
by Rémi Bernon Nov. 13, 2025

Nov. 13, 2025

As much as possible at least. Some applications expect the GL_RENDERER strings to be shorter than 64 chars, as they copy it to some local buffer. They also seem to crash after GL_VENDOR call, possibly expecting native names too, so lets try to match both. The GPU name table is taken from wined3d, and it should then be possible to get rid of the names there and always use the descriptions from the registry / GL_RENDERER strings instead. The vendor names differ between OpenGL and the registry on Windows, GL_VENDOR names are taken from https://feedback.wildfiregames.com/report/opengl/feature/GL_VENDOR. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58831 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9448

1 5

[PATCH 0/1] MR9445: opencl: Silence warnings on macOS for functions deprecated by OpenCL 1.2.
by Brendan Shanks (＠bshanks) Nov. 12, 2025

Nov. 12, 2025

There are 6 functions that were deprecated in OpenCL 1.2 and are marked deprecated in the macOS headers (without any way to disable the `deprecated` attribute other than `#pragma diagnostic ignored`). Fixes these warnings: ``` ../dlls/opencl/unix_thunks.c:38:25: warning: 'clCreateImage2D' is deprecated: first deprecated in macOS 10.8 [-Wdeprecated-declarations] 38 | *params->__retval = clCreateImage2D( params->context, params->flags, params->image_format, params->image_width, params->image_height, params->image_row_pitch, params->host_ptr, params->errcode_ret ); | ^ /Applications/Xcode26.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/System/Library/Frameworks/OpenCL.framework/Headers/cl.h:1170:1: note: 'clCreateImage2D' has been explicitly marked deprecated here 1170 | clCreateImage2D(cl_context /* context */, | ^ ../dlls/opencl/unix_thunks.c:46:25: warning: 'clCreateImage3D' is deprecated: first deprecated in macOS 10.8 [-Wdeprecated-declarations] 46 | *params->__retval = clCreateImage3D( params->context, params->flags, params->image_format, params->image_width, params->image_height, params->image_depth, params->image_row_pitch, params->image_slice_pitch, params->host_ptr, params->errcode_ret ); | ^ /Applications/Xcode26.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/System/Library/Frameworks/OpenCL.framework/Headers/cl.h:1180:1: note: 'clCreateImage3D' has been explicitly marked deprecated here 1180 | clCreateImage3D(cl_context /* context */, | ^ ../dlls/opencl/unix_thunks.c:124:12: warning: 'clEnqueueBarrier' is deprecated: first deprecated in macOS 10.8 [-Wdeprecated-declarations] 124 | return clEnqueueBarrier( params->command_queue ); | ^ /Applications/Xcode26.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/System/Library/Frameworks/OpenCL.framework/Headers/cl.h:1201:1: note: 'clEnqueueBarrier' has been explicitly marked deprecated here 1201 | clEnqueueBarrier(cl_command_queue /* command_queue */) CL_EXT_SUFFIX__VERSION_1_1_DEPRECATED; | ^ ../dlls/opencl/unix_thunks.c:203:12: warning: 'clEnqueueMarker' is deprecated: first deprecated in macOS 10.8 [-Wdeprecated-declarations] 203 | return clEnqueueMarker( params->command_queue, params->event ); | ^ /Applications/Xcode26.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/System/Library/Frameworks/OpenCL.framework/Headers/cl.h:1192:1: note: 'clEnqueueMarker' has been explicitly marked deprecated here 1192 | clEnqueueMarker(cl_command_queue /* command_queue */, | ^ ../dlls/opencl/unix_thunks.c:266:12: warning: 'clEnqueueWaitForEvents' is deprecated: first deprecated in macOS 10.8 [-Wdeprecated-declarations] 266 | return clEnqueueWaitForEvents( params->command_queue, params->num_events, params->event_list ); | ^ /Applications/Xcode26.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/System/Library/Frameworks/OpenCL.framework/Headers/cl.h:1196:1: note: 'clEnqueueWaitForEvents' has been explicitly marked deprecated here 1196 | clEnqueueWaitForEvents(cl_command_queue /* command_queue */, | ^ ../dlls/opencl/unix_thunks.c:551:12: warning: 'clUnloadCompiler' is deprecated: first deprecated in macOS 10.8 [-Wdeprecated-declarations] 551 | return clUnloadCompiler(); | ^ /Applications/Xcode26.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/System/Library/Frameworks/OpenCL.framework/Headers/cl.h:1204:1: note: 'clUnloadCompiler' has been explicitly marked deprecated here 1204 | clUnloadCompiler(void) CL_EXT_SUFFIX__VERSION_1_1_DEPRECATED; | ^ 6 warnings generated. ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9445

2 1