|
 |
f0b236 |
From c9c326e1878a377ce4193aaa4b1b41cb711b5e48 Mon Sep 17 00:00:00 2001
|
|
|
f0b236 |
From: Rob Crittenden <rcritten@redhat.com>
|
|
|
f0b236 |
Date: Thu, 30 Apr 2020 12:46:41 -0400
|
|
|
f0b236 |
Subject: [PATCH] Address an include issue discovered by coverity
|
|
|
f0b236 |
|
|
|
f0b236 |
nspr.h isn't included so use PORT_ErrorToString() instead
|
|
|
f0b236 |
of PR_ErrorToString(), and remain consistent with the
|
|
|
f0b236 |
other PORT calls even though they directly translate
|
|
|
f0b236 |
to their NSPR equivalents.
|
|
|
f0b236 |
|
|
|
f0b236 |
Also remove a couple of unused variables in pkcs7.c
|
|
|
f0b236 |
---
|
|
|
f0b236 |
src/pkcs7.c | 8 +++++---
|
|
|
f0b236 |
1 file changed, 5 insertions(+), 3 deletions(-)
|
|
|
f0b236 |
|
|
|
f0b236 |
diff --git a/src/pkcs7.c b/src/pkcs7.c
|
|
|
f0b236 |
index f81174f..a569256 100644
|
|
|
f0b236 |
--- a/src/pkcs7.c
|
|
|
f0b236 |
+++ b/src/pkcs7.c
|
|
|
f0b236 |
@@ -57,6 +57,9 @@
|
|
|
f0b236 |
#define _(_text) (_text)
|
|
|
f0b236 |
#endif
|
|
|
f0b236 |
|
|
|
f0b236 |
+/* taken from nspr4.h */
|
|
|
f0b236 |
+#define PR_LANGUAGE_I_DEFAULT 0 /* i-default, the default language */
|
|
|
f0b236 |
+
|
|
|
f0b236 |
/* Return 0 if we think "issuer" could have issued "issued", which includes
|
|
|
f0b236 |
* self-signing. */
|
|
|
f0b236 |
static int
|
|
|
f0b236 |
@@ -289,7 +292,7 @@ log_pkcs7_errors(int level, char *msg)
|
|
|
f0b236 |
}
|
|
|
f0b236 |
nss_err = PORT_GetError();
|
|
|
f0b236 |
if (nss_err < 0) {
|
|
|
f0b236 |
- cm_log(level, "%d: %s\n", nss_err, PR_ErrorToString(nss_err, 0));
|
|
|
f0b236 |
+ cm_log(level, "%d: %s\n", nss_err, PORT_ErrorToString(nss_err));
|
|
|
f0b236 |
}
|
|
|
f0b236 |
}
|
|
|
f0b236 |
|
|
|
f0b236 |
@@ -929,9 +932,8 @@ cm_pkcs7_verify_signed(unsigned char *data, size_t length,
|
|
|
f0b236 |
PKCS7_SIGNER_INFO *si;
|
|
|
f0b236 |
BIO *in, *out = NULL;
|
|
|
f0b236 |
const unsigned char *u;
|
|
|
f0b236 |
- char *s, buf[LINE_MAX], *p, *q;
|
|
|
f0b236 |
+ char *s, *p, *q;
|
|
|
f0b236 |
int ret = -1, i;
|
|
|
f0b236 |
- long error;
|
|
|
f0b236 |
|
|
|
f0b236 |
if (digest != NULL) {
|
|
|
f0b236 |
*digest = NULL;
|
|
|
f0b236 |
--
|
|
|
f0b236 |
2.21.1
|
|
|
f0b236 |
|