[Bug 57463] New: winebus always crashing with unknown since 8b41c2cfddba1f9973246f61e39d4a4d92912da5
https://bugs.winehq.org/show_bug.cgi?id=57463 Bug ID: 57463 Summary: winebus always crashing with unknown since 8b41c2cfddba1f9973246f61e39d4a4d92912da5 Product: Wine Version: unspecified Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: l12436.tw(a)gmail.com Distribution: --- Created attachment 77463 --> https://bugs.winehq.org/attachment.cgi?id=77463 crash log of winebus.sys It will crash at winedevice.exe even execute winecfg I revert de265dfa0bc41fefec3525cd6a1db5d1e1d57029 and 8b41c2cfddba1f9973246f61e39d4a4d92912da5 in previous version of wine will fix this crash. -- 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=57463 --- Comment #1 from TOM <l12436.tw(a)gmail.com> --- This seems fix the crash issue. diff --git a/dlls/winebus.sys/main.c b/dlls/winebus.sys/main.c index 2522e2e6b8b..cac2eed6a12 100644 --- a/dlls/winebus.sys/main.c +++ b/dlls/winebus.sys/main.c @@ -597,7 +597,8 @@ static void process_hid_report(DEVICE_OBJECT *device, BYTE *report_buf, DWORD re if (!ext->collection_desc.ReportIDs[0].ReportID) last_report = ext->last_reports[0]; else last_report = ext->last_reports[report_buf[0]]; - memcpy(last_report->buffer, report_buf, report_len); + if (last_report != NULL) + memcpy(last_report->buffer, report_buf, report_len); if ((irp = pop_pending_read(ext))) { -- 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=57463 Fabian Maurer <dark.shadow4(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression Regression SHA1| |de265dfa0bc41fefec3525cd6a1 | |db5d1e1d57029 CC| |dark.shadow4(a)web.de -- 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=57463 Fabian Maurer <dark.shadow4(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1|de265dfa0bc41fefec3525cd6a1 |8b41c2cfddba1f9973246f61e39 |db5d1e1d57029 |d4a4d92912da5 -- 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=57463 Rémi Bernon <rbernon(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rbernon(a)codeweavers.com --- Comment #2 from Rémi Bernon <rbernon(a)codeweavers.com> --- Created attachment 77468 --> https://bugs.winehq.org/attachment.cgi?id=77468 Additional traces Thanks for the report, would you mind building with the attached patch to add more traces, then making a log when it crashes with WINEDEBUG=+hid? -- 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=57463 --- Comment #3 from TOM <l12436.tw(a)gmail.com> --- Created attachment 77469 --> https://bugs.winehq.org/attachment.cgi?id=77469 log for +hid with patched -- 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=57463 --- Comment #4 from Fabian Maurer <dark.shadow4(a)web.de> --- Is this fixed with https://gitlab.winehq.org/wine/wine/-/commit/c9592bae7f475c1b208de0a72ed29e9... -- 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=57463 --- Comment #5 from TOM <l12436.tw(a)gmail.com> --- Yes it fixed, no longer crash -- 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=57463 Fabian Maurer <dark.shadow4(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |c9592bae7f475c1b208de0a72ed | |29e94e3017094 Resolution|--- |FIXED Status|UNCONFIRMED |RESOLVED --- Comment #6 from Fabian Maurer <dark.shadow4(a)web.de> --- Thanks, marking fixed then. Thanks Rémi! -- 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=57463 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #7 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 10.0-rc1. -- 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