The default bug view has changed. See this FAQ.

"Assertion failure: pn_type < PNK_LIMIT,"

RESOLVED FIXED in mozilla17

Status

()

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

People

(Reporter: gkw, Assigned: u443197)

Tracking

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

Trunk
mozilla17
x86_64
Mac OS X
assertion, regression, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [jsbugmon:update])

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
Created attachment 654629 [details]
stack

odeURIL:(function(){})

asserts js debug shell on m-c changeset 198ca6edd0ae without any CLI arguments at Assertion failure: pn_type < PNK_LIMIT,

Due to skipped revisions, the first bad revision could be any of:
changeset:   103113:d7568555261c
user:        Alex Crichton
date:        Sat Aug 18 21:33:20 2012 -0700
summary:     Bug 433529: Part 1 - Statically resolve names for anonymous functions. r=jimb

changeset:   103114:4ebcfba34c38
user:        Alex Crichton
date:        Sat Aug 18 21:37:09 2012 -0700
summary:     Bug 433529: Part 2 - Add a displayAtom to JSFunction to store resolved names. r=jimb

(setting bisect in JSBugMon to reconfirm this bisection result)
(Reporter)

Updated

5 years ago
Hardware: x86 → x86_64
Whiteboard: [jsbugmon:update,bisect] → [jsbugmon:update]
JSBugMon: Bisection requested, result:
Due to skipped revisions, the first bad revision could be any of:
changeset:   103113:d7568555261c
user:        Alex Crichton
date:        Sat Aug 18 21:33:20 2012 -0700
summary:     Bug 433529: Part 1 - Statically resolve names for anonymous functions. r=jimb

changeset:   103114:4ebcfba34c38
user:        Alex Crichton
date:        Sat Aug 18 21:37:09 2012 -0700
summary:     Bug 433529: Part 2 - Add a displayAtom to JSFunction to store resolved names. r=jimb
(Assignee)

Comment 2

5 years ago
Created attachment 654673 [details] [diff] [review]
patch

This is mostly the extra additions you already had in bug 433529 Jim, although tweaked slightly to just skip PNK_COLON for labels because there won't be an assignment after that.
Attachment #654673 - Flags: review?(jimb)

Comment 3

5 years ago
Comment on attachment 654673 [details] [diff] [review]
patch

Review of attachment 654673 [details] [diff] [review]:
-----------------------------------------------------------------

Seems familiar, somehow. Thanks for the quick turnaround!
Attachment #654673 - Flags: review?(jimb) → review+
(Assignee)

Comment 4

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/fcbf82dee1db
https://hg.mozilla.org/mozilla-central/rev/fcbf82dee1db
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
(Assignee)

Updated

5 years ago
Assignee: general → alex
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.