Closed
Bug 416637
Opened 17 years ago
Closed 17 years ago
"ASSERTION: Invalid char passed in" with paragraph separator and combining acute accent
Categories
(Core :: Graphics, defect, P2)
Tracking
()
VERIFIED
FIXED
People
(Reporter: jruderman, Assigned: roc)
References
Details
(Keywords: assertion, testcase)
Attachments
(2 files)
|
101 bytes,
text/html
|
Details | |
|
1.72 KB,
patch
|
vlad
:
review+
|
Details | Diff | Splinter Review |
Loading the testcase triggers:
###!!! ASSERTION: Invalid char passed in: '!gfxFontGroup::IsInvalidChar(aString[firstOffset/2])', file /Users/jruderman/trunk/mozilla/gfx/thebes/src/gfxAtsuiFonts.cpp, line 877
The characters in the testcase are:
U+2029, paragraph separator
U+0301, combining acute accent
|
Assignee | |
Comment 1•17 years ago
|
||
This could probably lead to crazy stuff in the font system
Assignee: nobody → roc
Flags: blocking1.9?
Flags: blocking1.9? → blocking1.9+
|
|
Assignee | |
Comment 2•17 years ago
|
||
Simple fix ... even though the combining mark(s) are at the "start of the text", there may be invalid characters before them that we skipped on purpose, so the string might not be at offset 0.
Attachment #303197 -
Flags: review?(vladimir)
|
|
Assignee | |
Updated•17 years ago
|
Whiteboard: [needs review]
|
||
Updated•17 years ago
|
Priority: -- → P2
Attachment #303197 -
Flags: review?(vladimir) → review+
|
|
||
Updated•17 years ago
|
Flags: tracking1.9+ → blocking1.9+
|
|
Assignee | |
Updated•17 years ago
|
Whiteboard: [needs review] → [needs landing]
|
|
Assignee | |
Comment 3•17 years ago
|
||
checked in with test
Status: NEW → RESOLVED
Closed: 17 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [needs landing]
|
||
Comment 4•17 years ago
|
||
verified fixed using Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9b5pre) Gecko/2008032615 Firefox/3.0b5pre - no assertion when i load the testcase
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•