> -----Original Message----- > From: Akhil Goyal [mailto:akhil.go...@nxp.com] > Sent: Friday, October 13, 2017 4:45 PM > To: alok.makhar...@nxp.com; dev@dpdk.org > Cc: De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>; > hemant.agra...@nxp.com > Subject: Re: [PATCH v2] crypto/dpaa_sec: add check for segmented buffer > > On 10/13/2017 9:07 PM, alok.makhar...@nxp.com wrote: > > From: Alok Makhariya <alok.makhar...@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.makhar...@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.go...@nxp.com>
Applied to dpdk-next-crypto. Thanks, Pablo