summaryrefslogtreecommitdiff
path: root/libnativeloader/native_loader_namespace.cpp
diff options
context:
space:
mode:
authorKiyoung Kim <kiyoungkim@google.com>2020-01-29 16:09:38 +0900
committerKiyoung Kim <kiyoungkim@google.com>2020-02-07 03:25:40 +0000
commit99c19ca707391f0908313c7cee2ef315d2b5cb3c (patch)
treec5213c3eb212732c88cc455e8ca67c1990d29767 /libnativeloader/native_loader_namespace.cpp
parentcf0c6ef642517fba3bc9a211acaed742ff39b86d (diff)
Update platform namespace name
Platform namespace has been renamed as 'system' from linkerconfig generator. To meet this requirement, libnativeloader should search for namespace 'system' rather than namespace 'platform'. Bug: 147987608 Test: m -j passed Test: atest libnativeloader_test passed Change-Id: I23d865ac71a80619f291eb9ae0761a2cad5df352
Diffstat (limited to 'libnativeloader/native_loader_namespace.cpp')
-rw-r--r--libnativeloader/native_loader_namespace.cpp20
1 files changed, 10 insertions, 10 deletions
diff --git a/libnativeloader/native_loader_namespace.cpp b/libnativeloader/native_loader_namespace.cpp
index fff3191a75..6faa78cdde 100644
--- a/libnativeloader/native_loader_namespace.cpp
+++ b/libnativeloader/native_loader_namespace.cpp
@@ -35,7 +35,7 @@ namespace android {
namespace {
constexpr const char* kDefaultNamespaceName = "default";
-constexpr const char* kPlatformNamespaceName = "platform";
+constexpr const char* kSystemNamespaceName = "system";
std::string GetLinkerError(bool is_bridged) {
const char* msg = is_bridged ? NativeBridgeGetError() : dlerror();
@@ -63,11 +63,11 @@ Result<NativeLoaderNamespace> NativeLoaderNamespace::GetExportedNamespace(const
return Errorf("namespace {} does not exist or exported", name);
}
-// The platform namespace is called "default" for binaries in /system and
-// "platform" for those in the Runtime APEX. Try "platform" first since
+// The system namespace is called "default" for binaries in /system and
+// "system" for those in the Runtime APEX. Try "system" first since
// "default" always exists.
-Result<NativeLoaderNamespace> NativeLoaderNamespace::GetPlatformNamespace(bool is_bridged) {
- auto ns = GetExportedNamespace(kPlatformNamespaceName, is_bridged);
+Result<NativeLoaderNamespace> NativeLoaderNamespace::GetSystemNamespace(bool is_bridged) {
+ auto ns = GetExportedNamespace(kSystemNamespaceName, is_bridged);
if (ns) return ns;
ns = GetExportedNamespace(kDefaultNamespaceName, is_bridged);
if (ns) return ns;
@@ -93,12 +93,12 @@ Result<NativeLoaderNamespace> NativeLoaderNamespace::Create(
is_bridged = NativeBridgeIsPathSupported(search_paths.c_str());
}
- // Fall back to the platform namespace if no parent is set.
- auto platform_ns = GetPlatformNamespace(is_bridged);
- if (!platform_ns) {
- return platform_ns.error();
+ // Fall back to the system namespace if no parent is set.
+ auto system_ns = GetSystemNamespace(is_bridged);
+ if (!system_ns) {
+ return system_ns.error();
}
- const NativeLoaderNamespace& effective_parent = parent != nullptr ? *parent : *platform_ns;
+ const NativeLoaderNamespace& effective_parent = parent != nullptr ? *parent : *system_ns;
// All namespaces for apps are isolated
uint64_t type = ANDROID_NAMESPACE_TYPE_ISOLATED;