[PATCH net-next] selftests: disable rp_filter for icmp_redirect.sh

[PATCH net-next] selftests: disable rp_filter for icmp_redirect.sh

[David Ahern]

h1 is initially configured to reach h2 via r1 rather than the
more direct path through r2. If rp_filter is set and inherited
for r2, forwarding fails since the source address of h1 is
reachable from eth0 vs the packet coming to it via r1 and eth1.
Since rp_filter setting affects the test, explicitly reset it.

Signed-off-by: David Ahern <dsahern@kernel.org>
---
 tools/testing/selftests/net/icmp_redirect.sh | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tools/testing/selftests/net/icmp_redirect.sh b/tools/testing/selftests/net/icmp_redirect.sh
index 18c5de53558a..bf361f30d6ef 100755
--- a/tools/testing/selftests/net/icmp_redirect.sh
+++ b/tools/testing/selftests/net/icmp_redirect.sh
@@ -180,6 +180,8 @@ setup()
 			;;
 		r[12]) ip netns exec $ns sysctl -q -w net.ipv4.ip_forward=1
 		       ip netns exec $ns sysctl -q -w net.ipv4.conf.all.send_redirects=1
+		       ip netns exec $ns sysctl -q -w net.ipv4.conf.default.rp_filter=0
+		       ip netns exec $ns sysctl -q -w net.ipv4.conf.all.rp_filter=0
 
 		       ip netns exec $ns sysctl -q -w net.ipv6.conf.all.forwarding=1
 		       ip netns exec $ns sysctl -q -w net.ipv6.route.mtu_expires=10
-- 
2.17.1

Re: [PATCH net-next] selftests: disable rp_filter for icmp_redirect.sh

[David Miller]

From: David Ahern <dsahern@kernel.org>
Date: Mon, 17 Aug 2020 09:43:33 -0600

> h1 is initially configured to reach h2 via r1 rather than the
> more direct path through r2. If rp_filter is set and inherited
> for r2, forwarding fails since the source address of h1 is
> reachable from eth0 vs the packet coming to it via r1 and eth1.
> Since rp_filter setting affects the test, explicitly reset it.
> 
> Signed-off-by: David Ahern <dsahern@kernel.org>

As a bug fix I've applied this to 'net', thanks David.