renamed to StaticMapObserver, code fmt

src/executors/mod.rs

@@ -178,19 +178,20 @@ impl InMemoryExecutor {
 #[cfg(test)]
 mod tests {
     use crate::executors::{Executor, ExitKind, InMemoryExecutor};
-    use crate::observers::Observer;
     use crate::inputs::Input;
+    use crate::observers::Observer;
     use crate::AflError;
 
     struct NopInput {}
     impl Input for NopInput {
-        fn serialize(&self) -> Result<&[u8], AflError> {Ok("NOP".as_bytes())}
+        fn serialize(&self) -> Result<&[u8], AflError> {
+            Ok("NOP".as_bytes())
+        }
         fn deserialize(&mut self, _buf: &[u8]) -> Result<(), AflError> {
             Ok(())
         }
     }
 
-
     struct Nopserver {}
 
     impl Observer for Nopserver {

src/feedbacks/mod.rs

@@ -2,7 +2,5 @@ use crate::corpus::Testcase;
 use crate::executors::Executor;
 
 pub trait Feedback {
-
     fn is_interesting(&mut self, executor: &dyn Executor, entry: &dyn Testcase) -> f64;
-
 }

src/observers/mod.rs

@@ -1,6 +1,8 @@
 use crate::AflError;
 use std::slice::from_raw_parts_mut;
 
+/// Observers observe different information about the target.
+/// They can then be used by various sorts of feedback.
 pub trait Observer {
     fn flush(&mut self) -> Result<(), AflError> {
         Ok(())
@@ -13,33 +15,28 @@ pub trait Observer {
     }
 }
 
+/// A MapObserver contains a map with values, collected from the child.
 pub trait MapObserver<MapT>: Observer {
-
-    // TODO: Rust
     fn get_map(&self) -> &[MapT];
-    //fn get_map_mut(&mut self) -> &mut Vec<MapT>;
+    fn get_map_mut(&mut self) -> &mut [MapT];
-
-    fn get_map_mut(&mut self) -> &mut [u8];
-
 }
 
-pub struct U8MapObserver {
+/// A staticMapObserver observes the static map, as oftentimes used for afl-like coverage information
-    
+pub struct StaticMapObserver {
     map: &'static mut [u8],
-
 }
 
-impl Observer for U8MapObserver {
+impl Observer for StaticMapObserver {
     fn reset(&mut self) -> Result<(), AflError> {
-
+        // Normal memset, see https://rust.godbolt.org/z/Trs5hv
-        // TODO: Clear
+        for i in self.map.iter_mut() {
-        Err(AflError::Unknown)
+            *i = 0;
-
+        }
+        Ok(())
     }
 }
 
-impl MapObserver<u8> for U8MapObserver {
+impl MapObserver<u8> for StaticMapObserver {
-
     fn get_map(&self) -> &[u8] {
         self.map
     }
@@ -47,13 +44,15 @@ impl MapObserver<u8> for U8MapObserver {
     fn get_map_mut(&mut self) -> &mut [u8] {
         self.map
     }
-
 }
 
-impl U8MapObserver {
+impl StaticMapObserver {
+    /// Creates a new StaticMapObserver from a raw pointer.
     pub fn new(map_ptr: *mut u8, len: usize) -> Self {
         unsafe {
-            U8MapObserver{map: from_raw_parts_mut(map_ptr, len)}
+            StaticMapObserver {
+                map: from_raw_parts_mut(map_ptr, len),
+            }
         }
     }
 }