commit e93b038972d43fd703b3c68603fb4d02bec6504e
author Paul Stewart <pstew@chromium.org> Tue Apr 24 13:11:28 2012 -0700
committer Gerrit <chrome-bot@google.com> Wed Apr 25 12:50:13 2012 -0700
tree 1c8d43d238604bfa01f4bdcdf26940a96ad70e12
parent ff14b0261ac867f98b96db9ba39490fd70269fcd

shill: Connection: UnPin routes on destruction

Tag pinned routes with the interface index associated with the request,
so they can be removed when the connection is destroyed.  Also move
PinHostRoute() out of the VPN code and into the Connection.

BUG=chromium-os:29911
TEST=New unit tests

Change-Id: I46019255276469929642db4a6395e64f53e3b7d5
Reviewed-on: https://gerrit.chromium.org/gerrit/20982
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 a5314ca..71cec11 100644
--- a/connection.cc
+++ b/connection.cc
@@ -45,6 +45,7 @@
 
   DCHECK(!routing_request_count_);
   routing_table_->FlushRoutes(interface_index_);
+  routing_table_->FlushRoutesWithTag(interface_index_);
   device_info_->FlushAddresses(interface_index_);
 }
 
@@ -52,6 +53,11 @@
   SLOG(Connection, 2) << __func__ << " " << interface_name_;
 
   const IPConfig::Properties &properties = config->properties();
+  if (!properties.trusted_ip.empty() && !PinHostRoute(properties)) {
+    LOG(ERROR) << "Unable to pin host route to " << properties.trusted_ip;
+    return;
+  }
+
   IPAddress local(properties.address_family);
   if (!local.SetAddressFromString(properties.address)) {
     LOG(ERROR) << "Local address " << properties.address << " is invalid";
@@ -166,7 +172,10 @@
 
   // Do not set interface_index_ since this may not be the
   // default route through which this destination can be found.
-  if (!routing_table_->RequestRouteToHost(address_prefix, -1)) {
+  // However, we should tag the created route with our interface
+  // index so we can clean this route up when this connection closes.
+  if (!routing_table_->RequestRouteToHost(address_prefix, -1,
+                                          interface_index_)) {
     LOG(ERROR) << "Could not request route to " << address.ToString();
     return false;
   }
@@ -211,4 +220,20 @@
   return is_default ? kDefaultMetric : kNonDefaultMetricBase + interface_index_;
 }
 
+bool Connection::PinHostRoute(const IPConfig::Properties &properties) {
+  SLOG(Connection, 2) << __func__;
+  if (properties.gateway.empty() || properties.trusted_ip.empty()) {
+    return false;
+  }
+
+  IPAddress trusted_ip(properties.address_family);
+  if (!trusted_ip.SetAddressFromString(properties.trusted_ip)) {
+    LOG(ERROR) << "Failed to parse trusted_ip "
+               << properties.trusted_ip << "; ignored.";
+    return false;
+  }
+
+  return RequestHostRoute(trusted_ip);
+}
+
 }  // namespace shill