[dpdk-dev] crypto/openssl: fix aad capabilities for AES-GCM

Message ID 1493131248-20969-1-git-send-email-arkadiuszx.kusztal@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Pablo de Lara Guarch
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Arkadiusz Kusztal April 25, 2017, 2:40 p.m. UTC
  Fix aad capabilities for AES-GCM authtentication, aad is changed from
range 8-12 to 0-65535

Fixes: 8a9867a635c0 ("crypto/openssl: rename libcrypto to openssl")

Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@intel.com>
---
 drivers/crypto/openssl/rte_openssl_pmd_ops.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
  

Comments

De Lara Guarch, Pablo April 26, 2017, 12:12 p.m. UTC | #1
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Arek Kusztal
> Sent: Tuesday, April 25, 2017 3:41 PM
> To: dev@dpdk.org
> Cc: Doherty, Declan; Kusztal, ArkadiuszX
> Subject: [dpdk-dev] [PATCH] crypto/openssl: fix aad capabilities for AES-
> GCM
> 
> Fix aad capabilities for AES-GCM authtentication, aad is changed from
> range 8-12 to 0-65535
> 
> Fixes: 8a9867a635c0 ("crypto/openssl: rename libcrypto to openssl")
> 
> Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@intel.com>

Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
  
De Lara Guarch, Pablo April 26, 2017, 2:16 p.m. UTC | #2
> -----Original Message-----
> From: De Lara Guarch, Pablo
> Sent: Wednesday, April 26, 2017 1:13 PM
> To: Kusztal, ArkadiuszX; dev@dpdk.org
> Cc: Doherty, Declan; Kusztal, ArkadiuszX
> Subject: RE: [dpdk-dev] [PATCH] crypto/openssl: fix aad capabilities for AES-
> GCM
> 
> 
> 
> > -----Original Message-----
> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Arek Kusztal
> > Sent: Tuesday, April 25, 2017 3:41 PM
> > To: dev@dpdk.org
> > Cc: Doherty, Declan; Kusztal, ArkadiuszX
> > Subject: [dpdk-dev] [PATCH] crypto/openssl: fix aad capabilities for AES-
> > GCM
> >
> > Fix aad capabilities for AES-GCM authtentication, aad is changed from
> > range 8-12 to 0-65535
> >
> > Fixes: 8a9867a635c0 ("crypto/openssl: rename libcrypto to openssl")
> >
> > Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@intel.com>
> 
> Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

Added Cc: stable@dpdk.org

Applied to dpdk-next-crypto.
Thanks,

Pablo
  

Patch

diff --git a/drivers/crypto/openssl/rte_openssl_pmd_ops.c b/drivers/crypto/openssl/rte_openssl_pmd_ops.c
index 25d1a4b..7cb21f8 100644
--- a/drivers/crypto/openssl/rte_openssl_pmd_ops.c
+++ b/drivers/crypto/openssl/rte_openssl_pmd_ops.c
@@ -350,9 +350,9 @@  static const struct rte_cryptodev_capabilities openssl_pmd_capabilities[] = {
 					.increment = 0
 				},
 				.aad_size = {
-					.min = 8,
-					.max = 12,
-					.increment = 4
+					.min = 0,
+					.max = 65535,
+					.increment = 1
 				}
 			}, }
 		}, }