New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 751730 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: ----



Sign in to add a comment

browser_tests failing on chromium.memory/Linux MSan Tests

Project Member Reported by loonyb...@chromium.org, Aug 2 2017

Issue description

browser_tests failing on chromium.memory/Linux MSan Tests

Builders failed on: 
- Linux MSan Tests: 
  https://build.chromium.org/p/chromium.memory/builders/Linux%20MSan%20Tests

Test seems flaky, with deleted value being used.

nyquist@, it seems like you wrote most part of DomDistillerTabUtilsBrowserTest.TestSwapWebContents

Could you please help take a look?

Thanks

 
Labels: Pri-1
Status: Assigned (was: Available)

Comment 3 by st...@chromium.org, Aug 2 2017

Labels: -Sheriff-Chromium
Removed Sheriff-Chromium label as an owner is assigned.
[ RUN      ] DomDistillerTabUtilsBrowserTest.TestSwapWebContents
Xlib:  extension "RANDR" missing on display ":99".
[26281:26281:0807/025210.905134:WARNING:password_store_factory.cc(241)] Using basic (unencrypted) store for password storage. See https://chromium.googlesource.com/chromium/src/+/master/docs/linux_password_storage.md for more information about password storage options.
(browser_tests:26281): LIBDBUSMENU-GLIB-WARNING **: Unable to get session bus: Unknown or unsupported transport 'disabled' for address 'disabled:'
[26281:26358:0807/025211.530078:WARNING:embedded_test_server.cc(219)] Request not handled. Returning 404: /favicon.ico
[26281:26281:0807/025212.128766:INFO:CONSOLE(140)] "DomDistiller debug level: 0", source:  (140)
==26281==WARNING: MemorySanitizer: use-of-uninitialized-value
    #0 0xc988f45 in __lower_bound<const void *> buildtools/third_party/libc++/trunk/include/__tree:2499:5
    #1 0xc988f45 in find<const void *> buildtools/third_party/libc++/trunk/include/__tree:2421:0
    #2 0xc988f45 in find buildtools/third_party/libc++/trunk/include/map:1212:0
    #3 0xc988f45 in base::SupportsUserData::GetUserData(void const*) const base/supports_user_data.cc:17:0
    #4 0x5f2ab2f in resource_coordinator::mojom::TabSignalObserverStubDispatch::Accept(resource_coordinator::mojom::TabSignalObserver*, mojo::Message*) ./out/Release/gen/services/resource_coordinator/public/interfaces/tab_signal.mojom.cc:106:13
    #5 0xd6dd3a6 in resource_coordinator::mojom::TabSignalObserverStub<mojo::RawPtrImplRefTraits<resource_coordinator::mojom::TabSignalObserver> >::Accept(mojo::Message*) ./out/Release/gen/services/resource_coordinator/public/interfaces/tab_signal.mojom.h:178:12
    #6 0x1219dac7 in mojo::InterfaceEndpointClient::HandleValidatedMessage(mojo::Message*) mojo/public/cpp/bindings/lib/interface_endpoint_client.cc:420:32
    #7 0x121ecce6 in mojo::FilterChain::Accept(mojo::Message*) mojo/public/cpp/bindings/lib/filter_chain.cc:40:17
    #8 0x121bdbfa in mojo::internal::MultiplexRouter::ProcessIncomingMessage(mojo::Message*, mojo::internal::MultiplexRouter::ClientCallBehavior, base::SequencedTaskRunner*) mojo/public/cpp/bindings/lib/multiplex_router.cc:886:22
    #9 0x121bc188 in mojo::internal::MultiplexRouter::Accept(mojo::Message*) mojo/public/cpp/bindings/lib/multiplex_router.cc:613:25
    #10 0x121ecce6 in mojo::FilterChain::Accept(mojo::Message*) mojo/public/cpp/bindings/lib/filter_chain.cc:40:17
    #11 0x12199ceb in mojo::Connector::ReadSingleMessage(unsigned int*) mojo/public/cpp/bindings/lib/connector.cc:440:51
    #12 0x1219c2f6 in mojo::Connector::ReadAllAvailableMessages() mojo/public/cpp/bindings/lib/connector.cc:469:10
    #13 0x5e7665e in Run base/callback.h:80:12
    #14 0x5e7665e in mojo::SimpleWatcher::DiscardReadyState(base::Callback<void (unsigned int), (base::internal::CopyMode)1, (base::internal::RepeatMode)1> const&, unsigned int, mojo::HandleSignalsState const&) mojo/public/cpp/system/simple_watcher.h:193:0
    #15 0x121fa7fd in Run base/callback.h:80:12
    #16 0x121fa7fd in mojo::SimpleWatcher::OnHandleReady(int, unsigned int, mojo::HandleSignalsState const&) mojo/public/cpp/system/simple_watcher.cc:276:0
    #17 0xc7c5580 in Run base/callback.h:91:12
    #18 0xc7c5580 in base::debug::TaskAnnotator::RunTask(char const*, base::PendingTask*) base/debug/task_annotator.cc:59:0
    #19 0xc865d18 in base::MessageLoop::RunTask(base::PendingTask*) base/message_loop/message_loop.cc:410:19
    #20 0xc869083 in base::MessageLoop::DeferOrRunPendingTask(base::PendingTask) base/message_loop/message_loop.cc:421:5
    #21 0xc86a0a2 in base::MessageLoop::DoWork() base/message_loop/message_loop.cc:528:13
    #22 0xc877b7b in base::MessagePumpGlib::Run(base::MessagePump::Delegate*) base/message_loop/message_pump_glib.cc:313:49
    #23 0xc92ca1b in base::RunLoop::Run() base/run_loop.cc:123:14
    #24 0x101c9dc in dom_distiller::DomDistillerTabUtilsBrowserTest_TestSwapWebContents_Test::RunTestOnMainThread() chrome/browser/dom_distiller/tab_utils_browsertest.cc:116:25
    #25 0xe37160a in content::BrowserTestBase::ProxyRunTestOnMainThreadLoop() content/public/test/browser_test_base.cc:315:5
    #26 0xddbdf48 in Run base/callback.h:80:12
    #27 0xddbdf48 in ChromeBrowserMainParts::PreMainMessageLoopRunImpl() chrome/browser/chrome_browser_main.cc:1857:0
    #28 0xddb9419 in ChromeBrowserMainParts::PreMainMessageLoopRun() chrome/browser/chrome_browser_main.cc:1210:18
    #29 0x6a86a19 in content::BrowserMainLoop::PreMainMessageLoopRun() content/browser/browser_main_loop.cc:1151:13
    #30 0x7ddce5d in Run base/callback.h:80:12
    #31 0x7ddce5d in content::StartupTaskRunner::RunAllTasksNow() content/browser/startup_task_runner.cc:45:0
    #32 0x6a7e637 in content::BrowserMainLoop::CreateStartupTasks() content/browser/browser_main_loop.cc:935:25
    #33 0x6a945d1 in content::BrowserMainRunnerImpl::Initialize(content::MainFunctionParams const&) content/browser/browser_main_runner.cc:127:17
    #34 0x6a716b9 in content::BrowserMain(content::MainFunctionParams const&) content/browser/browser_main.cc:42:32
    #35 0xc738b9b in content::RunNamedProcessTypeMain(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, content::MainFunctionParams const&, content::ContentMainDelegate*) content/app/content_main_runner.cc:408:14
    #36 0xc73b8e6 in content::ContentMainRunnerImpl::Run() content/app/content_main_runner.cc:690:12
    #37 0x147f6fba in service_manager::Main(service_manager::MainParams const&) services/service_manager/embedder/main.cc:469:29
    #38 0xc7352be in content::ContentMain(content::ContentMainParams const&) content/app/content_main.cc:19:10
    #39 0xe37012e in content::BrowserTestBase::SetUp() content/public/test/browser_test_base.cc:271:3
    #40 0xcba8c4b in InProcessBrowserTest::SetUp() chrome/test/base/in_process_browser_test.cc:271:20
    #41 0x4e99b87 in HandleExceptionsInMethodIfSupported<testing::Test, void> third_party/googletest/src/googletest/src/gtest.cc:2455:12
    #42 0x4e99b87 in testing::Test::Run() third_party/googletest/src/googletest/src/gtest.cc:2467:0
    #43 0x4e9d790 in testing::TestInfo::Run() third_party/googletest/src/googletest/src/gtest.cc:2653:11
    #44 0x4e9f169 in testing::TestCase::Run() third_party/googletest/src/googletest/src/gtest.cc:2771:28
    #45 0x4ec0803 in testing::internal::UnitTestImpl::RunAllTests() third_party/googletest/src/googletest/src/gtest.cc:4648:43
    #46 0x4ebf67a in HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool> third_party/googletest/src/googletest/src/gtest.cc:2455:12
    #47 0x4ebf67a in testing::UnitTest::Run() third_party/googletest/src/googletest/src/gtest.cc:4256:0
    #48 0xcc09958 in RUN_ALL_TESTS third_party/googletest/src/googletest/include/gtest/gtest.h:2237:46
    #49 0xcc09958 in base::TestSuite::Run() base/test/test_suite.cc:270:0
    #50 0xc784771 in ChromeTestSuiteRunner::RunTestSuite(int, char**) chrome/test/base/chrome_test_launcher.cc:68:38
    #51 0xe4c7e35 in content::LaunchTests(content::TestLauncherDelegate*, unsigned long, int, char**) content/public/test/test_launcher.cc:520:31
    #52 0xc7844c6 in main chrome/test/base/browser_tests_main.cc:21:10
    #53 0x7f991993bf44 in __libc_start_main /build/eglibc-MjiXCM/eglibc-2.19/csu/libc-start.c:287:0
    #54 0x840e5f in _start ??:0:0
  Uninitialized value was created by a heap deallocation
    #0 0x8acff9 in operator delete(void*) ??:0:0
    #1 0xe0d8bcf in ~SourcePageHandleWebContents components/dom_distiller/content/browser/distiller_page_web_contents.cc:39:5
    #2 0xe0d8bcf in dom_distiller::SourcePageHandleWebContents::~SourcePageHandleWebContents() components/dom_distiller/content/browser/distiller_page_web_contents.cc:37:0
    #3 0xe0d982e in operator() buildtools/third_party/libc++/trunk/include/memory:2272:5
    #4 0xe0d982e in reset buildtools/third_party/libc++/trunk/include/memory:2585:0
    #5 0xe0d982e in ~unique_ptr buildtools/third_party/libc++/trunk/include/memory:2539:0
    #6 0xe0d982e in dom_distiller::DistillerPageWebContents::~DistillerPageWebContents() components/dom_distiller/content/browser/distiller_page_web_contents.cc:81:0
    #7 0xe0d999c in dom_distiller::DistillerPageWebContents::~DistillerPageWebContents() components/dom_distiller/content/browser/distiller_page_web_contents.cc:80:55
    #8 0x149c9a9c in dom_distiller::DistillerImpl::~DistillerImpl() components/dom_distiller/core/distiller.cc:65:33
    #9 0xc7c5580 in Run base/callback.h:91:12
    #10 0xc7c5580 in base::debug::TaskAnnotator::RunTask(char const*, base::PendingTask*) base/debug/task_annotator.cc:59:0
    #11 0xc865d18 in base::MessageLoop::RunTask(base::PendingTask*) base/message_loop/message_loop.cc:410:19
    #12 0xc869083 in base::MessageLoop::DeferOrRunPendingTask(base::PendingTask) base/message_loop/message_loop.cc:421:5
    #13 0xc86a0a2 in base::MessageLoop::DoWork() base/message_loop/message_loop.cc:528:13
    #14 0xc878869 in HandleDispatch base/message_loop/message_pump_glib.cc:267:25
    #15 0xc878869 in base::(anonymous namespace)::WorkSourceDispatch(_GSource*, int (*)(void*), void*) base/message_loop/message_pump_glib.cc:109:0
    #16 0x7f9918244fce in g_main_dispatch /home/tom/chromium/src/third_party/instrumented_libraries/scripts/out/Instrumented-msan-chained-origins-trusty/gen/third_party/instrumented_libraries/libglib2.0-0/glib2.0-2.40.2/glib/gmain.c:3064:27
    #17 0x7f9918244fce in g_main_context_dispatch /home/tom/chromium/src/third_party/instrumented_libraries/scripts/out/Instrumented-msan-chained-origins-trusty/gen/third_party/instrumented_libraries/libglib2.0-0/glib2.0-2.40.2/glib/gmain.c:3663:0
    #18 0x7f991824625c in g_main_context_iterate /home/tom/chromium/src/third_party/instrumented_libraries/scripts/out/Instrumented-msan-chained-origins-trusty/gen/third_party/instrumented_libraries/libglib2.0-0/glib2.0-2.40.2/glib/gmain.c:3734:5
    #19 0x7f99182465ed in g_main_context_iteration /home/tom/chromium/src/third_party/instrumented_libraries/scripts/out/Instrumented-msan-chained-origins-trusty/gen/third_party/instrumented_libraries/libglib2.0-0/glib2.0-2.40.2/glib/gmain.c:3795:12
    #20 0xc877e03 in base::MessagePumpGlib::Run(base::MessagePump::Delegate*) base/message_loop/message_pump_glib.cc:309:30
    #21 0xc92ca1b in base::RunLoop::Run() base/run_loop.cc:123:14
    #22 0x101c9dc in dom_distiller::DomDistillerTabUtilsBrowserTest_TestSwapWebContents_Test::RunTestOnMainThread() chrome/browser/dom_distiller/tab_utils_browsertest.cc:116:25
    #23 0xe37160a in content::BrowserTestBase::ProxyRunTestOnMainThreadLoop() content/public/test/browser_test_base.cc:315:5
    #24 0xddbdf48 in Run base/callback.h:80:12
    #25 0xddbdf48 in ChromeBrowserMainParts::PreMainMessageLoopRunImpl() chrome/browser/chrome_browser_main.cc:1857:0
    #26 0xddb9419 in ChromeBrowserMainParts::PreMainMessageLoopRun() chrome/browser/chrome_browser_main.cc:1210:18
    #27 0x6a86a19 in content::BrowserMainLoop::PreMainMessageLoopRun() content/browser/browser_main_loop.cc:1151:13
SUMMARY: MemorySanitizer: use-of-uninitialized-value (/b/s/w/ir/out/Release/browser_tests+0xc988f45)
Exiting
Project Member

Comment 5 by bugdroid1@chromium.org, Aug 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/067d2c7f8fc65efd7debca8673fc11225cce9430

commit 067d2c7f8fc65efd7debca8673fc11225cce9430
Author: Vasilii Sukhanov <vasilii@chromium.org>
Date: Mon Aug 07 11:11:07 2017

Disable DomDistillerTabUtilsBrowserTest.TestSwapWebContents on Linux

TBR=nyquist@chromium.org

Bug: 751730
Change-Id: Ic0cda4bca3dbfbe78a96816710001e8754581f37
Reviewed-on: https://chromium-review.googlesource.com/603310
Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#492297}
[modify] https://crrev.com/067d2c7f8fc65efd7debca8673fc11225cce9430/chrome/browser/dom_distiller/tab_utils_browsertest.cc

Cc: mdjones@chromium.org nyquist@chromium.org
Owner: wychen@chromium.org
wychen: Would you be able to take a look at this?

Sign in to add a comment