Bug 171651

Summary: webassembly: Safari Crashes on opening "Show Web Inspector" on a webpage using wasm in webworker
Product: WebKit Reporter: Gaurav Dewan <gauravdewan007>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED CONFIGURATION CHANGED    
Severity: Normal CC: jfbastien, saam, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Mac   
OS: macOS 10.12   
Attachments:
Description Flags
Crash log of Worker thread (webassembly)
none
crash log none

Gaurav Dewan
Reported 2017-05-04 05:07:48 PDT
Created attachment 309032 [details] Crash log of Worker thread (webassembly) webassembly: Safari Crashes on opening "Show Web Inspector" on a webpage using wasm in webworker Thread 5 (operationCallArityCheck called from JSC::callWebAssemblyFunction (crash sigsegv: invalid address access at address 0x28 - address very near to null) Crash log is attached in Untitled.rtf Discussed at https://groups.google.com/forum/#!topic/emscripten-discuss/i_jJJrN-lnk
Attachments
Crash log of Worker thread (webassembly) (88.69 KB, application/msword)
2017-05-04 05:07 PDT, Gaurav Dewan 		no flags
Details
crash log (87.71 KB, text/plain)
2017-05-04 22:56 PDT, Alexey Proskuryakov 		no flags
Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2017-05-04 22:56:08 PDT
Created attachment 309146 [details] crash log Same crash log, as plain text.
Alexey Proskuryakov
Comment 2 2017-05-04 22:57:00 PDT
Thank you for the report! Do you have a reproducible test case?
Radar WebKit Bug Importer
Comment 3 2017-05-04 22:57:18 PDT
<rdar://problem/32008848>
Saam Barati
Comment 4 2017-05-09 17:29:01 PDT
Hi Guarav, I think I fixed this in: https://trac.webkit.org/changeset/216481/webkit Can you download a WebKit nightly and see if the bug still reproduces for you? You can download th nightly build here: https://webkit.org/downloads/
Gaurav Dewan
Comment 5 2017-05-10 04:44:31 PDT
Yes, this bug looks to be fixed in latest nightly build. Opening console does not cause any crash
JF Bastien
Comment 6 2017-05-10 08:13:21 PDT
Great! Closing since this seems fixed.
Note You need to log in before you can comment on or make changes to this bug.