On Thu, Apr 30, 2020 at 6:45 AM Lance Richardson <
lance.richard...@broadcom.com> wrote:
> On Thu, Apr 30, 2020 at 9:37 AM Yuan Linsi <yuanlins...@baidu.com> wrote:
> >
> > From: yuanlinsi01 <yuanlins...@baidu.com>
> >
> > We see a stack smashing as a result of defensive code missing. Once the
> > nb_pkts is less than RTE_BNXT_DESCS_PER_LOOP, it will be modified to
> > zero after doing a floor align, and we can not exit the following
> > receiving packets loop. And the buffers will be overwrite, then the
> > stack frame was ruined.
> >
> > Fix the problem by adding defensive code, once the nb_pkts is zero, just
> > directly return with no packets.
> >
> > Fixes: bc4a000f2 ("net/bnxt: implement SSE vector mode")
> > Cc: sta...@dpdk.org
> >
> > Signed-off-by: yuanlinsi01 <yuanlins...@baidu.com>
> > Signed-off-by: rongdongsheng <rongdongsh...@baidu.com>
>
> Thanks for the fix!
>
> Acked-by: Lance Richardson <lance.richard...@broadcom.com>
>
Patch applied to dpdk-next-net-brcm. Thanks
