From 04cdcc69734ecfd563cdc8aaedde3a818a235cb2 Mon Sep 17 00:00:00 2001 From: Tamir Duberstein Date: Sat, 6 Apr 2024 13:30:53 +0100 Subject: [PATCH] Appease clippy ``` warning: transmute used without annotations --> aya-log-common/src/lib.rs:220:41 | 220 | let bytes = unsafe { core::mem::transmute::<_, [u8; 16]>(self) }; | ^^^^^^^^^^^^^^^^^^^^^^^^ help: consider adding missing annotations: `transmute::<[u16; 8], [u8; 16]>` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_transmute_annotations = note: `#[warn(clippy::missing_transmute_annotations)]` on by default error: transmute used without annotations --> aya/src/maps/mod.rs:1130:52 | 1130 | .copy_from_slice(unsafe { mem::transmute(TEST_NAME) }); | ^^^^^^^^^ help: consider adding missing annotations: `transmute::<&str, &[i8]>` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_transmute_annotations note: the lint level is defined here --> aya/src/lib.rs:41:5 | 41 | clippy::all, | ^^^^^^^^^^^ = note: `#[deny(clippy::missing_transmute_annotations)]` implied by `#[deny(clippy::all)]` ``` --- aya-log-common/src/lib.rs | 2 +- aya/src/maps/mod.rs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/aya-log-common/src/lib.rs b/aya-log-common/src/lib.rs index 472090ee..24f41610 100644 --- a/aya-log-common/src/lib.rs +++ b/aya-log-common/src/lib.rs @@ -217,7 +217,7 @@ impl WriteToBuf for [u16; 8] { // compile-time constant. #[inline(never)] fn write(self, buf: &mut [u8]) -> Option { - let bytes = unsafe { core::mem::transmute::<_, [u8; 16]>(self) }; + let bytes = unsafe { core::mem::transmute::<[u16; 8], [u8; 16]>(self) }; write(Argument::ArrU16Len8.into(), &bytes, buf) } } diff --git a/aya/src/maps/mod.rs b/aya/src/maps/mod.rs index 192c988b..155c02a7 100644 --- a/aya/src/maps/mod.rs +++ b/aya/src/maps/mod.rs @@ -1127,7 +1127,7 @@ mod tests { ); let map_info = unsafe { &mut *(attr.info.info as *mut bpf_map_info) }; map_info.name[..TEST_NAME.len()] - .copy_from_slice(unsafe { mem::transmute(TEST_NAME) }); + .copy_from_slice(unsafe { mem::transmute::<&[u8], &[i8]>(TEST_NAME.as_bytes()) }); Ok(0) } _ => Err((-1, io::Error::from_raw_os_error(EFAULT))),