Bug 23509 - Crash at -[WebCoreAXObject doAXNextSentenceEndTextMarkerForTextMarker:]
Summary: Crash at -[WebCoreAXObject doAXNextSentenceEndTextMarkerForTextMarker:]
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: Macintosh OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-01-23 14:40 PST by chris fleizach
Modified: 2009-01-23 17:21 PST (History)
0 users

See Also:


Attachments
patch to stop a crasher (2.49 KB, patch)
2009-01-23 14:42 PST, chris fleizach
bdakin: review+
Details | Formatted Diff | Diff
new patch based on review (3.13 KB, patch)
2009-01-23 16:23 PST, chris fleizach
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description chris fleizach 2009-01-23 14:40:33 PST
WebKit can crash at this position when startOfLine returns null
Comment 1 chris fleizach 2009-01-23 14:42:01 PST
Created attachment 26984 [details]
patch to stop a crasher
Comment 2 chris fleizach 2009-01-23 14:42:56 PST
no good steps to reproduce listed in the bug, but the backtrace is pretty clear as to what's happening. as a result, i could not devise a layout test to that elicited the crash
Comment 3 chris fleizach 2009-01-23 15:03:44 PST
http://trac.webkit.org/changeset/40191
Comment 4 Darin Adler 2009-01-23 16:12:35 PST
Comment on attachment 26984 [details]
patch to stop a crasher

Seems to me that makeRange should probably return 0 if passed null positions. And plainText already handles the case where the range is 0. So I would suggest fixing this in the makeRange function rather than in the AccessibilityObject class.
Comment 5 chris fleizach 2009-01-23 16:23:47 PST
Created attachment 26987 [details]
new patch based on review
Comment 6 chris fleizach 2009-01-23 16:24:12 PST
updated patch based on Darin's review
Comment 7 chris fleizach 2009-01-23 16:24:48 PST
Darin, is this what you had in mind

(In reply to comment #4)
> (From update of attachment 26984 [details] [review])
> Seems to me that makeRange should probably return 0 if passed null positions.
> And plainText already handles the case where the range is 0. So I would suggest
> fixing this in the makeRange function rather than in the AccessibilityObject
> class.
> 

Comment 8 Darin Adler 2009-01-23 17:18:02 PST
Comment on attachment 26987 [details]
new patch based on review

r=me
Comment 9 chris fleizach 2009-01-23 17:21:52 PST
http://trac.webkit.org/changeset/40202