666003 - Assertion failure: size_t((regs.fp()->hasImacropc() ? regs.fp()->imacropc() : regs.pc) - script->code) < script->length, at jsinterp.cpp:6534

Status: RESOLVED FIXED

Description

[Jan de Mooij [:jandem]]

--
function f() {
    f = function() { g(); };
    f();
}
g = f;
f();
--
$ ./js test.js
Assertion failure: size_t((regs.fp()->hasImacropc() ? regs.fp()->imacropc() : regs.pc) - script->code) < script->length, at jsinterp.cpp:6534

Revision 2ff6092157a5, 32-bit OS X.

Comment 1

[Luke Wagner [:luke]]

Assigning to 'script' before guaranteed success.  Nice testcase, thanks.

Comment 2

[Luke Wagner [:luke]]

Created attachment 540866 [details] [diff] [review]
fix and test

Comment 3

[Jesse Ruderman]

The first bad revision is:
changeset:   ce10e78d030d
user:        Luke Wagner
date:        Tue Jun 14 16:36:13 2011 -0700
summary:     Bug 664252 - Turn lemons (LLVM-only build bustage) into lemonade (rewrite JSOP_CALL/EVAL/NEW to have less goto madness) (r=waldo)

Comment 4

[Luke Wagner [:luke]]

http://hg.mozilla.org/tracemonkey/rev/d416abec8cd3

Comment 5

[Chris Leary [:cdleary] (not checking bugmail)]

cdleary-bot mozilla-central merge info:
http://hg.mozilla.org/mozilla-central/rev/d416abec8cd3

Comment 6

[Christian Holler (:decoder)]

A testcase for this bug was automatically identified at js/src/jit-test/tests/basic/testBug666003.js.