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> --- 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