The default bug view has changed. See this FAQ.

Too-much-recursion through array_forEach

RESOLVED FIXED

Status

()

Core
JavaScript Engine
--
critical
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: Jesse Ruderman, Assigned: Waldo)

Tracking

(Blocks: 1 bug, {crash, testcase})

Trunk
crash, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: js-triage-needed)

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
Created attachment 546103 [details]
testcase (crashes Firefox when loaded)
(Reporter)

Comment 1

6 years ago
Created attachment 546104 [details]
stack trace (repeating portion only)
Hey, this is why we should self-host forEach!  ;)
Might need a recursion check in bound functions, or might need it for array extras.  Will investigate soon.
Assignee: general → jwalden+bmo
Status: NEW → ASSIGNED
OS: Windows 7 → All
Hardware: x86_64 → All
Whiteboard: js-triage-needed
(Reporter)

Comment 6

5 years ago
See also bug 684462, with Array.sort.
Jesse, bug 779215 likely fixed a lot of these bugs. Perhaps you'd like to retest?
(Reporter)

Comment 8

5 years ago
The testcase now correctly throws a JavaScript exception, "too much recursion" :)
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.