commit 14c29ec4815e8ecda47b7ce5efdaa444580e20ce
author Darin Petkov <petkov@chromium.org> Fri Mar 02 11:34:19 2012 +0100
committer Gerrit <chrome-bot@google.com> Mon Mar 05 12:34:48 2012 -0800
tree 2ddd55c1d1a760ba6cb78a815f0deb593441eee7
parent 24c6c62bc68431990da988ae8f9711787e31f5eb

shill: vpn: Parse openvpn IP configuration options.

The options are pushed to shill over DBus through the .Task.notify service
method.

BUG=chromium-os:27056
TEST=unit tests

Change-Id: I98d7d49c275f49daba5a6dbe0af0878bf82038a6
Reviewed-on: https://gerrit.chromium.org/gerrit/17219
Tested-by: Darin Petkov <petkov@chromium.org>
Reviewed-by: mukesh agrawal <quiche@chromium.org>
Commit-Ready: Darin Petkov <petkov@chromium.org>

dhcp_config.cc

diff --git a/dhcp_config.cc b/dhcp_config.cc
index 7435432..c0bcc67 100644
--- a/dhcp_config.cc
+++ b/dhcp_config.cc
@@ -38,6 +38,7 @@
 const char DHCPConfig::kDHCPCDPath[] = "/sbin/dhcpcd";
 const char DHCPConfig::kDHCPCDPathFormatLease[] = "var/run/dhcpcd-%s.lease";
 const char DHCPConfig::kDHCPCDPathFormatPID[] = "var/run/dhcpcd-%s.pid";
+const int DHCPConfig::kMinMTU = 576;
 const char DHCPConfig::kReasonBound[] = "BOUND";
 const char DHCPConfig::kReasonFail[] = "FAIL";
 const char DHCPConfig::kReasonRebind[] = "REBIND";
@@ -276,7 +277,7 @@
       properties->domain_search = value.operator vector<string>();
     } else if (key == kConfigurationKeyMTU) {
       int mtu = value.reader().get_uint16();
-      if (mtu >= 576) {
+      if (mtu >= kMinMTU) {
         properties->mtu = mtu;
       }
     } else {