DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] examples/fips_validation: fix parsing of cipher length for AES-GCM
@ 2020-01-06  9:33 Archana Muniganti
  2020-01-06  9:38 ` Akhil Goyal
  2020-01-07  7:16 ` Anoob Joseph
  0 siblings, 2 replies; 5+ messages in thread
From: Archana Muniganti @ 2020-01-06  9:33 UTC (permalink / raw)
  To: marko.kovacevic, roy.fan.zhang, akhil.goyal
  Cc: Sucharitha Sarananaga, anoobj, akamaluddin, jerinj, dev, stable,
	Archana Muniganti

From: Sucharitha Sarananaga <ssarananaga@marvell.com>

Cipher length need to be updated in case of AES-GCM decryption.

Fixes: 4aaad2995e13 ("examples/fips_validation: support GCM parsing")

Signed-off-by: Archana Muniganti <marchana@marvell.com>
Signed-off-by: Sucharitha Sarananaga <ssarananaga@marvell.com>
---
 examples/fips_validation/fips_validation_gcm.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/examples/fips_validation/fips_validation_gcm.c b/examples/fips_validation/fips_validation_gcm.c
index ea48ddf..f295025 100644
--- a/examples/fips_validation/fips_validation_gcm.c
+++ b/examples/fips_validation/fips_validation_gcm.c
@@ -19,6 +19,7 @@
 #define PTLEN_STR	"PTlen = "
 #define AADLEN_STR	"AADlen = "
 #define TAGLEN_STR	"Taglen = "
+#define CTLEN_STR	"PTlen = "
 
 #define COUNT_STR	"Count = "
 #define KEY_STR		"Key = "
@@ -46,6 +47,7 @@ struct fips_test_callback gcm_interim_vectors[] = {
 		{KEYLEN_STR, parser_read_uint32_bit_val, &vec.aead.key},
 		{IVLEN_STR, parser_read_uint32_bit_val, &vec.iv},
 		{PTLEN_STR, parser_read_uint32_bit_val, &vec.pt},
+		{CTLEN_STR, parser_read_uint32_bit_val, &vec.ct},
 		{AADLEN_STR, parser_read_uint32_bit_val, &vec.aead.aad},
 		{TAGLEN_STR, parser_read_uint32_bit_val,
 				&vec.aead.digest},
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] examples/fips_validation: fix parsing of cipher length for AES-GCM
  2020-01-06  9:33 [dpdk-dev] [PATCH] examples/fips_validation: fix parsing of cipher length for AES-GCM Archana Muniganti
@ 2020-01-06  9:38 ` Akhil Goyal
  2020-01-06 13:44   ` Archana Muniganti
  2020-01-07  7:16 ` Anoob Joseph
  1 sibling, 1 reply; 5+ messages in thread
From: Akhil Goyal @ 2020-01-06  9:38 UTC (permalink / raw)
  To: Archana Muniganti, marko.kovacevic, roy.fan.zhang
  Cc: Sucharitha Sarananaga, anoobj, akamaluddin, jerinj, dev, stable

> From: Sucharitha Sarananaga <ssarananaga@marvell.com>
> 
> Cipher length need to be updated in case of AES-GCM decryption.
> 
> Fixes: 4aaad2995e13 ("examples/fips_validation: support GCM parsing")
> 
> Signed-off-by: Archana Muniganti <marchana@marvell.com>
> Signed-off-by: Sucharitha Sarananaga <ssarananaga@marvell.com>
> ---
>  examples/fips_validation/fips_validation_gcm.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/examples/fips_validation/fips_validation_gcm.c
> b/examples/fips_validation/fips_validation_gcm.c
> index ea48ddf..f295025 100644
> --- a/examples/fips_validation/fips_validation_gcm.c
> +++ b/examples/fips_validation/fips_validation_gcm.c
> @@ -19,6 +19,7 @@
>  #define PTLEN_STR	"PTlen = "
>  #define AADLEN_STR	"AADlen = "
>  #define TAGLEN_STR	"Taglen = "
> +#define CTLEN_STR	"PTlen = "
Typo. Should be CTlen

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] examples/fips_validation: fix parsing of cipher length for AES-GCM
  2020-01-06  9:38 ` Akhil Goyal
@ 2020-01-06 13:44   ` Archana Muniganti
  0 siblings, 0 replies; 5+ messages in thread
From: Archana Muniganti @ 2020-01-06 13:44 UTC (permalink / raw)
  To: Akhil Goyal, marko.kovacevic, roy.fan.zhang
  Cc: Sucharitha Sarananaga, Anoob Joseph, Abed Mohammad Kamaluddin,
	Jerin Jacob Kollanukkaran, dev, stable

Hi Akhil,

Please see inline.

Thanks,
Archana

> -----Original Message-----
> From: Akhil Goyal <akhil.goyal@nxp.com>
> Sent: Monday, January 6, 2020 3:09 PM
> To: Archana Muniganti <marchana@marvell.com>;
> marko.kovacevic@intel.com; roy.fan.zhang@intel.com
> Cc: Sucharitha Sarananaga <ssarananaga@marvell.com>; Anoob Joseph
> <anoobj@marvell.com>; Abed Mohammad Kamaluddin
> <akamaluddin@marvell.com>; Jerin Jacob Kollanukkaran
> <jerinj@marvell.com>; dev@dpdk.org; stable@dpdk.org
> Subject: [EXT] RE: [PATCH] examples/fips_validation: fix parsing of cipher
> length for AES-GCM
> 
> External Email
> 
> ----------------------------------------------------------------------
> > From: Sucharitha Sarananaga <ssarananaga@marvell.com>
> >
> > Cipher length need to be updated in case of AES-GCM decryption.
> >
> > Fixes: 4aaad2995e13 ("examples/fips_validation: support GCM parsing")
> >
> > Signed-off-by: Archana Muniganti <marchana@marvell.com>
> > Signed-off-by: Sucharitha Sarananaga <ssarananaga@marvell.com>
> > ---
> >  examples/fips_validation/fips_validation_gcm.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/examples/fips_validation/fips_validation_gcm.c
> > b/examples/fips_validation/fips_validation_gcm.c
> > index ea48ddf..f295025 100644
> > --- a/examples/fips_validation/fips_validation_gcm.c
> > +++ b/examples/fips_validation/fips_validation_gcm.c
> > @@ -19,6 +19,7 @@
> >  #define PTLEN_STR	"PTlen = "
> >  #define AADLEN_STR	"AADlen = "
> >  #define TAGLEN_STR	"Taglen = "
> > +#define CTLEN_STR	"PTlen = "
> Typo. Should be CTlen
[Archana] NIST request file for AES-GCM test vectors contains 'PTlen =' field.
This field holds plain text length when encrypt test vectors are used. And same
field holds cipher text length when decrypt test vectors are used.
Hence added a new macro that will be used in AES-GCM 'decryption'

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] examples/fips_validation: fix parsing of cipher length for AES-GCM
  2020-01-06  9:33 [dpdk-dev] [PATCH] examples/fips_validation: fix parsing of cipher length for AES-GCM Archana Muniganti
  2020-01-06  9:38 ` Akhil Goyal
@ 2020-01-07  7:16 ` Anoob Joseph
  2020-02-05 12:29   ` Akhil Goyal
  1 sibling, 1 reply; 5+ messages in thread
From: Anoob Joseph @ 2020-01-07  7:16 UTC (permalink / raw)
  To: Archana Muniganti, marko.kovacevic, roy.fan.zhang, akhil.goyal
  Cc: Sucharitha Sarananaga, Abed Mohammad Kamaluddin,
	Jerin Jacob Kollanukkaran, dev, stable, Archana Muniganti


> -----Original Message-----
> From: Archana Muniganti <marchana@marvell.com>
> Sent: Monday, January 6, 2020 3:03 PM
> To: marko.kovacevic@intel.com; roy.fan.zhang@intel.com;
> akhil.goyal@nxp.com
> Cc: Sucharitha Sarananaga <ssarananaga@marvell.com>; Anoob Joseph
> <anoobj@marvell.com>; Abed Mohammad Kamaluddin
> <akamaluddin@marvell.com>; Jerin Jacob Kollanukkaran <jerinj@marvell.com>;
> dev@dpdk.org; stable@dpdk.org; Archana Muniganti
> <marchana@marvell.com>
> Subject: [PATCH] examples/fips_validation: fix parsing of cipher length for AES-
> GCM
> 
> From: Sucharitha Sarananaga <ssarananaga@marvell.com>
> 
> Cipher length need to be updated in case of AES-GCM decryption.
> 
> Fixes: 4aaad2995e13 ("examples/fips_validation: support GCM parsing")
> 
> Signed-off-by: Archana Muniganti <marchana@marvell.com>
> Signed-off-by: Sucharitha Sarananaga <ssarananaga@marvell.com>
 
Acked-by: Anoob Joseph <anoobj@marvell.com>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] examples/fips_validation: fix parsing of cipher length for AES-GCM
  2020-01-07  7:16 ` Anoob Joseph
@ 2020-02-05 12:29   ` Akhil Goyal
  0 siblings, 0 replies; 5+ messages in thread
From: Akhil Goyal @ 2020-02-05 12:29 UTC (permalink / raw)
  To: Anoob Joseph, Archana Muniganti, marko.kovacevic, roy.fan.zhang
  Cc: Sucharitha Sarananaga, Abed Mohammad Kamaluddin,
	Jerin Jacob Kollanukkaran, dev, stable, Archana Muniganti

> >
> > From: Sucharitha Sarananaga <ssarananaga@marvell.com>
> >
> > Cipher length need to be updated in case of AES-GCM decryption.
> >
> > Fixes: 4aaad2995e13 ("examples/fips_validation: support GCM parsing")
> >
> > Signed-off-by: Archana Muniganti <marchana@marvell.com>
> > Signed-off-by: Sucharitha Sarananaga <ssarananaga@marvell.com>
> 
> Acked-by: Anoob Joseph <anoobj@marvell.com>

Applied to dpdk-next-crypto

Thanks.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-02-05 12:29 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-06  9:33 [dpdk-dev] [PATCH] examples/fips_validation: fix parsing of cipher length for AES-GCM Archana Muniganti
2020-01-06  9:38 ` Akhil Goyal
2020-01-06 13:44   ` Archana Muniganti
2020-01-07  7:16 ` Anoob Joseph
2020-02-05 12:29   ` Akhil Goyal

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).