RESOLVED FIXED 149766
DumpRenderTree built with public iOS SDK crashes under -[WebPreferences(WebPrivate) _setCurrentNetworkLoaderSessionCookieAcceptPolicy:]
https://bugs.webkit.org/show_bug.cgi?id=149766
Summary DumpRenderTree built with public iOS SDK crashes under -[WebPreferences(WebPr...
Daniel Bates
Reported 2015-10-02 15:21:06 PDT
Created attachment 262356 [details] Crash Report Many tests crash on the Apple iOS 9 Simulator Release WK1 (Tests) with the following backtrace: Thread 0 Crashed:: Dispatch queue: com.apple.main-thread 0 com.apple.CFNetwork 0x0000000108429cab HTTPCookieStorage::setAcceptPolicy(int) + 17 1 com.apple.WebKitLegacy 0x0000000112cfebc5 +[WebPreferences(WebPrivate) _setCurrentNetworkLoaderSessionCookieAcceptPolicy:] + 37 2 ??? 0x0000000107ebba90 0 + 4427856528 3 ??? 0x0000000107eb9142 0 + 4427845954 4 ??? 0x0000000107eb8d84 0 + 4427844996 5 com.apple.Foundation 0x0000000108eaa607 __NSThreadPerformPerform + 283 6 com.apple.CoreFoundation 0x00000001087cc0a1 __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 17 7 com.apple.CoreFoundation 0x00000001087c1f47 __CFRunLoopDoSources0 + 423 8 com.apple.CoreFoundation 0x00000001087c1483 __CFRunLoopRun + 867 9 com.apple.CoreFoundation 0x00000001087c0e98 CFRunLoopRunSpecific + 488 10 com.apple.GraphicsServices 0x00000001092c9ad2 GSEventRunModal + 161 11 com.apple.UIKit 0x0000000109868676 UIApplicationMain + 171 12 ??? 0x0000000107eba6a1 0 + 4427851425 13 libdyld.dylib 0x000000010f0cd92d start + 1
Attachments
Crash Report (43.84 KB, application/octet-stream)
2015-10-02 15:21 PDT, Daniel Bates
no flags
Patch (2.04 KB, patch)
2015-10-02 15:27 PDT, Daniel Bates
no flags
Archive of layout-test-results from ews100 for mac-mavericks (614.57 KB, application/zip)
2015-10-02 16:01 PDT, Build Bot
no flags
Patch (4.78 KB, patch)
2015-10-05 11:23 PDT, Daniel Bates
ap: review+
Daniel Bates
Comment 1 2015-10-02 15:22:23 PDT
Notice that OpenSource iOS WebKit is built without USE(CFNETWORK). When WebKit is built without USE(CFNETWORK) we defer to the shared NSHTTPCookieStorage store for cookies instead of maintaining our own cookie store(s). So, NetworkStorageSession::cookieStorage() always returns a nullptr.
Daniel Bates
Comment 2 2015-10-02 15:27:12 PDT
Build Bot
Comment 3 2015-10-02 16:01:11 PDT
Comment on attachment 262357 [details] Patch Attachment 262357 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/237939 New failing tests: http/tests/cookies/third-party-cookie-relaxing.html http/tests/xmlhttprequest/cross-origin-cookie-storage.html
Build Bot
Comment 4 2015-10-02 16:01:15 PDT
Created attachment 262359 [details] Archive of layout-test-results from ews100 for mac-mavericks The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews100 Port: mac-mavericks Platform: Mac OS X 10.9.5
Daniel Bates
Comment 5 2015-10-02 16:46:43 PDT
Comment on attachment 262357 [details] Patch The patch is wrong.
Daniel Bates
Comment 6 2015-10-05 11:23:35 PDT
Created attachment 262450 [details] Patch I added an assert in +[WebPreferences _setCurrentNetworkLoaderSessionCookieAcceptPolicy:] to ensure we have a non-null pointer to a cookie store. Let me know if this assert is unnecessary.
Alexey Proskuryakov
Comment 7 2015-10-05 12:28:16 PDT
Comment on attachment 262450 [details] Patch EWS is still orange due to an unrelated issue.
Daniel Bates
Comment 8 2015-10-05 13:57:39 PDT
Note You need to log in before you can comment on or make changes to this bug.