diff options
| author | atishkov <a.tishkov@aladdin.ru> | 2023-07-17 13:10:44 +0300 |
|---|---|---|
| committer | Tomas Mraz <tomas@openssl.org> | 2023-07-19 14:47:33 +0200 |
| commit | 8c34367e434c6b9555f21cc4fc77a18d6ef84a85 (patch) | |
| tree | e790b3d84edbf277fc2d9ce575cb3494cbb88aeb /apps | |
| parent | 4ec53ad6e1791daafbe26bdbd539f2ba9172959a (diff) | |
apps/cms.c: Fix unreachable code in cms_main()
Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/21474)
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/cms.c | 23 |
1 files changed, 10 insertions, 13 deletions
diff --git a/apps/cms.c b/apps/cms.c index 97ad9165e0..b22848299e 100644 --- a/apps/cms.c +++ b/apps/cms.c @@ -802,6 +802,9 @@ int cms_main(int argc, char **argv) if ((operation & SMIME_IP) == 0 && contfile != NULL) BIO_printf(bio_err, "Warning: -contfile option is ignored for the given operation\n"); + if (operation != SMIME_ENCRYPT && *argv != NULL) + BIO_printf(bio_err, + "Warning: recipient certificate file parameters ignored for operation other than -encrypt\n"); if ((flags & CMS_BINARY) != 0) { if (!(operation & SMIME_OP)) @@ -829,19 +832,13 @@ int cms_main(int argc, char **argv) goto end; } - if (*argv != NULL) { - if (operation == SMIME_ENCRYPT) { - for (; *argv != NULL; argv++) { - cert = load_cert(*argv, FORMAT_UNDEF, - "recipient certificate file"); - if (cert == NULL) - goto end; - sk_X509_push(encerts, cert); - cert = NULL; - } - } else { - BIO_printf(bio_err, "Warning: recipient certificate file parameters ignored for operation other than -encrypt\n"); - } + for (; *argv != NULL; argv++) { + cert = load_cert(*argv, FORMAT_UNDEF, + "recipient certificate file"); + if (cert == NULL) + goto end; + sk_X509_push(encerts, cert); + cert = NULL; } } |