Closed
Bug 459044
Opened 17 years ago
Closed 17 years ago
UnicodeUtils::UTF8Count() with UTF-32 fail
Categories
(Tamarin Graveyard :: Tracing Virtual Machine, defect, P4)
Tracking
(Not tracked)
VERIFIED
FIXED
flash10.1
People
(Reporter: arcnorj, Assigned: cpeyer)
Details
User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.0.3) Gecko/2008092814 Iceweasel/3.0.1 (Debian-3.0.1-1)
Build Identifier: tamarin-central-441cba70ef00
I think in line 374 of UnicodeUtils.cpp:
outch = (c<<18 & 0x1C0000) |
in[1]<<12 & 0x3F000 |
in[2]<<6 & 0xFC0 &
in[3] & 0x3F);
...should read...
outch = (c<<18 & 0x1C0000) |
in[1]<<12 & 0x3F000 |
in[2]<<6 & 0xFC0 |
in[3] & 0x3F);
(note the & and | in the third line)
Reproducible: Always
|
Assignee | |
Comment 1•17 years ago
|
||
Needs investigation by vm engineer.
Status: UNCONFIRMED → NEW
Component: Self-hosting compiler (ESC) → Tracing Virtual Machine
Ever confirmed: true
Flags: flashplayer-qrb?
QA Contact: esc → tracing-vm
|
|
Assignee | |
Updated•17 years ago
|
Flags: in-testsuite?
Flags: flashplayer-triage+
|
|
||
Comment 2•17 years ago
|
||
Yes, that's a bug. I think this has already been fixed in Tamarin-Redux.
|
|
||
Comment 3•17 years ago
|
||
Chris, please verify.
Assignee: nobody → cpeyer
Status: NEW → ASSIGNED
Flags: flashplayer-qrb? → flashplayer-qrb+
Priority: -- → P4
Target Milestone: --- → flash10.x
|
|
||
Comment 4•17 years ago
|
||
This is already fixed in tamarin-redux/central:
http://hg.mozilla.org/tamarin-redux/file/tip/core/UnicodeUtils.cpp#l259
http://hg.mozilla.org/tamarin-central/file/tip/core/UnicodeUtils.cpp#l258
|
|
||
Updated•17 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•16 years ago
|
Status: RESOLVED → VERIFIED
|
|
Assignee | |
Comment 5•16 years ago
|
||
removing in-testsuite as there is no as testcase for this.
Flags: in-testsuite?
You need to log in
before you can comment on or make changes to this bug.
Description
•