Reviewed by Vicki.
authorsullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 15 Mar 2005 22:27:31 +0000 (22:27 +0000)
committersullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 15 Mar 2005 22:27:31 +0000 (22:27 +0000)
        - fixed <rdar://problem/4052246> crash in KWQKHTMLPart::createPart() trying to display local file in frame

        * kwq/KWQKHTMLPart.mm:
        (KWQKHTMLPart::createPart):
        Check part for nil before trying to ref. This was probably a longstanding code flaw revealed by
        the recent security fix.

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

WebCore/ChangeLog-2005-08-23
WebCore/kwq/KWQKHTMLPart.mm

index 9c7621d131c748176b1d98034d7f0d2a1e50c4ea..4225bda38c1b7e9ed19e8785fa3600d1c6dfb6fd 100644 (file)
@@ -1,3 +1,14 @@
+2005-03-15  John Sullivan  <sullivan@apple.com>
+
+        Reviewed by Vicki.
+        
+        - fixed <rdar://problem/4052246> crash in KWQKHTMLPart::createPart() trying to display local file in frame
+
+        * kwq/KWQKHTMLPart.mm:
+        (KWQKHTMLPart::createPart):
+        Check part for nil before trying to ref. This was probably a longstanding code flaw revealed by
+        the recent security fix.
+
 2005-03-15  Kevin Decker  <kdecker@apple.com>
 
         Reviewed by Ken and Maciej.
index 614fb588e855b214d91db0abd1ee7b858fb4e695..73510c9ceac89531f33f7665f78f8ec1106105f2 100644 (file)
@@ -826,8 +826,10 @@ ReadOnlyPart *KWQKHTMLPart::createPart(const ChildFrame &child, const KURL &url,
                                                        marginHeight:marginHeight];
        // This call needs to return an object with a ref, since the caller will expect to own it.
        // childBridge owns the only ref so far.
-       [childBridge part]->ref();
         part = [childBridge part];
+        if (part) {
+            part->ref();
+        }
     }
 
     return part;