Changeset 21427 in webkit

Timestamp:

May 12, 2007 1:35:46 AM (17 years ago)

Author:

eseidel

Message:

2007-05-12 Eric Seidel <​eric@webkit.org>

Reviewed by mjs.

Crash starting Safari with JavaScript disabled.
​http://bugs.webkit.org/show_bug.cgi?id=13691

Test: none possible.

• page/mac/FrameMac.mm: (WebCore::Frame::cleanupPlatformScriptObjects): NULL check d->m_script

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/mac/FrameMac.mm (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2007-05-12  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by mjs.
+
+        Crash starting Safari with JavaScript disabled.
+        http://bugs.webkit.org/show_bug.cgi?id=13691
+        
+        Test: none possible.
+
+        * page/mac/FrameMac.mm:
+        (WebCore::Frame::cleanupPlatformScriptObjects): NULL check d->m_script
+
 2007-05-11  David Hyatt  <hyatt@apple.com>
 

trunk/WebCore/page/mac/FrameMac.mm

@@ -733 +733 @@
     // persist across page loads.)
     removeDOMWrapper(reinterpret_cast<DOMObjectInternal*>(d->m_domWindow.get()));
-    if (d->m_jscript->haveInterpreter())
+    if (d->m_jscript && d->m_jscript->haveInterpreter())
         removeJSWrapper(KJS::Window::retrieveWindow(this));
     d->m_windowScriptObject = 0;