On Thu, May 4, 2023 at 10:54 AM <[email protected]> wrote: > > From: Jerin Jacob <[email protected]> > > Fix the following build issue by not allowing nb_ids to be zero. > nb_ids can be zero based on rte_rawdev_xstats_get() API > documentation but it is not valid for the case when second > argument is NULL. > > examples/ntb/ntb_fwd.c: In function 'ntb_stats_display': > examples/ntb/ntb_fwd.c:945:23: error: 'rte_rawdev_xstats_get' > accessing 8 bytes in a region of size 0 [-Werror=stringop-overflow=] > 945 | if (nb_ids != rte_rawdev_xstats_get(dev_id, ids, values, nb_ids)) { > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > examples/ntb/ntb_fwd.c:945:23: note: referencing argument 3 > of type 'uint64_t[0]' {aka 'long unsigned int[]'} > In file included from ../examples/ntb/ntb_fwd.c:17: > lib/rawdev/rte_rawdev.h:504:1: note: in a call to function > 'rte_rawdev_xstats_get' > 504 | rte_rawdev_xstats_get(uint16_t dev_id, > > Fixes: 5194299d6ef5 ("examples/ntb: support more functions") > Cc: [email protected]
> Signed-off-by: Jerin Jacob <[email protected]> > Tested-by: Ali Alnubani <[email protected]> Acked-by: Junfeng Guo <[email protected]> Tested-by: Daxue Gao <[email protected]> Applied, thanks. -- David Marchand
