Closed
Bug 712480
Opened 13 years ago
Closed 12 years ago
Assert in most JSAPI functions that we're not inside a GC
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla12
People
(Reporter: billm, Assigned: billm)
References
Details
Attachments
(1 file)
75.87 KB,
patch
|
Waldo
:
review+
|
Details | Diff | Splinter Review |
Bug 711794 is a case where a finalizer is calling a JSAPI function that can GC. We should be able to assert against this. We would have to identify a subset of the API that is safe. Things like getting reserved slots probably should be allowed during finalization. But most things should not. It probably makes sense to add asserts on every JSAPI function and then remove them as needed to get a browser running. Brian, do you think Sixgill could help here? One problem I can foresee is that it's not obvious from Ben Turner's patch in the bug that it completely eliminates the possibility of calling XPCOM release code--in particular if the runtime is null. However, if we could annotate certain choke points like this one as being okay, then maybe it would be practical.
Assignee | ||
Comment 1•13 years ago
|
||
With Ben's patch applied, this is green on tryserver.
Attachment #584629 -
Flags: review?(luke)
Assignee | ||
Comment 2•13 years ago
|
||
Comment on attachment 584629 [details] [diff] [review] patch Switching to Waldo for a quicker review.
Attachment #584629 -
Flags: review?(luke) → review?(jwalden+bmo)
Comment 3•13 years ago
|
||
Comment on attachment 584629 [details] [diff] [review] patch Review of attachment 584629 [details] [diff] [review]: ----------------------------------------------------------------- r=me if it passes try. And even if it does, I expect fallout for awhile from this. But it is a Good Thing even still.
Attachment #584629 -
Flags: review?(jwalden+bmo) → review+
Assignee | ||
Comment 4•12 years ago
|
||
Yeah, we'll have to keep a close watch on this. https://hg.mozilla.org/integration/mozilla-inbound/rev/edffd43801d9
Target Milestone: --- → mozilla12
Comment 5•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/edffd43801d9
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•