[dpdk-stable] patch 'test/crypto: fix build with GCC 11' has been queued to stable release 19.11.9

Christian Ehrhardt christian.ehrhardt at canonical.com
Thu Jun 10 14:06:32 CEST 2021


Hi,

FYI, your patch has been queued to stable release 19.11.9

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 06/12/21. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Queued patches are on a temporary branch at:
https://github.com/cpaelzer/dpdk-stable-queue

This queued commit can be viewed at:
https://github.com/cpaelzer/dpdk-stable-queue/commit/0c3ad62e9ea44fd59b81872ed10e0653f0d18e2c

Thanks.

Christian Ehrhardt <christian.ehrhardt at canonical.com>

---
>From 0c3ad62e9ea44fd59b81872ed10e0653f0d18e2c Mon Sep 17 00:00:00 2001
From: Kevin Traynor <ktraynor at redhat.com>
Date: Wed, 5 May 2021 09:53:14 +0100
Subject: [PATCH] test/crypto: fix build with GCC 11

[ upstream commit 8f73a72378333a68ada948ad648f9c6f925af58b ]

Fix the allocation for sessions, to prevent an array-bounds
warning with GCC 11. Set the not created session to NULL.

Fixes: 202d375c60bc ("app/test: add cryptodev unit and performance tests")

Signed-off-by: Kevin Traynor <ktraynor at redhat.com>
Acked-by: Akhil Goyal <gakhil at marvell.com>
---
 app/test/test_cryptodev.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index 84666b5b23..1047c5e038 100644
--- a/app/test/test_cryptodev.c
+++ b/app/test/test_cryptodev.c
@@ -9661,8 +9661,8 @@ test_multi_session(void)
 	rte_cryptodev_info_get(ts_params->valid_devs[0], &dev_info);
 
 	sessions = rte_malloc(NULL,
-			(sizeof(struct rte_cryptodev_sym_session *) *
-			MAX_NB_SESSIONS) + 1, 0);
+			sizeof(struct rte_cryptodev_sym_session *) *
+			(MAX_NB_SESSIONS + 1), 0);
 
 	/* Create multiple crypto sessions*/
 	for (i = 0; i < MAX_NB_SESSIONS; i++) {
@@ -9707,6 +9707,7 @@ test_multi_session(void)
 		}
 	}
 
+	sessions[i] = NULL;
 	/* Next session create should fail */
 	rte_cryptodev_sym_session_init(ts_params->valid_devs[0],
 			sessions[i], &ut_params->auth_xform,
-- 
2.31.1

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2021-06-10 14:05:00.136672335 +0200
+++ 0044-test-crypto-fix-build-with-GCC-11.patch	2021-06-10 14:04:58.086024792 +0200
@@ -1 +1 @@
-From 8f73a72378333a68ada948ad648f9c6f925af58b Mon Sep 17 00:00:00 2001
+From 0c3ad62e9ea44fd59b81872ed10e0653f0d18e2c Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 8f73a72378333a68ada948ad648f9c6f925af58b ]
+
@@ -10 +11,0 @@
-Cc: stable at dpdk.org
@@ -19 +20 @@
-index c68684b80b..39db52b17a 100644
+index 84666b5b23..1047c5e038 100644
@@ -22 +23 @@
-@@ -10748,8 +10748,8 @@ test_multi_session(void)
+@@ -9661,8 +9661,8 @@ test_multi_session(void)
@@ -33 +34 @@
-@@ -10794,6 +10794,7 @@ test_multi_session(void)
+@@ -9707,6 +9707,7 @@ test_multi_session(void)


More information about the stable mailing list