Summary: | REGRESSION (r225537): Crash in WebCore::SWServerWorker::setHasPendingEvents(bool) + 68 | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryan Haddad <ryanhaddad> | ||||||
Component: | New Bugs | Assignee: | youenn fablet <youennf> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | beidson, cdumez, commit-queue, jlewis3, lforschler, webkit-bug-importer, youennf | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | Other | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=180261 | ||||||||
Attachments: |
|
Description
Ryan Haddad
2017-12-06 10:24:35 PST
Similar crash seen with imported/w3c/web-platform-tests/service-workers/service-worker/fetch-event.https.html here: https://build.webkit.org/results/Apple%20High%20Sierra%20Release%20WK2%20(Tests)/r225573%20(1574)/results.html Thread 0 Crashed:: Dispatch queue: com.apple.main-thread 0 com.apple.WebCore 0x0000000107871f7d WebCore::SWServerWorker::setHasPendingEvents(bool) + 77 (SWServerRegistration.h:58) 1 com.apple.WebKit 0x0000000105522423 void IPC::handleMessage<Messages::WebSWServerToContextConnection::SetServiceWorkerHasPendingEvents, WebKit::WebSWServerToContextConnection, void (WebCore::SWServerToContextConnection::*)(WTF::ObjectIdentifier<WebCore::ServiceWorkerIdentifierType>, bool)>(IPC::Decoder&, WebKit::WebSWServerToContextConnection*, void (WebCore::SWServerToContextConnection::*)(WTF::ObjectIdentifier<WebCore::ServiceWorkerIdentifierType>, bool)) + 77 (HandleMessage.h:127) 2 com.apple.WebKit 0x0000000105242ddd IPC::Connection::dispatchMessage(std::__1::unique_ptr<IPC::Decoder, std::__1::default_delete<IPC::Decoder> >) + 119 (Connection.cpp:930) 3 com.apple.WebKit 0x0000000105245916 IPC::Connection::dispatchOneMessage() + 176 (Connection.cpp:959) 4 com.apple.JavaScriptCore 0x000000010abc73b8 WTF::RunLoop::performWork() + 376 (RunLoop.cpp:123) 5 com.apple.JavaScriptCore 0x000000010abc7522 WTF::RunLoop::performWork(void*) + 34 (RunLoopCF.cpp:39) 6 com.apple.CoreFoundation 0x00007fff306bf711 __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 17 7 com.apple.CoreFoundation 0x00007fff3077739c __CFRunLoopDoSource0 + 108 8 com.apple.CoreFoundation 0x00007fff306a2700 __CFRunLoopDoSources0 + 208 9 com.apple.CoreFoundation 0x00007fff306a1b7d __CFRunLoopRun + 1293 10 com.apple.CoreFoundation 0x00007fff306a13d7 CFRunLoopRunSpecific + 487 11 com.apple.Foundation 0x00007fff3279bb76 -[NSRunLoop(NSRunLoop) runMode:beforeDate:] + 277 12 com.apple.Foundation 0x00007fff3279ba4e -[NSRunLoop(NSRunLoop) run] + 76 13 libxpc.dylib 0x00007fff58263403 _xpc_objc_main + 536 14 libxpc.dylib 0x00007fff58262082 xpc_main + 417 15 com.apple.WebKit.Storage 0x00000001051f36a1 main + 490 16 libdyld.dylib 0x00007fff57f96115 start + 1 Probably due to https://trac.webkit.org/changeset/225537/webkit Found a crash on a debug bot: ASSERTION FAILED: registration /Volumes/Data/slave/highsierra-debug/build/Source/WebCore/workers/service/server/SWServerWorker.cpp(144) : void WebCore::SWServerWorker::setHasPendingEvents(bool) https://build.webkit.org/results/Apple%20El%20Capitan%20Debug%20WK2%20(Tests)/r225572%20(4486)/results.html Thanks Ryan, I will fix it shortly by adding an if null check. We might want to do future refactoring to improve on those issues. I would expect a SWServerWorker to have a related registration but there is no such guarantee right now. Created attachment 328587 [details]
Patch
Comment on attachment 328587 [details] Patch Clearing flags on attachment: 328587 Committed r225581: <https://trac.webkit.org/changeset/225581> All reviewed patches have been landed. Closing bug. *** Bug 180501 has been marked as a duplicate of this bug. *** |