Skip to content

Commit

Permalink
Add support to indicate bitness of public library.
Browse files Browse the repository at this point in the history
For public vendor libraries, a vendor might have only a 32 bit
or only a 64 bit version of the library. Add a way to indicate this
in the public.libraries.txt files. The new format is:

  library.so 32

This indicates that this is a 32 bit only public library.

  library.so 64

This indicates that this is a 64 bit only public library.

Bug: 29370721
Bug: 29512261
(cherry picked from commit 6664a805cae6983c7a19caf8abbe88ad2d918bcb)

Change-Id: Id52c8b61d5c802ce62edda7e25f2755f6f93d6b7
  • Loading branch information
cferris1000 committed Jun 22, 2016
1 parent 57bbedc commit 39da84b
Showing 1 changed file with 27 additions and 4 deletions.
31 changes: 27 additions & 4 deletions libnativeloader/native_loader.cpp
Expand Up @@ -133,9 +133,10 @@ class LibraryNamespaces {
std::string public_native_libraries_system_config =
root_dir + kPublicNativeLibrariesSystemConfigPathFromRoot;

LOG_ALWAYS_FATAL_IF(!ReadConfig(public_native_libraries_system_config, &sonames),
std::string error_msg;
LOG_ALWAYS_FATAL_IF(!ReadConfig(public_native_libraries_system_config, &sonames, &error_msg),
"Error reading public native library list from \"%s\": %s",
public_native_libraries_system_config.c_str(), strerror(errno));
public_native_libraries_system_config.c_str(), error_msg.c_str());

// For debuggable platform builds use ANDROID_ADDITIONAL_PUBLIC_LIBRARIES environment
// variable to add libraries to the list. This is intended for platform tests only.
Expand Down Expand Up @@ -172,20 +173,42 @@ class LibraryNamespaces {
}

private:
bool ReadConfig(const std::string& configFile, std::vector<std::string>* sonames) {
bool ReadConfig(const std::string& configFile, std::vector<std::string>* sonames,
std::string* error_msg = nullptr) {
// Read list of public native libraries from the config file.
std::string file_content;
if(!base::ReadFileToString(configFile, &file_content)) {
if (error_msg) *error_msg = strerror(errno);
return false;
}

std::vector<std::string> lines = base::Split(file_content, "\n");

for (const auto& line : lines) {
for (auto& line : lines) {
auto trimmed_line = base::Trim(line);
if (trimmed_line[0] == '#' || trimmed_line.empty()) {
continue;
}
size_t space_pos = trimmed_line.rfind(' ');
if (space_pos != std::string::npos) {
std::string type = trimmed_line.substr(space_pos + 1);
if (type != "32" && type != "64") {
if (error_msg) *error_msg = "Malformed line: " + line;
return false;
}
#if defined(__LP64__)
// Skip 32 bit public library.
if (type == "32") {
continue;
}
#else
// Skip 64 bit public library.
if (type == "64") {
continue;
}
#endif
trimmed_line.resize(space_pos);
}

sonames->push_back(trimmed_line);
}
Expand Down

0 comments on commit 39da84b

Please sign in to comment.