commit c733b946b0dd224a8bfa69e16cda3e11a01b6964
author pkasting@chromium.org <pkasting@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc> Tue May 05 19:19:23 2009 +0000
committer pkasting@chromium.org <pkasting@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc> Tue May 05 19:19:23 2009 +0000
tree ef4dfd01147e0f7b6aaec50e139f8301cb63e389
parent 62502aff5fe3f28503904fb8b2a0b6ba36fba288

2009-05-05  Peter Kasting  <pkasting@google.com>

        Reviewed by Brady Eidson.

        Safety-check m_documentLoader before dereferencing.  While it seems
        unlikely this could fail (as Safari 3 shipped without this),
        technically almost any call can change or reset m_documentLoader.

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::receivedFirstData):



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@43248 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index c7b31ff..624d3ee 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,14 @@
+2009-05-05  Peter Kasting  <pkasting@google.com>
+
+        Reviewed by Brady Eidson.
+
+        Safety-check m_documentLoader before dereferencing.  While it seems
+        unlikely this could fail (as Safari 3 shipped without this),
+        technically almost any call can change or reset m_documentLoader.
+
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::receivedFirstData):
+
 2009-05-05  Dan Bernstein  <mitz@apple.com>
 
         - Tiger build fix

WebCore/loader/FrameLoader.cpp

diff --git a/WebCore/loader/FrameLoader.cpp b/WebCore/loader/FrameLoader.cpp
index ca541d2..0bbad2b 100644
--- a/WebCore/loader/FrameLoader.cpp
+++ b/WebCore/loader/FrameLoader.cpp
@@ -878,10 +878,12 @@
     dispatchDidCommitLoad();
     dispatchWindowObjectAvailable();
     
-    String ptitle = m_documentLoader->title();
-    // If we have a title let the WebView know about it.
-    if (!ptitle.isNull())
-        m_client->dispatchDidReceiveTitle(ptitle);
+    if (m_documentLoader) {
+        String ptitle = m_documentLoader->title();
+        // If we have a title let the WebView know about it.
+        if (!ptitle.isNull())
+            m_client->dispatchDidReceiveTitle(ptitle);
+    }
 
     m_workingURL = KURL();