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

Issue 663854 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 663557
Owner: ----
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

TSAN remoting_unittests races in SecurityKeyIpc{Client,Server}Test

Project Member Reported by thestig@chromium.org, Nov 9 2016

Issue description

Started at https://build.chromium.org/p/chromium.memory.full/builders/Linux%20TSan%20Tests/builds/3376 so likely r430557.

WARNING: ThreadSanitizer: lock-order-inversion (potential deadlock) (pid=6107)
  Cycle in lock order graph: M1289 (0x7b14000036c8) => M1282 (0x7b4800000718) => M1289

  Mutex M1282 acquired here while holding mutex M1289 in main thread:
    #0 pthread_mutex_lock <null> (remoting_unittests+0x00000049cd3e)
    #1 base::internal::LockImpl::Lock() base/synchronization/lock_impl_posix.cc:64:12 (remoting_unittests+0x0000008d2a25)
    #2 Acquire base/synchronization/lock.h:26:26 (remoting_unittests+0x0000012c4288)
    #3 AutoLock base/synchronization/lock.h:115 (remoting_unittests+0x0000012c4288)
    #4 IPC::(anonymous namespace)::ChannelAssociatedGroupController::Accept(mojo::Message*) ipc/ipc_mojo_bootstrap.cc:604 (remoting_unittests+0x0000012c4288)
    #5 non-virtual thunk to IPC::(anonymous namespace)::ChannelAssociatedGroupController::Accept(mojo::Message*) ipc/ipc_mojo_bootstrap.cc:595:8 (remoting_unittests+0x0000012c49a4)
    #6 mojo::FilterChain::Accept(mojo::Message*) mojo/public/cpp/bindings/lib/filter_chain.cc:40:17 (remoting_unittests+0x0000012ceabd)

  Mutex M1289 previously acquired by the same thread here:
    #0 pthread_mutex_lock <null> (remoting_unittests+0x00000049cd3e)
    #1 base::internal::LockImpl::Lock() base/synchronization/lock_impl_posix.cc:64:12 (remoting_unittests+0x0000008d2a25)
    #2 Acquire base/synchronization/lock.h:26:26 (remoting_unittests+0x0000018c8d11)
    #3 AutoLock base/synchronization/lock.h:115 (remoting_unittests+0x0000018c8d11)
    #4 mojo::edk::Watcher::MaybeInvokeCallback(unsigned int, mojo::edk::HandleSignalsState const&, unsigned int) mojo/edk/system/watcher.cc:20 (remoting_unittests+0x0000018c8d11)

  Mutex M1289 acquired here while holding mutex M1282 in main thread:
    #0 pthread_mutex_lock <null> (remoting_unittests+0x00000049cd3e)
    #1 base::internal::LockImpl::Lock() base/synchronization/lock_impl_posix.cc:64:12 (remoting_unittests+0x0000008d2a25)
    #2 Acquire base/synchronization/lock.h:26:26 (remoting_unittests+0x0000018c8d11)
    #3 AutoLock base/synchronization/lock.h:115 (remoting_unittests+0x0000018c8d11)
    #4 mojo::edk::Watcher::MaybeInvokeCallback(unsigned int, mojo::edk::HandleSignalsState const&, unsigned int) mojo/edk/system/watcher.cc:20 (remoting_unittests+0x0000018c8d11)
    #5 mojo::edk::RequestContext::~RequestContext() mojo/edk/system/request_context.cc:59:22 (remoting_unittests+0x0000018c1a08)
    #6 mojo::edk::Core::WriteMessageNew(unsigned int, unsigned long, unsigned int) mojo/edk/system/core.cc:702:1 (remoting_unittests+0x0000018a2c16)

  Mutex M1282 previously acquired by the same thread here:
    #0 pthread_mutex_lock <null> (remoting_unittests+0x00000049cd3e)
    #1 base::internal::LockImpl::Lock() base/synchronization/lock_impl_posix.cc:64:12 (remoting_unittests+0x0000008d2a25)
    #2 Acquire base/synchronization/lock.h:26:26 (remoting_unittests+0x0000012c3bb9)
    #3 AutoLock base/synchronization/lock.h:115 (remoting_unittests+0x0000012c3bb9)
    #4 IPC::(anonymous namespace)::ChannelAssociatedGroupController::CloseEndpointHandle(unsigned int, bool) ipc/ipc_mojo_bootstrap.cc:168 (remoting_unittests+0x0000012c3bb9)
    #5 reset mojo/public/cpp/bindings/lib/scoped_interface_endpoint_handle.cc:38:22 (remoting_unittests+0x0000012d358b)
    #6 mojo::ScopedInterfaceEndpointHandle::~ScopedInterfaceEndpointHandle() mojo/public/cpp/bindings/lib/scoped_interface_endpoint_handle.cc:23 (remoting_unittests+0x0000012d358b)

 
Cc: benwells@chromium.org joedow@chromium.org dcheng@chromium.org
Mergedinto: 663557
Status: Duplicate (was: Untriaged)
Treating this as a duplicate of  issue 663557  since it's essentially the same underlying issue.

Sign in to add a comment