commit aabe014b4b52b043d3c58df8474617c616f188a5
author Arun Johnson <arunjohnson@google.com> Mon Jul 18 19:16:04 2022 +0000
committer Arun Johnson <arunjohnson@google.com> Wed Aug 03 21:43:59 2022 +0000
tree e580c2c32eb76a7e036fa5d7206da6faf17096cc
parent 9e2e53d5bfa45513db63c9030acdb2d965abbc5c

Adding null check for IResourceManagerClient in dump()

bug: 191613308
Change-Id: I1694f2d1c13163a9b8162630377de71303bc517f

services/mediaresourcemanager/ResourceManagerService.cpp

diff --git a/services/mediaresourcemanager/ResourceManagerService.cpp b/services/mediaresourcemanager/ResourceManagerService.cpp
index 0167cba..6a60aad 100644
--- a/services/mediaresourcemanager/ResourceManagerService.cpp
+++ b/services/mediaresourcemanager/ResourceManagerService.cpp
@@ -246,10 +246,9 @@
             snprintf(buffer, SIZE, "        Id: %lld\n", (long long)infos[j].clientId);
             result.append(buffer);
 
-            std::string clientName;
-            Status status = infos[j].client->getName(&clientName);
-            if (!status.isOk()) {
-                clientName = "<unknown client>";
+            std::string clientName = "<unknown client>";
+            if (infos[j].client != nullptr) {
+                Status status = infos[j].client->getName(&clientName);
             }
             snprintf(buffer, SIZE, "        Name: %s\n", clientName.c_str());
             result.append(buffer);