fixed deepsource issues

Author: Akshit Maheshwary

api_app/analyzers_manager/file_analyzers/capa_info.py

@@ -67,12 +67,20 @@ def _download_signatures(cls) -> None:
         for signature in signatures_list:
             try:
                 process = subprocess.run(
-                    ["wget", "-O", SIGNATURE_LOCATION, signature["download_url"]]
+                    [
+                        "/usr/bin/wget",
+                        "-O",
+                        SIGNATURE_LOCATION,
+                        signature["download_url"],
+                    ],
+                    check=True,
                 )
+
             except subprocess.CalledProcessError as e:
+                stderr = process.stderr
                 logger.error(f"Failed to download signature: {e}")
                 raise AnalyzerRunException(
-                    f"Failed to update signatures due to error: {process.stderr}"
+                    f"Failed to update signatures due to error: {stderr}"
                 )
         logger.info("Successfully updated singatures")
 
@@ -109,7 +117,7 @@ def run(self):
                     "Couldn't update capa rules or signatures successfully"
                 )
 
-            command: list[str] = ["capa", "--quiet", "--json"]
+            command: list[str] = ["/usr/local/bin/capa", "--quiet", "--json"]
             shell_code_arch = "sc64" if self.arch == "64" else "sc32"
             if self.shellcode:
                 command.append("-f")
@@ -128,17 +136,21 @@ def run(self):
             logger.info(f"Starting CAPA analysis for {self.filename}")
 
             process: subprocess.CompletedProcess = subprocess.run(
-                command, capture_output=True, text=True, timeout=self.timeout
+                command,
+                capture_output=True,
+                text=True,
+                timeout=self.timeout,
+                check=True,
             )
 
-            process.check_returncode()
             result = json.loads(process.stdout)
             logger.info("CAPA analysis successfully completed")
 
         except subprocess.CalledProcessError as e:
+            stderr = process.stderr
             logger.info(f"Capa Info failed to run for {self.filename} with command {e}")
             raise AnalyzerRunException(
-                f" Analyzer for {self.filename} failed with error: {process.stderr}"
+                f" Analyzer for {self.filename} failed with error: {stderr}"
             )
 
         return result

api_app/analyzers_manager/file_analyzers/floss.py

@@ -38,19 +38,26 @@ def run(self):
         # retrieve them with more simple tools
         try:
             process: subprocess.CompletedProcess = subprocess.run(
-                ["floss", "--json", "--no", "static", "--", self.filepath],
+                [
+                    "/usr/local/bin/floss",
+                    "--json",
+                    "--no",
+                    "static",
+                    "--",
+                    self.filepath,
+                ],
                 capture_output=True,
                 text=True,
+                check=True,
             )
 
-            process.check_returncode()
-
             result = loads(process.stdout)
 
         except subprocess.CalledProcessError as e:
+            stderr = process.stderr
             logger.info(f"Floss failed to run for {self.filename} with command {e}")
             raise AnalyzerRunException(
-                f" Analyzer for {self.filename} failed with error: {process.stderr}"
+                f" Analyzer for {self.filename} failed with error: {stderr}"
             )
 
         result["exceeded_max_number_of_strings"] = {}