shill: vpn: Don't delete tun interfaces claimed by VPNProvider.
... and delete then if they are not claimed.
BUG=chromium-os:27717
TEST=000VPNGenesis, unit tests
Change-Id: Icf08ebc7d90c22d3ed8a8a737f394b20bb250650
Reviewed-on: https://gerrit.chromium.org/gerrit/17961
Commit-Ready: Darin Petkov <petkov@chromium.org>
Reviewed-by: Darin Petkov <petkov@chromium.org>
Tested-by: Darin Petkov <petkov@chromium.org>
diff --git a/device_info.cc b/device_info.cc
index 31d6c39..5df6be3 100644
--- a/device_info.cc
+++ b/device_info.cc
@@ -265,9 +265,9 @@
// Tunnel devices are managed by the VPN code.
VLOG(2) << "Tunnel link " << link_name << " at index " << dev_index
<< " -- notifying VPNProvider.";
- if (manager_->vpn_provider()->OnDeviceInfoAvailable(link_name,
+ if (!manager_->vpn_provider()->OnDeviceInfoAvailable(link_name,
dev_index)) {
- // VPN does not know anything about this tunnel, it is probably
+ // If VPN does not know anything about this tunnel, it is probably
// left over from a previous instance and should not exist.
VLOG(2) << "Tunnel link is unused. Deleting.";
DeleteInterface(dev_index);