summaryrefslogtreecommitdiff
path: root/cmds/servicemanager/ServiceManager.cpp
diff options
context:
space:
mode:
author Steven Moreland <smoreland@google.com> 2019-08-20 16:58:25 -0700
committer Steven Moreland <smoreland@google.com> 2019-08-20 16:58:25 -0700
commitbb7951d252146e899fafda31f76a5025c73bd3f5 (patch)
tree8e6e804b9edb5641b4cd2ef87d8ce4d9f3b108f0 /cmds/servicemanager/ServiceManager.cpp
parent73523fdcb32bf22d55bad91c7068be7808a81fb2 (diff)
servicemanager: Allow '/' character in service names.
To work as instance name separater consistently w/ HIDL. In HIDL, people are used to writing something like: android.hardware.light@2.0::ILight/default In AIDL, the same thing would look like: android.hardware.light.ILight/default Bug: 136027762 Test: use '/' in service name Change-Id: Ie108029bd37b09647b761c5c5ce3a901dbfd4f19
Diffstat (limited to 'cmds/servicemanager/ServiceManager.cpp')
-rw-r--r--cmds/servicemanager/ServiceManager.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/cmds/servicemanager/ServiceManager.cpp b/cmds/servicemanager/ServiceManager.cpp
index f35f360dec..4d5c76ab95 100644
--- a/cmds/servicemanager/ServiceManager.cpp
+++ b/cmds/servicemanager/ServiceManager.cpp
@@ -67,7 +67,7 @@ bool isValidServiceName(const std::string& name) {
if (name.size() > 127) return false;
for (char c : name) {
- if (c == '_' || c == '-' || c == '.') continue;
+ if (c == '_' || c == '-' || c == '.' || c == '/') continue;
if (c >= 'a' && c <= 'z') continue;
if (c >= 'A' && c <= 'Z') continue;
if (c >= '0' && c <= '9') continue;