From ce8d5bc635d26fd5c6d8bbc50fb5d38d39881531 Mon Sep 17 00:00:00 2001 From: Jeremy Long Date: Tue, 9 Dec 2014 06:36:04 -0500 Subject: [PATCH] hack resolution to issue #172 - more information is needed to fully resolve the issue though Former-commit-id: 366299924689cca0e277fd82b9f7bd40b9c19490 --- .../org/owasp/dependencycheck/dependency/Dependency.java | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/dependency-check-core/src/main/java/org/owasp/dependencycheck/dependency/Dependency.java b/dependency-check-core/src/main/java/org/owasp/dependencycheck/dependency/Dependency.java index 2f72b3d84..a82fcd076 100644 --- a/dependency-check-core/src/main/java/org/owasp/dependencycheck/dependency/Dependency.java +++ b/dependency-check-core/src/main/java/org/owasp/dependencycheck/dependency/Dependency.java @@ -610,7 +610,14 @@ public class Dependency implements Serializable, Comparable { * @param dependency a reference to the related dependency */ public void addRelatedDependency(Dependency dependency) { - relatedDependencies.add(dependency); + if (this == dependency) { + LOGGER.warning("Attempted to add a circular reference - please post the log file to issue #172 here " + + "https://github.com/jeremylong/DependencyCheck/issues/172 "); + LOGGER.log(Level.FINE, "this: {0}", this.toString()); + LOGGER.log(Level.FINE, "dependency: {0}", dependency.toString()); + } else { + relatedDependencies.add(dependency); + } } /**