summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authormiod <>2015-07-16 02:16:19 +0000
committermiod <>2015-07-16 02:16:19 +0000
commit207ad0ebbc1371cabc99dd9599f264886bba4d1a (patch)
tree7eb7ce1a157ce58b9472212b1e27a95bfd50b537
parent6e62847ce043c93089d341b8940940fcca3a4685 (diff)
downloadopenbsd-207ad0ebbc1371cabc99dd9599f264886bba4d1a.tar.gz
openbsd-207ad0ebbc1371cabc99dd9599f264886bba4d1a.tar.bz2
openbsd-207ad0ebbc1371cabc99dd9599f264886bba4d1a.zip
Check return value of all used functions in OCSP_REQUEST_print(); covers
Coverity CID 78796; ok beck@
-rw-r--r--src/lib/libcrypto/ocsp/ocsp_prn.c14
-rw-r--r--src/lib/libssl/src/crypto/ocsp/ocsp_prn.c14
2 files changed, 18 insertions, 10 deletions
diff --git a/src/lib/libcrypto/ocsp/ocsp_prn.c b/src/lib/libcrypto/ocsp/ocsp_prn.c
index 362d480926..37d033adb6 100644
--- a/src/lib/libcrypto/ocsp/ocsp_prn.c
+++ b/src/lib/libcrypto/ocsp/ocsp_prn.c
@@ -1,4 +1,4 @@
1/* $OpenBSD: ocsp_prn.c,v 1.7 2014/06/12 15:49:30 deraadt Exp $ */ 1/* $OpenBSD: ocsp_prn.c,v 1.8 2015/07/16 02:16:19 miod Exp $ */
2/* Written by Tom Titchener <Tom_Titchener@groove.net> for the OpenSSL 2/* Written by Tom Titchener <Tom_Titchener@groove.net> for the OpenSSL
3 * project. */ 3 * project. */
4 4
@@ -174,11 +174,15 @@ OCSP_REQUEST_print(BIO *bp, OCSP_REQUEST* o, unsigned long flags)
174 inf->requestExtensions, flags, 4)) 174 inf->requestExtensions, flags, 4))
175 goto err; 175 goto err;
176 if (sig) { 176 if (sig) {
177 X509_signature_print(bp, sig->signatureAlgorithm, 177 if (X509_signature_print(bp, sig->signatureAlgorithm,
178 sig->signature); 178 sig->signature) == 0)
179 goto err;
179 for (i = 0; i < sk_X509_num(sig->certs); i++) { 180 for (i = 0; i < sk_X509_num(sig->certs); i++) {
180 X509_print(bp, sk_X509_value(sig->certs, i)); 181 if (X509_print(bp, sk_X509_value(sig->certs, i)) == 0)
181 PEM_write_bio_X509(bp, sk_X509_value(sig->certs, i)); 182 goto err;
183 if (PEM_write_bio_X509(bp,
184 sk_X509_value(sig->certs, i)) == 0)
185 goto err;
182 } 186 }
183 } 187 }
184 return 1; 188 return 1;
diff --git a/src/lib/libssl/src/crypto/ocsp/ocsp_prn.c b/src/lib/libssl/src/crypto/ocsp/ocsp_prn.c
index 362d480926..37d033adb6 100644
--- a/src/lib/libssl/src/crypto/ocsp/ocsp_prn.c
+++ b/src/lib/libssl/src/crypto/ocsp/ocsp_prn.c
@@ -1,4 +1,4 @@
1/* $OpenBSD: ocsp_prn.c,v 1.7 2014/06/12 15:49:30 deraadt Exp $ */ 1/* $OpenBSD: ocsp_prn.c,v 1.8 2015/07/16 02:16:19 miod Exp $ */
2/* Written by Tom Titchener <Tom_Titchener@groove.net> for the OpenSSL 2/* Written by Tom Titchener <Tom_Titchener@groove.net> for the OpenSSL
3 * project. */ 3 * project. */
4 4
@@ -174,11 +174,15 @@ OCSP_REQUEST_print(BIO *bp, OCSP_REQUEST* o, unsigned long flags)
174 inf->requestExtensions, flags, 4)) 174 inf->requestExtensions, flags, 4))
175 goto err; 175 goto err;
176 if (sig) { 176 if (sig) {
177 X509_signature_print(bp, sig->signatureAlgorithm, 177 if (X509_signature_print(bp, sig->signatureAlgorithm,
178 sig->signature); 178 sig->signature) == 0)
179 goto err;
179 for (i = 0; i < sk_X509_num(sig->certs); i++) { 180 for (i = 0; i < sk_X509_num(sig->certs); i++) {
180 X509_print(bp, sk_X509_value(sig->certs, i)); 181 if (X509_print(bp, sk_X509_value(sig->certs, i)) == 0)
181 PEM_write_bio_X509(bp, sk_X509_value(sig->certs, i)); 182 goto err;
183 if (PEM_write_bio_X509(bp,
184 sk_X509_value(sig->certs, i)) == 0)
185 goto err;
182 } 186 }
183 } 187 }
184 return 1; 188 return 1;