| From c2ed1880fd61a998e3ce40254a99a2ad000f1a7d Mon Sep 17 00:00:00 2001 |
| From: Mantas M <grawity@gmail.com> |
| Date: Fri, 16 Dec 2016 10:30:59 +0200 |
| Subject: net: ipv6: check route protocol when deleting routes |
| MIME-Version: 1.0 |
| Content-Type: text/plain; charset=UTF-8 |
| Content-Transfer-Encoding: 8bit |
| |
| From: Mantas M <grawity@gmail.com> |
| |
| commit c2ed1880fd61a998e3ce40254a99a2ad000f1a7d upstream. |
| |
| The protocol field is checked when deleting IPv4 routes, but ignored for |
| IPv6, which causes problems with routing daemons accidentally deleting |
| externally set routes (observed by multiple bird6 users). |
| |
| This can be verified using `ip -6 route del <prefix> proto something`. |
| |
| Signed-off-by: Mantas Mikulėnas <grawity@gmail.com> |
| Signed-off-by: David S. Miller <davem@davemloft.net> |
| Cc: Ben Hutchings <ben@decadent.org.uk> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| net/ipv6/route.c | 2 ++ |
| 1 file changed, 2 insertions(+) |
| |
| --- a/net/ipv6/route.c |
| +++ b/net/ipv6/route.c |
| @@ -2166,6 +2166,8 @@ static int ip6_route_del(struct fib6_con |
| continue; |
| if (cfg->fc_metric && cfg->fc_metric != rt->rt6i_metric) |
| continue; |
| + if (cfg->fc_protocol && cfg->fc_protocol != rt->rt6i_protocol) |
| + continue; |
| dst_hold(&rt->dst); |
| read_unlock_bh(&table->tb6_lock); |
| |
