summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Eddie Hung <eddie.hung@mediatek.com> 2018-09-26 13:30:13 +0800
committer Isaac Lee <isaac.lee@mediatek.com> 2018-09-28 13:14:23 +0800
commitab87b44dca7ccf9c5f48bc9daebf9edf94a1f390 (patch)
tree9626b410d8c2e9c3d35e751064883935cc43893e
parentbc5c593e2ef31b5e7406a396c7afb9beb2987dd4 (diff)
Fix UsbDeviceManager null object reference
STORAGE_SERVICE is not ready or not available to use. UsbDeviceManager will get null value and cause null object reference exception. Check return value before use. Test: No exception when start USBService Change-Id: Icdbc532894d3ca1548f327d9f63d495baa549c68
-rw-r--r--services/usb/java/com/android/server/usb/UsbDeviceManager.java3
1 files changed, 2 insertions, 1 deletions
diff --git a/services/usb/java/com/android/server/usb/UsbDeviceManager.java b/services/usb/java/com/android/server/usb/UsbDeviceManager.java
index 36f8063cca52..4da7285bcf2d 100644
--- a/services/usb/java/com/android/server/usb/UsbDeviceManager.java
+++ b/services/usb/java/com/android/server/usb/UsbDeviceManager.java
@@ -539,7 +539,8 @@ public class UsbDeviceManager implements ActivityManagerInternal.ScreenObserver
// We do not show the USB notification if the primary volume supports mass storage.
// The legacy mass storage UI will be used instead.
final StorageManager storageManager = StorageManager.from(mContext);
- final StorageVolume primary = storageManager.getPrimaryVolume();
+ final StorageVolume primary =
+ storageManager != null ? storageManager.getPrimaryVolume() : null;
boolean massStorageSupported = primary != null && primary.allowMassStorage();
mUseUsbNotification = !massStorageSupported && mContext.getResources().getBoolean(