Changeset 20855 in webkit

Timestamp:

Apr 11, 2007 6:38:41 PM (17 years ago)

Author:

beidson

Message:

Reviewed by Ada

<rdar://problem/5080987> - Crash in DocumentLoader::frameLoader() const

This was free nil checking lost during the ObjC -> C++ conversion

• loader/FrameLoader.cpp: (WebCore::FrameLoader::checkLoadCompleteForThisFrame): Null check the DocumentLoader

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• loader/FrameLoader.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2007-04-11  Brady Eidson  <beidson@apple.com>
+
+        Reviewed by Ada
+
+        <rdar://problem/5080987> - Crash in DocumentLoader::frameLoader() const
+
+        This was free nil checking lost during the ObjC -> C++ conversion
+
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::checkLoadCompleteForThisFrame): Null check the DocumentLoader
+
 2007-04-11  Justin Garcia  <justin.garcia@apple.com>
 

trunk/WebCore/loader/FrameLoader.cpp

@@ -2676 +2676 @@
         case FrameStateCommittedPage: {
             DocumentLoader* dl = m_documentLoader.get();            
-            if (dl->isLoadingInAPISense())
+            if (!dl || dl->isLoadingInAPISense())
                 return;