12/11/2018 10:29, Gaëtan Rivet: > On Mon, Nov 12, 2018 at 12:58:56AM +0100, Thomas Monjalon wrote: > > If the last part of the PCI address (function number) is missing, > > the parsing was successful, assuming function 0. > > The call to strtoul is not returning an error in such a case, > > so an explicit check is inserted before. > > > > This bug has always been there in older parsing macros: > > - GET_PCIADDR_FIELD > > - GET_BLACKLIST_FIELD > > > > Returning 0, that's understandable, but errno not being set > is rather lame from strtoul. > > > Fixes: af75078fece3 ("first public release") > > Cc: sta...@dpdk.org > > > > Reported-by: Wisam Jaddo <wis...@mellanox.com> > > Signed-off-by: Thomas Monjalon <tho...@monjalon.net> > Acked-by: Gaetan Rivet <gaetan.ri...@6wind.com>
Applied