[#ASC-1739] String.split() incorrect

ActionScript Compiler (ASC)

String.split() incorrect

Created: 11/07/06 05:38 PM Updated: 07/24/08 08:35 AM

Component/s:

ECMA Compliance

Security Level:

Public (All JIRA Users )

Issue Links:

Duplicate

This issue is duplicated by:
	~~ASC-2936~~ String.split() reuses matched characters

Severity:	Incorrectly Functioning
Reproducibility:	Every Time
Found in Version:	FP 9.1 - Code Complete - AVMPLUS_1_0_0_d690
Affected OS(s):	Windows - XP
Steps to Reproduce:	« Hide trace("raaan".split("aa")); traces: ["r","","n"] Expected result: traces ["r","an"] Show » trace("raaan".split("aa")); traces: ["r","","n"] Expected result: traces ["r","an"]
Language Found:	English
Bugbase Id:	190739
Needs Release Note:	No
Triaged:	Yes
Regression:	No
QA Owner:	Chris Peyer
Resolved by:	Lars Hansen
Participants:	Beta App, Brent Baker, BugDB Migration, Chris Peyer, JIRA Migration Admin and Lars Hansen
Bugzilla Bug Number:	408084

All

Comments

Bugzilla Comments

Sort Order:

[ Permlink | « Hide ]

JIRA Migration Admin - [06/04/07 12:28 PM ]

Move from BugDB issue number 190739

[ Permlink | « Hide ]

JIRA Migration Admin - [06/04/07 12:28 PM ]

Milestone ID = 1334
Milestone = FP 9.1 - Code Complete
Build ID = 23384
Build = AVMPLUS_1_0_0_d690
Fix Build ID = null
Fix Build = null

[ Permlink | « Hide ]

BugDB Migration - [06/04/07 12:28 PM ]

[Keith Peters 11/7/06] Entered Bug.
[dansmith 11/8/06] Chris Nuuja says: is a bug which recently came in from the Blaze beta list. Using String.split followed by String.join is a popular AS1/2 way of replacing items in a string. This bug will likely affect people trying to migrate such code from AS2 to AS3. I think we should consider fixing it for the .1 release, though I probably wouldn’t rebuild just for it.

[ Permlink | « Hide ]

Lars Hansen - [12/03/07 03:46 PM ]

The function has other problems too. "brabraxxx".split("brabra") will dereference memory before the start of the string data. For sufficiently long "bra" this could probably make the AVM crash.

[ Permlink | « Hide ]

Lars Hansen - [12/06/07 10:41 AM ]

Patch submitted: https://bugzilla.mozilla.org/show_bug.cgi?id=407156

[ Permlink | « Hide ]

Chris Peyer - [12/12/07 12:38 PM ]

Issue transfered to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=408084

[ Permlink | « Hide ]

Lars Hansen - [12/13/07 09:52 AM ]

Tamarin changeset 254:7f46ea6f47ec

[ Permlink | « Hide ]

Brent Baker - [07/21/08 06:50 AM ]

Confirmed fixed in TC 254:7f46ea6f47ec