Bug 187927

Summary: CrashTracer: com.apple.WebKit.Storage at WebCore: WebCore::encodeKey
Product: WebKit Reporter: Sihui Liu <sihui_liu>
Component: New BugsAssignee: Sihui Liu <sihui_liu>
Status: RESOLVED FIXED    
Severity: Normal CC: alecflett, beidson, cdumez, commit-queue, ddkilzer, ews-watchlist, jsbell, rniwa, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews102 for mac-sierra
none
Archive of layout-test-results from ews106 for mac-sierra-wk2
none
Archive of layout-test-results from ews115 for mac-sierra
none
Archive of layout-test-results from ews124 for ios-simulator-wk2
none
Patch for landing none

Description Sihui Liu 2018-07-23 14:50:03 PDT 
1   0x117b89069 WTFCrash
2   0x10ad51a6b WebCore::serializedTypeForKeyType(WebCore::IndexedDB::KeyType)
3   0x10ad35b73 WebCore::encodeKey(WTF::Vector<char, 0ul, WTF::CrashOnOverflow, 16ul>&, WebCore::IDBKeyData const&)
4   0x10ad35e08 WebCore::encodeKey(WTF::Vector<char, 0ul, WTF::CrashOnOverflow, 16ul>&, WebCore::IDBKeyData const&)
5   0x10ad35a60 WebCore::serializeIDBKeyData(WebCore::IDBKeyData const&)
6   0x10adb1316 WebCore::IDBServer::SQLiteIDBBackingStore::uncheckedPutIndexRecord(long long, long long, WebCore::IDBKeyData const&, WebCore::IDBKeyData const&, long long)
7   0x10adb10ec WebCore::IDBServer::SQLiteIDBBackingStore::uncheckedPutIndexKey(WebCore::IDBIndexInfo const&, WebCore::IDBKeyData const&, WebCore::IndexKey const&, long long)
8   0x10adb4b36 WebCore::IDBServer::SQLiteIDBBackingStore::updateAllIndexesForAddRecord(WebCore::IDBObjectStoreInfo const&, WebCore::IDBKeyData const&, WebCore::ThreadSafeDataBuffer const&, long long)
9   0x10adb5872 WebCore::IDBServer::SQLiteIDBBackingStore::addRecord(WebCore::IDBResourceIdentifier const&, WebCore::IDBObjectStoreInfo const&, WebCore::IDBKeyData const&, WebCore::IDBValue const&)
10  0x10adefcb0 WebCore::IDBServer::UniqueIDBDatabase::performPutOrAdd(unsigned long long, WebCore::IDBResourceIdentifier const&, unsigned long long, WebCore::IDBKeyData const&, WebCore::IDBValue const&, WebCore::IndexedDB::ObjectStoreOverwriteMode)
11  0x10ae204f4 void WTF::callMemberFunctionForCrossThreadTaskImpl<WebCore::IDBServer::UniqueIDBDatabase, void (WebCore::IDBServer::UniqueIDBDatabase::*)(unsigned long long, WebCore::IDBResourceIdentifier const&, unsigned long long, WebCore::IDBKeyData const&, WebCore::IDBValue const&, WebCore::IndexedDB::ObjectStoreOverwriteMode), std::__1::tuple<unsigned long long, WebCore::IDBResourceIdentifier, unsigned long long, WebCore::IDBKeyData, WebCore::IDBValue, WebCore::IndexedDB::ObjectStoreOverwriteMode>, 0ul, 1ul, 2ul, 3ul, 4ul, 5ul>(WebCore::IDBServer::UniqueIDBDatabase*, void (WebCore::IDBServer::UniqueIDBDatabase::*)(unsigned long long, WebCore::IDBResourceIdentifier const&, unsigned long long, WebCore::IDBKeyData const&, WebCore::IDBValue const&, WebCore::IndexedDB::ObjectStoreOverwriteMode), std::__1::tuple<unsigned long long, WebCore::IDBResourceIdentifier, unsigned long long, WebCore::IDBKeyData, WebCore::IDBValue, WebCore::IndexedDB::ObjectStoreOverwriteMode>&&, std::__1::integer_sequence<unsigned long, 0ul, 1ul, 2ul, 3ul, 4ul, 5ul>)
12  0x10ae20320 void WTF::callMemberFunctionForCrossThreadTask<WebCore::IDBServer::UniqueIDBDatabase, void (WebCore::IDBServer::UniqueIDBDatabase::*)(unsigned long long, WebCore::IDBResourceIdentifier const&, unsigned long long, WebCore::IDBKeyData const&, WebCore::IDBValue const&, WebCore::IndexedDB::ObjectStoreOverwriteMode), std::__1::tuple<unsigned long long, WebCore::IDBResourceIdentifier, unsigned long long, WebCore::IDBKeyData, WebCore::IDBValue, WebCore::IndexedDB::ObjectStoreOverwriteMode>, std::__1::integer_sequence<unsigned long, 0ul, 1ul, 2ul, 3ul, 4ul, 5ul> >(WebCore::IDBServer::UniqueIDBDatabase*, void (WebCore::IDBServer::UniqueIDBDatabase::*)(unsigned long long, WebCore::IDBResourceIdentifier const&, unsigned long long, WebCore::IDBKeyData const&, WebCore::IDBValue const&, WebCore::IndexedDB::ObjectStoreOverwriteMode), std::__1::tuple<unsigned long long, WebCore::IDBResourceIdentifier, unsigned long long, WebCore::IDBKeyData, WebCore::IDBValue, WebCore::IndexedDB::ObjectStoreOverwriteMode>&&)
13  0x10ae202b4 WTF::CrossThreadTask WTF::createCrossThreadTask<WebCore::IDBServer::UniqueIDBDatabase, unsigned long long, WebCore::IDBResourceIdentifier const&, unsigned long long, WebCore::IDBKeyData const&, WebCore::IDBValue const&, WebCore::IndexedDB::ObjectStoreOverwriteMode, unsigned long long, WebCore::IDBResourceIdentifier, unsigned long long, WebCore::IDBKeyData, WebCore::IDBValue, WebCore::IndexedDB::ObjectStoreOverwriteMode>(WebCore::IDBServer::UniqueIDBDatabase&, void (WebCore::IDBServer::UniqueIDBDatabase::*)(unsigned long long, WebCore::IDBResourceIdentifier const&, unsigned long long, WebCore::IDBKeyData const&, WebCore::IDBValue const&, WebCore::IndexedDB::ObjectStoreOverwriteMode), unsigned long long const&, WebCore::IDBResourceIdentifier const&, unsigned long long const&, WebCore::IDBKeyData const&, WebCore::IDBValue const&, WebCore::IndexedDB::ObjectStoreOverwriteMode const&)::'lambda'()::operator()()
14  0x10ae20039 WTF::Function<void ()>::CallableWrapper<WTF::CrossThreadTask WTF::createCrossThreadTask<WebCore::IDBServer::UniqueIDBDatabase, unsigned long long, WebCore::IDBResourceIdentifier const&, unsigned long long, WebCore::IDBKeyData const&, WebCore::IDBValue const&, WebCore::IndexedDB::ObjectStoreOverwriteMode, unsigned long long, WebCore::IDBResourceIdentifier, unsigned long long, WebCore::IDBKeyData, WebCore::IDBValue, WebCore::IndexedDB::ObjectStoreOverwriteMode>(WebCore::IDBServer::UniqueIDBDatabase&, void (WebCore::IDBServer::UniqueIDBDatabase::*)(unsigned long long, WebCore::IDBResourceIdentifier const&, unsigned long long, WebCore::IDBKeyData const&, WebCore::IDBValue const&, WebCore::IndexedDB::ObjectStoreOverwriteMode), unsigned long long const&, WebCore::IDBResourceIdentifier const&, unsigned long long const&, WebCore::IDBKeyData const&, WebCore::IDBValue const&, WebCore::IndexedDB::ObjectStoreOverwriteMode const&)::'lambda'()>::call()
15  0x1098552cb WTF::Function<void ()>::operator()() const
16  0x10ad0c2a5 WTF::CrossThreadTask::performTask()
17  0x10adf99f2 WebCore::IDBServer::UniqueIDBDatabase::executeNextDatabaseTask()
Comment 1 Sihui Liu 2018-07-23 14:57:44 PDT 
Created attachment 345611 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2018-07-23 14:59:17 PDT 
<rdar://problem/42516426>
Comment 3 Sihui Liu 2018-07-23 15:02:53 PDT 
<rdar://problem/41629014>
Comment 4 Sihui Liu 2018-07-23 15:04:47 PDT 
(In reply to Sihui Liu from comment #3)
> <rdar://problem/41629014>

Wrong radar number, should be <rdar://problem/42388498>.
Comment 5 Chris Dumez 2018-07-23 15:10:01 PDT 
Comment on attachment 345611 [details]
Patch

Matches what is done in IDBKey::isValid(), r=me.
Comment 6 EWS Watchlist 2018-07-23 16:04:12 PDT 
Comment on attachment 345611 [details]
Patch

Attachment 345611 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/8631444

New failing tests:
storage/indexeddb/index-multientry-private.html
imported/w3c/web-platform-tests/IndexedDB/key-conversion-exceptions.htm
Comment 7 EWS Watchlist 2018-07-23 16:04:14 PDT 
Created attachment 345619 [details]
Archive of layout-test-results from ews102 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 8 EWS Watchlist 2018-07-23 16:24:27 PDT 
Comment on attachment 345611 [details]
Patch

Attachment 345611 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/8631529

New failing tests:
storage/indexeddb/index-multientry-private.html
imported/w3c/web-platform-tests/IndexedDB/key-conversion-exceptions.htm
Comment 9 EWS Watchlist 2018-07-23 16:24:29 PDT 
Created attachment 345622 [details]
Archive of layout-test-results from ews106 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 10 EWS Watchlist 2018-07-23 16:43:31 PDT 
Comment on attachment 345611 [details]
Patch

Attachment 345611 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/8631515

New failing tests:
storage/indexeddb/index-multientry-private.html
imported/w3c/web-platform-tests/IndexedDB/key-conversion-exceptions.htm
Comment 11 EWS Watchlist 2018-07-23 16:43:33 PDT 
Created attachment 345623 [details]
Archive of layout-test-results from ews115 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews115  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 12 EWS Watchlist 2018-07-23 16:51:55 PDT 
Comment on attachment 345611 [details]
Patch

Attachment 345611 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/8631524

New failing tests:
storage/indexeddb/index-multientry-private.html
imported/w3c/web-platform-tests/IndexedDB/key-conversion-exceptions.htm
Comment 13 EWS Watchlist 2018-07-23 16:51:57 PDT 
Created attachment 345624 [details]
Archive of layout-test-results from ews124 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews124  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.4
Comment 14 Sihui Liu 2018-07-23 16:56:27 PDT 
(In reply to Build Bot from comment #12)
> Comment on attachment 345611 [details]
> Patch
> 
> Attachment 345611 [details] did not pass ios-sim-ews (ios-simulator-wk2):
> Output: https://webkit-queues.webkit.org/results/8631524
> 
> New failing tests:
> storage/indexeddb/index-multientry-private.html
Updated test expectation as a new test case has been added in the js file it uses.

> imported/w3c/web-platform-tests/IndexedDB/key-conversion-exceptions.htm
Updated test expectation as a check becomes PASS after the patch.
Comment 15 Sihui Liu 2018-07-23 16:57:06 PDT 
Created attachment 345625 [details]
Patch for landing
Comment 16 WebKit Commit Bot 2018-07-23 17:35:10 PDT 
Comment on attachment 345625 [details]
Patch for landing

Clearing flags on attachment: 345625

Committed r234125: <https://trac.webkit.org/changeset/234125>
Comment 17 WebKit Commit Bot 2018-07-23 17:35:12 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 18 David Kilzer (:ddkilzer) 2018-07-24 01:38:16 PDT 
Fixing radar linkage:

<rdar://problem/42388498>