Fixed spurrious eror message

GSA · Feb 8, 2021 · 7ddfaa8 · 7ddfaa8
1 parent 08f601e
commit 7ddfaa8
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/...b/src/main/java/gov/gsa/pivconformance/conformancelib/tests/PKIX_X509DataObjectTests.java b/...b/src/main/java/gov/gsa/pivconformance/conformancelib/tests/PKIX_X509DataObjectTests.java
@@ -257,12 +257,16 @@ void PKIX_Test_6(String oid, String containersAndPolicyOids, TestReporter report
 					s_logger.debug(validator.toString());
 					// Validate cert to trust anchor in default keystore
 					valid = validator.isValid(eeCert, allowedPolicies[1], null);
-					s_logger.debug(validator.toString());
-					if (validator.getCertPath() != null) {
-						validator.dumpCertPath(true);
+					if (valid == true)  {
+						s_logger.debug("Certificate is valid");
+						s_logger.debug(validator.toString());
+						if (validator.getCertPath() != null) {
+							validator.dumpCertPath(true);
+						} else {
+							failMsg = "Certificate is invalid";
+							s_logger.error(failMsg);
+						}
 					}
-					failMsg = "Certificate is invalid";
-					s_logger.error(failMsg);
 				} catch (Exception e) {
 					failMsg = e.getMessage();
 					s_logger.error(failMsg);