Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: f19f83b2829ce723f5b7c6a667805bf2ab3085a1
https://github.com/WebKit/WebKit/commit/f19f83b2829ce723f5b7c6a667805bf2ab3085a1
Author: David Kilzer <[email protected]>
Date: 2026-03-30 (Mon, 30 Mar 2026)
Changed paths:
M Source/WebCore/rendering/RenderLayerBacking.cpp
M Source/WebCore/rendering/RenderLayerBacking.h
Log Message:
-----------
Null deref crash under RenderLayerBacking::updateAncestorClipping()
<https://bugs.webkit.org/show_bug.cgi?id=311060>
<rdar://150587917>
Reviewed by Simon Fraser.
`RenderLayerBacking::updateAncestorClipping()` crashes due to a
null deref under
`m_owningLayer.page().scrollingCoordinator()` when tearing down
the render tree during Back/Forward Cache eviction.
`CachedFrame::destroy()` calls `detachFromPage()` before
`willBeRemovedFromFrame()`, so the page pointer is null by the
time the render tree is destroyed.
Add a simpler `clearAncestorClippingStack()` to be called from
`~RenderLayerBacking()` that just destroys the GraphicsLayers
without accessing the page. This follows the same pattern as
commit 307858@main for Bug 308230.
Unable to create a layout test to reproduce the crash.
* Source/WebCore/rendering/RenderLayerBacking.cpp:
(WebCore::RenderLayerBacking::~RenderLayerBacking):
(WebCore::RenderLayerBacking::clearAncestorClippingStack): Add.
* Source/WebCore/rendering/RenderLayerBacking.h:
Canonical link: https://commits.webkit.org/310278@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
