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

RESOLVED FIXED

Status

()

Core
JavaScript Engine
--
critical
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: gkw, Unassigned)

Tracking

(Blocks: 1 bug, {assertion, regression, testcase})

Trunk
assertion, regression, testcase
Points:
---

Firefox Tracking Flags

(blocking2.0 .x+)

Details

(Reporter)

Description

7 years ago
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,
(Reporter)

Updated

7 years ago
blocking2.0: --- → ?
blocking2.0: ? → .x
(Reporter)

Comment 1

7 years ago
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
(Reporter)

Updated

6 years ago
OS: Mac OS X → All
Hardware: x86 → All
(Reporter)

Comment 3

6 years ago
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
Last Resolved: 6 years ago
Resolution: --- → FIXED

Comment 4

6 years ago
Does the autoBisect result change if you use -a?
(Reporter)

Comment 5

6 years ago
(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.