Closed
Bug 478501
Opened 16 years ago
Closed 16 years ago
E4X truncate String when it found the '\0'.
Categories
(Tamarin Graveyard :: Virtual Machine, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: jodyer, Assigned: daumling)
References
Details
Attachments
(1 file, 1 obsolete file)
1.21 KB,
patch
|
lhansen
:
review+
stejohns
:
superreview+
|
Details | Diff | Splinter Review |
Steps to reproduce:
write following ActionScript code.
var s = "a" + String.fromCharCode(0) + "a";
trace(s.length);
trace(<e a={s}/>.toXMLString());
Actual Results:
3
<e a="a"/>
Expected Results:
3
<e a="a�a"/>
Workaround (if any):
Transferred Comments:
Chris Peyer - Thu Apr 10 11:40:41 CDT 2008
Verified valid bug by Werner Sharp.
Trevor Baker - Tue Jan 27 16:30:13 CST 2009
send to internal review for prioritization
This bug transferred from: http://bugs.adobe.com/jira/browse/ASC-3116
Updated•16 years ago
|
Blocks: AS3_Builtins
Assignee | ||
Comment 1•16 years ago
|
||
The simple fix is to supply a length when creating the resulting string in AvmCore::EscapeAttributeValue(). This fix would produce the same output as in SpiderMonkey (which is still wrong - this is also a SpiderMonkey bug). Jeff's expected value is not covered in ECMA-357, where 0x00 is just an ordinary character. It does not hurt IMHO to add 0x00 as a value to be escaped as Jeff's expected result suggests.
Updated•16 years ago
|
Attachment #382117 -
Flags: superreview?(stejohns)
Attachment #382117 -
Flags: review?(lhansen)
Attachment #382117 -
Flags: review+
Comment 2•16 years ago
|
||
Comment on attachment 382117 [details] [diff] [review]
Fis adds to ECMA-357
(Steven gets superreview on builtin fixes.)
Updated•16 years ago
|
Attachment #382117 -
Flags: superreview?(stejohns) → superreview+
Assignee | ||
Comment 3•16 years ago
|
||
Patch pushed as #2018:96dc103550a9
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Comment 4•15 years ago
|
||
Attachment #404629 -
Flags: review?(dschaffe)
Updated•15 years ago
|
Attachment #404629 -
Flags: review?(dschaffe) → review+
Comment 5•15 years ago
|
||
Comment on attachment 404629 [details] [diff] [review]
testcase
Pushed 2689:f822799a7fe3
Attachment #404629 -
Attachment is obsolete: true
Updated•15 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•