Harden PCM and ITP databases against crashes
<rdar://problem/86741319>
Created attachment 447646 [details] Patch
Comment on attachment 447646 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=447646&action=review r=me > Source/WebKit/NetworkProcess/DatabaseUtilities.cpp:279 > + WebCore::SQLiteTransaction transaction(m_database); beginTransactionIfNecessary ASSERTs when on the main thread. Should we add that assertion, too? ASSERT(!RunLoop::isMain());
Comment on attachment 447646 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=447646&action=review Thanks! >> Source/WebKit/NetworkProcess/DatabaseUtilities.cpp:279 >> + WebCore::SQLiteTransaction transaction(m_database); > > beginTransactionIfNecessary ASSERTs when on the main thread. Should we add that assertion, too? > > ASSERT(!RunLoop::isMain()); Yes, I can add that in.
Created attachment 447724 [details] Patch for landing
Created attachment 447725 [details] Patch for landing
Committed r287322 (245472@main): <https://commits.webkit.org/245472@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 447725 [details].