Removed JMP hooks until it's stable

memN0ps · Jan 28, 2024 · 010301e · 010301e
1 parent 6bf11a3
commit 010301e
Showing 1 changed file with 0 additions and 29 deletions.
diff --git a/hypervisor/src/utils/function_hook.rs b/hypervisor/src/utils/function_hook.rs
@@ -67,35 +67,6 @@ impl FunctionHook {
     pub fn new(original_address: u64, hook_address: u64, handler: *const ()) -> Option<Self> {
         log::debug!("Setting up hooks");
 
-        // Create the different trampolines. There's a few different ones available:
-        // - 1 Byte: CC shellcode
-        // - 14 Bytes: JMP shellcode
-        //
-        #[cfg(feature = "shellcode-hook")]
-        let (hook_type, trampoline) =
-            match Self::trampoline_shellcode(original_address, hook_address, JMP_SHELLCODE_LEN) {
-                Ok(trampoline) => (HookType::Jmp, trampoline),
-                Err(error) => {
-                    log::warn!("Failed to create jmp trampoline: {:?}", error);
-
-                    // If jmp trampoline didn't work, let's try this one:
-                    //
-                    let trampoline = Self::trampoline_shellcode(
-                        original_address,
-                        hook_address,
-                        BP_SHELLCODE_LEN,
-                    )
-                    .map_err(|e| {
-                        log::warn!("Failed to create bp trampoline: {:?}", e);
-                        e
-                    })
-                    .ok()?;
-
-                    (HookType::Breakpoint, trampoline)
-                }
-            };
-
-        #[cfg(not(feature = "shellcode-hook"))]
         let (hook_type, trampoline) = {
             let trampoline =
                 Self::trampoline_shellcode(original_address, hook_address, BP_SHELLCODE_LEN)