commit e78ec548060d969fecdb488d6fed711d843bba91
author Paul Stewart <pstew@chromium.org> Fri Jun 08 18:28:50 2012 -0700
committer Gerrit <chrome-bot@google.com> Sun Jun 10 20:01:00 2012 -0700
tree 298e4a33ba438ed9e9d27bea493e362df9ea9f9d
parent 92e6561671667edc1cfa2517ece57f9f38a51fe5

shill: connection: Flush routes when default connection changes

Also be more verbose with address configuration.

BUG=None
TEST=Unit tests

Change-Id: I7a419f416e6ed6b583d57a06b7be42ed37933593
Reviewed-on: https://gerrit.chromium.org/gerrit/24936
Commit-Ready: Paul Stewart <pstew@chromium.org>
Reviewed-by: Paul Stewart <pstew@chromium.org>
Tested-by: Paul Stewart <pstew@chromium.org>

connection.cc

diff --git a/connection.cc b/connection.cc
index 476216e..071ce8c 100644
--- a/connection.cc
+++ b/connection.cc
@@ -131,22 +131,27 @@
     return;
   }
 
-  IPAddress gateway_address(properties.address_family);
+  IPAddress gateway(properties.address_family);
   if (!properties.gateway.empty() &&
-      !gateway_address.SetAddressFromString(properties.gateway)) {
+      !gateway.SetAddressFromString(properties.gateway)) {
     LOG(ERROR) << "Gateway address " << properties.peer_address
                << " is invalid";
     return;
   }
 
-  if (!FixGatewayReachability(&local, &peer, gateway_address)) {
+  if (!FixGatewayReachability(&local, &peer, gateway)) {
     LOG(WARNING) << "Expect limited network connectivity.";
   }
 
+  LOG(INFO) << __func__ << ": Installing with parameters:"
+            << " local=" << local.ToString()
+            << " broadcast=" << broadcast.ToString()
+            << " peer=" << peer.ToString()
+            << " gateway=" << gateway.ToString();
   rtnl_handler_->AddInterfaceAddress(interface_index_, local, broadcast, peer);
 
-  if (gateway_address.IsValid()) {
-    routing_table_->SetDefaultRoute(interface_index_, gateway_address,
+  if (gateway.IsValid()) {
+    routing_table_->SetDefaultRoute(interface_index_, gateway,
                                     GetMetric(is_default_));
   }
 
@@ -185,6 +190,7 @@
       device->RequestPortalDetection();
     }
   }
+  routing_table_->FlushCache();
 }
 
 void Connection::RequestRouting() {