From 942c6a42ac9395d523bdebd3978d938915672aec Mon Sep 17 00:00:00 2001 From: Rowan Hart Date: Tue, 22 Aug 2023 23:23:52 -0700 Subject: [PATCH] Reset headers with a memcpy, not an assign from zeroed (#1443) --- libafl/src/observers/cmp.rs | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libafl/src/observers/cmp.rs b/libafl/src/observers/cmp.rs index 34dfc67f84..eca533190d 100644 --- a/libafl/src/observers/cmp.rs +++ b/libafl/src/observers/cmp.rs @@ -631,7 +631,6 @@ pub struct AFLppCmpHeader { #[bitfield(name = "reserved", ty = "u32", bits = "60..=63")] data: [u8; 8], } - /// The AFL++ `cmp_operands` struct #[derive(Default, Debug, Clone, Copy)] #[repr(C, packed)] @@ -814,8 +813,8 @@ impl CmpMap for AFLppCmpMap { fn reset(&mut self) -> Result<(), Error> { // For performance, we reset just the headers - self.headers = unsafe { core::mem::zeroed() }; - // self.vals.operands = unsafe { core::mem::zeroed() }; + self.headers.fill(AFLppCmpHeader { data: [0; 8] }); + Ok(()) } }