commit 4e21def0a045438d90037a1834e06f0f7ab2f9c6
author Devin Moore <devinmoore@google.com> Fri Feb 24 21:54:14 2023 +0000
committer Devin Moore <devinmoore@google.com> Tue Mar 07 18:27:51 2023 +0000
tree 445dc336536c6ae5d3dea246228e7ee0da0d649a
parent ae68df27154eecab181831d060cae39303ceb885

Add log for missing dump priority in addService

Test: none
Bug: none
Change-Id: If242067d33d0a81d72ce9f1cacc8e0d2668e96ee

cmds/servicemanager/ServiceManager.cpp

diff --git a/cmds/servicemanager/ServiceManager.cpp b/cmds/servicemanager/ServiceManager.cpp
index eadf678..a75c6b5 100644
--- a/cmds/servicemanager/ServiceManager.cpp
+++ b/cmds/servicemanager/ServiceManager.cpp
@@ -355,6 +355,10 @@
     }
 #endif  // !VENDORSERVICEMANAGER
 
+    if ((dumpPriority & DUMP_FLAG_PRIORITY_ALL) == 0) {
+        ALOGW("Dump flag priority is not set when adding %s", name.c_str());
+    }
+
     // implicitly unlinked when the binder is removed
     if (binder->remoteBinder() != nullptr &&
         binder->linkToDeath(sp<ServiceManager>::fromExisting(this)) != OK) {