[RTNETLINK]: ifindex 0 does not exist

ifindex == 0 does not exist and implies we should do a lookup by name if
one was given.

Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 3 insertions, 3 deletions

diff --git a/net/core/rtnetlink.c b/net/core/rtnetlink.c
index a8a5093a3001..02e8bf084277 100644
--- a/net/core/rtnetlink.c
+++ b/net/core/rtnetlink.c
@@ -580,7 +580,7 @@ static int rtnl_setlink(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg)
 
         err = -EINVAL;
         ifm = nlmsg_data(nlh);
-        if (ifm->ifi_index >= 0)
+        if (ifm->ifi_index > 0)
                 dev = dev_get_by_index(ifm->ifi_index);
         else if (tb[IFLA_IFNAME])
                 dev = dev_get_by_name(ifname);
@@ -672,7 +672,7 @@ static int rtnl_setlink(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg)
          * name provided implies that a name change has been
          * requested.
          */
-        if (ifm->ifi_index >= 0 && ifname[0]) {
+        if (ifm->ifi_index > 0 && ifname[0]) {
                 err = dev_change_name(dev, ifname);
                 if (err < 0)
                         goto errout_dev;
@@ -740,7 +740,7 @@ static int rtnl_getlink(struct sk_buff *skb, struct nlmsghdr* nlh, void *arg)
                 return err;
 
         ifm = nlmsg_data(nlh);
-        if (ifm->ifi_index >= 0) {
+        if (ifm->ifi_index > 0) {
                 dev = dev_get_by_index(ifm->ifi_index);
                 if (dev == NULL)
                         return -ENODEV;