diff options
author | Maciej Żenczykowski <maze@google.com> | 2024-03-08 07:55:37 +0000 |
---|---|---|
committer | Maciej Żenczykowski <maze@google.com> | 2024-03-08 07:55:53 +0000 |
commit | 278a712e6969c3ecae51e50fe1d0f08d0cec1a06 (patch) | |
tree | 5f12efadfa87ad5fa0954d98374e0c62516e550d /bpfloader/BpfLoader.cpp | |
parent | 6456428d38d813a027031e53a593a1bdce91cbaa (diff) | |
download | bpf-278a712e6969c3ecae51e50fe1d0f08d0cec1a06.tar.gz |
BpfLoader: remove dead code
Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: Iae17d41884c17cd061dbf487a48b523028cd67a7
Diffstat (limited to 'bpfloader/BpfLoader.cpp')
-rw-r--r-- | bpfloader/BpfLoader.cpp | 25 |
1 files changed, 0 insertions, 25 deletions
diff --git a/bpfloader/BpfLoader.cpp b/bpfloader/BpfLoader.cpp index 08fd72d..7a68f99 100644 --- a/bpfloader/BpfLoader.cpp +++ b/bpfloader/BpfLoader.cpp @@ -162,31 +162,6 @@ int createSysFsBpfSubDir(const char* const prefix) { return 0; } -// Technically 'value' doesn't need to be newline terminated, but it's best -// to include a newline to match 'echo "value" > /proc/sys/...foo' behaviour, -// which is usually how kernel devs test the actual sysctl interfaces. -int writeProcSysFile(const char *filename, const char *value) { - android::base::unique_fd fd(open(filename, O_WRONLY | O_CLOEXEC)); - if (fd < 0) { - const int err = errno; - ALOGE("open('%s', O_WRONLY | O_CLOEXEC) -> %s", filename, strerror(err)); - return -err; - } - int len = strlen(value); - int v = write(fd, value, len); - if (v < 0) { - const int err = errno; - ALOGE("write('%s', '%s', %d) -> %s", filename, value, len, strerror(err)); - return -err; - } - if (v != len) { - // In practice, due to us only using this for /proc/sys/... files, this can't happen. - ALOGE("write('%s', '%s', %d) -> short write [%d]", filename, value, len, v); - return -EINVAL; - } - return 0; -} - int main(int argc, char** argv) { (void)argc; android::base::InitLogging(argv, &android::base::KernelLogger); |