| Summary: | WTFCrash in ~CanMakeCheckedPtrBase of ~EventTarget | ||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Fujii Hironori <Hironori.Fujii> | ||||||||||||
| Component: | DOM | Assignee: | Fujii Hironori <Hironori.Fujii> | ||||||||||||
| Status: | RESOLVED FIXED | ||||||||||||||
| Severity: | Normal | CC: | cdumez, webkit-bug-importer | ||||||||||||
| Priority: | P2 | Keywords: | InRadar | ||||||||||||
| Version: | WebKit Nightly Build | ||||||||||||||
| Hardware: | Unspecified | ||||||||||||||
| OS: | Unspecified | ||||||||||||||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=261589 | ||||||||||||||
| Attachments: |
|
||||||||||||||
|
Description
Fujii Hironori
2023-11-07 21:14:17 PST
Can you reproduce the crash? If so, it would be helpful to set CHECKED_POINTER_DEBUG to 1 in CheckedRef.h and rebuild. It will print out on stderr which CheckedPtr/CheckedRef to the object are still live, before crashing. Otherwise, it is not super actionable. I'm keeping trying, but no luch so far. I conclude that this is not reproducible with CHECKED_POINTER_DEBUG=1.
But, it's easy to reproduce this crash without CHECKED_POINTER_DEBUG=1 on my PC.
> python .\Tools\Scripts\run-webkit-tests --release --no-retry --iter=100 -f http/tests/navigation/page-cache-iframe-provisional-load.html
Created attachment 468612 [details]
debugging patch
Created attachment 468613 [details] crash log with the debugging patch comment#5 (In reply to Fujii Hironori from comment #6) > Created attachment 468613 [details] > crash log with the debugging log Is this with `CHECKED_POINTER_DEBUG=1`, I don't see the allocation traces of the remaining CheckedPtrs / CheckedRefs like I would expect. Created attachment 468614 [details] WIP patch Partially reverting 268278@main (bug#261589) fixes the crash. I need to revert both CheckedRef (m_document and m_attachmentRoot). Created attachment 468615 [details]
WIP patch
destorying m_head after destorying m_document and m_attachmentRoot also fixed the crash.
Pull request: https://github.com/WebKit/WebKit/pull/20581 Committed 270813@main (b43c0f571e0a): <https://commits.webkit.org/270813@main> Reviewed commits have been landed. Closing PR #20581 and removing active labels. |