The default bug view has changed. See this FAQ.

Assertion failure: spoff == js_ReconstructStackDepth(cx_, fp_->script(), pc_), at vm/Stack.cpp:1012

RESOLVED FIXED in mozilla9

Status

()

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

People

(Reporter: decoder, Assigned: luke)

Tracking

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

Trunk
mozilla9
x86_64
Linux
assertion, testcase
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [js-triage-done][inbound])

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
The following code asserts on mozilla-inbound (revision 609f37c36bd7, options -j -m -a):


function toSource(arr) {
  for (i=0; i<len; i++) {}
}
test();
function test() {
  function gen() {
    var c = test;
    try {
      yield c;
    } finally {
      this.toSource();
    }
  }
  var iter = gen();
  for (i in iter) {
    500();
  }
}
(Assignee)

Updated

6 years ago
Assignee: general → luke
Whiteboard: js-triage-needed → js-triage-done
(Assignee)

Comment 1

6 years ago
Created attachment 552545 [details] [diff] [review]
fix and test

Looks like there is a bug where the mjit's exception handling doesn't update the current pc to match the updated sp when closing open iterators (which is observable since this can run finalizers).  I suspect this is debug-only failure; I can't think of how it would manifest a real problem.
Attachment #552545 - Flags: review?(dvander)
Attachment #552545 - Flags: review?(dvander) → review+
(Assignee)

Comment 2

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/5bbc3615e387
Whiteboard: js-triage-done → [js-triage-done][inbound]
http://hg.mozilla.org/mozilla-central/rev/5bbc3615e387

the bug number in the changeset is wrong
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
(Reporter)

Comment 4

4 years ago
Automatically extracted testcase for this bug was committed:

https://hg.mozilla.org/mozilla-central/rev/efaf8960a929
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.