patch 'test/crypto: fix AES-ECB test lengths' has been queued to stable release 24.11.2

Kevin Traynor ktraynor at redhat.com
Fri Mar 7 13:47:06 CET 2025


Hi,

FYI, your patch has been queued to stable release 24.11.2

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 03/12/25. 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/kevintraynor/dpdk-stable

This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable/commit/171bc627e1dca1e5b9de3abaee4ca468e7912ef5

Thanks.

Kevin

---
>From 171bc627e1dca1e5b9de3abaee4ca468e7912ef5 Mon Sep 17 00:00:00 2001
From: Shaokai Zhang <felix.zhang at jaguarmicro.com>
Date: Thu, 16 Jan 2025 16:35:48 +0800
Subject: [PATCH] test/crypto: fix AES-ECB test lengths

[ upstream commit 4d6b2633b030f6b469c78fd00f012c8192979332 ]

The test vector aes_test_data_19 plaintext is 192 bytes and this vector
is used to test 192-byte plaintext, but test vector plaintext.len
value is incorrectly assigned to 64.

The test vector aes_test_data_20 plaintext length is 256 bytes, but
testcase comment incorrectly describes it as "192-byte plaintext".

Fixes: c94c520b4163 ("crypto/aesni_mb: support AES-ECB")

Signed-off-by: Shaokai Zhang <felix.zhang at jaguarmicro.com>
Reviewed-by: Joey Xing <joey.xing at jaguarmicro.com>
Acked-by: Akhil Goyal <gakhil at marvell.com>
---
 .mailmap                                   | 1 +
 app/test/test_cryptodev_aes_test_vectors.h | 6 +++---
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/.mailmap b/.mailmap
index 0ee46fdb0f..ab00a189cb 100644
--- a/.mailmap
+++ b/.mailmap
@@ -1389,4 +1389,5 @@ Shani Peretz <shperetz at nvidia.com>
 Shannon Nelson <snelson at pensando.io>
 Shannon Zhao <zhaoshenglong at huawei.com>
+Shaokai Zhang <felix.zhang at jaguarmicro.com>
 Shaopeng He <shaopeng.he at intel.com>
 Shaowei Sun <1819846787 at qq.com>
diff --git a/app/test/test_cryptodev_aes_test_vectors.h b/app/test/test_cryptodev_aes_test_vectors.h
index 7127156cc4..505e603b7e 100644
--- a/app/test/test_cryptodev_aes_test_vectors.h
+++ b/app/test/test_cryptodev_aes_test_vectors.h
@@ -834,9 +834,9 @@ static const struct blockcipher_test_data aes_test_data_19 = {
 	.plaintext = {
 		.data = plaintext_aes192ecb_192bytes,
-		.len = 64
+		.len = 192
 	},
 	.ciphertext = {
 		.data = ciphertext_aes192ecb_192bytes,
-		.len = 64
+		.len = 192
 	}
 };
@@ -5104,5 +5104,5 @@ static const struct blockcipher_test_case aes_cipheronly_test_cases[] = {
 	},
 	{
-		.test_descr = "AES-256-ECB Decryption (192-byte plaintext)",
+		.test_descr = "AES-256-ECB Decryption (256-byte plaintext)",
 		.test_data = &aes_test_data_20,
 		.op_mask = BLOCKCIPHER_TEST_OP_DECRYPT,
-- 
2.48.1

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2025-03-07 11:02:58.119535669 +0000
+++ 0033-test-crypto-fix-AES-ECB-test-lengths.patch	2025-03-07 11:02:56.902335837 +0000
@@ -1 +1 @@
-From 4d6b2633b030f6b469c78fd00f012c8192979332 Mon Sep 17 00:00:00 2001
+From 171bc627e1dca1e5b9de3abaee4ca468e7912ef5 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 4d6b2633b030f6b469c78fd00f012c8192979332 ]
+
@@ -14 +15,0 @@
-Cc: stable at dpdk.org
@@ -25 +26 @@
-index 3ea758ca12..b3ffe83f46 100644
+index 0ee46fdb0f..ab00a189cb 100644
@@ -28 +29 @@
-@@ -1397,4 +1397,5 @@ Shani Peretz <shperetz at nvidia.com>
+@@ -1389,4 +1389,5 @@ Shani Peretz <shperetz at nvidia.com>



More information about the stable mailing list