Closed Bug 520513 Opened 11 years ago Closed 11 years ago

"Assertion failure: !(evalcg->flags & TCF_IN_FOR_INIT), at ../jsemit.cpp" or "Assertion failure: 0, at ../jsemit.cpp"

Categories

(Core :: JavaScript Engine, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla1.9.2
Tracking Status
status1.9.2 --- beta3-fixed
status1.9.1 --- ?

People

(Reporter: gkw, Assigned: brendan)

References

Details

(Keywords: assertion, regression, testcase, Whiteboard: fixed-in-tracemonkey)

Attachments

(1 file)

(function(){
  var c;
  eval("var c; for each(var c in s);");
})()


asserts js dbg shell on TM tip without -j at Assertion failure: !(evalcg->flags & TCF_IN_FOR_INIT), at ../jsemit.cpp:2171

autoBisecting soon...
autoBisect shows this is probably related to bug 452498:

The first bad revision is:
changeset:   26784:2cf0bbe3772a
user:        Brendan Eich
date:        Sun Apr 05 21:17:22 2009 -0700
summary:     upvar2, aka the big one take 2 (452498, r=mrbkap).

The assertion then was Assertion failure: 0, at ../jsemit.cpp:4579 which then morphed into the present one sometime later.
Blocks: upvar2
status1.9.1: --- → ?
Flags: blocking1.9.2?
Summary: "Assertion failure: !(evalcg->flags & TCF_IN_FOR_INIT), at ../jsemit.cpp" → "Assertion failure: !(evalcg->flags & TCF_IN_FOR_INIT), at ../jsemit.cpp" or "Assertion failure: 0, at ../jsemit.cpp"
Assignee: general → brendan
Status: NEW → ASSIGNED
OS: Mac OS X → All
Priority: -- → P2
Hardware: x86 → All
Target Milestone: --- → mozilla1.9.2
Attached patch fixSplinter Review
Similar bail-out from BindNameToSlot in the upvar-not-from-eval-in-function code was added earlier in BindNameToSlot for bug 470758. I cited both that bug and this one in the comment here.

/be
Attachment #404775 - Flags: review?(mrbkap)
This should go into 1.9.2 as an upvar2 fix.

/be
Priority: P2 → P1
Attachment #404775 - Flags: review?(mrbkap) → review+
Flags: blocking1.9.2? → blocking1.9.2+
http://hg.mozilla.org/tracemonkey/rev/f0e770f882c9

/be
Whiteboard: fixed-in-tracemonkey
http://hg.mozilla.org/mozilla-central/rev/f0e770f882c9
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Automatically extracted testcase for this bug was committed:

https://hg.mozilla.org/mozilla-central/rev/efaf8960a929
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.