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

Issue 836427 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Last visit > 30 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

DCHECK() hit in BluetoothAdapterBlueZ::RemoveDiscoverySession()

Project Member Reported by khorimoto@chromium.org, Apr 24 2018

Issue description

Eve on ToT. DCHECK() is hit upon login.

Thread 1 "chrome" received signal SIGABRT, Aborted.
#0  0x00007d6492c59dd2 in __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:54
#1  0x00007d6492c5bbf6 in __GI_abort () at abort.c:89
#2  0x00005cf805edb095 in base::debug::BreakDebugger() () at ../../base/debug/debugger_posix.cc:258
#3  0x00005cf805e4ecb4 in logging::LogMessage::~LogMessage() () at ../../base/logging.cc:855
#4  0x00005cf807647e9d in bluez::BluetoothAdapterBlueZ::RemoveDiscoverySession(device::BluetoothDiscoveryFilter*, base::RepeatingCallback<void ()> const&, base::OnceCallback<void (device::UMABluetoothDiscoverySessionOutcome)>) () at ../../device/bluetooth/bluez/bluetooth_adapter_bluez.cc:1449
#5  0x00005cf807637a25 in device::BluetoothDiscoverySession::Stop(base::RepeatingCallback<void ()> const&, base::RepeatingCallback<void ()> const&) () at ../../device/bluetooth/bluetooth_discovery_session.cc:70
#6  0x00005cf8076374bf in device::BluetoothDiscoverySession::~BluetoothDiscoverySession() () at ../../device/bluetooth/bluetooth_discovery_session.cc:29
#7  0x00005cf80763765e in device::BluetoothDiscoverySession::~BluetoothDiscoverySession() () at ../../device/bluetooth/bluetooth_discovery_session.cc:27
#8  0x00005cf8048c92dc in arc::ArcBluetoothBridge::OnDiscoveryStarted(std::__1::unique_ptr<device::BluetoothDiscoverySession, std::__1::default_delete<device::BluetoothDiscoverySession> >) ()
    at /usr/local/google/src/chromium2/.cros_cache/chrome-sdk/tarballs/eve+10612.0.0+target_toolchain/usr/bin/../include/c++/v1/memory:2399
#9  0x00005cf803a1933f in void base::internal::Invoker<base::internal::BindState<void (cryptauth::weave::BluetoothLowEnergyWeaveClientConnection::*)(std::__1::unique_ptr<device::BluetoothGattNotifySession, std::__1::default_delete<device::BluetoothGattNotifySession> >), base::WeakPtr<cryptauth::weave::BluetoothLowEnergyWeaveClientConnection> >, void (std::__1::unique_ptr<device::BluetoothGattNotifySession, std::__1::default_delete<device::BluetoothGattNotifySession> >)>::RunImpl<void (cryptauth::weave::BluetoothLowEnergyWeaveClientConnection::* const&)(std::__1::unique_ptr<device::BluetoothGattNotifySession, std::__1::default_delete<device::BluetoothGattNotifySession> >), std::__1::tuple<base::WeakPtr<cryptauth::weave::BluetoothLowEnergyWeaveClientConnection> > const&, 0ul>(void (cryptauth::weave::BluetoothLowEnergyWeaveClientConnection::* const&)(std::__1::unique_ptr<device::BluetoothGattNotifySession, std::__1::default_delete<device::BluetoothGattNotifySession> >), std::__1::tuple<base::WeakPtr<cryptauth::weave::BluetoothLowEnergyWeaveClientConnection> > const&, std::__1::integer_sequence<unsigned long, 0ul>, std::__1::unique_ptr<device::BluetoothGattNotifySession, std::__1::default_delete<device::BluetoothGattNotifySession> >&&) ()
    at ../../base/bind_internal.h:447
#10 0x00005cf80762f7a1 in device::BluetoothAdapter::OnStartDiscoverySession(std::__1::unique_ptr<device::BluetoothDiscoveryFilter, std::__1::default_delete<device::BluetoothDiscoveryFilter> >, base::RepeatingCallback<void (std::__1::unique_ptr<device::BluetoothDiscoverySession, std::__1::default_delete<device::BluetoothDiscoverySession> >)> const&) () at ../../base/callback.h:125
#11 0x00005cf807632498 in void base::internal::InvokeHelper<true, void>::MakeItSo<void (device::BluetoothAdapter::* const&)(std::__1::unique_ptr<device::BluetoothDiscoveryFilter, std::__1::default_delete<device::BluetoothDiscoveryFilter> >, base::RepeatingCallback<void (std::__1::unique_ptr<device::BluetoothDiscoverySession, std::__1::default_delete<device::BluetoothDiscoverySession> >)> const&), base::WeakPtr<device::BluetoothAdapter> const&, std::__1::unique_ptr<device::BluetoothDiscoveryFilter, std::__1::default_delete<device::BluetoothDiscoveryFilter> >, base::RepeatingCallback<void (std::__1::unique_ptr<device::BluetoothDiscoverySession, std::__1::default_delete<device::BluetoothDiscoverySession> >)> const&>(void (device::BluetoothAdapter::* const&)(std::__1::unique_ptr<device::BluetoothDiscoveryFilter, std::__1::default_delete<device::BluetoothDiscoveryFilter> >, base::RepeatingCallback<void (std::__1::unique_ptr<device::BluetoothDiscoverySession, std::__1::default_delete<device::BluetoothDiscoverySession> >)> const&), base::WeakPtr<device::BluetoothAdapter> const&, std::__1::unique_ptr<device::BluetoothDiscoveryFilter, std::__1::default_delete<device::BluetoothDiscoveryFilter> >&&, base::RepeatingCallback<void (std::__1::unique_ptr<device::BluetoothDiscoverySession, std::__1::default_delete<device::BluetoothDiscoverySession> >)> const&) () at ../../base/bind_internal.h:447
#12 0x00005cf807632365 in base::internal::Invoker<base::internal::BindState<void (device::BluetoothAdapter::*)(std::__1::unique_ptr<device::BluetoothDiscoveryFilter, std::__1::default_delete<device::BluetoothDiscoveryFilter> >, base::RepeatingCallback<void (std::__1::unique_ptr<device::BluetoothDiscoverySession, std::__1::default_delete<device::BluetoothDiscoverySession> >)> const&), base::WeakPtr<device::BluetoothAdapter>, base::internal::PassedWrapper<std::__1::unique_ptr<device::BluetoothDiscoveryFilter, std::__1::default_delete<device::BluetoothDiscoveryFilter> > >, base::RepeatingCallback<void (std::__1::unique_ptr<device::BluetoothDiscoverySession, std::__1::default_delete<device::BluetoothDiscoverySession> >)> >, void ()>::Run(base::internal::BindStateBase*) () at ../../base/bind_internal.h:604
#13 0x00005cf807648a5c in bluez::BluetoothAdapterBlueZ::SetDiscoveryFilter(std::__1::unique_ptr<device::BluetoothDiscoveryFilter, std::__1::default_delete<device::BluetoothDiscoveryFilter> >, base::RepeatingCallback<void ()> const&, base::OnceCallback<void (device::UMABluetoothDiscoverySessionOutcome)>) ()
    at ../../base/callback.h:125
#14 0x00005cf807647050 in bluez::BluetoothAdapterBlueZ::AddDiscoverySession(device::BluetoothDiscoveryFilter*, base::RepeatingCallback<void ()> const&, base::OnceCallback<void (device::UMABluetoothDiscoverySessionOutcome)>) () at ../../device/bluetooth/bluez/bluetooth_adapter_bluez.cc:1392
#15 0x00005cf807649321 in bluez::BluetoothAdapterBlueZ::ProcessQueuedDiscoveryRequests() () at ../../device/bluetooth/bluez/bluetooth_adapter_bluez.cc:1729
#16 0x00005cf807647b23 in bluez::BluetoothAdapterBlueZ::OnStartDiscovery(base::RepeatingCallback<void ()> const&, base::OnceCallback<void (device::UMABluetoothDiscoverySessionOutcome)>) () at ../../device/bluetooth/bluez/bluetooth_adapter_bluez.cc:1579
#17 0x00005cf80764aff0 in void base::internal::FunctorTraits<void (bluez::BluetoothAdapterBlueZ::*)(base::RepeatingCallback<void ()> const&, base::OnceCallback<void (device::UMABluetoothDiscoverySessionOutcome)>), void>::Invoke<base::WeakPtr<bluez::BluetoothAdapterBlueZ> const&, base::RepeatingCallback<void ()> const&, base::RepeatingCallback<void (device::UMABluetoothDiscoverySessionOutcome)> const&>(void (bluez::BluetoothAdapterBlueZ::*)(base::RepeatingCallback<void ()> const&, base::OnceCallback<void (device::UMABluetoothDiscoverySessionOutcome)>), base::WeakPtr<bluez::BluetoothAdapterBlueZ> const&, base::RepeatingCallback<void ()> const&, base::RepeatingCallback<void (device::UMABluetoothDiscoverySessionOutcome)> const&) () at ../../base/bind_internal.h:447
#18 0x00005cf807663ca1 in bluez::BluetoothAdapterClientImpl::OnSuccess(base::RepeatingCallback<void ()> const&, dbus::Response*) () at ../../base/callback.h:125
#19 0x00005cf803d1ffc4 in base::internal::Invoker<base::internal::BindState<void (ntp_tiles::IconCacherImpl::*)(GURL const&, favicon_base::LargeIconResult const&), base::WeakPtr<ntp_tiles::IconCacherImpl>, GURL>, void (favicon_base::LargeIconResult const&)>::Run(base::internal::BindStateBase*, favicon_base::LargeIconResult const&) () at ../../base/bind_internal.h:447
#20 0x00005cf806bb74fb in base::internal::Invoker<base::internal::BindState<dbus::ObjectProxy::CallMethodWithErrorCallback(dbus::MethodCall*, int, base::OnceCallback<void (dbus::Response*)>, base::OnceCallback<void (dbus::ErrorResponse*)>)::$_0, base::OnceCallback<void (dbus::Response*)>, base::OnceCallback<void (dbus::ErrorResponse*)> >, void (dbus::Response*, dbus::ErrorResponse*)>::RunOnce(base::internal::BindStateBase*, dbus::Response*, dbus::ErrorResponse*) () at ../../base/callback.h:96
#21 0x00005cf806bb3294 in dbus::ObjectProxy::RunResponseOrErrorCallback(dbus::ObjectProxy::ReplyCallbackHolder, base::TimeTicks, dbus::Response*, dbus::ErrorResponse*) () at ../../base/callback.h:96
#22 0x00005cf806bb7f68 in void base::internal::FunctorTraits<void (dbus::ObjectProxy::*)(dbus::ObjectProxy::ReplyCallbackHolder, base::TimeTicks, dbus::Response*, dbus::ErrorResponse*), void>::Invoke<scoped_refptr<dbus::ObjectProxy>, dbus::ObjectProxy::ReplyCallbackHolder, base::TimeTicks, dbus::Response*, dbus::ErrorResponse*>(void (dbus::ObjectProxy::*)(dbus::ObjectProxy::ReplyCallbackHolder, base::TimeTicks, dbus::Response*, dbus::ErrorResponse*), scoped_refptr<dbus::ObjectProxy>&&, dbus::ObjectProxy::ReplyCallbackHolder&&, base::TimeTicks&&, dbus::Response*&&, dbus::ErrorResponse*&&) () at ../../base/bind_internal.h:447
#23 0x00005cf806bb7e70 in base::internal::Invoker<base::internal::BindState<void (dbus::ObjectProxy::*)(dbus::ObjectProxy::ReplyCallbackHolder, base::TimeTicks, dbus::Response*, dbus::ErrorResponse*), scoped_refptr<dbus::ObjectProxy>, dbus::ObjectProxy::ReplyCallbackHolder, base::TimeTicks, dbus::Response*, dbus::ErrorResponse*>, void ()>::RunOnce(base::internal::BindStateBase*) () at ../../base/bind_internal.h:530
#24 0x00005cf805ea85e0 in base::(anonymous namespace)::PostTaskAndReplyRelay::RunTaskAndPostReply(base::(anonymous namespace)::PostTaskAndReplyRelay) () at ../../base/callback.h:96
#25 0x00005cf805ea8a86 in base::internal::Invoker<base::internal::BindState<void (*)(base::(anonymous namespace)::PostTaskAndReplyRelay), base::(anonymous namespace)::PostTaskAndReplyRelay>, void ()>::RunOnce(base::internal::BindStateBase*) () at ../../base/bind_internal.h:402
#26 0x00005cf805e37ace in base::debug::TaskAnnotator::RunTask(char const*, base::PendingTask*) () at ../../base/callback.h:96
#27 0x00005cf805f02726 in base::internal::IncomingTaskQueue::RunTask(base::PendingTask*) () at ../../base/message_loop/incoming_task_queue.cc:124
#28 0x00005cf805e56986 in base::MessageLoop::RunTask(base::PendingTask*) () at ../../base/message_loop/message_loop.cc:319
#29 0x00005cf805e56d97 in base::MessageLoop::DeferOrRunPendingTask(base::PendingTask) () at ../../base/message_loop/message_loop.cc:329
#30 0x00005cf805e57018 in base::MessageLoop::DoWork() () at ../../base/message_loop/message_loop.cc:373
#31 0x00005cf805ef0009 in base::MessagePumpLibevent::Run(base::MessagePump::Delegate*) () at ../../base/message_loop/message_pump_libevent.cc:210
#32 0x00005cf805e562b1 in base::MessageLoop::Run(bool) () at ../../base/message_loop/message_loop.cc:271
#33 0x00005cf805e81726 in base::RunLoop::Run() () at ../../base/run_loop.cc:130
#34 0x00005cf805a702d4 in ChromeBrowserMainParts::MainMessageLoopRun(int*) () at ../../chrome/browser/chrome_browser_main.cc:2122
#35 0x00005cf803bff694 in content::BrowserMainLoop::RunMainMessageLoopParts() () at ../../content/browser/browser_main_loop.cc:975
#36 0x00005cf803c023b3 in content::BrowserMainRunnerImpl::Run() () at ../../content/browser/browser_main_runner.cc:161
#37 0x00005cf803bfb55d in content::BrowserMain(content::MainFunctionParams const&) () at ../../content/browser/browser_main.cc:46
#38 0x00005cf805a55b94 in content::RunNamedProcessTypeMain(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, content::MainFunctionParams const&, content::ContentMainDelegate*) () at ../../content/app/content_main_runner.cc:633
#39 0x00005cf805a56e83 in content::ContentMainRunnerImpl::Run() () at ../../content/app/content_main_runner.cc:922
#40 0x00005cf805a63504 in service_manager::Main(service_manager::MainParams const&) () at ../../services/service_manager/embedder/main.cc:452
#41 0x00005cf805a54781 in content::ContentMain(content::ContentMainParams const&) () at ../../content/app/content_main.cc:19
#42 0x00005cf802dd5cbf in ChromeMain () at ../../chrome/app/chrome_main.cc:101
#43 0x00007d6492c46736 in __libc_start_main (main=0x5cf802dd5c20 <main>, argc=36, argv=0x7ffd86933e68, init=<optimized out>, fini=<optimized out>, rtld_fini=<optimized out>, stack_end=0x7ffd86933e58) at ../csu/libc-start.c:289
#44 0x00005cf802dd5af9 in _start ()

 

Comment 1 by r...@chromium.org, Apr 24 2018

Components: IO>Bluetooth
Labels: M-68
Owner: xiaoyinh@chromium.org
Status: Assigned (was: Untriaged)
Owner: sonnysasaka@chromium.org
Assigning to me as this is related to a change I made.
Project Member

Comment 3 by bugdroid1@chromium.org, Apr 28 2018

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

commit 50e31da7dd033916b4c9be6d1c326b7648c387e7
Author: Sonny Sasaka <sonnysasaka@chromium.org>
Date: Sat Apr 28 02:10:59 2018

device/bluetooth: Remove DCHECK(IsDiscovering()) until BlueZ is fixed.

BlueZ bug makes it wrong to assume that the Discovering property is
updated when StartDiscovery/StopDiscovery returns (crbug.com/822104).
BlueZ bug fixing is deprioritized so here we remove the DCHECK first
until BlueZ bug is fixed.

This is a left-over from https://crrev.com/c/963845.

BUG=814595,836427
TEST=No longer crashes

Change-Id: I81167db3c3d31225279798e37bede79c56037622
Reviewed-on: https://chromium-review.googlesource.com/1033455
Reviewed-by: Miao-chen Chou <mcchou@chromium.org>
Commit-Queue: Sonny Sasaka <sonnysasaka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554610}
[modify] https://crrev.com/50e31da7dd033916b4c9be6d1c326b7648c387e7/device/bluetooth/bluez/bluetooth_adapter_bluez.cc

Sign in to add a comment