[Webkit-unassigned] [Bug 24398] BackForwardList doesn't initialize m_client, which causes a crash

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Mar 9 13:03:44 PDT 2009


https://bugs.webkit.org/show_bug.cgi?id=24398





------- Comment #4 from darin at chromium.org  2009-03-09 13:03 PDT -------
(From update of attachment 28315)
>Index: WebCore/ChangeLog
...
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Fix a crash when loading a non html page (like a svg file) in test_shell
>+        and reloading the page.
>+
>+        * history/BackForwardListChromium.cpp:

please add a bug link to the ChangeLog.


>Index: WebCore/history/BackForwardListChromium.cpp
...
>+    if (m_client)
>+        m_client->close();

we should probably null check each usage and probably add some ASSERTs.  wdyt?


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the webkit-unassigned mailing list