crypto: fixed vnet_crypto_is_set_handler() 90/30690/5
authorPiotrX Kleski <piotrx.kleski@intel.com>
Mon, 11 Jan 2021 08:59:31 +0000 (08:59 +0000)
committerNeale Ranns <neale@graphiant.com>
Wed, 20 Jan 2021 11:04:46 +0000 (11:04 +0000)
Type: fix

vnet_crypto_is_set_handler() was checking if op handler is set by
using alg instead of op id.
This fix changes the check to use op ids from crypto_main's alg_data
for provided alg.

Signed-off-by: PiotrX Kleski <piotrx.kleski@intel.com>
Change-Id: I5f0f00a889ddada44d9912af175ebad66677037a

src/vnet/crypto/crypto.c

index 26fc494..e52e68e 100644 (file)
@@ -189,8 +189,17 @@ int
 vnet_crypto_is_set_handler (vnet_crypto_alg_t alg)
 {
   vnet_crypto_main_t *cm = &crypto_main;
+  vnet_crypto_op_id_t opt = 0;
+  int i;
+
+  if (alg > vec_len (cm->algs))
+    return 0;
+
+  for (i = 0; i < VNET_CRYPTO_OP_N_TYPES; i++)
+    if ((opt = cm->algs[alg].op_by_type[i]) != 0)
+      break;
 
-  return (alg < vec_len (cm->ops_handlers) && NULL != cm->ops_handlers[alg]);
+  return NULL != cm->ops_handlers[opt];
 }
 
 void