summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLorenzo Colitti <lorenzo@google.com>2020-05-10 03:56:08 +0000
committerAndroid (Google) Code Review <android-gerrit@google.com>2020-05-10 03:56:08 +0000
commit1998f83dff81098310745fbf139ca8fa28cd65af (patch)
tree111d7195627ee549791f86f63e4dac686f7a401f
parent2369fc3481c7111da241b8fbd33374a7b2ec10e6 (diff)
parent3f518a4f0319bdfeab2d363a89f1ddf4856162f8 (diff)
downloadtests-1998f83dff81098310745fbf139ca8fa28cd65af.tar.gz
Merge "Fix srcaddr_selection_test when forwarding is on." into rvc-dev
-rwxr-xr-xnet/test/srcaddr_selection_test.py9
1 files changed, 9 insertions, 0 deletions
diff --git a/net/test/srcaddr_selection_test.py b/net/test/srcaddr_selection_test.py
index 45a8140..e57ce16 100755
--- a/net/test/srcaddr_selection_test.py
+++ b/net/test/srcaddr_selection_test.py
@@ -72,6 +72,9 @@ class IPv6SourceAddressSelectionTest(multinetwork_base.MultiNetworkBaseTest):
def SetUseOptimistic(self, ifname, value):
self.SetSysctl("/proc/sys/net/ipv6/conf/%s/use_optimistic" % ifname, value)
+ def SetForwarding(self, value):
+ self.SetSysctl("/proc/sys/net/ipv6/conf/all/forwarding", value)
+
def GetSourceIP(self, netid, mode="mark"):
s = self.BuildSocket(6, net_test.UDPSocket, netid, mode)
# Because why not...testing for temporary addresses is a separate thing.
@@ -163,6 +166,12 @@ class MultiInterfaceSourceAddressSelectionTest(IPv6SourceAddressSelectionTest):
# link-local address is generated.
self.WaitForDad(self.test_lladdr)
+ # Disable forwarding, because optimistic addresses don't work when
+ # forwarding is on. Forwarding will be re-enabled when the sysctls are
+ # restored by MultiNetworkBaseTest.tearDownClass.
+ # TODO: Fix this and remove this hack.
+ self.SetForwarding("0")
+
class TentativeAddressTest(MultiInterfaceSourceAddressSelectionTest):