[Bug 48458] New: Changes to scanf.c and scanf.h have resulted in Star Citizen incorrectly showing green lighting
https://bugs.winehq.org/show_bug.cgi?id=48458 Bug ID: 48458 Summary: Changes to scanf.c and scanf.h have resulted in Star Citizen incorrectly showing green lighting Product: Wine Version: unspecified Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: msvcrt Assignee: wine-bugs(a)winehq.org Reporter: michael.scott(a)quest-arts.co.uk Distribution: --- I was looking into this after a discussion with the LUG group in Star Citizen and discovered the green lighting issue was introduced in the following commit in wine-5.0_rc4: f99d307a3e1f9beb7fd9dc8892b5cfabbabf816b msvcrt: Use parse_double for scanf floats. I had to cherry pick the following additional commits, due to the error 30014 occurring when joining th PU without them: 1366dd69d5a0578299dc69365db972b0341456d2 msvcrt: Update consumed in all scanf variants on end of input. and c4ad7391956fcbfe7a1a9324ebda1e013e5f6edd msvcrt: Don't use ungetc when reading double in scanf. I confirmed this is the issue by taking dlls/msvcrt/scanf.c and dlls/msvcrt/scanf.h as they are after commit: 66c9c358ae5c50fc246cf4a4e280b401a5fc730b msvcrt: Use parse_double for wcstod. then replacing the latest wine.git HEAD with those files and recompiling. This resolved the green lighting 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.
https://bugs.winehq.org/show_bug.cgi?id=48458 Piotr Caban <piotr.caban(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |piotr.caban(a)gmail.com --- Comment #1 from Piotr Caban <piotr.caban(a)gmail.com> --- There is a different regression caused by the same patch that is fixed by: https://source.winehq.org/patches/data/176606 Could you please test if the game works with this patch? -- 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.
https://bugs.winehq.org/show_bug.cgi?id=48458 --- Comment #2 from Mike Scott <michael.scott(a)quest-arts.co.uk> --- Hi Piotr, Thank you for the quick reply. I have tried out the patch using the latest wine.git HEAD. The patch resolves this 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.
https://bugs.winehq.org/show_bug.cgi?id=48458 Mike Scott <michael.scott(a)quest-arts.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|unspecified |5.0-rc4 -- 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.
https://bugs.winehq.org/show_bug.cgi?id=48458 Gijs Vermeulen <gijsvrm(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Fixed by SHA1| |287dabd9b6887e94cabfa2a5f9b | |fe822522095e5 Status|UNCONFIRMED |RESOLVED --- Comment #3 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- Patch committed as <https://source.winehq.org/git/wine.git/commit/287dabd9b6887e94cabfa2a5f9bfe822522095e5> Marking FIXED. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=48458 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |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.
https://bugs.winehq.org/show_bug.cgi?id=48458 Piotr Caban <piotr.caban(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1| |f99d307a3e1f9beb7fd9dc8892b | |5cfabbabf816b -- 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.
https://bugs.winehq.org/show_bug.cgi?id=48458 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #4 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 5.0-rc6. -- 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