756243 - IonMonkey: Assertion failure: kind == GetGCThingTraceKind(*thingp), at gc/Marking.cpp:231

Status: VERIFIED FIXED

(Core :: JavaScript Engine, defect)

Description

[Christian Holler (:decoder)]

The following testcase asserts on ionmonkey revision 14735b4dbccc (run with --ion -n -m):


function enterFunc (funcName)
    funcName += "()";
var lfcode = new Array();
gczeal(2);
evaluate("test();\
function test() {\
  enterFunc ('test');\
  (new test('(a(b(c)))(d(e(f)))\\\\2\\\\5'));\
}\
");

Comment 1

[David Anderson [:dvander] - inactive, e-mail if emergency]

This is some kind of horrible bug involving invalidation, gc - we're restoring a value to the interpreter stack which has been freed. Investigating.

Comment 2

[David Anderson [:dvander] - inactive, e-mail if emergency]

Attachment: fix

Another simple off-by-N bug.

Comment 3

[David Anderson [:dvander] - inactive, e-mail if emergency]

Attachment: better fix

Comment 4

[Nicolas B. Pierron [:nbp]]

Comment on attachment 624958 [details] [diff] [review]
better fix

Review of attachment 624958 [details] [diff] [review]:
-----------------------------------------------------------------

Good, would be better if you can define

JSFunction *fun = maybeCalleeTokenToFunction(layout->calleeToken());

Comment 5

[David Anderson [:dvander] - inactive, e-mail if emergency]

http://hg.mozilla.org/projects/ionmonkey/rev/8c54899dae82

Comment 6

[Christian Holler (:decoder)]

JSBugMon: This bug has been automatically verified fixed.

Comment 7

[Christian Holler (:decoder)]

Early ion gc issue, in-testsuite-.