The default bug view has changed. See this FAQ.

"Assertion failure: !cx->isExceptionPending()" with isPrivilegeEnabled throwing

RESOLVED FIXED in mozilla11

Status

()

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

People

(Reporter: Jesse Ruderman, Assigned: sfink)

Tracking

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

Trunk
mozilla11
x86_64
Mac OS X
assertion, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

6 years ago
Created attachment 572363 [details]
testcase (asserts fatally when loaded)

Assertion failure: !cx->isExceptionPending(), at js/src/jscntxtinlines.h:300

(Should throw an exception along the lines of "not enough arguments".)
(Reporter)

Comment 1

6 years ago
Created attachment 572364 [details]
stack trace
(Assignee)

Comment 2

5 years ago
Created attachment 575353 [details] [diff] [review]
return false on error
Assignee: general → sphink
Status: NEW → ASSIGNED
Attachment #575353 - Flags: review?(bobbyholley+bmo)
Comment on attachment 575353 [details] [diff] [review]
return false on error

I'm not a peer in this code, but this seems like a reasonable change. r=bholley.

NB: This entire API is going away soon.
Attachment #575353 - Flags: review?(bobbyholley+bmo) → review+
(Assignee)

Comment 4

5 years ago
(In reply to Bobby Holley (:bholley) from comment #3)
> I'm not a peer in this code, but this seems like a reasonable change.
> r=bholley.

Whoops, sorry. I didn't really pay attention to where this fix ended up.

> NB: This entire API is going away soon.

For that reason, and because it's such a simple fix, and because I'm a bad person, I think I'll land it anyway.
(Assignee)

Comment 5

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/d048ad9ebe24

looked good on try: https://tbpl.mozilla.org/?tree=Try&rev=51ba3fc4f12f
Target Milestone: --- → mozilla11
https://hg.mozilla.org/mozilla-central/rev/d048ad9ebe24
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.