On Thu, Apr 30, 2020 at 6:45 AM Lance Richardson < [email protected]> wrote:
> On Thu, Apr 30, 2020 at 9:37 AM Yuan Linsi <[email protected]> wrote: > > > > From: yuanlinsi01 <[email protected]> > > > > 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: [email protected] > > > > Signed-off-by: yuanlinsi01 <[email protected]> > > Signed-off-by: rongdongsheng <[email protected]> > > Thanks for the fix! > > Acked-by: Lance Richardson <[email protected]> > Patch applied to dpdk-next-net-brcm. Thanks

