The default bug view has changed. See this FAQ.

"Assertion failure: pnu->isUsed(),"

VERIFIED FIXED in mozilla16

Status

()

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

People

(Reporter: gkw, Assigned: Benjamin)

Tracking

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

Trunk
mozilla16
x86
Windows 7
assertion, regression, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [js:t])

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

5 years ago
Created attachment 636029 [details]
testcase

The attached testcase asserts js debug shell on m-i changeset cb74a377095a without any CLI flag arguments at Assertion failure: pnu->isUsed(), when the testcase passed in as a CLI argument.

autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   97130:70acf458a32d
user:        Benjamin Peterson
date:        Sat Jun 02 20:16:24 2012 +0200
summary:     Bug 759498: fix defaults when arguments are bound as functions, r=jorendorff
(Assignee)

Comment 1

5 years ago
Created attachment 636128 [details] [diff] [review]
fix and assurance that it won't happen again

That was quite a subtle bug! I have no idea why the defaults provoked it; it seems to be a very old bug.
Assignee: general → bpeterson
Attachment #636128 - Flags: review?(jorendorff)
Whiteboard: js-triage-needed → [js:t]
Comment on attachment 636128 [details] [diff] [review]
fix and assurance that it won't happen again

Agreed. That's really strange.
Attachment #636128 - Flags: review?(jorendorff) → review+
(Assignee)

Comment 3

5 years ago
Created attachment 636468 [details] [diff] [review]
a few cosmetic changes
Attachment #636128 - Attachment is obsolete: true
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
(Assignee)

Updated

5 years ago
Attachment #636468 - Flags: review+
(Assignee)

Updated

5 years ago
Attachment #636468 - Flags: review+
(Reporter)

Comment 4

5 years ago
Comment on attachment 636468 [details] [diff] [review]
a few cosmetic changes

Bringing over r+ to avoid potential confusion that this may be a lingering patch in the future.

Landing soon.
Attachment #636468 - Flags: review+
(Reporter)

Comment 5

5 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/2ae1e57c5851
Target Milestone: --- → mozilla16
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/2ae1e57c5851
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Reporter)

Updated

5 years ago
Flags: in-testsuite+
(Reporter)

Comment 7

4 years ago
A type of test for this bug has already been landed because it is already marked in-testsuite+ -> VERIFIED.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.