diff options
| author | Pauli <ppzgs1@gmail.com> | 2021-03-19 09:35:05 +1000 |
|---|---|---|
| committer | Pauli <pauli@openssl.org> | 2021-03-26 08:46:02 +1000 |
| commit | 8cdcb63fc02239e92fc4bdcc5b97d38cc726c439 (patch) | |
| tree | f52bcd6cbdb933dda3e5c30b35da3d80e1852ed3 /apps | |
| parent | 3352a4f6fade5da0d0dfa9e7c9c9c261598c4bac (diff) | |
| download | openssl-8cdcb63fc02239e92fc4bdcc5b97d38cc726c439.tar.gz | |
apps: fix coverity 1455340: unchecked return value
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/14615)
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/lib/s_cb.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/apps/lib/s_cb.c b/apps/lib/s_cb.c index 4c209e76df..6824567c70 100644 --- a/apps/lib/s_cb.c +++ b/apps/lib/s_cb.c @@ -1276,12 +1276,14 @@ int config_ctx(SSL_CONF_CTX *cctx, STACK_OF(OPENSSL_STRING) *str, static int add_crls_store(X509_STORE *st, STACK_OF(X509_CRL) *crls) { X509_CRL *crl; - int i; + int i, ret = 1; + for (i = 0; i < sk_X509_CRL_num(crls); i++) { crl = sk_X509_CRL_value(crls, i); - X509_STORE_add_crl(st, crl); + if (!X509_STORE_add_crl(st, crl)) + ret = 0; } - return 1; + return ret; } int ssl_ctx_add_crls(SSL_CTX *ctx, STACK_OF(X509_CRL) *crls, int crl_download) |