crypto/cnxk: fix status code in asymmetric operation

Message ID 20250216142510.1467-1-gmuthukrishn@marvell.com (mailing list archive)
State Accepted
Delegated to: akhil goyal
Headers
Series crypto/cnxk: fix status code in asymmetric operation |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/github-robot: build success github build: passed
ci/iol-unit-arm64-testing success Testing PASS
ci/iol-unit-amd64-testing success Testing PASS
ci/iol-abi-testing success Testing PASS
ci/iol-compile-amd64-testing success Testing PASS
ci/iol-sample-apps-testing success Testing PASS
ci/iol-compile-arm64-testing success Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/intel-Functional success Functional PASS

Commit Message

Gowrishankar Muthukrishnan Feb. 16, 2025, 2:25 p.m. UTC
Return error code in an asymmetric operation status
when none of the known conditions met.

Fixes: d29c4e0a4bea ("crypto/cnxk: fix ECDH public key verification")

Signed-off-by: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com>
---
 drivers/crypto/cnxk/cn9k_cryptodev_ops.c | 2 ++
 1 file changed, 2 insertions(+)
  

Comments

Akhil Goyal March 2, 2025, 8:28 p.m. UTC | #1
> Subject: [PATCH] crypto/cnxk: fix status code in asymmetric operation
> 
> Return error code in an asymmetric operation status
> when none of the known conditions met.
> 
> Fixes: d29c4e0a4bea ("crypto/cnxk: fix ECDH public key verification")
> 
> Signed-off-by: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com>
> ---
Applied to dpdk-next-crypto
Cc: stable@dpdk.org
  

Patch

diff --git a/drivers/crypto/cnxk/cn9k_cryptodev_ops.c b/drivers/crypto/cnxk/cn9k_cryptodev_ops.c
index 8d10bc9f9b..2b1bd70c6d 100644
--- a/drivers/crypto/cnxk/cn9k_cryptodev_ops.c
+++ b/drivers/crypto/cnxk/cn9k_cryptodev_ops.c
@@ -529,6 +529,8 @@  cn9k_cpt_dequeue_post_process(struct cnxk_cpt_qp *qp, struct rte_crypto_op *cop,
 						cop->status = RTE_CRYPTO_OP_STATUS_SUCCESS;
 						return;
 					}
+				} else {
+					cop->status = RTE_CRYPTO_OP_STATUS_ERROR;
 				}
 			} else
 				cop->status = RTE_CRYPTO_OP_STATUS_ERROR;