Closed
Bug 387358
Opened 17 years ago
Closed 17 years ago
Stuck in a reflow loop that asserts: ###!!! ASSERTION: aPos out of range: '0 <= aPos && aPos < mCharacterCount', file ../../dist/include/thebes/gfxFont.h, line 556
Categories
(Core :: Layout, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: MatsPalmgren_bugz, Assigned: roc)
References
()
Details
(Keywords: assertion, hang, Whiteboard: [sg:low?])
Attachments
(3 files)
(deleted),
text/plain
|
Details | |
(deleted),
patch
|
smontagu
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
smontagu
:
review+
|
Details | Diff | Splinter Review |
Marking Security-Sensitive due to mentioning "Random Classes" fuzzer.
Stuck in a loop that asserts: ###!!! ASSERTION: aPos out of range: '0 <= aPos && aPos < mCharacterCount', file ../../dist/include/thebes/gfxFont.h, line 556
STEPS TO REPRODUCE
1. load http://store.apple.com/1-800-MY-APPLE/WebObjects/AppleStore
2. start "Random Classes 2.0" fuzzer (bug 331889) with args:
13, 32, 100, 400, 0, 0
ACTUAL RESULT
Hangs after a few seconds with the console filling up with the assertions
above. The stack doesn't grow uncontrollably but we never seem to finish
the reflow somehow.
PLATFORMS AND BUILDS TESTED
Bug occurs in a Firefox trunk debug build on Linux (Ubuntu-feisty/x86_64)
Flags: blocking1.9?
Reporter | ||
Comment 1•17 years ago
|
||
Not sure if this helps any, I just typed CTRL+C in a debugger and this
is the stack I got at that point...
Reporter | ||
Comment 2•17 years ago
|
||
BTW, sorry for using an old version of "Random Classes", I was triaging
bug 369971 and just thought I should spawn this off as a new bug...
Assignee | ||
Comment 3•17 years ago
|
||
I simply forgot to update the index variable.
Assignee | ||
Comment 4•17 years ago
|
||
Unfortunately it's hard to test this right now with a small testcase because whether the testcase is triggered or not depends very much on what fonts you have installed.
Assignee | ||
Updated•17 years ago
|
Attachment #271933 -
Flags: review? → review?(smontagu)
Updated•17 years ago
|
Attachment #271933 -
Flags: review?(smontagu) → review+
Comment 5•17 years ago
|
||
If without the patch we assert in a loop, we will still assert once with it, right?
Assignee | ||
Comment 6•17 years ago
|
||
You're right! There is a deeper issue here with mismatch between DOM offsets and textrun offsets.
Attachment #271965 -
Flags: review?(smontagu)
Updated•17 years ago
|
Attachment #271965 -
Flags: review?(smontagu) → review+
Assignee | ||
Comment 7•17 years ago
|
||
checked in
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•17 years ago
|
Flags: in-testsuite?
Comment 8•17 years ago
|
||
Since this involves Thebes am I right in assuming this is a 1.9-only bug?
This was reading out of range, right, not writing?
Whiteboard: [sg:low?]
Assignee | ||
Comment 9•17 years ago
|
||
Yeah, this is 1.9 only. Not sure of overall impact, but it doesn't really matter.
Reporter | ||
Updated•15 years ago
|
Flags: blocking1.9?
Updated•10 years ago
|
Group: core-security
You need to log in
before you can comment on or make changes to this bug.
Description
•