net/ipv6: Do not allow route add with a device that is down
authorDavid Ahern <dsahern@gmail.com>
Thu, 25 Jan 2018 03:45:29 +0000 (19:45 -0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 25 Jan 2018 21:22:02 +0000 (16:22 -0500)
IPv6 allows routes to be installed when the device is not up (admin up).
Worse, it does not mark it as LINKDOWN. IPv4 does not allow it and really
there is no reason for IPv6 to allow it, so check the flags and deny if
device is admin down.

Signed-off-by: David Ahern <dsahern@gmail.com>
Reviewed-by: Ido Schimmel <idosch@mellanox.com>
Reviewed-by: Roopa Prabhu <roopa@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/route.c

index f85da2f1e7291632cbdd90ca90c211e01c6a9236..aa4411c81e7e3fba1d0a1d3cd96e732354c548bb 100644 (file)
@@ -2734,6 +2734,12 @@ static struct rt6_info *ip6_route_info_create(struct fib6_config *cfg,
        if (!dev)
                goto out;
 
+       if (!(dev->flags & IFF_UP)) {
+               NL_SET_ERR_MSG(extack, "Nexthop device is not up");
+               err = -ENETDOWN;
+               goto out;
+       }
+
        if (!ipv6_addr_any(&cfg->fc_prefsrc)) {
                if (!ipv6_chk_addr(net, &cfg->fc_prefsrc, dev, 0)) {
                        NL_SET_ERR_MSG(extack, "Invalid source address");