diff --git a/hbvm/src/vm/mem/mod.rs b/hbvm/src/vm/mem/mod.rs index 88e82ca2..a10490dc 100644 --- a/hbvm/src/vm/mem/mod.rs +++ b/hbvm/src/vm/mem/mod.rs @@ -59,33 +59,29 @@ impl Memory { pub fn load(&self, addr: u64) -> Option { let lookup = self.page_lookup(addr)?; match lookup.perm { - Permission::Empty | Permission::Node => None, - Permission::Readonly | Permission::Write | Permission::Exec => { - let mut value = MaybeUninit::::zeroed(); - unsafe { - core::ptr::copy_nonoverlapping::(lookup.ptr, value.as_mut_ptr().cast(), 1); - Some(value.assume_init()) - } - } + Permission::Empty | Permission::Node => return None, + Permission::Readonly | Permission::Write | Permission::Exec => (), + } + + let mut value = MaybeUninit::::zeroed(); + unsafe { + core::ptr::copy_nonoverlapping::(lookup.ptr, value.as_mut_ptr().cast(), 1); + Some(value.assume_init()) } } /// Store value to an address pub fn store(&mut self, addr: u64, value: Value) -> Result<(), ()> { let lookup = self.page_lookup(addr).ok_or(())?; - match lookup.perm { - Permission::Write => { - unsafe { - core::ptr::copy_nonoverlapping::( - (&value as *const Value).cast(), - lookup.ptr, - 1, - ) - }; - Ok(()) - } - _ => Err(()), + if lookup.perm != Permission::Write { + return Err(()); } + + unsafe { + core::ptr::copy_nonoverlapping::((&value as *const Value).cast(), lookup.ptr, 1) + }; + + Ok(()) } #[inline]