Fix windows

.github/workflows/build_and_test.yml

@@ -38,8 +38,9 @@ jobs:
   Build-and-test-Fuzzer:
     strategy:
       matrix:
-        os: [ubuntu-latest] #[ubuntu-latest, windows-latest, macOS-latest]
-        # os: [ubuntu-latest, windows-latest] #[ubuntu-latest, windows-latest, macOS-latest]
+        # os: [ubuntu-latest] #[ubuntu-latest, windows-latest, macOS-latest]
+        os: [ubuntu-latest, windows-latest] #[ubuntu-latest, windows-latest, macOS-latest]
+        # os: [windows-latest] #[ubuntu-latest, windows-latest, macOS-latest]
     runs-on: ${{ matrix.os }}
     timeout-minutes: 45
     steps:
@@ -52,6 +53,10 @@ jobs:
       - uses: actions/checkout@v4
         name: Checkout repository
 
+      - name: use Windows developer command prompt
+        if: contains(matrix.os, 'windows-latest')
+        uses: ilammy/msvc-dev-cmd@v1
+
       - name: Install Rust stable
         uses: dtolnay/rust-toolchain@stable
 

crates/tauri-fuzz-cli/src/main.rs

@@ -1,5 +1,6 @@
 // Copyright 2023-2024 CrabNebula Ltd., Alexandre Dang
 // SPDX-License-Identifier: PolyForm-Noncommercial-1.0.0
+#![allow(clippy::single_match_else, clippy::semicolon_if_nothing_returned)]
 
 use clap::{ArgAction, Parser, Subcommand};
 use log::Level;
@@ -37,7 +38,7 @@ enum Commands {
     Fuzz(fuzz::Options),
 }
 
-fn main() -> anyhow::Result<()> {
+fn main() {
     let cli = Cli::parse_from(std::env::args_os().skip(1));
 
     if !cli.quiet {
@@ -55,8 +56,6 @@ fn main() -> anyhow::Result<()> {
         log::error!("{}", e);
         std::process::exit(1);
     }
-
-    Ok(())
 }
 
 fn setup_logger(cli: &Cli) -> anyhow::Result<()> {

crates/tauri-fuzz-cli/src/util.rs

@@ -72,7 +72,7 @@ pub fn tauri_dir() -> anyhow::Result<PathBuf> {
         } else if is_tauri_config_file(&path) {
             return path
                 .parent()
-                .map(|p| p.to_path_buf())
+                .map(Path::to_path_buf)
                 .context("failed to get parent from path");
         }
     }

crates/tauri-fuzz-policies/build.rs

@@ -15,7 +15,7 @@ fn add_manifest() {
     let mut manifest = std::env::current_dir().unwrap();
     manifest.push(WINDOWS_MANIFEST_FILE);
 
-    println!("cargo:rerun-if-changed={}", WINDOWS_MANIFEST_FILE);
+    println!("cargo:rerun-if-changed={WINDOWS_MANIFEST_FILE}");
     // Embed the Windows application manifest file.
     println!("cargo:rustc-link-arg=/MANIFEST:EMBED");
     println!(

crates/tauri-fuzz-policies/src/engine.rs

@@ -34,6 +34,12 @@ pub struct FunctionPolicy {
 
 impl FunctionPolicy {
     /// Check the function policy in the specified context and if the invocation should be blocked
+    ///
+    /// # Panics
+    /// This may panic if the `Rule` was not built correctly.
+    /// For example a `Rule` on entry with a context on exit.
+    /// Or trying to access registers that does not contain a parameter or a return value of the
+    /// target function
     pub fn should_block(&mut self, context: &Context) -> bool {
         let should_block = self.rule.should_block(context);
         match should_block {
@@ -62,17 +68,17 @@ impl FunctionPolicy {
 // maybe using cloneable `Box` can improve performance such as in this example:
 // https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=6ca48c4cff92370c907ecf4c548ee33c
 
-/// ConditionOnParameters is a closure on the registers containing the parameters of the function.
+/// `ConditionOnParameters` is a closure on the registers containing the parameters of the function.
 /// Registers can contain a value but also a pointer depending on the type of the parameters.
 pub type ConditionOnParameters = Arc<dyn Fn(&[usize]) -> Result<bool, RuleError>>;
-/// ConditionOnReturnValue is a closure on the value contained in the return value register
+/// `ConditionOnReturnValue` is a closure on the value contained in the return value register
 /// This can contain a value but also a pointer depending on the type of the return value.
 pub type ConditionOnReturnValue = Arc<dyn Fn(usize) -> Result<bool, RuleError>>;
-/// ConditionOnParameters but with an additional argument that can be used as storage to pass
+/// `ConditionOnParameters` but with an additional argument that can be used as storage to pass
 /// information for later usage
 pub type ConditionOnParametersWithStorage =
     Arc<dyn Fn(&[usize], &mut Option<usize>) -> Result<bool, RuleError>>;
-/// ConditionOnReturnValue but with an additional argument that can be used for the analysis
+/// `ConditionOnReturnValue` but with an additional argument that can be used for the analysis
 pub type ConditionOnReturnValueWithStorage =
     Arc<dyn Fn(usize, &mut Option<usize>) -> Result<bool, RuleError>>;
 
@@ -146,13 +152,12 @@ impl Debug for Context {
     fn fmt(&self, f: &mut Formatter<'_>) -> std::fmt::Result {
         match self {
             Context::EntryContext(parameters) => {
-                write!(f, "Function entry with parameters: {:?}", parameters)
+                write!(f, "Function entry with parameters: {parameters:?}")
             }
             Context::LeaveContext(return_value) => {
                 write!(
                     f,
-                    "Function exit with return value as usize: {:?}",
-                    return_value
+                    "Function exit with return value as usize: {return_value:?}"
                 )
             }
         }

crates/tauri-fuzz-policies/src/lib.rs

@@ -1,6 +1,14 @@
 // Copyright 2023-2024 CrabNebula Ltd., Alexandre Dang
 // SPDX-License-Identifier: PolyForm-Noncommercial-1.0.0
 
+#![allow(
+    clippy::must_use_candidate,
+    clippy::unnecessary_wraps,
+    clippy::missing_panics_doc,
+    clippy::wildcard_imports,
+    clippy::enum_glob_use
+)]
+
 pub mod engine;
 pub mod policies;
 pub use policies::*;