Fix crash during copy to dst (#333)

philgweber · web-flow · commit 5edc49ea735a · 2025-06-06T11:39:53.000-07:00
dest_slice will always be the max size and src_slice is a subset of this
length. Must not copy beyond the src_slice.len otherwise will access
invalid data and crash.
diff --git a/espi-service/src/espi_service.rs b/espi-service/src/espi_service.rs
@@ -218,7 +218,7 @@ pub async fn espi_service(mut espi: espi::Espi<'static>, memory_map_buffer: &'st
 
                     match result {
                         Ok(dest_slice) => {
-                            dest_slice.copy_from_slice(src_slice);
+                            dest_slice[..src_slice.len()].copy_from_slice(src_slice);
                         }
                         Err(_e) => {
                             #[cfg(feature = "defmt")]

Original file line number	Diff line number	Diff line change
`@@ -218,7 +218,7 @@ pub async fn espi_service(mut espi: espi::Espi<'static>, memory_map_buffer: &'st`
`218`	`218`
`219`	`219`	`match result {`
`220`	`220`	`Ok(dest_slice) => {`
`221`		`- dest_slice.copy_from_slice(src_slice);`
	`221`	`+ dest_slice[..src_slice.len()].copy_from_slice(src_slice);`
`222`	`222`	`}`
`223`	`223`	`Err(_e) => {`
`224`	`224`	`#[cfg(feature = "defmt")]`