Update zerocopy to v0.8

Cargo.toml

@@ -60,7 +60,7 @@ tokio-util = { version = "0.7", features = ["compat"] }
 usdt = "0.5.0"
 uuid = { version = "1.16", default-features = false }
 version_check = "0.9.5"
-zerocopy = "0.6.6"
+zerocopy = "0.8.25"
 zip = { version = "0.6.6", default-features = false, features = ["deflate", "bzip2"] }
 
 gateway-messages.path = "gateway-messages"

faux-mgs/src/main.rs

@@ -60,7 +60,7 @@ use std::sync::Arc;
 use std::time::Duration;
 use tokio::net::UdpSocket;
 use uuid::Uuid;
-use zerocopy::AsBytes;
+use zerocopy::IntoBytes;
 
 mod picocom_map;
 mod usart;

gateway-messages/Cargo.toml

@@ -14,7 +14,7 @@ smoltcp = { workspace = true, optional = true }
 static_assertions.workspace = true
 strum_macros.workspace = true
 uuid.workspace = true
-zerocopy.workspace = true
+zerocopy = { workspace = true, features = ["derive"] }
 
 [dev-dependencies]
 serde_json.workspace = true

gateway-messages/src/mgs_to_sp.rs

@@ -343,7 +343,9 @@ pub enum UnlockChallenge {
     PartialEq,
     Eq,
     Debug,
-    zerocopy::AsBytes,
+    zerocopy::IntoBytes,
+    zerocopy::KnownLayout,
+    zerocopy::Immutable,
 )]
 #[repr(C)]
 pub struct EcdsaSha2Nistp256Challenge {

gateway-messages/src/tlv.rs

@@ -10,8 +10,10 @@ use core::iter;
 use core::mem;
 use zerocopy::byteorder::LittleEndian;
 use zerocopy::byteorder::U32;
-use zerocopy::AsBytes;
 use zerocopy::FromBytes;
+use zerocopy::Immutable;
+use zerocopy::IntoBytes;
+use zerocopy::KnownLayout;
 
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum EncodeError<E> {
@@ -42,11 +44,31 @@ impl fmt::Display for DecodeError {
 #[cfg(feature = "std")]
 impl std::error::Error for DecodeError {}
 
-#[derive(Debug, Clone, Copy, PartialEq, Eq, AsBytes, FromBytes)]
+#[derive(
+    Debug,
+    Clone,
+    Copy,
+    PartialEq,
+    Eq,
+    IntoBytes,
+    FromBytes,
+    Immutable,
+    KnownLayout,
+)]
 #[repr(C)]
 pub struct Tag(pub [u8; 4]);
 
-#[derive(Debug, Clone, Copy, PartialEq, Eq, AsBytes, FromBytes)]
+#[derive(
+    Debug,
+    Clone,
+    Copy,
+    PartialEq,
+    Eq,
+    IntoBytes,
+    FromBytes,
+    Immutable,
+    KnownLayout,
+)]
 #[repr(C)]
 struct Header {
     tag: Tag,
@@ -103,9 +125,8 @@ where
 /// returns `(Tag, value, rest_of_buf)`.
 pub fn decode(buf: &[u8]) -> Result<(Tag, &[u8], &[u8]), DecodeError> {
     // Peel header off the front.
-    let header =
-        Header::read_from_prefix(buf).ok_or(DecodeError::BufferTooSmall)?;
-    let buf = &buf[mem::size_of::<Header>()..];
+    let (header, buf) = Header::read_from_prefix(buf)
+        .map_err(|_| DecodeError::BufferTooSmall)?;
 
     // Split remaining buffer at the end of our value.
     let length = header.length.get() as usize;