Bug 194378 – Infinite recursion via CachedResource::~CachedResource

• Status: RESOLVED
• Resolution: FIXED
• Component: Page Loading

[Antti Koivisto]

Infinite recursion via ~CachedResource

Thread 0 Crashed ↩:: Dispatch queue: com.apple.main-thread
0   com.apple.WebCore             	0x7fff5ea60a9a WebCore::CachedResourceHandle<WebCore::CachedResource> WTF::HashMap<WTF::String, WebCore::CachedResourceHandle<WebCore::CachedResource>, WTF::StringHash, WTF::HashTraits<WTF::String>, WTF::HashTraits<WebCore::CachedResourceHandle<WebCore::CachedResource> > >::get<WTF::IdentityHashTranslator<WTF::HashMap<WTF::String, WebCore::CachedResourceHandle<WebCore::CachedResource>, WTF::StringHash, WTF::HashTraits<WTF::String>, WTF::HashTraits<WebCore::CachedResourceHandle<WebCore::CachedResource> > >::KeyValuePairTraits, WTF::StringHash>, WTF::String>(WTF::String const&) const + 170 (/BuildRoot/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.Internal.sdk/usr/local/include/wtf/text/StringHash.h:55)
1   com.apple.WebCore             	0x7fff5ea544fc WebCore::CachedResourceLoader::removeCachedResource(WebCore::CachedResource&) + 236 (/BuildRoot/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.Internal.sdk/usr/local/include/wtf/HashMap.h:424)
2   com.apple.WebCore             	0x7fff5d93cc3f WebCore::CachedResource::~CachedResource() + 47 (./loader/cache/CachedResource.cpp:178)
3   com.apple.WebCore             	0x7fff5ea5455e WebCore::CachedResource::~CachedResource() + 14 (./loader/cache/CachedResource.cpp:165)
4   com.apple.WebCore             	0x7fff5d8dd470 WebCore::CachedResource::unregisterHandle(WebCore::CachedResourceHandleBase*) + 176 (./loader/cache/CachedResource.cpp:624)
5   com.apple.WebCore             	0x7fff5ea54511 WebCore::CachedResourceLoader::removeCachedResource(WebCore::CachedResource&) + 257 (./loader/cache/CachedResourceHandle.cpp:55)
6   com.apple.WebCore             	0x7fff5d93cc3f WebCore::CachedResource::~CachedResource() + 47 (./loader/cache/CachedResource.cpp:178)
7   com.apple.WebCore             	0x7fff5ea5455e WebCore::CachedResource::~CachedResource() + 14 (./loader/cache/CachedResource.cpp:165)
8   com.apple.WebCore             	0x7fff5d8dd470 WebCore::CachedResource::unregisterHandle(WebCore::CachedResourceHandleBase*) + 176 (./loader/cache/CachedResource.cpp:624)
9   com.apple.WebCore             	0x7fff5ea54511 WebCore::CachedResourceLoader::removeCachedResource(WebCore::CachedResource&) + 257 (./loader/cache/CachedResourceHandle.cpp:55)
10  com.apple.WebCore             	0x7fff5d93cc3f WebCore::CachedResource::~CachedResource() + 47 (./loader/cache/CachedResource.cpp:178)
11  com.apple.WebCore             	0x7fff5ea5455e WebCore::CachedResource::~CachedResource() + 14 (./loader/cache/CachedResource.cpp:165)
12  com.apple.WebCore             	0x7fff5d8dd470 WebCore::CachedResource::unregisterHandle(WebCore::CachedResourceHandleBase*) + 176 (./loader/cache/CachedResource.cpp:624)
13  com.apple.WebCore             	0x7fff5ea54511 WebCore::CachedResourceLoader::removeCachedResource(WebCore::CachedResource&) + 257 (./loader/cache/CachedResourceHandle.cpp:55)
14  com.apple.WebCore             	0x7fff5d93cc3f WebCore::CachedResource::~CachedResource() + 47 (./loader/cache/CachedResource.cpp:178)
15  com.apple.WebCore             	0x7fff5ea5455e WebCore::CachedResource::~CachedResource() + 14 (./loader/cache/CachedResource.cpp:165)
16  com.apple.WebCore             	0x7fff5d8dd470 WebCore::CachedResource::unregisterHandle(WebCore::CachedResourceHandleBase*) + 176 (./loader/cache/CachedResource.cpp:624)
17  com.apple.WebCore             	0x7fff5ea54511 WebCore::CachedResourceLoader::removeCachedResource(WebCore::CachedResource&) + 257 (./loader/cache/CachedResourceHandle.cpp:55)
18  com.apple.WebCore             	0x7fff5d93cc3f WebCore::CachedResource::~CachedResource() + 47 (./loader/cache/CachedResource.cpp:178)
19  com.apple.WebCore             	0x7fff5ea5455e WebCore::CachedResource::~CachedResource() + 14 (./loader/cache/CachedResource.cpp:165)
20  com.apple.WebCore             	0x7fff5d8dd470 WebCore::CachedResource::unregisterHandle(WebCore::CachedResourceHandleBase*) + 176 (./loader/cache/CachedResource.cpp:624)
21  com.apple.WebCore             	0x7fff5ea54511 WebCore::CachedResourceLoader::removeCachedResource(WebCore::CachedResource&) + 257 (./loader/cache/CachedResourceHandle.cpp:55)
22  com.apple.WebCore             	0x7fff5d93cc3f WebCore::CachedResource::~CachedResource() + 47 (./loader/cache/CachedResource.cpp:178)
23  com.apple.WebCore             	0x7fff5ea5455e WebCore::CachedResource::~CachedResource() + 14 (./loader/cache/CachedResource.cpp:165)
24  com.apple.WebCore             	0x7fff5d8dd470 WebCore::CachedResource::unregisterHandle(WebCore::CachedResourceHandleBase*) + 176 (./loader/cache/CachedResource.cpp:624)
25  com.apple.WebCore             	0x7fff5ea54511 WebCore::CachedResourceLoader::removeCachedResource(WebCore::CachedResource&) + 257 (./loader/cache/CachedResourceHandle.cpp:55)
26  com.apple.WebCore             	0x7fff5d93cc3f WebCore::CachedResource::~CachedResource() + 47 (./loader/cache/CachedResource.cpp:178)
27  com.apple.WebCore             	0x7fff5ea5455e WebCore::CachedResource::~CachedResource() + 14 (./loader/cache/CachedResource.cpp:165)

[Antti Koivisto]

<rdar://problem/42023295>

[Antti Koivisto]

Created attachment 361387 [details]
patch

[WebKit Commit Bot]

Comment on attachment 361387 [details]
patch

Clearing flags on attachment: 361387

Committed r241121: <https://trac.webkit.org/changeset/241121>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.