Bug 234528

Summary: Harden PCM and ITP databases against crashes
Product: WebKit Reporter: Kate Cheney <katherine_cheney>
Component: WebKit Misc.Assignee: Kate Cheney <katherine_cheney>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, bfulgham, webkit-bug-importer, wilander
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch for landing
none
Patch for landing none

Kate Cheney
Reported 2021-12-20 14:41:11 PST
Harden PCM and ITP databases against crashes
Attachments
Patch (5.69 KB, patch)
2021-12-20 14:51 PST, Kate Cheney
no flags
Patch for landing (7.92 KB, patch)
2021-12-21 09:28 PST, Kate Cheney
no flags
Patch for landing (5.82 KB, patch)
2021-12-21 09:32 PST, Kate Cheney
no flags
Radar WebKit Bug Importer
Comment 1 2021-12-20 14:41:27 PST
Kate Cheney
Comment 2 2021-12-20 14:51:11 PST
Brent Fulgham
Comment 3 2021-12-20 15:18:38 PST
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());
Kate Cheney
Comment 4 2021-12-21 09:25:24 PST
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.
Kate Cheney
Comment 5 2021-12-21 09:28:07 PST
Created attachment 447724 [details] Patch for landing
Kate Cheney
Comment 6 2021-12-21 09:32:11 PST
Created attachment 447725 [details] Patch for landing
EWS
Comment 7 2021-12-21 10:32:49 PST
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].
Note You need to log in before you can comment on or make changes to this bug.