Variable "cx" tracked as NULL was passed to a function that dereferences it [@ ThrowJSException - nsJSObjWrapper::NP_SetProperty]

RESOLVED FIXED

Status

()

Core
Plug-ins
--
critical
RESOLVED FIXED
12 years ago
7 years ago

People

(Reporter: timeless, Assigned: Alfred Peng)

Tracking

({coverity, crash})

Trunk
x86
Linux
coverity, crash
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(crash signature, URL)

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
 
(Assignee)

Comment 1

12 years ago
Created attachment 218815 [details] [diff] [review]
Patch v1

Add the checking for cx before we throw JS exceptions.
Attachment #218815 - Flags: review?(jst)
Comment on attachment 218815 [details] [diff] [review]
Patch v1

r+sr=jst
Attachment #218815 - Flags: superreview+
Attachment #218815 - Flags: review?(jst)
Attachment #218815 - Flags: review+
(Assignee)

Comment 3

12 years ago
jst, could you help check in the patch?
(Assignee)

Comment 4

12 years ago
=> Leon checked in.
(Assignee)

Updated

12 years ago
Assignee: nobody → alfred.peng
(Assignee)

Comment 5

12 years ago
=> Tinderbox ok, FIXED.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Crash Signature: [@ ThrowJSException - nsJSObjWrapper::NP_SetProperty]
You need to log in before you can comment on or make changes to this bug.