Merge pull request #6 from AikidoSec/clarify-return-statement-for-she…

…ll-injection Clarify return statement for detect_shell_injection
AikidoSec · Sep 20, 2024 · 991cebc · 991cebc
2 parents 543a618 + 6c4b698
commit 991cebc
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/ffi_bindings/mod.rs b/src/ffi_bindings/mod.rs
@@ -34,8 +34,11 @@ pub extern "C" fn detect_shell_injection(
 
     let command_str = str::from_utf8(command_bytes).unwrap();
     let userinput_str = str::from_utf8(userinput_bytes).unwrap();
+
+    // Returns an integer value, representing a boolean (1 = true, 0 = false).
     if detect_shell_injection_stringified(command_str, userinput_str) {
-        return 1;
+        1
+    } else {
+        0
     }
-    0
 }