commit 16fb3f9e425bc511ec5f8952d3130c633dee8ddc
author Jon Spivack <spivack@google.com> Fri Jul 26 13:14:42 2019 -0700
committer Jon Spivack <spivack@google.com> Mon Sep 09 15:24:42 2019 -0700
tree c1684bdc95e9ffb95580ca645f643d9a256c1b93
parent 983f76b3c632b29ca6ff858d986378a9028493c2

Allow AIDL interfaces in service parsing

Bug: 138756857
Test: Manual (using mediaextractor as a test service)
Change-Id: Ice2c695fca7062d6a115df13a6ac1d6fe82a3a98

init/interface_utils.cpp

diff --git a/init/interface_utils.cpp b/init/interface_utils.cpp
index a54860f..ddbacd7 100644
--- a/init/interface_utils.cpp
+++ b/init/interface_utils.cpp
@@ -77,6 +77,12 @@
                                                 const InterfaceInheritanceHierarchyMap& hierarchy) {
     std::set<FQName> interface_fqnames;
     for (const std::string& instance : instances) {
+        // There is insufficient build-time information on AIDL interfaces to check them here
+        // TODO(b/139307527): Rework how services store interfaces to avoid excess string parsing
+        if (base::Split(instance, "/")[0] == "aidl") {
+            continue;
+        }
+
         FqInstance fqinstance;
         if (!fqinstance.setTo(instance)) {
             return Error() << "Unable to parse interface instance '" << instance << "'";