[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