diff options
author | Jerry Zhang <zhangjerry@google.com> | 2018-03-26 14:59:39 -0700 |
---|---|---|
committer | Jerry Zhang <zhangjerry@google.com> | 2018-03-27 00:01:33 +0000 |
commit | 2ecbc7acdb6f87e34228e67e36333f64eb021f82 (patch) | |
tree | b0ceb5b09377a0f002853cc08329ae717c3f6071 /media/java/android/mtp/MtpDatabase.java | |
parent | 642c0be3f35354df689e40010f106526411d2f15 (diff) |
Guard against null server
If MtpService stops before MtpServer
shuts down, the server will be null,
so don't attempt to use it in that
case.
Bug: 76433619
Test: no errors
Change-Id: I6744cb78908e6a0663a4b60107d5c9c18dda62bb
Diffstat (limited to 'media/java/android/mtp/MtpDatabase.java')
-rwxr-xr-x | media/java/android/mtp/MtpDatabase.java | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/media/java/android/mtp/MtpDatabase.java b/media/java/android/mtp/MtpDatabase.java index 32a00d597e62..f15de2c15792 100755 --- a/media/java/android/mtp/MtpDatabase.java +++ b/media/java/android/mtp/MtpDatabase.java @@ -314,7 +314,9 @@ public class MtpDatabase implements AutoCloseable { public void addStorage(StorageVolume storage) { MtpStorage mtpStorage = mManager.addMtpStorage(storage); mStorageMap.put(storage.getPath(), mtpStorage); - mServer.addStorage(mtpStorage); + if (mServer != null) { + mServer.addStorage(mtpStorage); + } } public void removeStorage(StorageVolume storage) { @@ -322,7 +324,9 @@ public class MtpDatabase implements AutoCloseable { if (mtpStorage == null) { return; } - mServer.removeStorage(mtpStorage); + if (mServer != null) { + mServer.removeStorage(mtpStorage); + } mManager.removeMtpStorage(mtpStorage); mStorageMap.remove(storage.getPath()); } |