commit 92424d5e388402fb9e309dd8ad37f40e1484408f
author Maciej Żenczykowski <maze@google.com> Fri Aug 09 14:29:12 2019 -0700
committer Maciej Żenczykowski <maze@google.com> Sat Aug 10 01:30:15 2019 +0000
tree dbe1130d3dd00656835e16c1c458585d2e5341d6
parent 07590428db74a2f7f9171efeb03cfd8b77599ddb

fix InterfaceController::setIPv6AddrGenMode(INetd::IPV6_ADDR_GEN_MODE_EUI64)

to actually do something... writing an empty string is a no-op by definition,
furthermore the way to switch to EUI64 mode is via 'addr_gen_mode' file,
not via the stable_secret file.

Test: builds and TreeHugger
Bug: 138428295
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: I77515cfec315136e0d8641fa1457917cc1efe70d

server/InterfaceController.cpp

diff --git a/server/InterfaceController.cpp b/server/InterfaceController.cpp
index a2d6192..a67b4e1 100644
--- a/server/InterfaceController.cpp
+++ b/server/InterfaceController.cpp
@@ -289,9 +289,9 @@
 
     switch (mode) {
         case INetd::IPV6_ADDR_GEN_MODE_EUI64:
-            // Ignore return value. If /proc/.../stable_secret is
+            // Ignore return value. If /proc/.../addr_gen_mode is
             // missing we're probably in EUI64 mode already.
-            writeValueToPath(ipv6_proc_path, interface.c_str(), "stable_secret", "");
+            writeValueToPath(ipv6_proc_path, interface.c_str(), "addr_gen_mode", "0");
             break;
         case INetd::IPV6_ADDR_GEN_MODE_STABLE_PRIVACY: {
             return enableStablePrivacyAddresses(interface, getProperty, setProperty);