From 6e5efb27b33f9c3b6478728ece5e89d80f7e8994 Mon Sep 17 00:00:00 2001 From: Akash Saxena Date: Thu, 25 Oct 2018 10:01:01 +0000 Subject: [PATCH] test/crypto: remove redundant RSA verification Change unit test app to check only for op->status = RTE_CRYPTO_OP_STATUS_SUCCESS/ERROR instead of calling rsa_verify(). as the cryptodev API is expected to return error incase of data mismatch. Signed-off-by: Ayuj Verma Signed-off-by: Akash Saxena Signed-off-by: Shally Verma Acked-by: Akhil Goyal --- test/test/test_cryptodev_asym.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/test/test/test_cryptodev_asym.c b/test/test/test_cryptodev_asym.c index 2fdfc1df03..a899f99735 100644 --- a/test/test/test_cryptodev_asym.c +++ b/test/test/test_cryptodev_asym.c @@ -153,10 +153,13 @@ test_rsa_sign_verify(void) goto error_exit; } status = TEST_SUCCESS; - int ret = 0; - ret = rsa_verify(&rsaplaintext, result_op); - if (ret) + if (result_op->status != RTE_CRYPTO_OP_STATUS_SUCCESS) { + RTE_LOG(ERR, USER1, + "line %u FAILED: %s", + __LINE__, "Failed to process asym crypto op"); status = TEST_FAILED; + goto error_exit; + } error_exit: -- 2.20.1