The default bug view has changed. See this FAQ.

"Assertion failure: 0,"

VERIFIED FIXED in Firefox 10

Status

()

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

People

(Reporter: gkw, Assigned: luke)

Tracking

(Blocks: 1 bug, 4 keywords)

Trunk
mozilla11
x86
Mac OS X
assertion, regression, testcase, verified-beta
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox10 verified)

Details

(Whiteboard: js-triage-done [qa!])

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

6 years ago
Created attachment 573376 [details]
stack

String(
    (function() {
        function k({
            d: {
                length
            }
        }) {}
    }), "\
    (function() {\
        function k({\
            d: {\
                length\
            }\
        }) {}\
    })\
")

asserts js debug shell on m-c changeset 4fb61ebbf8ff with patch v1 from bug 697279 without any CLI arguments at Assertion failure: 0,

autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   79903:a6c390aa3bf2
user:        Luke Wagner
date:        Mon Nov 07 11:46:25 2011 -0800
summary:     Bug 696813 - Simplify Decompile (r=waldo)
(Assignee)

Comment 1

6 years ago
Created attachment 573384 [details] [diff] [review]
fix

Mmm destructuring.
Assignee: general → luke
Status: NEW → ASSIGNED
Attachment #573384 - Flags: review?(jwalden+bmo)
(Assignee)

Comment 2

6 years ago
Created attachment 573396 [details] [diff] [review]
fix

Oops, patch should not include nsJSEnvironment hackery...
Attachment #573384 - Attachment is obsolete: true
Attachment #573384 - Flags: review?(jwalden+bmo)
Attachment #573396 - Flags: review?(jwalden+bmo)
(Assignee)

Updated

6 years ago
Whiteboard: js-triage-needed → js-triage-done
Attachment #573396 - Flags: review?(jwalden+bmo) → review+
(Assignee)

Comment 3

6 years ago
Comment on attachment 573396 [details] [diff] [review]
fix

Low-risk fix (backing out overzealous removal).
Attachment #573396 - Flags: approval-mozilla-aurora?
(Assignee)

Comment 4

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/b7d2f71b4c87
Target Milestone: --- → mozilla11
https://hg.mozilla.org/mozilla-central/rev/b7d2f71b4c87
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED

Updated

5 years ago
Attachment #573396 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 6

5 years ago
https://hg.mozilla.org/releases/mozilla-aurora/rev/dc811a039a7a
(Assignee)

Updated

5 years ago
status-firefox10: --- → fixed
Whiteboard: js-triage-done → js-triage-done [qa+]
OS; Mac OS X 10.6

I used the beta source with the latest revision (mozilla-beta-226c1a05e0b6) and built the js debug shell; then I run the test from comment #0 (./js testcaseFile) and no message was displayed. 
Is this what was needed to verify this issue?
Thanks!
(In reply to Mihaela Velimiroviciu [QA] from comment #7)
 
> Is this what was needed to verify this issue?

Yes that sounds alright. In general if you are not sure if you're testing correctly, try building the debug shell in both the most current and unfixed revisions to see the assertion going away :)
Setting VERIFIED based on previous comment. Thank you!
Status: RESOLVED → VERIFIED
status-firefox10: fixed → verified
Keywords: verified-beta
Whiteboard: js-triage-done [qa+] → js-triage-done [qa!]
A testcase for this bug was automatically identified at js/src/jit-test/tests/basic/testBug701244.js.
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.