summaryrefslogtreecommitdiff
path: root/drivers/net/mlx5/mlx5_ethdev.c
diff options
context:
space:
mode:
authorAdrien Mazarguil <adrien.mazarguil@6wind.com>2018-06-27 11:20:52 +0200
committerFerruh Yigit <ferruh.yigit@intel.com>2018-07-03 01:35:58 +0200
commitf264c7980ca5b5f574c5f4c93644daf702124e4d (patch)
tree5f2024bbd3e09e20d8bf7e4c46876d005f73025f /drivers/net/mlx5/mlx5_ethdev.c
parentf9aaa6ac444615586729e6f25b04a6689e9d1a44 (diff)
downloaddpdk-f264c7980ca5b5f574c5f4c93644daf702124e4d.zip
dpdk-f264c7980ca5b5f574c5f4c93644daf702124e4d.tar.gz
dpdk-f264c7980ca5b5f574c5f4c93644daf702124e4d.tar.xz
net/mlx5: fix invalid error check
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: stable@dpdk.org Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com> Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
Diffstat (limited to 'drivers/net/mlx5/mlx5_ethdev.c')
-rw-r--r--drivers/net/mlx5/mlx5_ethdev.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/mlx5/mlx5_ethdev.c b/drivers/net/mlx5/mlx5_ethdev.c
index 90488af..ebe5cb6 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;
}