Reviewed by Tim.
authorandersca <andersca@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 16 Nov 2006 19:49:12 +0000 (19:49 +0000)
committerandersca <andersca@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 16 Nov 2006 19:49:12 +0000 (19:49 +0000)
        <rdar://problem/4841123>
        REGRESSION: Crash in WebCore::Range::boundaryPointsValid when replying to a mail Message

        * Misc/WebNSAttributedStringExtras.m:
        (+[NSAttributedString _web_attributedStringFromRange:]):
        If the range passed in is null, return null. When this function was in the bridge, it would never get called
        with a null range when nothing was selected. Instead, the range would just have invalid boundary points.

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

WebKit/ChangeLog
WebKit/Misc/WebNSAttributedStringExtras.m

index 47ac1cd..fc7c671 100644 (file)
@@ -1,3 +1,15 @@
+2006-11-16  Anders Carlsson  <acarlsson@apple.com>
+
+        Reviewed by Tim.
+
+        <rdar://problem/4841123>
+        REGRESSION: Crash in WebCore::Range::boundaryPointsValid when replying to a mail Message
+        
+        * Misc/WebNSAttributedStringExtras.m:
+        (+[NSAttributedString _web_attributedStringFromRange:]):
+        If the range passed in is null, return null. When this function was in the bridge, it would never get called
+        with a null range when nothing was selected. Instead, the range would just have invalid boundary points.
+
 2006-11-15  Adam Roben  <aroben@apple.com>
 
         Reviewed by Anders.
index c3be170..e9beef8 100644 (file)
@@ -157,7 +157,7 @@ static NSFileWrapper *fileWrapperForElement(Element* e)
     NSMutableAttributedString *result;
     BEGIN_BLOCK_OBJC_EXCEPTIONS;
 
-    if (!range->boundaryPointsValid())
+    if (!range || !range->boundaryPointsValid())
         return nil;
     
     Node* firstNode = range->startNode();