Add process.destroyForcibly()

jfrog · Jun 19, 2023 · 1df3414 · 1df3414
1 parent 93ef08c
commit 1df3414
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/build-info-extractor/src/main/java/org/jfrog/build/extractor/executor/CommandExecutor.java b/build-info-extractor/src/main/java/org/jfrog/build/extractor/executor/CommandExecutor.java
@@ -159,6 +159,9 @@ public CommandResults exeCommand(File execDir, List<String> args, List<String> c
                 boolean outputReaderTerminatedProperly = service.awaitTermination(TIMEOUT_SECONDS, TimeUnit.SECONDS);
                 boolean terminatedProperly = executionTerminatedProperly && outputReaderTerminatedProperly;
                 return getCommandResults(terminatedProperly, args, inputStreamReader.getOutput(), errorStreamReader.getOutput(), process.exitValue());
+            } finally {
+                // Ensure termination of the subprocess we have created.
+                process.destroyForcibly();
             }
         } finally {
             service.shutdownNow();