Reviewed by Dave.
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 27 Feb 2004 09:25:16 +0000 (09:25 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 27 Feb 2004 09:25:16 +0000 (09:25 +0000)
- fixed <rdar://problem/3571356>: REGRESSION: Crash in isPointInsideSelection clicking on link in Apple web directory frame

        * khtml/khtml_part.cpp:
        (KHTMLPart::isPointInsideSelection): Don't crash if innerNode is
NULL, as this can happen mousing over a frame that's in the
process of loading and currently empty. Since such a frame
obviously can't be inside the selection, this seems like a good
enough fix.

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

WebCore/ChangeLog-2005-08-23
WebCore/khtml/khtml_part.cpp

index e8e7a3d..a54bfd5 100644 (file)
@@ -1,3 +1,16 @@
+2004-02-27  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Dave.
+
+       - fixed <rdar://problem/3571356>: REGRESSION: Crash in isPointInsideSelection clicking on link in Apple web directory frame
+       
+        * khtml/khtml_part.cpp:
+        (KHTMLPart::isPointInsideSelection): Don't crash if innerNode is
+       NULL, as this can happen mousing over a frame that's in the
+       process of loading and currently empty. Since such a frame
+       obviously can't be inside the selection, this seems like a good
+       enough fix.
+       
 2004-02-26  David Hyatt  <hyatt@apple.com>
 
        Fix for 3528839, wide table cut off because of shorts being used instead of ints.
index 8f0568c..9682822 100644 (file)
@@ -4426,7 +4426,7 @@ bool KHTMLPart::isPointInsideSelection(int x, int y)
     RenderObject::NodeInfo nodeInfo(true, true);
     xmlDocImpl()->renderer()->layer()->nodeAtPoint(nodeInfo, x, y);
     DOM::NodeImpl* innerNode = nodeInfo.innerNode();
-    if (!innerNode->renderer()) {
+    if (!innerNode || !innerNode->renderer()) {
         return false;
     }