Closed Bug 620407 Opened 11 years ago Closed 10 years ago

JM: "Assertion failure: tpa == TPA_Error,"

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
critical

Tracking

()

RESOLVED FIXED
Tracking Status
blocking2.0 --- .x+

People

(Reporter: gkw, Unassigned)

Details

(Keywords: assertion, regression, testcase)

JSON.parse(function() {},
    function() {
        for (a = 0; a < 1; a++) {}
    }
)

asserts js debug shell on TM changeset 35697ebafa6c with -m and -j at Assertion failure: tpa == TPA_Error,
blocking2.0: --- → ?
blocking2.0: ? → .x
autoBisect shows this is probably related to bug 603044:

Due to skipped revisions, the first bad revision could be any of:
changeset:   55675:e000b5963fde
user:        David Anderson
date:        Fri Oct 15 11:36:56 2010 -0700
summary:     Remove JSOP_BEGIN and fix tracer integration issues (bug 603044, r=luke+dmandelin).

changeset:   55676:ae031ec5ad63
user:        David Anderson
date:        Mon Oct 18 20:30:36 2010 -0700
summary:     Build bustage fix.
Blocks: 603044
This bug is somewhere in json.cpp - it looks like parsing fails, setting cx->throwing in the process. But it gets revived in js_FinishJSONParse, which executes code with a pending exception already set.
No longer blocks: 603044
OS: Mac OS X → All
Hardware: x86 → All
Resolving FIXED, the assertion no longer shows. After the changeset below landed, it now shows a message related to SyntaxError: JSON.parse

autoBisect shows this is probably related to the following changeset:

The first good revision is:
changeset:   62353:f569d49576bb
user:        Bill McCloskey
date:        Fri Feb 11 16:31:32 2011 -0800
summary:     Bug 631951 - Shrink methodjit memory usage by interpreting a few times before compiling (r=dvander)
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Does the autoBisect result change if you use -a?
(In reply to Jesse Ruderman from comment #4)
> Does the autoBisect result change if you use -a?

Yes, it does, good point:

autoBisect shows this is probably related to the following changeset:

The first good revision is:
changeset:   67934:6c8becdd1574
user:        Jeff Walden
date:        Wed Mar 23 16:34:53 2011 -0700
summary:     Bug 589664 - Rewrite the JSON parser.  r=njn, anticipating more review but getting it in-tree now for simplicity, even if more changes need to be made later
You need to log in before you can comment on or make changes to this bug.