$ JS_GC_ZEAL=2 ./default-optdebug-build/jsapi-tests testGCOutOfMemory testGCOutOfMemory <no filename>:0:out of memory TEST-UNEXPECTED-FAIL | testGCOutOfMemory | Failed to initialize. WARNING: YOU ARE LEAKING THE WORLD (at least one JSRuntime and everything alive inside it, that is) AT JS_ShutDown TIME. FIX THIS! 1 unexpected failure. Assertion failure: CurrentThreadCanAccessRuntime(cx->runtime()), at /Users/jon/work/compacting/js/src/jscntxt.cpp:1341 Segmentation fault: 11 There are two problems here: firstly that the test fails, secondly that it crashes the test harness.

This is a patch to fix the first problem. I moved the definition of uninit() next to init() and updated it to make it clean up properly if init() failed. The important change though is to call uninit() immediately if init() fails rather than waiting for this to happen in the test destructor, by which time JS_ShutDown() has already run. Output is now merely: testGCOutOfMemory <no filename>:0:out of memory TEST-UNEXPECTED-FAIL | testGCOutOfMemory | Failed to initialize. 1 unexpected failure.

(In reply to Jon Coppeard (:jonco) from comment #1) s/first problem/second problem/ above.

I independently found the second problem and fixed it in bug 1135141.

Removing leave-open keyword from resolved bugs, per :sylvestre.