diff options
| author | Jon Spivack <spivack@google.com> | 2019-07-26 13:14:42 -0700 |
|---|---|---|
| committer | Jon Spivack <spivack@google.com> | 2019-09-09 15:24:42 -0700 |
| commit | 16fb3f9e425bc511ec5f8952d3130c633dee8ddc (patch) | |
| tree | c1684bdc95e9ffb95580ca645f643d9a256c1b93 /init/interface_utils.cpp | |
| parent | 983f76b3c632b29ca6ff858d986378a9028493c2 (diff) | |
Allow AIDL interfaces in service parsing
Bug: 138756857
Test: Manual (using mediaextractor as a test service)
Change-Id: Ice2c695fca7062d6a115df13a6ac1d6fe82a3a98
Diffstat (limited to 'init/interface_utils.cpp')
| -rw-r--r-- | init/interface_utils.cpp | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/init/interface_utils.cpp b/init/interface_utils.cpp index a54860fa4..ddbacd763 100644 --- a/init/interface_utils.cpp +++ b/init/interface_utils.cpp @@ -77,6 +77,12 @@ Result<void> CheckInterfaceInheritanceHierarchy(const std::set<std::string>& ins 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 << "'"; |