[Bug 45622] New: Overwatch crashes when trying to save highlights
https://bugs.winehq.org/show_bug.cgi?id=45622 Bug ID: 45622 Summary: Overwatch crashes when trying to save highlights Product: Wine Version: 3.13 Hardware: x86 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: sashok.olen(a)gmail.com Distribution: --- Created attachment 62052 --> https://bugs.winehq.org/attachment.cgi?id=62052 terminal output Whenever I try to save a highlight through in-game feature, the game crashes, even with hardware encoding option disabled. It doesn't crash with a wine debugger dialog but I attached a part of the terminal output of when it crashes. The game also creates its own dump when it crashes but I am not sure it's useful to wine developers. However, if it is, just ask for it and I will upload it. -- 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=45622 zzzzzyzz(a)hacari.org changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |zzzzzyzz(a)hacari.org -- 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=45622 Zebediah Figura <z.figura12(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |z.figura12(a)gmail.com Component|-unknown |mfplat Summary|Overwatch crashes when |Overwatch crashes when |trying to save highlights |trying to save highlights | |(needs | |mfplat.MFTRegisterLocal) --- Comment #1 from Zebediah Figura <z.figura12(a)gmail.com> --- Looks like this: wine: Call from 0x7b44cfd7 to unimplemented function mfplat.dll.MFTRegisterLocal, aborting Disabling mfplat may work around the issue, if the game is smart enough to fall back to quartz or something similar. -- 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=45622 --- Comment #2 from Alexandr Oleynikov <sashok.olen(a)gmail.com> --- The game is smart enough. Though that disables the ability to encode to MP4 and with hardware encoding. -- 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=45622 --- Comment #3 from Alexandr Oleynikov <sashok.olen(a)gmail.com> --- There are free weekends starting from August 23rd till August 27-th for the game, so it's a good opportunity to test and debug everything if somebody maybe wants to fix this :) -- 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=45622 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Ever confirmed|0 |1 CC| |leslie_alistair(a)hotmail.com Staged patchset| |https://github.com/wine-sta | |ging/wine-staging/tree/mast | |er/patches/mfplat-MFTRegist | |erLocal Status|UNCONFIRMED |STAGED -- 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=45622 --- Comment #4 from Alexandr Oleynikov <sashok.olen(a)gmail.com> --- Still crashes, even when the latest commit that stubs it out. Please change back to "Unconfirmed" or "New". -- 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=45622 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Fixed by SHA1| |5b8565d60760f17ecac7bccf9f2 | |c4477943de544 Status|STAGED |RESOLVED --- Comment #5 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- Fixed by https://source.winehq.org/git/wine.git/?a=commit;h=5b8565d60760f17ecac7bccf9... -- 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=45622 Alexandr Oleynikov <sashok.olen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|FIXED |--- Status|RESOLVED |REOPENED --- Comment #6 from Alexandr Oleynikov <sashok.olen(a)gmail.com> --- It wasn't fixed by a stub, it still crashes even with that. -- 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=45622 Gijs Vermeulen <gijsvrm(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |NEW Summary|Overwatch crashes when |Overwatch crashes when |trying to save highlights |trying to save highlights |(needs |(needs |mfplat.MFTRegisterLocal) |mfplat.MFTRegisterLocal | |implementation) --- Comment #7 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- Changing title to reflect requirement. -- 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=45622 --- Comment #8 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- Please provide a new +mfplat log. -- 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=45622 --- Comment #9 from Alexandr Oleynikov <sashok.olen(a)gmail.com> --- Created attachment 62264 --> https://bugs.winehq.org/attachment.cgi?id=62264 staging-3.15 This is with staging-3.15. -- 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=45622 --- Comment #10 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- (In reply to Alexandr Oleynikov from comment #9)
Created attachment 62264 [details] staging-3.15
This is with staging-3.15.
This doesn't have the patch applied wine: Call from 0x7b44fd36 to unimplemented function mfplat.dll.MFTRegisterLocal, aborting -- 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=45622 --- Comment #11 from Alexandr Oleynikov <sashok.olen(a)gmail.com> --- Weird, the patch was released before 3.15. Let me test again but with some other build. -- 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=45622 --- Comment #12 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- (In reply to Alistair Leslie-Hughes from comment #10) It does, but the patchset in staging still has the function as stub in the spec file. -- 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=45622 --- Comment #13 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- (In reply to Alistair Leslie-Hughes from comment #10) This is also wrong upstream: https://source.winehq.org/git/wine.git/blobdiff/42c20ac0b838b351257b572db4d9... -- 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=45622 --- Comment #14 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- (In reply to Gijs Vermeulen from comment #13)
(In reply to Alistair Leslie-Hughes from comment #10)
This is also wrong upstream: https://source.winehq.org/git/wine.git/blobdiff/ 42c20ac0b838b351257b572db4d96fbfed7c7291.. 5b8565d60760f17ecac7bccf9f2c4477943de544:/dlls/mfplat/mfplat.spec
Thanks, Ill send a patch shortly. -- 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=45622 --- Comment #15 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- This should now be fixed upstream with https://source.winehq.org/git/wine.git/?a=commit;h=24279f1cf26832dad45d5bc85... Please retest. -- 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=45622 --- Comment #16 from Alexandr Oleynikov <sashok.olen(a)gmail.com> --- This seems to have fixed it, it doesn't crash anymore (still fails to save it, but that's expected). -- 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=45622 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Fixed by SHA1|5b8565d60760f17ecac7bccf9f2 |24279f1cf26832dad45d5bc850b |c4477943de544 |5a193f408f521 Resolution|--- |FIXED --- Comment #17 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- Thanks for testing. Marking as 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=45622 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #18 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 3.16. -- 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=45622 Storm Engineer <hewanci(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |hewanci(a)gmail.com --- Comment #19 from Storm Engineer <hewanci(a)gmail.com> --- (In reply to Alexandr Oleynikov from comment #16)
This seems to have fixed it, it doesn't crash anymore (still fails to save it, but that's expected).
What do you mean "that's expected"? Before the crash issue started I was successfully saving highlights. -- 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=45622 --- Comment #20 from Alexandr Oleynikov <sashok.olen(a)gmail.com> --- Even though the crash issue itself is fixed, the game will still fail to save Highlights in MP4 as the “fix” is just a stub. But now it won’t crash and will just display its own error message inside the application. But if you previously saved Highlights in WebM format, this won’t affect it at all. -- 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=45622 --- Comment #21 from Storm Engineer <hewanci(a)gmail.com> --- (In reply to Alexandr Oleynikov from comment #20)
the game will still fail to save Highlights in MP4 Oh! I didn't realize the issue was format specific.
I had the format setting on Auto. That defaults to MP4 now apparently, not sure what it defaulted to back then. But I have just confirmed that I can successfully save highlights as WEBM! \o/ Thank you very much for this critical piece of information! Meanwhile I created a ticket yesterday about the failure to save (bug 45862), I'm editing that now to reflect that it's for MP4 only. -- 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=45622 --- Comment #22 from Storm Engineer <hewanci(a)gmail.com> --- Just checked my old saved highlights, they are WEBM too. That settles it. -- 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=45622 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |3.0.x -- 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=45622 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|3.0.x |--- --- Comment #23 from Michael Stefaniuc <mstefani(a)winehq.org> --- Removing the 3.0.x milestone from bug fixes included in 3.0.4. -- 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)
-
wine-bugs＠winehq.org