Closed
Bug 513018
Opened 15 years ago
Closed 15 years ago
parseFloat accepts illegal number syntax
Categories
(Tamarin Graveyard :: Virtual Machine, defect, P2)
Tamarin Graveyard
Virtual Machine
Tracking
(Not tracked)
RESOLVED
FIXED
Q3 11 - Serrano
People
(Reporter: brbaker, Assigned: lhansen)
References
Details
(Whiteboard: has-patch)
Attachments
(1 file, 1 obsolete file)
|
7.27 KB,
patch
|
stejohns
:
review+
|
Details | Diff | Splinter Review |
Steps to reproduce:
trace(parseFloat("12..34..56")); (note the two decimal points between each pair of numbers)
Actual Results:
1234.56
Expected Results, this is what firefox returns:
12
Transferred from JIRA ASC-3822
http://bugs.adobe.com/jira/browse/ASC-3822
Flags: flashplayer-triage+
Flags: flashplayer-qrb?
Assignee: nobody → rreitmai
Blocks: AS3_Builtins
Flags: flashplayer-qrb? → flashplayer-qrb+
Target Milestone: --- → flash10.1
|
Assignee | |
Updated•15 years ago
|
Priority: P3 → P4
|
|
Assignee | |
Comment 2•15 years ago
|
||
Note, not a duplicate of bug #510380.
|
|
Assignee | |
Updated•15 years ago
|
Status: NEW → ASSIGNED
Summary: parseFloat does not return correct value → parseFloat accepts illegal number syntax
|
|
Assignee | |
Comment 3•15 years ago
|
||
(Patch will not land without the required discussion about versioning, pls review the code for functionality :-)
The bug is that the number is scanned correctly, but then the information about where the scanning stopped is discarded and the string's length is incorrectly used during conversion to double.
The patch changes to code so that it makes use of the information about the scanned length.
Attachment #407005 -
Flags: review?(stejohns)
|
|
Assignee | |
Updated•15 years ago
|
Whiteboard: Has patch
|
||
Updated•15 years ago
|
Attachment #407005 -
Flags: review?(stejohns) → review+
|
|
||
Comment 4•15 years ago
|
||
Any fix for this would likely have to be version-checked for backwards compatibility.
|
Reporter | |
Updated•15 years ago
|
Flags: in-testsuite?
|
|
Assignee | |
Updated•15 years ago
|
Whiteboard: Has patch → Has patch; versioning
|
|
Assignee | |
Updated•15 years ago
|
Whiteboard: Has patch; versioning → Has patch
|
|
Assignee | |
Updated•15 years ago
|
Assignee: lhansen → nobody
Priority: P4 → --
Target Milestone: flash10.1 → Future
|
||
Comment 5•15 years ago
|
||
This is a mass change. Every comment has "assigned-to-new" in it.
I didn't look through the bugs, so I'm sorry if I change a bug which shouldn't be changed. But I guess these bugs are just bugs that were once assigned and people forgot to change the Status back when unassigning.
Status: ASSIGNED → NEW
|
|
Assignee | |
Updated•15 years ago
|
Priority: -- → P2
Whiteboard: Has patch → has-patch
Target Milestone: Future → flash10.2
|
|
Assignee | |
Comment 6•15 years ago
|
||
Attachment #407005 -
Attachment is obsolete: true
Attachment #465152 -
Flags: review?(stejohns)
|
|
Assignee | |
Updated•15 years ago
|
Status: NEW → ASSIGNED
|
|
||
Updated•15 years ago
|
Attachment #465152 -
Flags: review?(stejohns) → review+
|
|
Assignee | |
Comment 7•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 8•15 years ago
|
||
Annotated the JIRA bug as well.
|
||
Updated•14 years ago
|
Flags: flashplayer-bug+
You need to log in
before you can comment on or make changes to this bug.
Description
•