From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerin Jacob Subject: [PATCH 2/2] app/crypto-perf: fix memcpy source range Date: Wed, 16 May 2018 12:09:55 +0530 Message-ID: <20180516063955.6193-2-jerin.jacob@caviumnetworks.com> References: <20180516063955.6193-1-jerin.jacob@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain Cc: thomas@monjalon.net, Jerin Jacob , pablo.de.lara.guarch@intel.com, declan.doherty@intel.com, andy@warmcat.com, stable@dpdk.org To: dev@dpdk.org Return-path: In-Reply-To: <20180516063955.6193-1-jerin.jacob@caviumnetworks.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Since arm64 was using plain memcpy for rte_memcpy, gcc 8.1, could detect size was more than source address range. /export/dpdk.org/test/test/test_cryptodev.c: In function 'test_multi_session_random_usage': /export/dpdk.org/build/include/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)) ^~~~~~~~~~~~~~~~~~~~~ /export/dpdk.org/test/test/test_cryptodev.c:6618:3: note: in expansion of macro 'rte_memcpy' rte_memcpy(&ut_paramz[i].ut_params, &testsuite_params, ^~~~~~~~~~ /export/dpdk.org/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: pablo.de.lara.guarch@intel.com Cc: declan.doherty@intel.com Cc: andy@warmcat.com Cc: stable@dpdk.org Signed-off-by: Jerin Jacob --- 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 680082f2b..169ecdef3 100644 --- a/test/test/test_cryptodev.c +++ b/test/test/test_cryptodev.c @@ -6616,7 +6616,7 @@ test_multi_session_random_usage(void) ts_params->session_mpool); rte_memcpy(&ut_paramz[i].ut_params, &testsuite_params, - sizeof(struct crypto_unittest_params)); + sizeof(struct crypto_testsuite_params)); test_AES_CBC_HMAC_SHA512_decrypt_create_session_params( &ut_paramz[i].ut_params, -- 2.17.0