[dpdk-stable] [PATCH] examples/l2fwd-crypto: fix digest with AEAD algorithms
De Lara Guarch, Pablo
pablo.de.lara.guarch at intel.com
Mon Jul 23 19:56:16 CEST 2018
> -----Original Message-----
> From: Luca Boccassi [mailto:bluca at debian.org]
> Sent: Monday, July 23, 2018 6:09 PM
> To: De Lara Guarch, Pablo <pablo.de.lara.guarch at intel.com>;
> ankur.dwivedi at cavium.com; Doherty, Declan <declan.doherty at intel.com>
> Cc: stable at dpdk.org
> Subject: Re: [dpdk-stable] [PATCH] examples/l2fwd-crypto: fix digest with AEAD
> algorithms
>
> On Mon, 2018-07-16 at 07:25 +0100, Pablo de Lara wrote:
> > When performing authentication verification (both for AEAD algorithms,
> > such as AES-GCM, or for authentication algorithms, such as SHA1-
> > HMAC), the digest address is calculated based on the packet size and
> > the algorithm used (substracting digest size and IP header to the
> > packet size).
> >
> > However, for AEAD algorithms, this was not calculated correctly, since
> > the digest size was not being substracted.
> >
> > Bugzilla ID: 44
> > Fixes: 2661f4fbe93d ("examples/l2fwd-crypto: add AEAD parameters")
> > Cc: stable at dpdk.org
> >
> > Reported-by: Ankur Dwivedi <ankur.dwivedi at cavium.com>
> > Signed-off-by: Pablo de Lara <pablo.de.lara.guarch at intel.com>
> > ---
> > examples/l2fwd-crypto/main.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/examples/l2fwd-crypto/main.c b/examples/l2fwd-
> > crypto/main.c index 9d6bb7857..9ac06a697 100644
> > --- a/examples/l2fwd-crypto/main.c
> > +++ b/examples/l2fwd-crypto/main.c
> > @@ -408,7 +408,7 @@ l2fwd_simple_crypto_enqueue(struct rte_mbuf *m,
> > /* Zero pad data to be crypto'd so it is block aligned */
> > data_len = rte_pktmbuf_data_len(m) - ipdata_offset;
> >
> > - if (cparams->do_hash && cparams->hash_verify)
> > + if ((cparams->do_hash || cparams->do_aead) && cparams-
> > >hash_verify)
> > data_len -= cparams->digest_length;
> >
> > if (cparams->do_cipher) {
>
> Hi Pablo,
>
> For which stable release branch is this patch intended? Has it (or an
> equivalent) been merged into mainline?
Hi Luca,
This is not merged yet into mainline, only in dpdk-next-crypto.
This patch was applied in 17.08, so this should target all stable branches from that version.
>
> --
> Kind regards,
> Luca Boccassi
More information about the stable
mailing list