From 887a5d50a4d997c629fd91b776471982b540d713 Mon Sep 17 00:00:00 2001 From: Will Stranathan Date: Sat, 1 Mar 2014 07:23:46 -0500 Subject: [PATCH] Added proper waitfor to the process. Need to add a watchdog to this Former-commit-id: 51bea69bb73e379fa7aac776171110fa1f75b8d4 --- .../org/owasp/dependencycheck/analyzer/AssemblyAnalyzer.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dependency-check-core/src/main/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzer.java b/dependency-check-core/src/main/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzer.java index 063a89f7f..b2c38857a 100644 --- a/dependency-check-core/src/main/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzer.java +++ b/dependency-check-core/src/main/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzer.java @@ -200,7 +200,7 @@ public class AssemblyAnalyzer extends AbstractAnalyzer { final Document doc = DocumentBuilderFactory.newInstance().newDocumentBuilder().parse(p.getInputStream()); final XPath xpath = XPathFactory.newInstance().newXPath(); final String error = xpath.evaluate("/assembly/error", doc); - if (p.exitValue() != 1 || error == null || "".equals(error)) { + if (p.waitFor() != 1 || error == null || "".equals(error)) { LOG.warning("An error occured with the .NET AssemblyAnalyzer, please see the log for more details."); LOG.fine("GrokAssembly.exe is not working properly"); grokAssemblyExe = null;