From 873dac8ec608ca02d59214b4a3022b2f25de717b Mon Sep 17 00:00:00 2001 From: Jerin Jacob Date: Wed, 16 May 2018 17:58:04 +0530 Subject: [PATCH] app/crypto-perf: fix parameters copy Since arm64 was using plain memcpy for rte_memcpy, gcc 8.1, could detect size was more than source address range. In this case, the source was wrong. test/test/test_cryptodev.c: In function 'test_multi_session_random_usage': rte_memcpy_64.h:364:29: error: 'memcpy' forming offset [113, 184] is out of the bounds [0, 112] of object 'testsuite_params' with type 'struct crypto_testsuite_params' [-Werror=array-bounds] #define rte_memcpy(d, s, n) memcpy((d), (s), (n)) ^~~~~~~~~~~~~~~~~~~~~ test/test/test_cryptodev.c:6618:3: note: in expansion of macro 'rte_memcpy' rte_memcpy(&ut_paramz[i].ut_params, &testsuite_params, ^~~~~~~~~~ test/test/test_cryptodev.c:140:39: note: 'testsuite_params' declared here static struct crypto_testsuite_params testsuite_params = { NULL }; Fixes: ffbe3be0d4b5 ("app/test: add libcrypto") Cc: stable@dpdk.org Suggested-by: Pablo de Lara Signed-off-by: Jerin Jacob Acked-by: Pablo de Lara --- test/test/test_cryptodev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/test/test_cryptodev.c b/test/test/test_cryptodev.c index 680082f2bf..940e3378a1 100644 --- a/test/test/test_cryptodev.c +++ b/test/test/test_cryptodev.c @@ -6615,7 +6615,7 @@ test_multi_session_random_usage(void) sessions[i] = rte_cryptodev_sym_session_create( ts_params->session_mpool); - rte_memcpy(&ut_paramz[i].ut_params, &testsuite_params, + rte_memcpy(&ut_paramz[i].ut_params, &unittest_params, sizeof(struct crypto_unittest_params)); test_AES_CBC_HMAC_SHA512_decrypt_create_session_params( -- 2.20.1