diff options
| author | Dimitry Ivanov <dimitry@google.com> | 2016-05-10 16:21:25 -0700 |
|---|---|---|
| committer | Dimitry Ivanov <dimitry@google.com> | 2016-05-10 17:32:38 -0700 |
| commit | d0b1531929d76411d964d4077d441d751e2c01fb (patch) | |
| tree | aa8e0c632ad9f3adc296dc09c68d249d9b352bb3 /libnativeloader/native_loader.cpp | |
| parent | f334cbf0e1425633bef96a21b0ce9e30f4c6ffa9 (diff) | |
nativeloader: Ignore empty java_permitted_path
Passing empty permitted path should result in no-op.
This addresses review comment on CL with commit
f334cbf0e1425633bef96a21b0ce9e30f4c6ffa9
Bug: http://b/28639227
Change-Id: I4a4540e522e90a145a374939921932c86f35e88d
Diffstat (limited to 'libnativeloader/native_loader.cpp')
| -rw-r--r-- | libnativeloader/native_loader.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/libnativeloader/native_loader.cpp b/libnativeloader/native_loader.cpp index 0b49b4106..8103b37a0 100644 --- a/libnativeloader/native_loader.cpp +++ b/libnativeloader/native_loader.cpp @@ -78,7 +78,9 @@ class LibraryNamespaces { if (java_permitted_path != nullptr) { ScopedUtfChars path(env, java_permitted_path); - permitted_path = permitted_path + ":" + path.c_str(); + if (path.c_str() != nullptr && path.size() > 0) { + permitted_path = permitted_path + ":" + path.c_str(); + } } if (!initialized_ && !InitPublicNamespace(library_path.c_str(), target_sdk_version)) { |