- Wine-GitLab - list.winehq.org

Re: [PATCH v7 0/2] MR10291: Fix GetModuleFileName string termination
by Akihiro Sagawa (＠sgwaki) March 12, 2026

March 12, 2026

Akihiro Sagawa (@sgwaki) commented about dlls/kernel32/tests/loader.c: > +{ > + WCHAR dll_name[MAX_PATH]; > + int dll_name_len, i; > + > + dll_name[0] = 0; > + GetModuleFileNameW(NULL, dll_name, MAX_PATH); > + dll_name_len = wcslen(dll_name); > + ok(dll_name_len > 0, "can't get path for NULL module\n"); > + ok(dll_name_len < MAX_PATH, "unterminated path for NULL module\n"); > + > + memset(dll_name, 0x9, sizeof(dll_name)); > + GetModuleFileNameW(NULL, dll_name, dll_name_len); > + for (i = 0; i < MAX_PATH && dll_name[i]; i++) > + ; > + ok(i > 0, "can't get path for NULL module with short buffer\n"); > + ok(i < MAX_PATH, "unterminated path for NULL module with short buffer\n"); You might want to compare this with the return value here as well. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10291#note_132020

1 0

Re: [PATCH v7 0/2] MR10291: Fix GetModuleFileName string termination
by Akihiro Sagawa (＠sgwaki) March 12, 2026

March 12, 2026

Akihiro Sagawa (@sgwaki) commented about dlls/kernel32/tests/loader.c: > > +static void test_getmodulefilename_string_termination(void) > +{ > + WCHAR dll_name[MAX_PATH]; > + int dll_name_len, i; > + > + dll_name[0] = 0; > + GetModuleFileNameW(NULL, dll_name, MAX_PATH); > + dll_name_len = wcslen(dll_name); > + ok(dll_name_len > 0, "can't get path for NULL module\n"); > + ok(dll_name_len < MAX_PATH, "unterminated path for NULL module\n"); > + > + memset(dll_name, 0x9, sizeof(dll_name)); > + GetModuleFileNameW(NULL, dll_name, dll_name_len); > + for (i = 0; i < MAX_PATH && dll_name[i]; i++) > + ; You can replace this implementation with `wcsnlen()`. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10291#note_132019

1 0

Re: [PATCH v7 0/2] MR10291: Fix GetModuleFileName string termination
by Akihiro Sagawa (＠sgwaki) March 12, 2026

March 12, 2026

Akihiro Sagawa (@sgwaki) commented about dlls/kernel32/tests/loader.c: > DeleteFileA( long_path ); > } > > +static void test_getmodulefilename_string_termination(void) > +{ > + WCHAR dll_name[MAX_PATH]; > + int dll_name_len, i; > + > + dll_name[0] = 0; > + GetModuleFileNameW(NULL, dll_name, MAX_PATH); > + dll_name_len = wcslen(dll_name); You might want to add a return‑value check and compare it with `wcslen()` result to determine success or failure. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10291#note_132018

1 0

Re: [PATCH v7 0/2] MR10291: Fix GetModuleFileName string termination
by Akihiro Sagawa (＠sgwaki) March 12, 2026

March 12, 2026

Akihiro Sagawa (@sgwaki) commented about dlls/kernel32/tests/loader.c: > DeleteFileA( long_path ); > } > > +static void test_getmodulefilename_string_termination(void) > +{ > + WCHAR dll_name[MAX_PATH]; > + int dll_name_len, i; > + > + dll_name[0] = 0; This assignment is redundant. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10291#note_132016

1 0

Re: [PATCH v7 0/2] MR10291: Fix GetModuleFileName string termination
by Akihiro Sagawa (＠sgwaki) March 12, 2026

March 12, 2026

Akihiro Sagawa (@sgwaki) commented about dlls/kernel32/tests/loader.c: > DeleteFileA( long_path ); > } > > +static void test_getmodulefilename_string_termination(void) > +{ > + WCHAR dll_name[MAX_PATH]; > + int dll_name_len, i; > + > + dll_name[0] = 0; > + GetModuleFileNameW(NULL, dll_name, MAX_PATH); > + dll_name_len = wcslen(dll_name); > + ok(dll_name_len > 0, "can't get path for NULL module\n"); > + ok(dll_name_len < MAX_PATH, "unterminated path for NULL module\n"); It's unclear what the actual value of `dll_name_len` is. See [[Writing good error messages|Wine-Developer's-Guide/Writing-Conformance-Tests#writing-good-error-messages]] for details. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10291#note_132017

1 0

[PATCH v12 0/1] MR10244: vbscript: Fix Sub first argument parentheses handling.
by Francis De Brabandere (＠francisdb) March 12, 2026

March 12, 2026

Closes https://bugs.winehq.org/show_bug.cgi?id=54177 Fix parsing of expressions like `Sub (i - 20) * 8,y` where a space-separated parenthesized expression followed by a binary operator(*` /` \` ^` &`) was incorrectly treated as call argument parentheses instead of expression grouping. If you look at all the extra test cases in `lang.vbs` you will see which parentheses related issues this solves. I have been testing with a repo full of scripts: https://github.com/sverrewl/vpxtable_scripts There are still quite some compiler issues left but this addresses the biggest parser issues. -- v12: vbscript: Fix Sub first argument parentheses handling. https://gitlab.winehq.org/wine/wine/-/merge_requests/10244

2 1

[PATCH v6 0/1] MR10302: vbscript: Collect const declarations upfront.
by Francis De Brabandere (＠francisdb) March 12, 2026

March 12, 2026

fixes https://bugs.winehq.org/show_bug.cgi?id=56931 -- v6: vbscript: Collect const declarations upfront. https://gitlab.winehq.org/wine/wine/-/merge_requests/10302

2 1

[PATCH v16 0/4] MR8394: Fix Clip Studio Paint crash with Windows 10 compatibility mode
by Ignacy Kuchciński March 12, 2026

March 12, 2026

This fixes the crash when launching ClipStudioPaint.exe with Windows 10 compatiblity mode by adding the minimum required windows.ui and windows.storage definitions and stubs. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58254 -- v16: windows.storage: Add stubs for StorageFile class. include: Add Windows.Storage.StorageFile missing definitions. windows.ui: Add stubs for RadialController classes. include: Add Windows.UI.Input.RadialController definitions. https://gitlab.winehq.org/wine/wine/-/merge_requests/8394

2 5

[PATCH v7 0/1] MR10308: vbscript: Fix chained array indexing on call expression results.
by Francis De Brabandere (＠francisdb) March 12, 2026

March 12, 2026

fixes * https://bugs.winehq.org/show_bug.cgi?id=58026 * https://bugs.winehq.org/show_bug.cgi?id=58056 -- v7: vbscript: Fix chained array indexing on call expression results. https://gitlab.winehq.org/wine/wine/-/merge_requests/10308

2 1

[PATCH v3 0/1] MR10312: vbscript: fix dot member access after line continuation
by Francis De Brabandere (＠francisdb) March 12, 2026

March 12, 2026

fixes https://bugs.winehq.org/show_bug.cgi?id=56480 The lexer distinguishes '.' (member access) from tDOT (with-statement expression) by checking the character before the dot. After a line continuation (_), ptr[-1] is a newline or whitespace instead of the actual preceding token, causing the dot to be incorrectly treated as a with-expression. Track when a line continuation was just consumed and use last_token to resolve the ambiguity in that case. -- v3: vbscript: Fix dot member access after line continuation. https://gitlab.winehq.org/wine/wine/-/merge_requests/10312

2 1