1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 22:37:35 +00:00

LibWeb: Make BrowsingContext::m_top_level_browsing_context a WeakPtr

At the moment, nested browsing contexts expect that there's always a
top-level browsing context at some higher level. That's okay, but let's
keep the top-level pointer in a WeakPtr to make it easier to catch
mistakes (as this turns UAF into a null dereference.)
This commit is contained in:
Andreas Kling 2021-09-08 11:17:43 +02:00
parent ef371596a4
commit e90ccf6a20

View file

@ -35,7 +35,7 @@ public:
void register_viewport_client(ViewportClient&);
void unregister_viewport_client(ViewportClient&);
bool is_top_level() const { return this == &m_top_level_browsing_context; }
bool is_top_level() const { return this == &top_level_browsing_context(); }
bool is_focused_context() const;
DOM::Document const* document() const { return m_document; }
@ -63,8 +63,8 @@ public:
void scroll_to_anchor(String const&);
BrowsingContext& top_level_browsing_context() { return m_top_level_browsing_context; }
BrowsingContext const& top_level_browsing_context() const { return m_top_level_browsing_context; }
BrowsingContext& top_level_browsing_context() { return *m_top_level_browsing_context; }
BrowsingContext const& top_level_browsing_context() const { return *m_top_level_browsing_context; }
DOM::Element* host_element() { return m_host_element; }
DOM::Element const* host_element() const { return m_host_element; }
@ -98,7 +98,10 @@ private:
void reset_cursor_blink_cycle();
WeakPtr<Page> m_page;
BrowsingContext& m_top_level_browsing_context;
// NOTE: We expect there to always be a top-level browsing context as long as we exist.
// The use of WeakPtr is for safety in case we get something wrong.
WeakPtr<BrowsingContext> m_top_level_browsing_context;
FrameLoader m_loader;
EventHandler m_event_handler;