Closed Bug 650753 Opened 10 years ago Closed 10 years ago
Crash [@ js::gc::Cell::compartment] with Weak
Map, forced GC
Tested version: http://hg.mozilla.org/tracemonkey/rev/63a06fbd23e0 1. Install 'DOM Fuzz Lite' from https://www.squarefree.com/extensions/domFuzzLite.xpi 2. Load the testcase. Result: crash [@ js::gc::Cell::compartment] I couldn't get it to crash without the extension, which is surprising because the forced GC is at the end of the testcase. I'm curious what's going on here.
null is a gcThing for historically reasons according to luke. isMarkable is what we want.
Assignee: general → anygregor
Comment on attachment 526790 [details] [diff] [review] patch Doh. Thanks.
Attachment #526790 - Flags: review?(gal) → review+
jorendorff, this is probably the crash we had during the aurora landing
I'm so glad this turned up sooner rather than later. Though, this patch absolutely should not have gotten an r+ without a js test. Don't worry about it, I'll push one.
> I'm so glad this turned up sooner rather than later. I found this by adding special WeakMap stuff to my DOM fuzzer and running it on TM branch (which I don't normally do). Just for you <3
Why does this bug only happen with a forced GC, and not with normal GCs or shutdown GC?
(In reply to comment #8) > Why does this bug only happen with a forced GC, and not with normal GCs or > shutdown GC? This path is only executed when the key is alive. I guess the keys are already unreachable if we wait for the normal GC.
Interesting. So we might want to check in several testcases, with various combinations of keeping the key and the map alive.
Pushed: http://hg.mozilla.org/tracemonkey/rev/332284d2b284 Jesse: <3
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Crash Signature: [@ js::gc::Cell::compartment]
A testcase for this bug was automatically identified at js/src/tests/js1_8_5/extensions/regress-650753.js.
You need to log in before you can comment on or make changes to this bug.