summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Jerome Poichet <jpoichet@google.com> 2015-06-25 12:03:02 -0700
committer Jérôme Poichet <jpoichet@google.com> 2015-06-25 19:20:38 +0000
commitcd62b10bd612e0c49083cc03f4ac63376a52fe8c (patch)
tree142cdb1139e9f928b14b6b60c24b27fb7662730b
parentf9c6dbb6d1c2001c0ccd7479668eb59b1d4b7c5c (diff)
Preventing leak
When transitioning from ON to OFF with LE Advertisers, advertiser do not get a chance to unregister themselves as the stopAdvertising checks the state of the stack and throws before unregistering the object. It will then never remove the callback objects causing a leak. b/22092678 | Remote service crash after switching to restricted profile Change-Id: I04817026a524d10d60abdd8b533554a71a0112e2
-rw-r--r--core/java/android/bluetooth/le/BluetoothLeAdvertiser.java1
1 files changed, 0 insertions, 1 deletions
diff --git a/core/java/android/bluetooth/le/BluetoothLeAdvertiser.java b/core/java/android/bluetooth/le/BluetoothLeAdvertiser.java
index 67d9de594090..eaf20d8e19b3 100644
--- a/core/java/android/bluetooth/le/BluetoothLeAdvertiser.java
+++ b/core/java/android/bluetooth/le/BluetoothLeAdvertiser.java
@@ -152,7 +152,6 @@ public final class BluetoothLeAdvertiser {
*/
public void stopAdvertising(final AdvertiseCallback callback) {
synchronized (mLeAdvertisers) {
- BluetoothLeUtils.checkAdapterStateOn(mBluetoothAdapter);
if (callback == null) {
throw new IllegalArgumentException("callback cannot be null");
}