From 458297bf56db6cb4c6cdfcdfa0ad619fb1d74a21 Mon Sep 17 00:00:00 2001 From: Will Stranathan Date: Sat, 1 Mar 2014 07:13:00 -0500 Subject: [PATCH 1/5] Fixed logging to JULI Former-commit-id: 5f5d9b29a6ba76f9193d47aa485b11cadb47bb67 --- .../dependencycheck/analyzer/AssemblyAnalyzerTest.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java b/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java index facb498be..a81d10395 100644 --- a/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java +++ b/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java @@ -17,7 +17,6 @@ */ package org.owasp.dependencycheck.analyzer; -import org.mortbay.log.Log; import org.owasp.dependencycheck.analyzer.exception.AnalysisException; import java.io.File; @@ -36,6 +35,8 @@ import org.owasp.dependencycheck.dependency.Dependency; import org.owasp.dependencycheck.dependency.Evidence; import org.owasp.dependencycheck.utils.Settings; +import com.sun.istack.internal.logging.Logger; + /** * Tests for the AssemblyAnalyzer. * @@ -43,6 +44,8 @@ import org.owasp.dependencycheck.utils.Settings; * */ public class AssemblyAnalyzerTest { + + private static final Logger LOGGER = Logger.getLogger(AssemblyAnalyzerTest.class); AssemblyAnalyzer analyzer; @@ -57,7 +60,7 @@ public class AssemblyAnalyzerTest { analyzer = new AssemblyAnalyzer(); analyzer.initialize(); } catch (Exception e) { - Log.warn("Exception setting up AssemblyAnalyzer. Tests will be incomplete"); + LOGGER.warning("Exception setting up AssemblyAnalyzer. Tests will be incomplete", e); Assume.assumeNoException("Is mono installed? TESTS WILL BE INCOMPLETE", e); } } From 3bc17e7b832b79bf38cac62f41d90909710f06dc Mon Sep 17 00:00:00 2001 From: Will Stranathan Date: Sat, 1 Mar 2014 07:16:38 -0500 Subject: [PATCH 2/5] Fixed JULI Logging (stupid fix imports) Former-commit-id: 65aa4381b27717ec7bc766f37705fd63941849d1 --- .../analyzer/AssemblyAnalyzerTest.java | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java b/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java index a81d10395..d640f9cb8 100644 --- a/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java +++ b/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java @@ -17,26 +17,24 @@ */ package org.owasp.dependencycheck.analyzer; -import org.owasp.dependencycheck.analyzer.exception.AnalysisException; - -import java.io.File; - -import org.junit.After; - import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; import static org.junit.Assume.assumeFalse; +import java.io.File; +import java.util.logging.Level; +import java.util.logging.Logger; + +import org.junit.After; import org.junit.Assume; import org.junit.Before; import org.junit.Test; +import org.owasp.dependencycheck.analyzer.exception.AnalysisException; import org.owasp.dependencycheck.dependency.Confidence; import org.owasp.dependencycheck.dependency.Dependency; import org.owasp.dependencycheck.dependency.Evidence; import org.owasp.dependencycheck.utils.Settings; -import com.sun.istack.internal.logging.Logger; - /** * Tests for the AssemblyAnalyzer. * @@ -45,7 +43,7 @@ import com.sun.istack.internal.logging.Logger; */ public class AssemblyAnalyzerTest { - private static final Logger LOGGER = Logger.getLogger(AssemblyAnalyzerTest.class); + private static final Logger LOGGER = Logger.getLogger(AssemblyAnalyzerTest.class.getName()); AssemblyAnalyzer analyzer; @@ -60,7 +58,7 @@ public class AssemblyAnalyzerTest { analyzer = new AssemblyAnalyzer(); analyzer.initialize(); } catch (Exception e) { - LOGGER.warning("Exception setting up AssemblyAnalyzer. Tests will be incomplete", e); + LOGGER.log(Level.WARNING, "Exception setting up AssemblyAnalyzer. Tests will be incomplete", e); Assume.assumeNoException("Is mono installed? TESTS WILL BE INCOMPLETE", e); } } From 03b06eee67b625e9888e5f66de6dc6cbc059950d Mon Sep 17 00:00:00 2001 From: Will Stranathan Date: Sat, 1 Mar 2014 07:13:00 -0500 Subject: [PATCH 3/5] Fixed logging to JULI Former-commit-id: b8155251fa7120e33a042115f36ecac05fc7dce2 --- .../dependencycheck/analyzer/AssemblyAnalyzerTest.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java b/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java index facb498be..a81d10395 100644 --- a/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java +++ b/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java @@ -17,7 +17,6 @@ */ package org.owasp.dependencycheck.analyzer; -import org.mortbay.log.Log; import org.owasp.dependencycheck.analyzer.exception.AnalysisException; import java.io.File; @@ -36,6 +35,8 @@ import org.owasp.dependencycheck.dependency.Dependency; import org.owasp.dependencycheck.dependency.Evidence; import org.owasp.dependencycheck.utils.Settings; +import com.sun.istack.internal.logging.Logger; + /** * Tests for the AssemblyAnalyzer. * @@ -43,6 +44,8 @@ import org.owasp.dependencycheck.utils.Settings; * */ public class AssemblyAnalyzerTest { + + private static final Logger LOGGER = Logger.getLogger(AssemblyAnalyzerTest.class); AssemblyAnalyzer analyzer; @@ -57,7 +60,7 @@ public class AssemblyAnalyzerTest { analyzer = new AssemblyAnalyzer(); analyzer.initialize(); } catch (Exception e) { - Log.warn("Exception setting up AssemblyAnalyzer. Tests will be incomplete"); + LOGGER.warning("Exception setting up AssemblyAnalyzer. Tests will be incomplete", e); Assume.assumeNoException("Is mono installed? TESTS WILL BE INCOMPLETE", e); } } From bf3bc83fd8fdfd031cd840769125118dd02786ff Mon Sep 17 00:00:00 2001 From: Will Stranathan Date: Sat, 1 Mar 2014 07:16:38 -0500 Subject: [PATCH 4/5] Fixed JULI Logging (stupid fix imports) Former-commit-id: d29bba7ac4ccdf648d9a945e728e9d0dbc301b6e --- .../analyzer/AssemblyAnalyzerTest.java | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java b/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java index a81d10395..d640f9cb8 100644 --- a/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java +++ b/dependency-check-core/src/test/java/org/owasp/dependencycheck/analyzer/AssemblyAnalyzerTest.java @@ -17,26 +17,24 @@ */ package org.owasp.dependencycheck.analyzer; -import org.owasp.dependencycheck.analyzer.exception.AnalysisException; - -import java.io.File; - -import org.junit.After; - import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; import static org.junit.Assume.assumeFalse; +import java.io.File; +import java.util.logging.Level; +import java.util.logging.Logger; + +import org.junit.After; import org.junit.Assume; import org.junit.Before; import org.junit.Test; +import org.owasp.dependencycheck.analyzer.exception.AnalysisException; import org.owasp.dependencycheck.dependency.Confidence; import org.owasp.dependencycheck.dependency.Dependency; import org.owasp.dependencycheck.dependency.Evidence; import org.owasp.dependencycheck.utils.Settings; -import com.sun.istack.internal.logging.Logger; - /** * Tests for the AssemblyAnalyzer. * @@ -45,7 +43,7 @@ import com.sun.istack.internal.logging.Logger; */ public class AssemblyAnalyzerTest { - private static final Logger LOGGER = Logger.getLogger(AssemblyAnalyzerTest.class); + private static final Logger LOGGER = Logger.getLogger(AssemblyAnalyzerTest.class.getName()); AssemblyAnalyzer analyzer; @@ -60,7 +58,7 @@ public class AssemblyAnalyzerTest { analyzer = new AssemblyAnalyzer(); analyzer.initialize(); } catch (Exception e) { - LOGGER.warning("Exception setting up AssemblyAnalyzer. Tests will be incomplete", e); + LOGGER.log(Level.WARNING, "Exception setting up AssemblyAnalyzer. Tests will be incomplete", e); Assume.assumeNoException("Is mono installed? TESTS WILL BE INCOMPLETE", e); } } From eefc6a5567b03841a9ae2f7bb42cdad0c5c6b266 Mon Sep 17 00:00:00 2001 From: Will Stranathan Date: Sat, 1 Mar 2014 07:23:46 -0500 Subject: [PATCH 5/5] Added proper waitfor to the process. Need to add a watchdog to this Former-commit-id: ff4b0b1f2c1254449e63eb660c1dbd31cef21c3b --- .../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;