summaryrefslogtreecommitdiffstats
path: root/apps
diff options
context:
space:
mode:
authorSimo Sorce <simo@redhat.com>2022-11-14 10:25:15 -0500
committerTomas Mraz <tomas@openssl.org>2022-11-16 17:07:05 +0100
commit5aaa3458e5d21898cec119f80ffec0b0023ce9c9 (patch)
tree99b1d7001767dbdaaefaba3dea59e5929b0b3f25 /apps
parent31d6565c846f84a0b3cb64c90cfc295c6f92ce6f (diff)
Drop explicit check for engines in opt_legacy_okay
The providers indication should always indicate that this is not a legacy request. This makes a check for engines redundant as the default return is that legacy is ok if there are no explicit providers. Fixes #19662 Signed-off-by: Simo Sorce <simo@redhat.com> Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com> Reviewed-by: Paul Dale <pauli@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/19671) (cherry picked from commit 2fea56832780248af2aba2e4433ece2d18428515)
Diffstat (limited to 'apps')
-rw-r--r--apps/lib/apps.c8
1 files changed, 0 insertions, 8 deletions
diff --git a/apps/lib/apps.c b/apps/lib/apps.c
index dd72b7fea0..78acaccad4 100644
--- a/apps/lib/apps.c
+++ b/apps/lib/apps.c
@@ -3371,14 +3371,6 @@ int opt_legacy_okay(void)
{
int provider_options = opt_provider_option_given();
int libctx = app_get0_libctx() != NULL || app_get0_propq() != NULL;
-#ifndef OPENSSL_NO_ENGINE
- ENGINE *e = ENGINE_get_first();
-
- if (e != NULL) {
- ENGINE_free(e);
- return 1;
- }
-#endif
/*
* Having a provider option specified or a custom library context or
* property query, is a sure sign we're not using legacy.