commit a781cee3f456a4487618753bdbf91a45ddcc6137
author Chong Zhang <chz@google.com> Wed Mar 17 11:42:33 2021 -0700
committer Chong Zhang <chz@google.com> Wed Mar 17 11:57:52 2021 -0700
tree 07b4cca82177330bb63cc0ade877de1bc4cc0ed6
parent da52d86663542d1860adb0bba7f49902f0e67ac9

Do not register with null observer

bug: 168307955
test: unit tests.
Change-Id: Iadef8660c95f2c545a212dfabf06c264b7c719a6

services/mediaresourcemanager/ResourceObserverService.cpp

diff --git a/services/mediaresourcemanager/ResourceObserverService.cpp b/services/mediaresourcemanager/ResourceObserverService.cpp
index 9cc6fe4..4e97406 100644
--- a/services/mediaresourcemanager/ResourceObserverService.cpp
+++ b/services/mediaresourcemanager/ResourceObserverService.cpp
@@ -165,6 +165,10 @@
         return Status::fromServiceSpecificError(PERMISSION_DENIED);
     }
 
+    if (in_observer == nullptr) {
+        return Status::fromServiceSpecificError(BAD_VALUE);
+    }
+
     ::ndk::SpAIBinder binder = in_observer->asBinder();
 
     {
@@ -220,6 +224,10 @@
         return Status::fromServiceSpecificError(PERMISSION_DENIED);
     }
 
+    if (in_observer == nullptr) {
+        return Status::fromServiceSpecificError(BAD_VALUE);
+    }
+
     ::ndk::SpAIBinder binder = in_observer->asBinder();
 
     {