1166041 - Intermittent LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer or js_pod_malloc, AllocateObjectBuffer, js::NativeObject::growSlots, js::NativeObject::updateSlotsForSpan

Reporter

Description

•

10 years ago

11:36:35 INFO - ==1906==ERROR: LeakSanitizer: detected memory leaks 11:36:35 INFO - Direct leak of 64 byte(s) in 1 object(s) allocated from: 11:36:35 INFO - #0 0x472111 in malloc /builds/slave/moz-toolchain/src/llvm/projects/compiler-rt/lib/asan/asan_malloc_linux.cc:74 11:36:35 INFO - #1 0x7f8b25e85a7e in js_malloc /builds/slave/fx-team-l64-asan-0000000000000/build/src/obj-firefox/js/src/../../dist/include/js/Utility.h:119 11:36:35 INFO - #2 0x7f8b25e85a7e in js_pod_malloc<unsigned char> /builds/slave/fx-team-l64-asan-0000000000000/build/src/obj-firefox/js/src/../../dist/include/js/Utility.h:274 11:36:35 INFO - #3 0x7f8b25e85a7e in pod_malloc<unsigned char> /builds/slave/fx-team-l64-asan-0000000000000/build/src/js/src/vm/MallocProvider.h:63 11:36:35 INFO - #4 0x7f8b25e85a7e in js::Nursery::allocateBuffer(JSObject*, unsigned int) /builds/slave/fx-team-l64-asan-0000000000000/build/src/js/src/gc/Nursery.cpp:263 11:36:35 INFO - #5 0x7f8b25fe750d in AllocateObjectBuffer<js::HeapSlot> /builds/slave/fx-team-l64-asan-0000000000000/build/src/js/src/gc/Nursery-inl.h:55 11:36:35 INFO - #6 0x7f8b25fe750d in js::NativeObject::growSlots(js::ExclusiveContext*, unsigned int, unsigned int) /builds/slave/fx-team-l64-asan-0000000000000/build/src/js/src/vm/NativeObject.cpp:399 11:36:36 INFO - #7 0x7f8b25fe6581 in js::NativeObject::updateSlotsForSpan(js::ExclusiveContext*, unsigned long, unsigned long) /builds/slave/fx-team-l64-asan-0000000000000/build/src/js/src/vm/NativeObject.cpp:262 11:36:36 INFO - #8 0x7f8b25fe61ca in js::NativeObject::setLastProperty(js::ExclusiveContext*, js::Shape*) /builds/slave/fx-team-l64-asan-0000000000000/build/src/js/src/vm/NativeObject.cpp:298:10

Comment hidden (Legacy TBPL/Treeherder Robot)

Andrew McCreight [:mccr8]

Updated

•

10 years ago

Blocks: LSan

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=fx-team&job_id=3119536 repository: fx-team start_time: 2015-05-18T18:49:08 who: philringnalda[at]gmail[dot]com machine: tst-linux64-spot-731 buildname: Ubuntu ASAN VM 12.04 x64 fx-team opt test mochitest-e10s-browser-chrome-1 revision: 602cec468f2b SUMMARY: AddressSanitizer: 64 byte(s) leaked in 1 allocation(s). TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer 624 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 1] - Got focus: browser2, expected blur: button2 625 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 2] - Got blur: button2, expected blur: window2-document 626 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 3] - Got blur: window2-document, expected blur: window2-window 628 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element focusedElement - Got browser2, expected null 630 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element getFocusedElementForWindow - Got browser2, expected null 633 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element activeElement - Got browser2, expected main-window 634 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedElement after switch to chrome with no focused element - Got [object XULElement], expected null 635 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 0] - Got blur: browser2, expected focus: browser1 636 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 1] - Got blur: window2-window, expected focus: window1-document 637 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 2] - Got focus: window2-document, expected focus: window1-window 639 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus focusedElement - Got null, expected browser1 641 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus getFocusedElementForWindow - Got null, expected browser1 644 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus activeElement - Got main-window, expected browser1 647 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focus button events [event 0] - Got focus: window2-window, expected focus: button1 SUMMARY: AddressSanitizer: 432 byte(s) leaked in 4 allocation(s). SUMMARY: AddressSanitizer: 432 byte(s) leaked in 6 allocation(s).

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=try&job_id=7649430 repository: try start_time: 2015-05-18T21:16:09 who: bballo[at]mozilla[dot]com machine: tst-linux64-spot-1253 buildname: Ubuntu ASAN VM 12.04 x64 try opt test mochitest-e10s-browser-chrome-1 revision: c2ab9f61a833 SUMMARY: AddressSanitizer: 64 byte(s) leaked in 1 allocation(s). TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer 624 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 1] - Got focus: browser2, expected blur: button2 625 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 2] - Got blur: button2, expected blur: window2-document 626 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 3] - Got blur: window2-document, expected blur: window2-window 628 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element focusedElement - Got browser2, expected null 630 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element getFocusedElementForWindow - Got browser2, expected null 633 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element activeElement - Got browser2, expected main-window 634 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedElement after switch to chrome with no focused element - Got [object XULElement], expected null 635 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 0] - Got blur: browser2, expected focus: browser1 636 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 1] - Got blur: window2-window, expected focus: window1-document 637 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 2] - Got focus: window2-document, expected focus: window1-window 639 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus focusedElement - Got null, expected browser1 641 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus getFocusedElementForWindow - Got null, expected browser1 644 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus activeElement - Got main-window, expected browser1 647 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focus button events [event 0] - Got focus: window2-window, expected focus: button1 SUMMARY: AddressSanitizer: 432 byte(s) leaked in 4 allocation(s). SUMMARY: AddressSanitizer: 720 byte(s) leaked in 10 allocation(s).

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=fx-team&job_id=3123451 repository: fx-team start_time: 2015-05-19T06:08:29 who: tomcat[at]mozilla[dot]com machine: tst-linux64-spot-1820 buildname: Ubuntu ASAN VM 12.04 x64 fx-team opt test mochitest-browser-chrome-1 revision: 59fa3030fe7a SUMMARY: AddressSanitizer: 144 byte(s) leaked in 2 allocation(s). SUMMARY: AddressSanitizer: 432 byte(s) leaked in 6 allocation(s). 1965 INFO TEST-UNEXPECTED-FAIL | browser/components/test/browser_bug538331.js | Update notification box should have been displayed - 1975 INFO TEST-UNEXPECTED-FAIL | browser/components/test/browser_bug538331.js | Update notification box should have been displayed - 1979 INFO TEST-UNEXPECTED-FAIL | browser/components/test/browser_bug538331.js | Found an unexpected unknown window at the end of test run - expected PASS SUMMARY: AddressSanitizer: 85451 byte(s) leaked in 1304 allocation(s). TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, js::RegExpStatics::create, js::GlobalObject::getRegExpStatics, js::frontend::Parser TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_new, HashChildren, js::PropertyTree::insertChild, js::PropertyTree::getChild TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at AllocateProtoAndIfaceCache, xpc::CreateGlobalObject, xpc::CreateSandboxObject, XPCJSRuntime::InitSingletonScopes TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, js::ObjectGroup::newPlainObject, js::frontend::ParseNode::getConstantValue, js::frontend::ParseNode::getConstantValue TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, js::frontend::CreateScriptSourceObject, js::frontend::CompileScript, Evaluate TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, js::TenuringTracer::moveSlotsToTenured, js::TenuringTracer::moveObjectToTenured, js::TenuringTracer::moveToTenured TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, js::NewStringCopyNDontDeflate, AtomizeAndCopyChars, js::AtomizeChars TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_calloc, zone, JSScript::makeTypes, ensureHasTypes TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, entryCount, js::Shape::hashify, js::NativeObject::toDictionaryMode TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at nsCycleCollector_startup, NS_InitXPCOM2, Initialize, XREMain::XRE_main TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_calloc, pod_calloc, AllocScriptData, js::CloneScript TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at xpc::CreateSandboxObject, XPCJSRuntime::InitSingletonScopes, xpcModuleCtor TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, js::frontend::CreateScriptSourceObject, js::CloneScript, JSRuntime::cloneSelfHostedFunctionScript TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, tryNewTenuredObject, js::Allocate, JSObject::create TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, js::NewStringCopyNDontDeflate, AtomizeAndCopyChars, js::Atomize TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, entryCount, js::Shape::hashify, js::Shape::search TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, _ZL17NewStringDeflatedILN2js7AllowGCE0EEP12JSFlatStringPNS0_16ExclusiveContextEPKDsm, AtomizeAndCopyChars, js::AtomizeString TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, CreateFunctionPrototype, js::GlobalObject::resolveConstructor, js::GlobalObject::ensureConstructor TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, _ZL17NewStringDeflatedILN2js7AllowGCE0EEP12JSFlatStringPNS0_16ExclusiveContextEPKDsm, AtomizeAndCopyChars, js::AtomizeChars TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_realloc, js::Nursery::reallocateBuffer, ReallocateObjectBuffer, js::NativeObject::growSlots TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, js::NativeIterator::allocateIterator, js::GlobalObject::initIteratorClasses, js::InitIteratorClasses TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_calloc, pod_calloc, AllocScriptData, JSScript::partiallyInit TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, js::frontend::CreateScriptSourceObject, js::frontend::CompileScript, JS::Compile

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=9987320 repository: mozilla-inbound start_time: 2015-05-19T19:02:06 who: philringnalda[at]gmail[dot]com machine: tst-linux64-spot-1930 buildname: Ubuntu ASAN VM 12.04 x64 mozilla-inbound opt test mochitest-e10s-browser-chrome-1 revision: 13f7e6d0e909 SUMMARY: AddressSanitizer: 64 byte(s) leaked in 1 allocation(s). TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer 76 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug1045809.js | Mixed Content is NOT being blocked in Test2 - Got undefined, expected 0 389 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug822367.js | OK: Mixed Content is being blocked - 395 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug822367.js | Mixed Content Doorhanger did not appear for mixed display content! - 402 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug822367.js | OK: Mixed Content is being blocked in test 3 - 413 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug822367.js | OK: Mixed Content is being blocked in Test 4 - 420 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug822367.js | OK: Mixed Content is being blocked in test 4 - 424 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug822367.js | OK: Mixed Content is being blocked in Test 5 - 429 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug822367.js | OK: Mixed Content is being blocked in Test 6 - 445 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug902156.js | OK: Mixed Content is being blocked in Test1A! - 458 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug902156.js | OK: Mixed Content is being blocked in Test 2A! - 471 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug902156.js | OK: Mixed Content is being blocked in Test 3A - 543 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_mixedcontent_securityflags.js | OK: Mixed Content is being blocked - 747 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 1] - Got focus: browser2, expected blur: button2 748 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 2] - Got blur: button2, expected blur: window2-document 749 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 3] - Got blur: window2-document, expected blur: window2-window 751 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element focusedElement - Got browser2, expected null 753 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element getFocusedElementForWindow - Got browser2, expected null 756 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element activeElement - Got browser2, expected main-window 757 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedElement after switch to chrome with no focused element - Got [object XULElement], expected null 758 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 0] - Got blur: browser2, expected focus: browser1 759 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 1] - Got blur: window2-window, expected focus: window1-document 760 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 2] - Got focus: window2-document, expected focus: window1-window 762 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus focusedElement - Got null, expected browser1 764 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus getFocusedElementForWindow - Got null, expected browser1 767 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus activeElement - Got main-window, expected browser1 770 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focus button events [event 0] - Got focus: window2-window, expected focus: button1 SUMMARY: AddressSanitizer: 432 byte(s) leaked in 4 allocation(s). SUMMARY: AddressSanitizer: 144 byte(s) leaked in 2 allocation(s). SUMMARY: AddressSanitizer: 864 byte(s) leaked in 12 allocation(s). SUMMARY: AddressSanitizer: 144 byte(s) leaked in 2 allocation(s).

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=fx-team&job_id=3133827 repository: fx-team start_time: 2015-05-19T23:49:08 who: tomcat[at]mozilla[dot]com machine: tst-linux64-spot-989 buildname: Ubuntu ASAN VM 12.04 x64 fx-team opt test mochitest-e10s-browser-chrome-1 revision: 09dde9554781 SUMMARY: AddressSanitizer: 64 byte(s) leaked in 1 allocation(s). TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer 624 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 1] - Got focus: browser2, expected blur: button2 625 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 2] - Got blur: button2, expected blur: window2-document 626 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 3] - Got blur: window2-document, expected blur: window2-window 628 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element focusedElement - Got browser2, expected null 630 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element getFocusedElementForWindow - Got browser2, expected null 633 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element activeElement - Got browser2, expected main-window 634 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedElement after switch to chrome with no focused element - Got [object XULElement], expected null 635 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 0] - Got blur: browser2, expected focus: browser1 636 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 1] - Got blur: window2-window, expected focus: window1-document 637 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 2] - Got focus: window2-document, expected focus: window1-window 639 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus focusedElement - Got null, expected browser1 641 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus getFocusedElementForWindow - Got null, expected browser1 644 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus activeElement - Got main-window, expected browser1 647 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focus button events [event 0] - Got focus: window2-window, expected focus: button1 SUMMARY: AddressSanitizer: 432 byte(s) leaked in 4 allocation(s). SUMMARY: AddressSanitizer: 288 byte(s) leaked in 4 allocation(s). SUMMARY: AddressSanitizer: 576 byte(s) leaked in 8 allocation(s). SUMMARY: AddressSanitizer: 144 byte(s) leaked in 2 allocation(s).

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=9995900 repository: mozilla-inbound start_time: 2015-05-20T03:08:35 who: tomcat[at]mozilla[dot]com machine: tst-linux64-spot-1864 buildname: Ubuntu ASAN VM 12.04 x64 mozilla-inbound opt test mochitest-e10s-browser-chrome-1 revision: 9665b9f50268 SUMMARY: AddressSanitizer: 64 byte(s) leaked in 1 allocation(s). TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer 622 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 1] - Got focus: browser2, expected blur: button2 623 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 2] - Got blur: button2, expected blur: window2-document 624 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 3] - Got blur: window2-document, expected blur: window2-window 626 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element focusedElement - Got browser2, expected null 628 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element getFocusedElementForWindow - Got browser2, expected null 631 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element activeElement - Got browser2, expected main-window 632 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedElement after switch to chrome with no focused element - Got [object XULElement], expected null 633 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 0] - Got blur: browser2, expected focus: browser1 634 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 1] - Got blur: window2-window, expected focus: window1-document 635 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 2] - Got focus: window2-document, expected focus: window1-window 637 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus focusedElement - Got null, expected browser1 639 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus getFocusedElementForWindow - Got null, expected browser1 642 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus activeElement - Got main-window, expected browser1 645 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focus button events [event 0] - Got focus: window2-window, expected focus: button1 SUMMARY: AddressSanitizer: 432 byte(s) leaked in 4 allocation(s). SUMMARY: AddressSanitizer: 288 byte(s) leaked in 4 allocation(s). SUMMARY: AddressSanitizer: 864 byte(s) leaked in 12 allocation(s).

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=10025136 repository: mozilla-inbound start_time: 2015-05-20T17:36:11 who: philringnalda[at]gmail[dot]com machine: tst-linux64-spot-1339 buildname: Ubuntu ASAN VM 12.04 x64 mozilla-inbound opt test mochitest-e10s-browser-chrome-1 revision: d98d6c08d2fd SUMMARY: AddressSanitizer: 64 byte(s) leaked in 1 allocation(s). TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer 622 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 1] - Got focus: browser2, expected blur: button2 623 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 2] - Got blur: button2, expected blur: window2-document 624 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 3] - Got blur: window2-document, expected blur: window2-window 626 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element focusedElement - Got browser2, expected null 628 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element getFocusedElementForWindow - Got browser2, expected null 631 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element activeElement - Got browser2, expected main-window 632 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedElement after switch to chrome with no focused element - Got [object XULElement], expected null 633 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 0] - Got blur: browser2, expected focus: browser1 634 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 1] - Got blur: window2-window, expected focus: window1-document 635 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 2] - Got focus: window2-document, expected focus: window1-window 637 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus focusedElement - Got null, expected browser1 639 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus getFocusedElementForWindow - Got null, expected browser1 642 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus activeElement - Got main-window, expected browser1 645 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focus button events [event 0] - Got focus: window2-window, expected focus: button1 SUMMARY: AddressSanitizer: 432 byte(s) leaked in 4 allocation(s). SUMMARY: AddressSanitizer: 144 byte(s) leaked in 2 allocation(s). SUMMARY: AddressSanitizer: 1008 byte(s) leaked in 14 allocation(s).

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=fx-team&job_id=3152125 repository: fx-team start_time: 2015-05-21T04:02:56 who: tomcat[at]mozilla[dot]com machine: tst-linux64-spot-005 buildname: Ubuntu ASAN VM 12.04 x64 fx-team opt test mochitest-e10s-browser-chrome-1 revision: b38860355bec SUMMARY: AddressSanitizer: 64 byte(s) leaked in 1 allocation(s). TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer 622 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 1] - Got focus: browser2, expected blur: button2 623 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 2] - Got blur: button2, expected blur: window2-document 624 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 3] - Got blur: window2-document, expected blur: window2-window 626 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element focusedElement - Got browser2, expected null 628 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element getFocusedElementForWindow - Got browser2, expected null 631 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element activeElement - Got browser2, expected main-window 632 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedElement after switch to chrome with no focused element - Got [object XULElement], expected null 633 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 0] - Got blur: browser2, expected focus: browser1 634 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 1] - Got blur: window2-window, expected focus: window1-document 635 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 2] - Got focus: window2-document, expected focus: window1-window 637 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus focusedElement - Got null, expected browser1 639 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus getFocusedElementForWindow - Got null, expected browser1 642 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus activeElement - Got main-window, expected browser1 645 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focus button events [event 0] - Got focus: window2-window, expected focus: button1 SUMMARY: AddressSanitizer: 432 byte(s) leaked in 4 allocation(s). SUMMARY: AddressSanitizer: 144 byte(s) leaked in 2 allocation(s). SUMMARY: AddressSanitizer: 1008 byte(s) leaked in 14 allocation(s).

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-central&job_id=1536710 repository: mozilla-central start_time: 2015-05-21T06:24:03 who: rvandermeulen[at]mozilla[dot]com machine: tst-linux64-spot-894 buildname: Ubuntu ASAN VM 12.04 x64 mozilla-central opt test mochitest-e10s-browser-chrome-1 revision: 91197924c93e SUMMARY: AddressSanitizer: 64 byte(s) leaked in 1 allocation(s). TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer 622 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 1] - Got focus: browser2, expected blur: button2 623 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 2] - Got blur: button2, expected blur: window2-document 624 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element events [event 3] - Got blur: window2-document, expected blur: window2-window 626 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element focusedElement - Got browser2, expected null 628 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element getFocusedElementForWindow - Got browser2, expected null 631 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | after switch to chrome with no focused element activeElement - Got browser2, expected main-window 632 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedElement after switch to chrome with no focused element - Got [object XULElement], expected null 633 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 0] - Got blur: browser2, expected focus: browser1 634 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 1] - Got blur: window2-window, expected focus: window1-document 635 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus events [event 2] - Got focus: window2-document, expected focus: window1-window 637 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus focusedElement - Got null, expected browser1 639 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus getFocusedElementForWindow - Got null, expected browser1 642 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focusedWindow after tab switch from no focus to no focus activeElement - Got main-window, expected browser1 645 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tabfocus.js | focus button events [event 0] - Got focus: browser1, expected focus: button1 SUMMARY: AddressSanitizer: 432 byte(s) leaked in 4 allocation(s). SUMMARY: AddressSanitizer: 144 byte(s) leaked in 2 allocation(s). SUMMARY: AddressSanitizer: 864 byte(s) leaked in 12 allocation(s). SUMMARY: AddressSanitizer: 288 byte(s) leaked in 4 allocation(s).

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Reporter

Comment 78

•

10 years ago

Ping?

Flags: needinfo?(terrence)

Terrence Cole [:terrence]

Comment 79

•

10 years ago

Appears to be new with Brian's rewrite of this subsystem.

Flags: needinfo?(terrence) → needinfo?(bhackett1024)

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=10166307 repository: mozilla-inbound start_time: 2015-05-26T14:32:36 who: wkocher[at]mozilla[dot]com machine: tst-linux64-spot-614 buildname: Ubuntu ASAN VM 12.04 x64 mozilla-inbound opt test mochitest-e10s-browser-chrome-1 revision: c02250c4d3c9 SUMMARY: AddressSanitizer: 64 byte(s) leaked in 1 allocation(s). TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer SUMMARY: AddressSanitizer: 432 byte(s) leaked in 4 allocation(s). 771 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_marks.js | uncaught exception - Error: operation not possible on dead CPOW at :0 775 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_marks.js | Test timed out - expected PASS 777 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_marks.js | Found a tab after previous test timed out: https://sub2.test1.example.com/browser/browser/base/content/test/social/social_activate.html - expected PASS 816 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_multiworker.js | all provider prefs uninstalled from previous test - Got 2, expected 0 817 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_multiworker.js | all providers uninstalled from previous test 2 - Got 2, expected 0 855 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_perwindowPB.js | all provider prefs uninstalled from previous test - Got 2, expected 0 856 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_perwindowPB.js | all providers uninstalled from previous test 2 - Got 2, expected 0 880 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_sidebar.js | no providers left - Got 2, expected 0 883 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_sidebar.js | all provider prefs uninstalled from previous test - Got 2, expected 0 884 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_sidebar.js | all providers uninstalled from previous test 2 - Got 2, expected 0 946 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_status.js | Should be in the default state when we finish - 954 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_status.js | all provider prefs uninstalled from previous test - Got 2, expected 0 955 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_status.js | all providers uninstalled from previous test 2 - Got 2, expected 0 956 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_status.js | A promise chain failed to handle a rejection: - at chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_marks.js:354 - ReferenceError: info is not defined 963 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | needs zero providers to start this test. - Got 2, expected 0 964 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | no providers are enabled - 966 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | providers are enabled - 971 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | Social:ToggleNotifications visible? - Got false, expected true 972 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | The Social UI tests succeeded. - Got 6, expected 7 974 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | Social:ToggleNotifications visible? - Got false, expected true 975 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | The Social UI tests succeeded. - Got 6, expected 7 995 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | social is disabled - 996 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | no providers - Got 2, expected 0 1058 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | social is disabled - 1059 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | no providers - Got 2, expected 0 1063 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | all provider prefs uninstalled from previous test - Got 2, expected 0 1064 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | all providers uninstalled from previous test 2 - Got 2, expected 0 SUMMARY: AddressSanitizer: 720 byte(s) leaked in 10 allocation(s).

Terrence Cole [:terrence]

Comment 83

•

10 years ago

At least I think it starts around the time that that landed. Brian, feel free to punt this back my way if you disagree.

Comment hidden (Legacy TBPL/Treeherder Robot)

Jon Coppeard (:jonco)

Comment 97

•

10 years ago

Attached patch bug1166041-nursery-leak — Details — Splinter Review

We actually intentionally leak slots memory on that path if we hit OOM inserting into a hash table. Here's a patch to tighten that up and fail the allocation instead.

Attachment #8612271 - Flags: review?(terrence)

Terrence Cole [:terrence]

Comment 98

•

10 years ago

Comment on attachment 8612271 [details] [diff] [review] bug1166041-nursery-leak Review of attachment 8612271 [details] [diff] [review]: ----------------------------------------------------------------- Yup, better to just raise the error.

Attachment #8612271 - Flags: review?(terrence) → review+

Terrence Cole [:terrence]

Updated

•

10 years ago

Assignee: nobody → jcoppeard

Status: NEW → ASSIGNED

Flags: needinfo?(bhackett1024)

Comment hidden (Legacy TBPL/Treeherder Robot)

Pulsebot

Comment 108

•

10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/531cb24dbb82

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Reporter

Comment 114

•

10 years ago

(In reply to Treeherder Robot from comment #113) This was on a rev after your push :(

Flags: needinfo?(jcoppeard)

Keywords: leave-open

Jon Coppeard (:jonco)

Comment 115

•

10 years ago

(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #114) Oh well it was worth fixing that first. The real cause must be something more subtle.

Ryan VanderMeulen [:RyanVM]

Reporter

Comment 116

•

10 years ago

https://hg.mozilla.org/mozilla-central/rev/531cb24dbb82

Comment hidden (Legacy TBPL/Treeherder Robot)

Jon Coppeard (:jonco)

Comment 210

•

10 years ago

Attached patch bug1166041-leak-asserts — Details — Splinter Review

Add some assertions to hopefully catch this.

Flags: needinfo?(jcoppeard)

Attachment #8620893 - Flags: review?(terrence)

Comment hidden (Legacy TBPL/Treeherder Robot)

Terrence Cole [:terrence]

Updated

•

10 years ago

Attachment #8620893 - Flags: review?(terrence) → review+

Comment hidden (Legacy TBPL/Treeherder Robot)

Pulsebot

Comment 218

•

10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/70e774f5c152

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Reporter

Comment 224

•

10 years ago

https://hg.mozilla.org/mozilla-central/rev/70e774f5c152

Comment hidden (Legacy TBPL/Treeherder Robot)

Jon Coppeard (:jonco)

Comment 304

•

10 years ago

Andrew, is it possible to increase the number of lines of backtrace we get for these failures? We don't have enough here to see beyond NativeObject::setLastProperty() which is still pretty generic.

Flags: needinfo?(continuation)

Andrew McCreight [:mccr8]

Comment 305

•

10 years ago

The current value was set by decoder to reduce memory usage. This is determined by the ASan option malloc_context_size[1]. I'm not sure how common this leak is, but you could try increasing it a bit and then doing a try run with a bunch of retriggers. I'm not sure how badly it would fall over, or if we use AWS machines with the same amount of RAM now or what. [1] http://mxr.mozilla.org/mozilla-central/search?string=malloc_context_size&filter=[Mm]alloc_context_size

Flags: needinfo?(continuation)

Jan de Mooij [:jandem]

Assignee

Comment 306

•

10 years ago

(In reply to Andrew McCreight [:mccr8] from comment #305) > The current value was set by decoder to reduce memory usage. Actually that was me, bug 977921 has some info. However that was before Mochitest-bc was split up and it was more than a year ago, so it's probably fine to increase that value now and see if anything breaks.

Comment hidden (Legacy TBPL/Treeherder Robot)

Jon Coppeard (:jonco)

Comment 314

•

10 years ago

I captured some more backtrace but it still doesn't tell us much: 05:03:31 INFO - Direct leak of 64 byte(s) in 1 object(s) allocated from: 05:03:31 INFO - #0 0x472231 in malloc /builds/slave/moz-toolchain/src/llvm/projects/compiler-rt/lib/asan/asan_malloc_linux.cc:74 05:03:31 INFO - #1 0x7f1ad0bfeb5e in js_malloc /builds/slave/try-l64-asan-00000000000000000/build/src/obj-firefox/js/src/../../dist/include/js/Utility.h:135 05:03:31 INFO - #2 0x7f1ad0bfeb5e in js_pod_malloc<unsigned char> /builds/slave/try-l64-asan-00000000000000000/build/src/obj-firefox/js/src/../../dist/include/js/Utility.h:290 05:03:31 INFO - #3 0x7f1ad0bfeb5e in pod_malloc<unsigned char> /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/MallocProvider.h:63 05:03:31 INFO - #4 0x7f1ad0bfeb5e in js::Nursery::allocateBuffer(JSObject*, unsigned int) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/gc/Nursery.cpp:264 05:03:31 INFO - #5 0x7f1ad0d8b81c in AllocateObjectBuffer<js::HeapSlot> /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/gc/Nursery-inl.h:60 05:03:31 INFO - #6 0x7f1ad0d8b81c in js::NativeObject::growSlots(js::ExclusiveContext*, unsigned int, unsigned int) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:402 05:03:31 INFO - #7 0x7f1ad0d8a72b in js::NativeObject::updateSlotsForSpan(js::ExclusiveContext*, unsigned long, unsigned long) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:262 05:03:31 INFO - #8 0x7f1ad0d8a2ed in js::NativeObject::setLastProperty(js::ExclusiveContext*, js::Shape*) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:298:10 05:03:31 INFO - #9 0x7f1ad0e236aa in js::NativeObject::getChildProperty(js::ExclusiveContext*, JS::Handle<js::NativeObject*>, JS::Handle<js::Shape*>, js::StackShape&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Shape.cpp:414 05:03:31 INFO - #10 0x7f1ad0e01831 in js::NativeObject::addPropertyInternal(js::ExclusiveContext*, JS::Handle<js::NativeObject*>, JS::Handle<jsid>, bool (*)(JSContext*, JS::Handle<JSObject*>, JS::Handle<jsid>, JS::MutableHandle<JS::Value>), bool (*)(JSContext*, JS::Handle<JSObject*>, JS::Handle<jsid>, JS::MutableHandle<JS::Value>, JS::ObjectOpResult&), unsigned int, unsigned int, unsigned int, js::ShapeTable::Entry*, bool) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Shape.cpp:581 05:03:31 INFO - #11 0x7f1ad0df9bb7 in js::NativeObject::putProperty(js::ExclusiveContext*, JS::Handle<js::NativeObject*>, JS::Handle<jsid>, bool (*)(JSContext*, JS::Handle<JSObject*>, JS::Handle<jsid>, JS::MutableHandle<JS::Value>), bool (*)(JSContext*, JS::Handle<JSObject*>, JS::Handle<jsid>, JS::MutableHandle<JS::Value>, JS::ObjectOpResult&), unsigned int, unsigned int, unsigned int) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Shape.cpp:728 05:03:31 INFO - #12 0x7f1ad0d96d1f in AddOrChangeProperty(js::ExclusiveContext*, JS::Handle<js::NativeObject*>, JS::Handle<jsid>, JS::Handle<JSPropertyDescriptor>) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:1151 05:03:31 INFO - #13 0x7f1ad0d94b6f in js::NativeDefineProperty(js::ExclusiveContext*, JS::Handle<js::NativeObject*>, JS::Handle<jsid>, JS::Handle<JSPropertyDescriptor>, JS::ObjectOpResult&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:1366

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=11750021 repository: mozilla-inbound start_time: 2015-07-15T18:10:43 who: rvandermeulen[at]mozilla[dot]com machine: tst-linux64-spot-731 buildname: Ubuntu ASAN VM 12.04 x64 mozilla-inbound opt test mochitest-e10s-browser-chrome-1 revision: bf1063b0719f SUMMARY: AddressSanitizer: 64 byte(s) leaked in 1 allocation(s). TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer 154 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_marks.js | uncaught exception - Error: operation not possible on dead CPOW at :0 157 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_marks.js | Test timed out - expected PASS 159 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_marks.js | Found a tab after previous test timed out: https://sub2.test1.example.com/browser/browser/base/content/test/social/social_activate.html - expected PASS 198 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_multiworker.js | all provider prefs uninstalled from previous test - Got 2, expected 0 199 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_multiworker.js | all providers uninstalled from previous test 2 - Got 2, expected 0 237 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_perwindowPB.js | all provider prefs uninstalled from previous test - Got 2, expected 0 238 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_perwindowPB.js | all providers uninstalled from previous test 2 - Got 2, expected 0 262 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_sidebar.js | no providers left - Got 2, expected 0 265 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_sidebar.js | all provider prefs uninstalled from previous test - Got 2, expected 0 266 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_sidebar.js | all providers uninstalled from previous test 2 - Got 2, expected 0 313 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_status.js | A promise chain failed to handle a rejection: - at chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_marks.js:354 - ReferenceError: info is not defined 327 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_status.js | Should be in the default state when we finish - 335 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_status.js | all provider prefs uninstalled from previous test - Got 2, expected 0 336 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_status.js | all providers uninstalled from previous test 2 - Got 2, expected 0 343 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | needs zero providers to start this test. - Got 2, expected 0 344 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | no providers are enabled - 346 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | providers are enabled - 351 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | Social:ToggleNotifications visible? - Got false, expected true 352 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | The Social UI tests succeeded. - Got 6, expected 7 354 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | Social:ToggleNotifications visible? - Got false, expected true 355 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | The Social UI tests succeeded. - Got 6, expected 7 375 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | social is disabled - 376 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | no providers - Got 2, expected 0 438 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | social is disabled - 439 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | no providers - Got 2, expected 0 443 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | all provider prefs uninstalled from previous test - Got 2, expected 0 444 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | all providers uninstalled from previous test 2 - Got 2, expected 0

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Reporter

Updated

•

10 years ago

Summary: Intermittent LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer → Intermittent LeakSanitizer | leak at js_pod_malloc, pod_malloc, js::Nursery::allocateBuffer, AllocateObjectBuffer or js_pod_malloc, AllocateObjectBuffer, js::NativeObject::growSlots, js::NativeObject::updateSlotsForSpan

Comment hidden (Legacy TBPL/Treeherder Robot)

Jon Coppeard (:jonco)

Comment 410

•

10 years ago

Captured more context. It might be a cross compartment wrapper whose slots we are leaking. 03:39:40 INFO - ==2132==ERROR: LeakSanitizer: detected memory leaks 03:39:40 INFO - Direct leak of 64 byte(s) in 1 object(s) allocated from: 03:39:40 INFO - #0 0x472271 in malloc /builds/slave/moz-toolchain/src/llvm/projects/compiler-rt/lib/asan/asan_malloc_linux.cc:74 03:39:40 INFO - #1 0x7fbdeaf66278 in js_malloc /builds/slave/try-l64-asan-00000000000000000/build/src/obj-firefox/js/src/../../dist/include/js/Utility.h:135 03:39:40 INFO - #2 0x7fbdeaf66278 in js_pod_malloc<unsigned char> /builds/slave/try-l64-asan-00000000000000000/build/src/obj-firefox/js/src/../../dist/include/js/Utility.h:290 03:39:40 INFO - #3 0x7fbdeaf66278 in unsigned char* js::MallocProvider<JS::Zone>::pod_malloc<unsigned char>(unsigned long) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/MallocProvider.h:63 03:39:40 INFO - #4 0x7fbdeaebbdcc in AllocateObjectBuffer<js::HeapSlot> /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/gc/Nursery-inl.h:60 03:39:40 INFO - #5 0x7fbdeaebbdcc in js::NativeObject::growSlots(js::ExclusiveContext*, unsigned int, unsigned int) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:402 03:39:40 INFO - #6 0x7fbdeaebacdb in js::NativeObject::updateSlotsForSpan(js::ExclusiveContext*, unsigned long, unsigned long) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:262 03:39:40 INFO - #7 0x7fbdeaeba89d in js::NativeObject::setLastProperty(js::ExclusiveContext*, js::Shape*) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:298:10 03:39:40 INFO - #8 0x7fbdeaf53daa in js::NativeObject::getChildProperty(js::ExclusiveContext*, JS::Handle<js::NativeObject*>, JS::Handle<js::Shape*>, js::StackShape&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Shape.cpp:414 03:39:40 INFO - #9 0x7fbdeaf31f51 in js::NativeObject::addPropertyInternal(js::ExclusiveContext*, JS::Handle<js::NativeObject*>, JS::Handle<jsid>, bool (*)(JSContext*, JS::Handle<JSObject*>, JS::Handle<jsid>, JS::MutableHandle<JS::Value>), bool (*)(JSContext*, JS::Handle<JSObject*>, JS::Handle<jsid>, JS::MutableHandle<JS::Value>, JS::ObjectOpResult&), unsigned int, unsigned int, unsigned int, js::ShapeTable::Entry*, bool) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Shape.cpp:581 03:39:40 INFO - #10 0x7fbdeaf2a2d7 in js::NativeObject::putProperty(js::ExclusiveContext*, JS::Handle<js::NativeObject*>, JS::Handle<jsid>, bool (*)(JSContext*, JS::Handle<JSObject*>, JS::Handle<jsid>, JS::MutableHandle<JS::Value>), bool (*)(JSContext*, JS::Handle<JSObject*>, JS::Handle<jsid>, JS::MutableHandle<JS::Value>, JS::ObjectOpResult&), unsigned int, unsigned int, unsigned int) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Shape.cpp:728 03:39:40 INFO - #11 0x7fbdeaec72cf in AddOrChangeProperty(js::ExclusiveContext*, JS::Handle<js::NativeObject*>, JS::Handle<jsid>, JS::Handle<JSPropertyDescriptor>) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:1151 03:39:40 INFO - #12 0x7fbdeaec511c in js::NativeDefineProperty(js::ExclusiveContext*, JS::Handle<js::NativeObject*>, JS::Handle<jsid>, JS::Handle<JSPropertyDescriptor>, JS::ObjectOpResult&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:1366 03:39:40 INFO - #13 0x7fbdeb9a3401 in js::DefineProperty(js::ExclusiveContext*, JS::Handle<JSObject*>, JS::Handle<jsid>, JS::Handle<JS::Value>, bool (*)(JSContext*, JS::Handle<JSObject*>, JS::Handle<jsid>, JS::MutableHandle<JS::Value>), bool (*)(JSContext*, JS::Handle<JSObject*>, JS::Handle<jsid>, JS::MutableHandle<JS::Value>, JS::ObjectOpResult&), unsigned int, JS::ObjectOpResult&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/jsobj.cpp:2626 03:39:40 INFO - #14 0x7fbdeaed2f5c in js::SetPropertyByDefining(JSContext*, JS::Handle<JSObject*>, JS::Handle<jsid>, JS::Handle<JS::Value>, JS::Handle<JS::Value>, bool, JS::ObjectOpResult&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:2119 03:39:40 INFO - #15 0x7fbdeaed7574 in SetNonexistentProperty(JSContext*, JS::Handle<js::NativeObject*>, JS::Handle<jsid>, JS::Handle<JS::Value>, JS::Handle<JS::Value>, js::QualifiedBool, JS::ObjectOpResult&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:2180 03:39:40 INFO - #16 0x7fbdeaed5eb8 in js::NativeSetProperty(JSContext*, JS::Handle<js::NativeObject*>, JS::Handle<jsid>, JS::Handle<JS::Value>, JS::Handle<JS::Value>, js::QualifiedBool, JS::ObjectOpResult&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.cpp:2353 03:39:40 INFO - #17 0x7fbdeadea43b in SetProperty /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/NativeObject.h:1434 03:39:40 INFO - #18 0x7fbdeadea43b in SetPropertyOperation /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Interpreter.cpp:317 03:39:40 INFO - #19 0x7fbdeadea43b in Interpret(JSContext*, js::RunState&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Interpreter.cpp:2777 03:39:40 INFO - #20 0x7fbdeadd7914 in js::RunScript(JSContext*, js::RunState&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Interpreter.cpp:661 03:39:40 INFO - #21 0x7fbdeadb8c90 in js::Invoke(JSContext*, JS::CallArgs, js::MaybeConstruct) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Interpreter.cpp:738 03:39:40 INFO - #22 0x7fbdeae08d8d in js::InvokeConstructor(JSContext*, JS::CallArgs) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Interpreter.cpp:805 03:39:40 INFO - #23 0x7fbdeae09641 in js::InvokeConstructor(JSContext*, JS::Value, unsigned int, JS::Value const*, bool, JS::MutableHandle<JS::Value>) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Interpreter.cpp:835 03:39:40 INFO - #24 0x7fbdeba9cd3b in construct /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/proxy/DirectProxyHandler.cpp:85 03:39:40 INFO - #25 0x7fbdeba9cd3b in js::CrossCompartmentWrapper::construct(JSContext*, JS::Handle<JSObject*>, JS::CallArgs const&) const /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/proxy/CrossCompartmentWrapper.cpp:309 03:39:40 INFO - #26 0x7fbdebaadf70 in js::Proxy::construct(JSContext*, JS::Handle<JSObject*>, JS::CallArgs const&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/proxy/Proxy.cpp:410 03:39:40 INFO - #27 0x7fbdebab09c2 in js::proxy_Construct(JSContext*, unsigned int, JS::Value*) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/proxy/Proxy.cpp:706 03:39:40 INFO - #28 0x7fbdeae08c9e in CallJSNative /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/jscntxtinlines.h:235 03:39:40 INFO - #29 0x7fbdeae08c9e in CallJSNativeConstructor /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/jscntxtinlines.h:268 03:39:40 INFO - #30 0x7fbdeae08c9e in js::InvokeConstructor(JSContext*, JS::CallArgs) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Interpreter.cpp:816 03:39:40 INFO - #31 0x7fbdeadf7519 in Interpret(JSContext*, js::RunState&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Interpreter.cpp:2969 03:39:40 INFO - #32 0x7fbdeadd7914 in js::RunScript(JSContext*, js::RunState&) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Interpreter.cpp:661 03:39:40 INFO - #33 0x7fbdeadb8c90 in js::Invoke(JSContext*, JS::CallArgs, js::MaybeConstruct) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Interpreter.cpp:738 03:39:40 INFO - #34 0x7fbdeb9172f1 in js::CallOrConstructBoundFunction(JSContext*, unsigned int, JS::Value*) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/jsfun.cpp:1545 03:39:40 INFO - #35 0x7fbdeadb84c3 in CallJSNative /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/jscntxtinlines.h:235 03:39:40 INFO - #36 0x7fbdeadb84c3 in js::Invoke(JSContext*, JS::CallArgs, js::MaybeConstruct) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Interpreter.cpp:720 03:39:40 INFO - #37 0x7fbdead5bcf3 in js::Invoke(JSContext*, JS::Value const&, JS::Value const&, unsigned int, JS::Value const*, JS::MutableHandle<JS::Value>) /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/vm/Interpreter.cpp:775 03:39:40 INFO - #38 0x7fbdeba9bf7b in call /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/proxy/DirectProxyHandler.cpp:77 03:39:40 INFO - #39 0x7fbdeba9bf7b in js::CrossCompartmentWrapper::call(JSContext*, JS::Handle<JSObject*>, JS::CallArgs const&) const /builds/slave/try-l64-asan-00000000000000000/build/src/js/src/proxy/CrossCompartmentWrapper.cpp:289

Comment hidden (Legacy TBPL/Treeherder Robot)

Jon Coppeard (:jonco)

Comment 488

•

10 years ago

Unassigning as I'm not actively working on this.

Assignee: jcoppeard → nobody

Status: ASSIGNED → NEW

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Reporter

Comment 493

•

10 years ago

Terrence, any other ideas? This happens too frequently to just ignore. We either need a fix or suppression here if that's deemed too difficult.

Flags: needinfo?(terrence)

Comment hidden (Legacy TBPL/Treeherder Robot)

Terrence Cole [:terrence]

Comment 499

•

10 years ago

I'll take a look.

Comment hidden (Legacy TBPL/Treeherder Robot)

Terrence Cole [:terrence]

Comment 530

•

10 years ago

Attached patch wait_on_nursery_free_end-v0.diff (obsolete) — Details — Splinter Review

We fire off the nursery's background mallocBuffer freeing thread at the start of the shutdown GC. It looks like we only wait for it when we (1) do the next nursery sweep, (2) OOM, or (3) AutoFinishGC. I don't think we do any of these before shutting down, so we might just be killing this thread before it finishes its work.

Assignee: nobody → terrence

Status: NEW → ASSIGNED

Flags: needinfo?(terrence)

Attachment #8640101 - Flags: review?(jcoppeard)

Terrence Cole [:terrence]

Comment 531

•

10 years ago

Attached patch wait_on_nursery_free_end-v1.diff — Details — Splinter Review

me: Hey me, I don't need to bother giving this trivial 1 line patch even the most basic modicum of testing, do I? me: Of course not! what could go wrong? Turns out runtime init can fail, in which case the task will be nullptr. Whoops.

Attachment #8640101 - Attachment is obsolete: true

Attachment #8640101 - Flags: review?(jcoppeard)

Attachment #8640111 - Flags: review?(jcoppeard)

Comment hidden (Legacy TBPL/Treeherder Robot)

Jon Coppeard (:jonco)

Comment 545

•

10 years ago

Comment on attachment 8640111 [details] [diff] [review] wait_on_nursery_free_end-v1.diff Review of attachment 8640111 [details] [diff] [review]: ----------------------------------------------------------------- Ah, good find.

Attachment #8640111 - Flags: review?(jcoppeard) → review+

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Reporter

Updated

•

10 years ago

status-firefox40: --- → unaffected

status-firefox42: --- → affected

status-firefox-esr38: --- → unaffected

Keywords: leave-open

Comment hidden (Legacy TBPL/Treeherder Robot)

Pulsebot

Comment 555

•

10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/6eebf397a8c2

Comment hidden (Legacy TBPL/Treeherder Robot)

Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)

Comment 561

•

10 years ago

Backed out with the other patches in that push for various Spidermonkey and Cpp test failures: https://treeherder.mozilla.org/logviewer.html#?job_id=12273235&repo=mozilla-inbound https://treeherder.mozilla.org/logviewer.html#?job_id=12277586&repo=mozilla-inbound https://hg.mozilla.org/integration/mozilla-inbound/rev/cb821c634326

Flags: needinfo?(terrence)

Comment hidden (Legacy TBPL/Treeherder Robot)

Pulsebot

Comment 583

•

10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/54e85cce2cad

Comment hidden (Legacy TBPL/Treeherder Robot)

Carsten Book [:Tomcat]

Comment 595

•

10 years ago

https://hg.mozilla.org/mozilla-central/rev/54e85cce2cad

Status: ASSIGNED → RESOLVED

Closed: 10 years ago

status-firefox42: affected → fixed

Resolution: --- → FIXED

Target Milestone: --- → mozilla42

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Reporter

Comment 601

•

10 years ago

(In reply to Treeherder Robot from comment #598) (In reply to Treeherder Robot from comment #599) (In reply to Treeherder Robot from comment #600) On the m-c merge no less :(

Status: RESOLVED → REOPENED

status-firefox42: fixed → affected

Resolution: FIXED → ---

Target Milestone: mozilla42 → ---

Comment hidden (Legacy TBPL/Treeherder Robot)

Terrence Cole [:terrence]

Updated

•

10 years ago

Flags: needinfo?(terrence)

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=12631843 repository: mozilla-inbound start_time: 2015-08-07T08:50:54 who: rvandermeulen[at]mozilla[dot]com machine: tst-linux64-spot-750 buildname: Ubuntu ASAN VM 12.04 x64 mozilla-inbound opt test mochitest-e10s-browser-chrome-1 revision: b4fed4b50af5 SUMMARY: AddressSanitizer: 64 byte(s) leaked in 1 allocation(s). TEST-UNEXPECTED-FAIL | LeakSanitizer | leak at js_pod_malloc, AllocateObjectBuffer, js::NativeObject::growSlots, js::NativeObject::updateSlotsForSpan 135 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_chatwindow_resize.js | Test timed out - expected PASS TypeError: node.receiveMessage is undefined -- chrome://browser/content/browser.js:8999 172 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_marks.js | mark button is disabled - Got undefined, expected false 183 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_marks.js | button is disabled - 190 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_marks.js | Test timed out - expected PASS 192 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_marks.js | Found a tab after previous test timed out: https://test1.example.com/browser/browser/base/content/test/social/social_activate.html - expected PASS 196 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_multiworker.js | Exception thrown - at resource://gre/modules/SocialService.jsm:401 - Error: SocialService.addProvider: provider with this origin already exists 201 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_perwindowPB.js | Exception thrown - at resource://gre/modules/SocialService.jsm:401 - Error: SocialService.addProvider: provider with this origin already exists 206 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_sidebar.js | Exception thrown - at resource://gre/modules/SocialService.jsm:401 - Error: SocialService.addProvider: provider with this origin already exists 211 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_status.js | Exception thrown - at resource://gre/modules/SocialService.jsm:401 - Error: SocialService.addProvider: provider with this origin already exists 218 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | needs zero providers to start this test. - Got 2, expected 0 219 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | no providers are enabled - 222 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | providers are enabled - 223 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | context-marklinkMenu menu exists for each mark provider - Got 1, expected 0 224 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | context-markpageMenu menu exists for each mark provider - Got 1, expected 0 225 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | The Social UI tests succeeded. - Got 5, expected 7 228 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | social sidebar active? - Got true, expected false 229 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | Social:ToggleNotifications visible? - Got false, expected true 230 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | The Social UI tests succeeded. - Got 5, expected 7 232 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | social sidebar active? - Got true, expected false 233 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | Social:ToggleNotifications visible? - Got false, expected true 234 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | The Social UI tests succeeded. - Got 5, expected 7 239 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | sub-test testEnabledStartup failed: Error: SocialService.addProvider: provider with this origin already exists 246 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | sub-test testGlobalState failed: Error: SocialService.addProvider: provider with this origin already exists 251 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | sub-test testPerWindowSidebar failed: Error: SocialService.addProvider: provider with this origin already exists 255 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | all provider prefs uninstalled from previous test - Got 2, expected 0 256 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_window.js | all providers uninstalled from previous test 2 - Got 2, expected 0

Comment hidden (Legacy TBPL/Treeherder Robot)

Jan de Mooij [:jandem]

Assignee

Updated

•

10 years ago

Depends on: 1194627

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Reporter

Comment 951

•

10 years ago

Looks like the fix in bug 1194627 is holding up!

Assignee: terrence → jdemooij

Status: REOPENED → RESOLVED

Closed: 10 years ago → 10 years ago

status-firefox43: --- → fixed

Resolution: --- → FIXED

Target Milestone: --- → mozilla43

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Reporter

Comment 960

•

9 years ago

No instances on the release branches since bug 1194627 got uplifted.

status-firefox41: affected → fixed

status-firefox42: affected → fixed

bug1166041-nursery-leak 10 years ago Jon Coppeard (:jonco) 1.83 KB, patch	terrence : review+	Details \| Diff \| Splinter Review
bug1166041-leak-asserts 10 years ago Jon Coppeard (:jonco) 3.38 KB, patch	terrence : review+	Details \| Diff \| Splinter Review
wait_on_nursery_free_end-v0.diff 10 years ago Terrence Cole [:terrence] 916 bytes, patch		Details \| Diff \| Splinter Review
wait_on_nursery_free_end-v1.diff 10 years ago Terrence Cole [:terrence] 1.47 KB, patch	jonco : review+	Details \| Diff \| Splinter Review