diff options
author | Andrew Walbran <qwandor@google.com> | 2023-07-31 17:58:48 +0100 |
---|---|---|
committer | Andrew Walbran <qwandor@google.com> | 2023-07-31 17:58:48 +0100 |
commit | f7bf5435aa6c36ee23c3b7c7bd812e207cd79d79 (patch) | |
tree | 0f45e7ec3286d0487dc9544d10fd1acd8c16b0dd /libatrace_rust | |
parent | 5f69572f99559928ab70d5cc7240f3dd3e54dc12 (diff) | |
download | extras-f7bf5435aa6c36ee23c3b7c7bd812e207cd79d79.tar.gz |
Add safety comments.
These will soon be required by a lint.
Bug: 290018030
Test: m rust
Change-Id: I4a2c338909c8f25e3727edc9a8ce61ae1a7e7ea3
Diffstat (limited to 'libatrace_rust')
-rw-r--r-- | libatrace_rust/benchmark/src/atrace_benchmark.rs | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/libatrace_rust/benchmark/src/atrace_benchmark.rs b/libatrace_rust/benchmark/src/atrace_benchmark.rs index 30c492a5..da9d0a49 100644 --- a/libatrace_rust/benchmark/src/atrace_benchmark.rs +++ b/libatrace_rust/benchmark/src/atrace_benchmark.rs @@ -23,6 +23,7 @@ extern "C" { } fn turn_tracing_off() { + // SAFETY: This call is always safe. unsafe { disable_app_atrace(); } @@ -30,6 +31,7 @@ fn turn_tracing_off() { fn turn_tracing_on() { let procname = std::ffi::CString::new(std::env::args().next().unwrap()).unwrap(); + // SAFETY: `procname` is a valid C string and the function doesn't store it after it returns. unsafe { enable_atrace_for_single_app(procname.as_ptr()); } |