diff options
author | Simo Sorce <simo@redhat.com> | 2022-11-14 10:25:15 -0500 |
---|---|---|
committer | Tomas Mraz <tomas@openssl.org> | 2022-11-16 17:07:05 +0100 |
commit | 5aaa3458e5d21898cec119f80ffec0b0023ce9c9 (patch) | |
tree | 99b1d7001767dbdaaefaba3dea59e5929b0b3f25 /apps | |
parent | 31d6565c846f84a0b3cb64c90cfc295c6f92ce6f (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.c | 8 |
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. |