641479 - "Assertion failure: !cx->isExceptionPending()" with JSON.parse

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jan de Mooij [:jandem]]

--
function f() {
    throw 2;
}
JSON.parse(f, f);
--
$ ./js test.js
Assertion failure: !cx->isExceptionPending(), at ../jsinterp.cpp:6178

In a release build this prints
uncaught exception: 2

Comment 1

[Jeff Walden [:Waldo]]

The new parser fixes this.  I'm not 100% sure why the old parser was broken here, might also possibly be the scaffolding around the parsing code that was at fault (and got reorganized slightly in the rewrite).

We need to add this test before closing this out as WFM:

try
{
  var r = JSON.parse("function f() {\n throw 2;\n}", function() { throw 2; });
  throw new Error("didn't throw, returned " + r);
}
catch (e)
{
  assertEq(e instanceof SyntaxError, true);
}

Comment 2

[Jeff Walden [:Waldo]]

Fixed by the new parser.