summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Natalie Silvanovich <natashenka@google.com> 2014-05-01 16:12:23 -0700
committer Natalie Silvanovich <natashenka@google.com> 2014-05-01 16:26:09 -0700
commit55db646c4e43fd24386bf522aef13c0b3075593c (patch)
tree9b3c05b301af695fc098d1dceb747ca994241fd2
parent61bc9f37cc0e0921d2e205dccdd45df36c353a9c (diff)
Null checks in register/unregister BT Adapters
Prevents system crash Bug: 13743852 Change-Id: I05bcb31fc8377866b93899a01c004d15e041f21f
-rw-r--r--services/core/java/com/android/server/BluetoothManagerService.java8
1 files changed, 8 insertions, 0 deletions
diff --git a/services/core/java/com/android/server/BluetoothManagerService.java b/services/core/java/com/android/server/BluetoothManagerService.java
index 0d6f548139c6..f22020f575db 100644
--- a/services/core/java/com/android/server/BluetoothManagerService.java
+++ b/services/core/java/com/android/server/BluetoothManagerService.java
@@ -307,6 +307,10 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
}
public IBluetooth registerAdapter(IBluetoothManagerCallback callback){
+ if (callback == null) {
+ Log.w(TAG, "Callback is null in registerAdapter");
+ return null;
+ }
Message msg = mHandler.obtainMessage(MESSAGE_REGISTER_ADAPTER);
msg.obj = callback;
mHandler.sendMessage(msg);
@@ -316,6 +320,10 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
}
public void unregisterAdapter(IBluetoothManagerCallback callback) {
+ if (callback == null) {
+ Log.w(TAG, "Callback is null in unregisterAdapter");
+ return;
+ }
mContext.enforceCallingOrSelfPermission(BLUETOOTH_PERM,
"Need BLUETOOTH permission");
Message msg = mHandler.obtainMessage(MESSAGE_UNREGISTER_ADAPTER);