[openssl] master update
Richard Levitte
levitte at openssl.org
Sat May 16 10:09:15 UTC 2020
The branch master has been updated
via 827f04d5105e9bec0af214c42b8ad799fba5bb0d (commit)
from 0b2b0be948404cefe7160c9b1096bc554e982f03 (commit)
- Log -----------------------------------------------------------------
commit 827f04d5105e9bec0af214c42b8ad799fba5bb0d
Author: Richard Levitte <levitte at openssl.org>
Date: Fri May 15 07:50:43 2020 +0200
CORE: Fix a couple of bugs in algorithm_do_this()
The call of ossl_provider_query_operation() used |data->operation_id|,
when |cur_operation| should be used.
If any ossl_provider_query_operation() call returned NULL, the loop
was stopped, when it should just continue on to the next operation.
Reviewed-by: Matt Caswell <matt at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/11837)
-----------------------------------------------------------------------
Summary of changes:
crypto/core_algorithm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/crypto/core_algorithm.c b/crypto/core_algorithm.c
index 2973b37604..79625fdea6 100644
--- a/crypto/core_algorithm.c
+++ b/crypto/core_algorithm.c
@@ -37,11 +37,11 @@ static int algorithm_do_this(OSSL_PROVIDER *provider, void *cbdata)
cur_operation <= last_operation;
cur_operation++) {
const OSSL_ALGORITHM *map =
- ossl_provider_query_operation(provider, data->operation_id,
+ ossl_provider_query_operation(provider, cur_operation,
&no_store);
if (map == NULL)
- break;
+ continue;
ok = 1; /* As long as we've found *something* */
while (map->algorithm_names != NULL) {
More information about the openssl-commits
mailing list