Disabling Mobile Data during an active Tether connection may cause crash |
|||||||||||||
Issue descriptionOS: Chrome What steps will reproduce the problem? (1) Enable Mobile Data and Bluetooth. (2) Connect to Tether network. (3) Disable Mobile Data via the toggle in the System Tray. What is the expected result? Bluetooth and Mobile Data will be disabled, active Tether connection will be disconnected. What happens instead? Crash.
,
Sep 6 2017
The failure of the first attempt is due to an unrelated issue, which has been filed at crbug.com/762345 .
,
Sep 8 2017
Leslie, were you using any debug code when you got this stack trace?
,
Sep 8 2017
No, as far as I know.
,
Sep 12 2017
,
Sep 12 2017
,
Sep 12 2017
More generally, this crash occurs when Mobile Data is disabled while a Tether network is connected. I've updated the summary and description to reflect this new information.
,
Sep 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7a98f47e043a5581eb610b58b047a23ba207a145 commit 7a98f47e043a5581eb610b58b047a23ba207a145 Author: Kyle Horimoto <khorimoto@google.com> Date: Wed Sep 13 17:28:24 2017 [CrOS Tether] Fix a crash during shutdown. The crash was caused because NetworkConfigurationRemover was destroyed before WifiHotspotDisconnector. Side note: there is a bug filed ( crbug.com/761532 ) to clean this up. It does not scale, and it is prone to errors like this crash. Bug: 762318 , 672263 Change-Id: Ie91976af96fde96dcd4bdf0902b26a8c6fae7300 Reviewed-on: https://chromium-review.googlesource.com/664481 Reviewed-by: Ryan Hansberry <hansberry@chromium.org> Commit-Queue: Ryan Hansberry <hansberry@chromium.org> Cr-Commit-Position: refs/heads/master@{#501676} [modify] https://crrev.com/7a98f47e043a5581eb610b58b047a23ba207a145/chromeos/components/tether/initializer_impl.cc
,
Sep 13 2017
,
Sep 13 2017
This bug requires manual review: Request affecting a post-stable build Please contact the milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), ketakid@(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Sep 13 2017
Consider this merge approved for 62, pending the CL making it past the Chrome PFQ on ToT.
,
Sep 13 2017
Approving merge to M61 and M62.
,
Sep 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/14103a0e23a2e5d54ecd0ee1f8b0c472ba85a691 commit 14103a0e23a2e5d54ecd0ee1f8b0c472ba85a691 Author: Kyle Horimoto <khorimoto@google.com> Date: Wed Sep 13 19:03:30 2017 [CrOS Tether] Fix a crash during shutdown. The crash was caused because NetworkConfigurationRemover was destroyed before WifiHotspotDisconnector. Side note: there is a bug filed ( crbug.com/761532 ) to clean this up. It does not scale, and it is prone to errors like this crash. TBR=khorimoto@google.com (cherry picked from commit 7a98f47e043a5581eb610b58b047a23ba207a145) Bug: 762318 , 672263 Change-Id: Ie91976af96fde96dcd4bdf0902b26a8c6fae7300 Reviewed-on: https://chromium-review.googlesource.com/664481 Reviewed-by: Ryan Hansberry <hansberry@chromium.org> Commit-Queue: Ryan Hansberry <hansberry@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#501676} Reviewed-on: https://chromium-review.googlesource.com/665598 Reviewed-by: Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/branch-heads/3202@{#207} Cr-Branched-From: fa6a5d87adff761bc16afc5498c3f5944c1daa68-refs/heads/master@{#499098} [modify] https://crrev.com/14103a0e23a2e5d54ecd0ee1f8b0c472ba85a691/chromeos/components/tether/initializer_impl.cc
,
Sep 13 2017
,
Sep 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9954055c59d6462c34cf3ed0850ed343e6964042 commit 9954055c59d6462c34cf3ed0850ed343e6964042 Author: Kyle Horimoto <khorimoto@google.com> Date: Wed Sep 13 19:06:36 2017 [CrOS Tether] Fix a crash during shutdown. The crash was caused because NetworkConfigurationRemover was destroyed before WifiHotspotDisconnector. Side note: there is a bug filed ( crbug.com/761532 ) to clean this up. It does not scale, and it is prone to errors like this crash. TBR=khorimoto@google.com (cherry picked from commit 7a98f47e043a5581eb610b58b047a23ba207a145) Bug: 762318 , 672263 Change-Id: Ie91976af96fde96dcd4bdf0902b26a8c6fae7300 Reviewed-on: https://chromium-review.googlesource.com/664481 Reviewed-by: Ryan Hansberry <hansberry@chromium.org> Commit-Queue: Ryan Hansberry <hansberry@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#501676} Reviewed-on: https://chromium-review.googlesource.com/665365 Reviewed-by: Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/branch-heads/3163@{#1188} Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528} [modify] https://crrev.com/9954055c59d6462c34cf3ed0850ed343e6964042/chromeos/components/tether/initializer_impl.cc
,
Jan 22 2018
,
Jan 23 2018
|
|||||||||||||
►
Sign in to add a comment |
|||||||||||||
Comment 1 by khorimoto@chromium.org
, Sep 5 2017