diff options
author | miod <> | 2015-07-16 02:16:19 +0000 |
---|---|---|
committer | miod <> | 2015-07-16 02:16:19 +0000 |
commit | 207ad0ebbc1371cabc99dd9599f264886bba4d1a (patch) | |
tree | 7eb7ce1a157ce58b9472212b1e27a95bfd50b537 | |
parent | 6e62847ce043c93089d341b8940940fcca3a4685 (diff) | |
download | openbsd-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.c | 14 | ||||
-rw-r--r-- | src/lib/libssl/src/crypto/ocsp/ocsp_prn.c | 14 |
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; |