Skip to content

Commit 64c6d08

Browse files
NicolasDichteldavem330
authored andcommitted
ipv6: del unreachable route when an addr is deleted on lo
When an address is added on loopback (ip -6 a a 2002::1/128 dev lo), two routes are added: - one in the local table: local 2002::1 via :: dev lo proto none metric 0 - one the in main table (for the prefix): unreachable 2002::1 dev lo proto kernel metric 256 error -101 When the address is deleted, the route inserted in the main table remains because we use rt6_lookup(), which returns NULL when dst->error is set, which is the case here! Thus, it is better to use ip6_route_lookup() to avoid this kind of filter. Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com> Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent 515e124 commit 64c6d08

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

net/ipv6/addrconf.c

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -788,10 +788,16 @@ static void ipv6_del_addr(struct inet6_ifaddr *ifp)
788788
struct in6_addr prefix;
789789
struct rt6_info *rt;
790790
struct net *net = dev_net(ifp->idev->dev);
791+
struct flowi6 fl6 = {};
792+
791793
ipv6_addr_prefix(&prefix, &ifp->addr, ifp->prefix_len);
792-
rt = rt6_lookup(net, &prefix, NULL, ifp->idev->dev->ifindex, 1);
794+
fl6.flowi6_oif = ifp->idev->dev->ifindex;
795+
fl6.daddr = prefix;
796+
rt = (struct rt6_info *)ip6_route_lookup(net, &fl6,
797+
RT6_LOOKUP_F_IFACE);
793798

794-
if (rt && addrconf_is_prefix_route(rt)) {
799+
if (rt != net->ipv6.ip6_null_entry &&
800+
addrconf_is_prefix_route(rt)) {
795801
if (onlink == 0) {
796802
ip6_del_rt(rt);
797803
rt = NULL;

0 commit comments

Comments
 (0)