commit 2cf04899dab04f401ded8c1c9d445d49bb862420
author Steven Moreland <smoreland@google.com> Thu Aug 29 09:47:37 2019 -0700
committer android-build-merger <android-build-merger@google.com> Thu Aug 29 09:47:37 2019 -0700
tree 320cf9975a9fb574620404b8053f6b0071a16197
parent a7df9f05f5221ada4e48fbfb8b67cf42dd9062b8
parent 8541be0e72d8dc6dcfca49064d3547d834f8847c

Lazy start remove redundant log. am: 52fde2119d am: d20a0bbcdd am: 0fd475648c am: e9a195ad3c
am: 8541be0e72

Change-Id: I84bde7bb8bc6fc3f45135f8f1301e83343015001

ServiceManager.cpp

diff --git a/ServiceManager.cpp b/ServiceManager.cpp
index 90b5dd8..6a9864f 100644
--- a/ServiceManager.cpp
+++ b/ServiceManager.cpp
@@ -306,6 +306,11 @@
 
     auto pidcon = getBinderCallingContext();
 
+    if (!mAcl.canAdd(IBase::descriptor, pidcon)) {
+        LOG(ERROR) << "Missing permissions to add IBase";
+        return false;
+    }
+
     auto ret = service->interfaceChain([&](const auto &interfaceChain) {
         addSuccess = addImpl(name, service, interfaceChain, pidcon);
     });