test/crypto: remove redundant RSA verification
authorAkash Saxena <akash.saxena@caviumnetworks.com>
Thu, 25 Oct 2018 10:01:01 +0000 (10:01 +0000)
committerAkhil Goyal <akhil.goyal@nxp.com>
Fri, 2 Nov 2018 11:25:39 +0000 (12:25 +0100)
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 <ayuj.verma@caviumnetworks.com>
Signed-off-by: Akash Saxena <akash.saxena@caviumnetworks.com>
Signed-off-by: Shally Verma <shally.verma@caviumnetworks.com>
Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
test/test/test_cryptodev_asym.c

index 2fdfc1d..a899f99 100644 (file)
@@ -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: