[dpdk-dev] [PATCH v2] crypto/dpaa_sec: add check for segmented buffer
De Lara Guarch, Pablo
pablo.de.lara.guarch at intel.com
Mon Oct 16 16:07:02 CEST 2017
> -----Original Message-----
> From: Akhil Goyal [mailto:akhil.goyal at nxp.com]
> Sent: Friday, October 13, 2017 4:45 PM
> To: alok.makhariya at nxp.com; dev at dpdk.org
> Cc: De Lara Guarch, Pablo <pablo.de.lara.guarch at intel.com>;
> hemant.agrawal at nxp.com
> Subject: Re: [PATCH v2] crypto/dpaa_sec: add check for segmented buffer
>
> On 10/13/2017 9:07 PM, alok.makhariya at nxp.com wrote:
> > From: Alok Makhariya <alok.makhariya at nxp.com>
> >
> > The code would crash in case of segmented buffer if no check
> >
> > Fixes: 89aaf708d44b ("crypto/dpaa_sec: add crypto driver for NXP DPAA
> platform")
> >
> > Signed-off-by: Alok Makhariya <alok.makhariya at nxp.com>
> > ---
> > drivers/crypto/dpaa_sec/dpaa_sec.c | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/drivers/crypto/dpaa_sec/dpaa_sec.c
> b/drivers/crypto/dpaa_sec/dpaa_sec.c
> > index 7b9a683..bd13d8c 100644
> > --- a/drivers/crypto/dpaa_sec/dpaa_sec.c
> > +++ b/drivers/crypto/dpaa_sec/dpaa_sec.c
> > @@ -893,6 +893,13 @@ dpaa_sec_enqueue_op(struct rte_crypto_op
> *op, struct dpaa_sec_qp *qp)
> > return ret;
> > }
> >
> > + /*
> > + * Segmented buffer is not supported.
> > + */
> > + if (!rte_pktmbuf_is_contiguous(op->sym->m_src)) {
> > + op->status = RTE_CRYPTO_OP_STATUS_ERROR;
> > + return -ENOTSUP;
> > + }
> > if (is_auth_only(ses)) {
> > cf = build_auth_only(op, ses);
> > } else if (is_cipher_only(ses)) {
> >
> Acked-by: Akhil Goyal <akhil.goyal at nxp.com>
Applied to dpdk-next-crypto.
Thanks,
Pablo
More information about the dev
mailing list