ligang/vpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

dpdk-cryptodev: fix coverity issues

Browse Source 
This patch addresses coverity issues CID 322716 and CID 322717.

Type: fix
Signed-off-by: Piotr Bronowski <piotrx.bronowski@intel.com>
Change-Id: I59d6f40c1af8e829d8cb3c042a52e144aeaf1e6b
This commit is contained in:
Piotr Bronowski
2023-08-21 13:36:38 +00:00
committed by Dave Wallace
parent 6d4b3fd19c
commit feda2ff640
2 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/plugins/dpdk/cryptodev/cryptodev.h
+1 -1
View File
@@ -293,7 +293,7 @@ cryptodev_mark_frame_fill_err (vnet_crypto_async_frame_t *f, u64 current_err,
for (i = index; i < (index + n); i++)
f->elts[i].status = op_s;
err |= (~(~(0u) << n) << index);
err |= (~(~(0ull) << n) << index);
return err;
}
src/plugins/dpdk/cryptodev/cryptodev_op_data_path.c
+7 -4
View File
@@ -515,10 +515,13 @@ cryptodev_frame_dequeue_internal (vlib_main_t *vm, u32 *nb_elts_processed,
fe[2].status = cryptodev_status_conversion[cop[2]->op.status];
fe[3].status = cryptodev_status_conversion[cop[3]->op.status];
err0 |= (fe[0].status == VNET_CRYPTO_OP_STATUS_COMPLETED) << n;
err1 |= (fe[1].status == VNET_CRYPTO_OP_STATUS_COMPLETED) << (n + 1);
err2 |= (fe[2].status == VNET_CRYPTO_OP_STATUS_COMPLETED) << (n + 2);
err3 |= (fe[3].status == VNET_CRYPTO_OP_STATUS_COMPLETED) << (n + 3);
err0 |= ((u64) (fe[0].status == VNET_CRYPTO_OP_STATUS_COMPLETED)) << n;
err1 |= ((u64) (fe[1].status == VNET_CRYPTO_OP_STATUS_COMPLETED))
<< (n + 1);
err2 |= ((u64) (fe[2].status == VNET_CRYPTO_OP_STATUS_COMPLETED))
<< (n + 2);
err3 |= ((u64) (fe[3].status == VNET_CRYPTO_OP_STATUS_COMPLETED))
<< (n + 3);
cop += 4;
fe += 4;