diff options
author | Steven Moreland <smoreland@google.com> | 2019-12-12 14:22:34 -0800 |
---|---|---|
committer | Steven Moreland <smoreland@google.com> | 2022-02-09 00:59:37 +0000 |
commit | 0f10f3fd9f961abf78030922ac5638150b8c2694 (patch) | |
tree | bbe47c668eaa33b1dd8841bdc961c75791d25a84 /bpfloader/BpfLoader.cpp | |
parent | 7429fcb8421ce0cd134bca8cc9141f5b64417ad1 (diff) | |
download | bpf-0f10f3fd9f961abf78030922ac5638150b8c2694.tar.gz |
bpfloader: also load from /vendor/etc/bpf/android-t-preview-2android-t-preview-1android-t-beta-3android-s-v2-beta-3android-s-qpr3-beta-1android-t-preview-1android-s-v2-beta-3android-s-qpr3-beta-1
Allow vendors to use bpf programs, but limit to tracepoints
for now (other types of programs, for instance skfilter, aren't
safe to expose, because the kernel gives us limited ways to
control which resources can have BPF programs attached, and
some shared resources only support a single BPF program at an
attach point).
Bug: 140330870
Bug: 162057235
Test: install bpf program to /vendor/etc/bpf/ and use it.
Test: atest libbpf_load_test
Change-Id: I6c876fe52739c38db73689ffd784167e7d35d58a
Diffstat (limited to 'bpfloader/BpfLoader.cpp')
-rw-r--r-- | bpfloader/BpfLoader.cpp | 38 |
1 files changed, 30 insertions, 8 deletions
diff --git a/bpfloader/BpfLoader.cpp b/bpfloader/BpfLoader.cpp index 3f86a56..5c24f0a 100644 --- a/bpfloader/BpfLoader.cpp +++ b/bpfloader/BpfLoader.cpp @@ -39,6 +39,7 @@ #include <sys/types.h> #include <android-base/logging.h> +#include <android-base/macros.h> #include <android-base/properties.h> #include <android-base/stringprintf.h> #include <android-base/strings.h> @@ -52,10 +53,22 @@ using android::base::EndsWith; using std::string; -struct { +// see b/162057235. For arbitrary program types, the concern is that due to the lack of +// SELinux access controls over BPF program attachpoints, we have no way to control the +// attachment of programs to shared resources (or to detect when a shared resource +// has one BPF program replace another that is attached there) +constexpr bpf_prog_type kVendorAllowedProgTypes[] = { + BPF_PROG_TYPE_TRACEPOINT, +}; + +struct Location { const char* const dir; const char* const prefix; -} locations[] = { + const bpf_prog_type* allowedProgTypes = nullptr; + size_t allowedProgTypesLength = 0; +}; + +const Location locations[] = { // Tethering mainline module: tether offload { .dir = "/apex/com.android.tethering/etc/bpf/", @@ -71,23 +84,32 @@ struct { .dir = "/system/etc/bpf/", .prefix = "", }, + // Vendor operating system + { + .dir = "/vendor/etc/bpf/", + .prefix = "vendor/", + .allowedProgTypes = kVendorAllowedProgTypes, + .allowedProgTypesLength = arraysize(kVendorAllowedProgTypes), + }, }; -int loadAllElfObjects(const char* const progDir, const char* const prefix) { +int loadAllElfObjects(const Location& location) { int retVal = 0; DIR* dir; struct dirent* ent; - if ((dir = opendir(progDir)) != NULL) { + if ((dir = opendir(location.dir)) != NULL) { while ((ent = readdir(dir)) != NULL) { string s = ent->d_name; if (!EndsWith(s, ".o")) continue; - string progPath(progDir); + string progPath(location.dir); progPath += s; bool critical; - int ret = android::bpf::loadProg(progPath.c_str(), &critical, prefix); + int ret = android::bpf::loadProg(progPath.c_str(), &critical, location.prefix, + location.allowedProgTypes, + location.allowedProgTypesLength); if (ret) { if (critical) retVal = ret; ALOGE("Failed to load object: %s, ret: %s", progPath.c_str(), std::strerror(-ret)); @@ -122,9 +144,9 @@ int main(int argc, char** argv) { android::base::InitLogging(argv, &android::base::KernelLogger); // Load all ELF objects, create programs and maps, and pin them - for (const auto location : locations) { + for (const auto& location : locations) { createSysFsBpfSubDir(location.prefix); - if (loadAllElfObjects(location.dir, location.prefix) != 0) { + if (loadAllElfObjects(location) != 0) { ALOGE("=== CRITICAL FAILURE LOADING BPF PROGRAMS FROM %s ===", location.dir); ALOGE("If this triggers reliably, you're probably missing kernel options or patches."); ALOGE("If this triggers randomly, you might be hitting some memory allocation " |