Blame SOURCES/0021-On-PKCS-7-verify-failures-log-the-PKCS-7-file-fix-va.patch

1495bf
From 6627c9d346b887016afa92664f690a0310d4ce00 Mon Sep 17 00:00:00 2001
1495bf
From: Rob Crittenden <rcritten@redhat.com>
1495bf
Date: Mon, 2 Apr 2018 13:31:32 -0400
1495bf
Subject: [PATCH 21/25] On PKCS#7 verify failures log the PKCS#7 file, fix
1495bf
 variable used
1495bf
1495bf
results was being used in place of results2.
1495bf
1495bf
In practice it would be the result of GetCACaps which means it would
1495bf
log _something_, just not the failed PKCS#7 file.
1495bf
---
1495bf
 src/scep.c | 7 +++++--
1495bf
 1 file changed, 5 insertions(+), 2 deletions(-)
1495bf
1495bf
diff --git a/src/scep.c b/src/scep.c
1495bf
index 0dbdcd7..5dd362d 100644
1495bf
--- a/src/scep.c
1495bf
+++ b/src/scep.c
1495bf
@@ -920,15 +920,18 @@ main(int argc, const char **argv)
1495bf
 			if (i != 0) {
1495bf
 				printf(_("Error: failed to verify signature on "
1495bf
 					 "server response.\n"));
1495bf
+				cm_log(1, "Error: failed to verify signature on "
1495bf
+					 "server response.\n");
1495bf
 				while ((error = ERR_get_error()) != 0) {
1495bf
 					memset(buf, '\0', sizeof(buf));
1495bf
 					ERR_error_string_n(error, buf, sizeof(buf));
1495bf
 					cm_log(1, "%s\n", buf);
1495bf
 				}
1495bf
-				s = cm_store_base64_from_bin(ctx, (unsigned char *) results,
1495bf
-							     results_length);
1495bf
+				s = cm_store_base64_from_bin(ctx, (unsigned char *) results2,
1495bf
+							     results_length2);
1495bf
 				s = cm_submit_u_pem_from_base64("PKCS7", 0, s);
1495bf
 				fprintf(stderr, "%s", s);
1495bf
+				cm_log(1, "%s", s);
1495bf
 				free(s);
1495bf
 				return CM_SUBMIT_STATUS_UNREACHABLE;
1495bf
 			}
1495bf
-- 
1495bf
1.8.3.1
1495bf