diff options
author | Michal Kubeček <mkubecek@suse.cz> | 2015-05-18 20:53:55 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-05-20 12:02:25 -0400 |
commit | 35f1b4e96b9258a3668872b1139c51e5a23eb876 (patch) | |
tree | 89234a81b657a4ad1e2ed07ab98736b8ee979e69 /net/ipv6/ip6_fib.c | |
parent | 892bd6291ac122e3cdb354ef39567c9127768916 (diff) | |
download | op-kernel-dev-35f1b4e96b9258a3668872b1139c51e5a23eb876.zip op-kernel-dev-35f1b4e96b9258a3668872b1139c51e5a23eb876.tar.gz |
ipv6: do not delete previously existing ECMP routes if add fails
If adding a nexthop of an IPv6 multipath route fails, comment in
ip6_route_multipath() says we are going to delete all nexthops already
added. However, current implementation deletes even the routes it
hasn't even tried to add yet. For example, running
ip route add 1234:5678::/64 \
nexthop via fe80::aa dev dummy1 \
nexthop via fe80::bb dev dummy1 \
nexthop via fe80::cc dev dummy1
twice results in removing all routes first command added.
Limit the second (delete) run to nexthops that succeeded in the first
(add) run.
Fixes: 51ebd3181572 ("ipv6: add support of equal cost multipath (ECMP)")
Signed-off-by: Michal Kubecek <mkubecek@suse.cz>
Acked-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/ip6_fib.c')
0 files changed, 0 insertions, 0 deletions