On Wed, Jun 27, 2018 at 11:20:52AM +0200, Adrien Mazarguil wrote: > Since its return type is unsigned, if_nametoindex() returns 0 in case of > error, never -1. > > Fixes: ccdcba53a3f4 ("net/mlx5: use Netlink to add/remove MAC addresses") > Cc: Nelio Laranjeiro <nelio.laranje...@6wind.com> > Cc: sta...@dpdk.org > > Signed-off-by: Adrien Mazarguil <adrien.mazarg...@6wind.com> Acked-by: Nelio Laranjeiro <nelio.laranje...@6wind.com>
> --- > drivers/net/mlx5/mlx5_ethdev.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/mlx5/mlx5_ethdev.c b/drivers/net/mlx5/mlx5_ethdev.c > index 90488af33..ebe5cb6e3 100644 > --- a/drivers/net/mlx5/mlx5_ethdev.c > +++ b/drivers/net/mlx5/mlx5_ethdev.c > @@ -192,13 +192,13 @@ int > mlx5_ifindex(const struct rte_eth_dev *dev) > { > char ifname[IF_NAMESIZE]; > - int ret; > + unsigned int ret; > > ret = mlx5_get_ifname(dev, &ifname); > if (ret) > return ret; > ret = if_nametoindex(ifname); > - if (ret == -1) { > + if (ret == 0) { > rte_errno = errno; > return -rte_errno; > } > -- > 2.11.0 Thanks, -- Nélio Laranjeiro 6WIND