http://bugs.winehq.org/show_bug.cgi?id=58811
Bug ID: 58811 Summary: Steam: Unable to display the intended GUI windows due to steamwebhelper being unresponsive. Product: Wine Version: 10.16 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: dwrite Assignee: wine-bugs@winehq.org Reporter: chiitoo@gentoo.org Regression SHA1: 2a2cc34db71a86899b69bf3ee1da52793c342019 Distribution: Gentoo
Created attachment 79466 --> http://bugs.winehq.org/attachment.cgi?id=79466 terminal
After 2a2cc34db71 [1], Steam appears to launch, showing its notification area icon, but no window will appear.
After a while from clicking said icon, the "Steamwebhelper is not responding" window [2] will appear, with a few different options to restart things with, of which none do any better.
I believe the
src\vgui2\vgui_surfacelib\Win32Font.cpp (963) : Couldn't get string length
line shows up only when this problem can be seen, which could be a sort of a hint at least.
Building before the mentioned commit, or reverting it, and efce37afd70 due to conflicts, on current 'git master' will make it work again.
Hacking on it a bit, removing the 'wcsupr(refkey->name);' line also makes it work, though maybe that is obvious in this case, and might more or less nullify the point of the uppercasing intended.
(Did not try a fresh prefix and clean install of Steam yet due to the cumbersome log-in process, but hopefully that is not necessary here.)
Last tested at 'wine-10.17'.
Thank you!
1. https://gitlab.winehq.org/wine/wine/-/commit/2a2cc34db71a86899b69bf3ee1da527... 2. https://help.steampowered.com/en/faqs/view/6C3E-9504-CC20-AF49
http://bugs.winehq.org/show_bug.cgi?id=58811
Chiitoo chiitoo@gentoo.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #79466|terminal |terminal output from a bad description| |run
http://bugs.winehq.org/show_bug.cgi?id=58811
Chiitoo chiitoo@gentoo.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download, regression URL| |https://cdn.fastly.steamsta | |tic.com/client/installer/St | |eamSetup.exe
http://bugs.winehq.org/show_bug.cgi?id=58811
--- Comment #1 from Zeb Figura z.figura12@gmail.com --- *** Bug 58808 has been marked as a duplicate of this bug. ***
http://bugs.winehq.org/show_bug.cgi?id=58811
--- Comment #2 from Zeb Figura z.figura12@gmail.com --- *** Bug 58809 has been marked as a duplicate of this bug. ***
http://bugs.winehq.org/show_bug.cgi?id=58811
--- Comment #3 from Zeb Figura z.figura12@gmail.com --- *** Bug 58810 has been marked as a duplicate of this bug. ***
http://bugs.winehq.org/show_bug.cgi?id=58811
--- Comment #4 from Chiitoo chiitoo@gentoo.org --- Whoa, I did notice the bug report go out before I was ready, so maybe pressed 'return' accidentally while adding the attachment... and perhaps it was down just long enough to manage creating all those...
Sorry about that!
http://bugs.winehq.org/show_bug.cgi?id=58811
random-nick@mail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |random-nick@mail.com
--- Comment #5 from random-nick@mail.com --- I can reproduce this bug and have bisected it and got the same regression commit and it shows up on the latest release (10.18) and the master (0fc31f4) too. I'm not sure if this bug appeared as soon as 10.16 came out or if it only appeared after a Steam update.
You do not have to log into Steam to reproduce but if you're not logged in the system tray icon doesn't appear (intended behaviour) so you can't tell if the window was supposed to appear or if the updater is still working. To reproduce this bug in a clean prefix you just need to install and run Steam and if the log in window appears after it is done updating then the bug is not present but if does not appear it is present and you will get those "Couldn't get string length" lines in C:\Program Files (x86)\Steam\logs\console_log.txt or on the terminal if you run steam.exe from the terminal. When the log in window does show up in a clean prefix (when using a build before this regression), it won't have any text but this is a separate issue that can be worked around by installing corefonts. Installing or not installing corefonts doesn't seem to affect this bug.
http://bugs.winehq.org/show_bug.cgi?id=58811
Hans Leidekker hans@meelstraat.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |info@maduranma.com
--- Comment #6 from Hans Leidekker hans@meelstraat.net --- *** Bug 58968 has been marked as a duplicate of this bug. ***
http://bugs.winehq.org/show_bug.cgi?id=58811
--- Comment #7 from Martín Durán info@maduranma.com --- Just so it does not get lost, mr. Rafał Mużyło answered in the closed duplicate #58968 this:
(In reply to Rafał Mużyło from comment #4)
We might be looking at something close to a compiler bug here, due to a type mismatch.
The way wcsupr is implemented in msvcrt is by calling _wcsupr_s_l with '-1' as size, but the type for size is size_t (unsigned). _wcsupr_s_l doesn't really account for such values...
Just to verify my idea is going in the right direction, try changing that implementation in _wcsupr_l to - let's say '_wcsupr_s_l( str, wcslen(str)+1, locale);' (likely won't be quite correct for NULL str, but otherwise OK...well, unless I messed something up, so caution advised).
Just pasting here as I'd probably not be read as closed.