Bug 189721 - Crash under WebProcessProxy::suspendedPageWasDestroyed(WebKit::SuspendedPageProxy&)
Summary: Crash under WebProcessProxy::suspendedPageWasDestroyed(WebKit::SuspendedPageP...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-09-18 15:41 PDT by Chris Dumez
Modified: 2018-09-20 09:14 PDT (History)
6 users (show)

See Also:


Attachments
Patch (4.97 KB, patch)
2018-09-18 15:51 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (8.84 KB, patch)
2018-09-18 16:16 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
ServiceWorkers.RestoreFromDiskNonDefaultStore crashlog (69.34 KB, text/plain)
2018-09-20 08:24 PDT, Ryan Haddad
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2018-09-18 15:41:51 PDT
Crash under WebProcessProxy::suspendedPageWasDestroyed(WebKit::SuspendedPageProxy&):
Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   com.apple.WebKit              	0x0000000105c6e11f WTF::KeyValuePair<unsigned long long, WebKit::SuspendedPageProxy*>* WTF::HashTable<unsigned long long, WTF::KeyValuePair<unsigned long long, WebKit::SuspendedPageProxy*>, WTF::KeyValuePairKeyExtractor<WTF::KeyValuePair<unsigned long long, WebKit::SuspendedPageProxy*> >, WTF::IntHash<unsigned long long>, WTF::HashMap<unsigned long long, WebKit::SuspendedPageProxy*, WTF::IntHash<unsigned long long>, WTF::HashTraits<unsigned long long>, WTF::HashTraits<WebKit::SuspendedPageProxy*> >::KeyValuePairTraits, WTF::HashTraits<unsigned long long> >::lookup<WTF::IdentityHashTranslator<WTF::HashMap<unsigned long long, WebKit::SuspendedPageProxy*, WTF::IntHash<unsigned long long>, WTF::HashTraits<unsigned long long>, WTF::HashTraits<WebKit::SuspendedPageProxy*> >::KeyValuePairTraits, WTF::IntHash<unsigned long long> >, unsigned long long>(unsigned long long const&) + 117
1   com.apple.WebKit              	0x0000000105c6e085 WTF::HashTableIterator<unsigned long long, WTF::KeyValuePair<unsigned long long, WebKit::SuspendedPageProxy*>, WTF::KeyValuePairKeyExtractor<WTF::KeyValuePair<unsigned long long, WebKit::SuspendedPageProxy*> >, WTF::IntHash<unsigned long long>, WTF::HashMap<unsigned long long, WebKit::SuspendedPageProxy*, WTF::IntHash<unsigned long long>, WTF::HashTraits<unsigned long long>, WTF::HashTraits<WebKit::SuspendedPageProxy*> >::KeyValuePairTraits, WTF::HashTraits<unsigned long long> > WTF::HashTable<unsigned long long, WTF::KeyValuePair<unsigned long long, WebKit::SuspendedPageProxy*>, WTF::KeyValuePairKeyExtractor<WTF::KeyValuePair<unsigned long long, WebKit::SuspendedPageProxy*> >, WTF::IntHash<unsigned long long>, WTF::HashMap<unsigned long long, WebKit::SuspendedPageProxy*, WTF::IntHash<unsigned long long>, WTF::HashTraits<unsigned long long>, WTF::HashTraits<WebKit::SuspendedPageProxy*> >::KeyValuePairTraits, WTF::HashTraits<unsigned long long> >::find<WTF::IdentityHashTranslator<WTF::HashMap<unsigned long long, WebKit::SuspendedPageProxy*, WTF::IntHash<unsigned long long>, WTF::HashTraits<unsigned long long>, WTF::HashTraits<WebKit::SuspendedPageProxy*> >::KeyValuePairTraits, WTF::IntHash<unsigned long long> >, unsigned long long>(unsigned long long const&) + 23
2   com.apple.WebKit              	0x0000000105c684e2 WTF::HashMap<unsigned long long, WebKit::SuspendedPageProxy*, WTF::IntHash<unsigned long long>, WTF::HashTraits<unsigned long long>, WTF::HashTraits<WebKit::SuspendedPageProxy*> >::remove(unsigned long long const&) + 14
3   com.apple.WebKit              	0x0000000105c684c5 WebKit::WebProcessProxy::suspendedPageWasDestroyed(WebKit::SuspendedPageProxy&) + 39
4   com.apple.WebKit              	0x0000000105aee6c8 WebKit::SuspendedPageProxy::~SuspendedPageProxy() + 44
5   com.apple.WebKit              	0x0000000105aee776 WebKit::SuspendedPageProxy::~SuspendedPageProxy() + 14
6   com.apple.WebKit              	0x00000001059312de WebKit::WebPageProxy::~WebPageProxy() + 222
7   com.apple.WebKit              	0x0000000105ceeb7f -[WKObject dealloc] + 25
8   com.apple.WebKit              	0x0000000105cca316 WebKit::WebViewImpl::~WebViewImpl() + 1082
9   com.apple.WebKit              	0x0000000105930f3a -[WKView dealloc] + 116
10  com.apple.Safari.framework    	0x000000010348e94b -[BrowserWKView dealloc] + 105
11  libobjc.A.dylib               	0x00007fff5c4d2c8c (anonymous namespace)::AutoreleasePoolPage::pop(void*) + 726
12  com.apple.CoreFoundation      	0x00007fff305a8e76 _CFAutoreleasePoolPop + 22
13  com.apple.AppKit              	0x00007fff2de06e6a _wrapRunLoopWithAutoreleasePoolHandler + 85
14  com.apple.CoreFoundation      	0x00007fff3063495d __CFRUNLOOP_IS_CALLING_OUT_TO_AN_OBSERVER_CALLBACK_FUNCTION__ + 23
15  com.apple.CoreFoundation      	0x00007fff30634892 __CFRunLoopDoObservers + 452
16  com.apple.CoreFoundation      	0x00007fff305d5d20 CFRunLoopRunSpecific + 523
17  com.apple.HIToolbox           	0x00007fff2f86f895 RunCurrentEventLoopInMode + 293
18  com.apple.HIToolbox           	0x00007fff2f86f5cb ReceiveNextEventCommon + 618
19  com.apple.HIToolbox           	0x00007fff2f86f348 _BlockUntilNextEventMatchingListInModeWithFilter + 64
20  com.apple.AppKit              	0x00007fff2db2c8df _DPSNextEvent + 997
21  com.apple.AppKit              	0x00007fff2db2b67e -[NSApplication(NSEvent) _nextEventMatchingEventMask:untilDate:inMode:dequeue:] + 1362
22  com.apple.Safari.framework    	0x000000010338a87d -[BrowserApplication nextEventMatchingMask:untilDate:inMode:dequeue:] + 273
23  com.apple.AppKit              	0x00007fff2db256e1 -[NSApplication run] + 699
24  com.apple.AppKit              	0x00007fff2db14e1b NSApplicationMain + 780
25  libdyld.dylib                 	0x00007fff5d5ab0a1 start + 1
Comment 1 Chris Dumez 2018-09-18 15:42:08 PDT
<rdar://problem/44359788>
Comment 2 Chris Dumez 2018-09-18 15:51:32 PDT
Created attachment 350070 [details]
Patch
Comment 3 Chris Dumez 2018-09-18 16:14:19 PDT
Comment on attachment 350070 [details]
Patch

I think I have a test.
Comment 4 Chris Dumez 2018-09-18 16:16:32 PDT
Created attachment 350072 [details]
Patch
Comment 5 Chris Dumez 2018-09-19 13:05:37 PDT
ping review?
Comment 6 Geoffrey Garen 2018-09-19 14:07:15 PDT
Comment on attachment 350072 [details]
Patch

r=me
Comment 7 WebKit Commit Bot 2018-09-19 14:33:59 PDT
Comment on attachment 350072 [details]
Patch

Clearing flags on attachment: 350072

Committed r236226: <https://trac.webkit.org/changeset/236226>
Comment 8 WebKit Commit Bot 2018-09-19 14:34:01 PDT
All reviewed patches have been landed.  Closing bug.
Comment 9 Ryan Haddad 2018-09-20 08:23:34 PDT
This change appears to have caused TestWebKitAPI.ServiceWorkers.RestoreFromDiskNonDefaultStore to crash:

https://build.webkit.org/builders/Apple%20Sierra%20Debug%20WK2%20%28Tests%29/builds/8049
Comment 10 Ryan Haddad 2018-09-20 08:24:54 PDT
Created attachment 350202 [details]
ServiceWorkers.RestoreFromDiskNonDefaultStore crashlog
Comment 11 Chris Dumez 2018-09-20 08:40:18 PDT
(In reply to Ryan Haddad from comment #10)
> Created attachment 350202 [details]
> ServiceWorkers.RestoreFromDiskNonDefaultStore crashlog

Looking now, thank you.
Comment 12 Chris Dumez 2018-09-20 09:14:04 PDT
(In reply to Chris Dumez from comment #11)
> (In reply to Ryan Haddad from comment #10)
> > Created attachment 350202 [details]
> > ServiceWorkers.RestoreFromDiskNonDefaultStore crashlog
> 
> Looking now, thank you.

Fixed in <https://trac.webkit.org/changeset/236262>.