[Bug 59944] New: BOINC's project dialog is messed up
http://bugs.winehq.org/show_bug.cgi?id=59944 Bug ID: 59944 Summary: BOINC's project dialog is messed up Product: Wine Version: 11.12 Hardware: x86-64 URL: https://github.com/BOINC/boinc/releases/download/clien t_release%2F7.22%2F7.22.2/boinc_7.22.2_windows_x86_64. exe OS: Linux Status: NEW Keywords: download, regression Severity: trivial Priority: P2 Component: ntdll Assignee: wine-bugs@list.winehq.org Reporter: imwellcushtymelike@gmail.com CC: julliard@winehq.org Regression SHA1: ad9dfe8bcc0ad048dc044694969e13235caadf46 Distribution: Ubuntu Created attachment 81294 --> http://bugs.winehq.org/attachment.cgi?id=81294 Screenshot of broken dialogs The dialog opened by BOINC so that you can choose a project is messed up visually. The description text is not wholly on the screen. The URL is vertical! (Though it still works) Bisected to ad9dfe8bcc0ad048dc044694969e13235caadf46. commit ad9dfe8bcc0ad048dc044694969e13235caadf46 Author: Alexandre Julliard <julliard@winehq.org> Date: Thu Oct 9 12:58:30 2025 +0200 ntdll: Use the export name if any to find the corresponding builtin. Same on 32- and 64-bit clients. https://github.com/BOINC/boinc/releases/download/client_release%2F7.22%2F7.2... -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 --- Comment #1 from Ken Sharp <imwellcushtymelike@gmail.com> --- Created attachment 81295 --> http://bugs.winehq.org/attachment.cgi?id=81295 Screenshot of working dialog -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 --- Comment #2 from Ken Sharp <imwellcushtymelike@gmail.com> --- Created attachment 81296 --> http://bugs.winehq.org/attachment.cgi?id=81296 Wine 11.12 console output -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 --- Comment #3 from Ken Sharp <imwellcushtymelike@gmail.com> --- Created attachment 81297 --> http://bugs.winehq.org/attachment.cgi?id=81297 Wine 11.12 +ntdll -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 Ken Sharp <imwellcushtymelike@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |source -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 --- Comment #4 from Ken Sharp <imwellcushtymelike@gmail.com> --- Created attachment 81298 --> http://bugs.winehq.org/attachment.cgi?id=81298 About About dialog too. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 Alexandre Julliard <julliard@winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1|ad9dfe8bcc0ad048dc044694969 | |e13235caadf46 | --- Comment #5 from Alexandre Julliard <julliard@winehq.org> --- That's some kind of comctl32 bug. The loader change is making it pick up the broken comctl32 from the prefix, but it's not causing the bug. To do a proper bisect you'd need to recreate the prefix and reinstall the app after every iteration. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 Ken Sharp <imwellcushtymelike@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1| |48bb5d586467895c329b840ed95 | |40525edf3b422 --- Comment #6 from Ken Sharp <imwellcushtymelike@gmail.com> --- Thanks, AJ. Ran a bisect multiple times and came up with different results. Finally, after nuking everything between each recompile I landed on: commit 48bb5d586467895c329b840ed9540525edf3b422 Author: Alexandre Julliard <julliard@winehq.org> Date: Wed Nov 12 13:43:14 2025 +0100 winebuild: Assign section file positions once the offset is known. Didn't trust the result so I ran multiple loops with both this commit, and the previous commit, compiling from scratch and installing in a clean wineprefix each time, and got consistent results. 48bb5d586467895c329b840ed9540525edf3b422 — Bug 48bb5d586467895c329b840ed9540525edf3b422^ — No bug For my sanity: wine boinc_7.14.2_windows_intelx86.exe /v/qn /S wine "c:\program files\boinc\boincmgr.exe" Tested with the 32-bit release to save some time. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 --- Comment #7 from Ken Sharp <imwellcushtymelike@gmail.com> ---
48bb5d586467895c329b840ed9540525edf3b422 — Bug
Just for clarity: I'm not saying Bug 48, that seems to be a Bugzilla issue. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 Ken Sharp <imwellcushtymelike@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Component|ntdll |tools -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 Alexandre Julliard <julliard@winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1|48bb5d586467895c329b840ed95 | |40525edf3b422 | --- Comment #8 from Alexandre Julliard <julliard@winehq.org> --- I don't think that's it either, I'm seeing the same issue in Wine 10.17, which is before the mentioned commit. I'm not convinced that this is actually a regression. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 --- Comment #9 from Ken Sharp <imwellcushtymelike@gmail.com> --- Created attachment 81326 --> http://bugs.winehq.org/attachment.cgi?id=81326 Wine 10.17 Screenshot Wine 10.17 gives me a different issue. The dialog doesn't load at all and the whole app is unusable. Ran multiple times again to confirm. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 Maotong Zhang <zmtong1988@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |zmtong1988@gmail.com --- Comment #10 from Maotong Zhang <zmtong1988@gmail.com> --- It looks like nFit being passed in is 0, which causes issues in the later calculations. static BOOL SYSLINK_WrapLine (LPWSTR Text, WCHAR BreakChar, int x, int *LineLen, int nFit, LPSIZE Extent) { int i; + if (nFit <= 1) nFit = (*LineLen > 1) ? *LineLen : 1; for (i = 0; i < nFit; i++) if (Text[i] == '\r' || Text[i] == '\n') break; ...... } After this fix, it should correctly render at least one line. Further research may be necessary. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 --- Comment #11 from Alexandre Julliard <julliard@winehq.org> --- (In reply to Ken Sharp from comment #9)
Created attachment 81326 [details] Wine 10.17 Screenshot
Wine 10.17 gives me a different issue. The dialog doesn't load at all and the whole app is unusable. Ran multiple times again to confirm.
There are other issues in older versions that break the install. Installing with latest Wine, a bisect yields 541d9be5ecda1bc0c8fce22f1318bf5d72572327 "comctl32: Separate v5 and v6". That's not necessarily meaningful in itself, it probably just means that with a proper comctl32 v6, the app (or Wine) takes a different code path somewhere. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 Ken Sharp <imwellcushtymelike@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #81297|0 |1 is obsolete| | Attachment #81326|0 |1 is obsolete| | --- Comment #12 from Ken Sharp <imwellcushtymelike@gmail.com> --- Created attachment 81332 --> http://bugs.winehq.org/attachment.cgi?id=81332 Screenshot with patch Patch dramatically improves the layout. Sick of trying to find a phantom regression: 9.0~repack-4build3 (Ubuntu’s wine-stable) — dialog never loads 9.0.1 (compiled in 2025) — no bug 9.0.1 (compiled today) — no bug 10.17 (compiled today) — empty dialog, app locks up 10.18 (compiled today) — no bug wine-10.18-297-g9250ecc5a6a (compiled today) ― no bug wine-10.18-298-g48bb5d58646 (compiled today) — this bug 11.0 (wine-stable) — this bug 11.11 (wine-devel) — this bug 11.11 (wine-staging) — this bug wine-11.12-31-g76be398f170 (compiled today) — this bug And here's a thrilling three hour video that proves nothing¹: https://youtu.be/BaHqlIQpm4Y ¹ Something weird occurs while compiling? I'll try changing a bunch of things. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 --- Comment #13 from Ken Sharp <imwellcushtymelike@gmail.com> --- Though BOINC 8 for amd64 has this issue, the arm64 version does not. They both have an issue maximising the event log, which may or may not be related. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 Bernhard Übelacker <bernhardu@mailbox.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bernhardu@mailbox.org --- Comment #14 from Bernhard Übelacker <bernhardu@mailbox.org> --- Created attachment 81585 --> http://bugs.winehq.org/attachment.cgi?id=81585 debugging.txt I tried to do some debugging here and got to this control gets created with a zero size, then a few times the text gets set, with a few LM_GETIDEALHEIGHT messages between. As far as I see the LM_GETIDEALSIZE calls SYSLINK_Render with a zero rectangle. The condition "rc.right - SL_LEFTMARGIN < 0" is not met because rc.right(0) minus SL_LEFTMARGIN(0) is only zero. Therefore the rendering is trying to have the minimal width and grows in height. The patch in following merge request changes this condition to "smaller or equal zero". That way the rendering is more greedy on the width than the height. https://gitlab.winehq.org/wine/wine/-/merge_requests/11481 Maybe you can give it a try, if it works for you. Unfortunately this is not explaining the randomness described previous comments. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 --- Comment #15 from Ken Sharp <imwellcushtymelike@gmail.com> --- Patch from MR looks good here. The dialog looks normal. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 Bernhard Übelacker <bernhardu@mailbox.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED Fixed by SHA1| |cdbb7f7d27066988caaedce1c38 | |af6dea6c65b64 --- Comment #16 from Bernhard Übelacker <bernhardu@mailbox.org> --- Merge request got accepted, should be fixed in wine-11.15: https://gitlab.winehq.org/wine/wine/-/commit/cdbb7f7d27066988caaedce1c38af6d... -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 --- Comment #17 from Ken Sharp <imwellcushtymelike@gmail.com> --- Created attachment 81745 --> http://bugs.winehq.org/attachment.cgi?id=81745 Wine 11.15 screenshot The patch was a partial fix. In fixing the text, it made the dialog way too big. See attached. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 Ken Sharp <imwellcushtymelike@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED |--- --- Comment #18 from Ken Sharp <imwellcushtymelike@gmail.com> --- Reopening -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 --- Comment #19 from Bernhard Übelacker <bernhardu@mailbox.org> --- (In reply to Ken Sharp from comment #18)
Reopening
Usually the target is to have only one issue per bug, therefore it may be helpful to create a followup-issue after one thing is fixed. But I guess we can just say if it is a different issue when it is solved, so here we go ... I retried some Debian packages, if they show the wide-dialog: 9.0.0~trixie-1 working 9.10~trixie-1 working 10.0.0~trixie-1 working 10.10~trixie-1 working 10.15~trixie-1 working 10.16~trixie-1 working 10.17~trixie-1 affected (link vertical) 10.18~trixie-1 affected (link vertical) 11.14~trixie-1 affected (link vertical) 11.15~trixie-1 working Unfortunately I see currently the wide-dialog only with a self-built wine-11.15 (with some custom patches for debugging). -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59944 --- Comment #20 from Bernhard Übelacker <bernhardu@mailbox.org> --- Some more debugging brought me to SYSLINK_Render again, which gets this time called for a string containing just a single space. Unfortunately in this case it currently returns values calculated from the uninitialized variable szDim. For testing setting szDim to some hight value should make this issue visible, even in the height of the dialog. I submitted this merge request that should fix that too: https://gitlab.winehq.org/wine/wine/-/merge_requests/11623 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
WineHQ Bugzilla