Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 1 | // Copyright 2019 The Chromium OS Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
Garrick Evans | 3388a03 | 2020-03-24 11:25:55 +0900 | [diff] [blame] | 5 | #include "patchpanel/crostini_service.h" |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 6 | |
| 7 | #include <memory> |
| 8 | #include <utility> |
| 9 | |
| 10 | #include <base/strings/string_number_conversions.h> |
| 11 | #include <base/strings/string_util.h> |
| 12 | #include <base/strings/stringprintf.h> |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 13 | #include "base/threading/thread_task_runner_handle.h" |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 14 | #include <chromeos/constants/vm_tools.h> |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 15 | #include <chromeos/dbus/service_constants.h> |
| 16 | #include <dbus/message.h> |
| 17 | #include <dbus/object_path.h> |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 18 | |
Garrick Evans | 3388a03 | 2020-03-24 11:25:55 +0900 | [diff] [blame] | 19 | #include "patchpanel/adb_proxy.h" |
| 20 | #include "patchpanel/device.h" |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 21 | |
Garrick Evans | 3388a03 | 2020-03-24 11:25:55 +0900 | [diff] [blame] | 22 | namespace patchpanel { |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 23 | namespace { |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 24 | constexpr int32_t kInvalidID = 0; |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 25 | constexpr int kDbusTimeoutMs = 200; |
Jason Jeremy Iman | 10314a8 | 2020-05-12 00:09:53 +0900 | [diff] [blame] | 26 | // The maximum number of ADB sideloading query failures before stopping. |
| 27 | constexpr int kAdbSideloadMaxTry = 5; |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 28 | constexpr base::TimeDelta kAdbSideloadUpdateDelay = |
| 29 | base::TimeDelta::FromMilliseconds(5000); |
Jason Jeremy Iman | bb8787e | 2019-12-13 14:08:20 +0900 | [diff] [blame] | 30 | |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 31 | std::string MakeKey(uint64_t vm_id, bool is_termina) { |
| 32 | return base::StringPrintf("%s:%s", is_termina ? "t" : "p", |
| 33 | base::NumberToString(vm_id).c_str()); |
| 34 | } |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 35 | |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 36 | } // namespace |
| 37 | |
Garrick Evans | 69b8587 | 2020-02-04 11:40:26 +0900 | [diff] [blame] | 38 | CrostiniService::CrostiniService(ShillClient* shill_client, |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 39 | AddressManager* addr_mgr, |
| 40 | Datapath* datapath, |
| 41 | TrafficForwarder* forwarder) |
| 42 | : shill_client_(shill_client), |
| 43 | addr_mgr_(addr_mgr), |
| 44 | datapath_(datapath), |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 45 | forwarder_(forwarder), |
| 46 | adb_sideloading_enabled_(false) { |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 47 | DCHECK(shill_client_); |
| 48 | DCHECK(addr_mgr_); |
Garrick Evans | f29f5a3 | 2019-12-06 11:34:25 +0900 | [diff] [blame] | 49 | DCHECK(datapath_); |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 50 | DCHECK(forwarder_); |
Garrick Evans | f29f5a3 | 2019-12-06 11:34:25 +0900 | [diff] [blame] | 51 | |
Jason Jeremy Iman | a7273a3 | 2020-08-04 11:25:31 +0900 | [diff] [blame] | 52 | dbus::Bus::Options options; |
| 53 | options.bus_type = dbus::Bus::SYSTEM; |
| 54 | |
| 55 | bus_ = new dbus::Bus(options); |
| 56 | if (!bus_->Connect()) { |
| 57 | LOG(ERROR) << "Failed to connect to system bus"; |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 58 | } else { |
| 59 | CheckAdbSideloadingStatus(); |
| 60 | } |
| 61 | |
Garrick Evans | 69b8587 | 2020-02-04 11:40:26 +0900 | [diff] [blame] | 62 | shill_client_->RegisterDefaultInterfaceChangedHandler(base::Bind( |
| 63 | &CrostiniService::OnDefaultInterfaceChanged, weak_factory_.GetWeakPtr())); |
Garrick Evans | f29f5a3 | 2019-12-06 11:34:25 +0900 | [diff] [blame] | 64 | } |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 65 | |
Garrick Evans | c1ac5c4 | 2020-03-31 15:31:22 +0900 | [diff] [blame] | 66 | CrostiniService::~CrostiniService() { |
| 67 | if (bus_) |
| 68 | bus_->ShutdownAndBlock(); |
| 69 | } |
| 70 | |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 71 | bool CrostiniService::Start(uint64_t vm_id, bool is_termina, int subnet_index) { |
| 72 | if (vm_id == kInvalidID) { |
| 73 | LOG(ERROR) << "Invalid VM id"; |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 74 | return false; |
| 75 | } |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 76 | |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 77 | const auto key = MakeKey(vm_id, is_termina); |
| 78 | if (taps_.find(key) != taps_.end()) { |
| 79 | LOG(WARNING) << "Already started for {id: " << vm_id << "}"; |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 80 | return false; |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 81 | } |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 82 | |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 83 | auto tap = AddTAP(is_termina, subnet_index); |
| 84 | if (!tap) { |
| 85 | LOG(ERROR) << "Cannot start for {id: " << vm_id << "}"; |
| 86 | return false; |
| 87 | } |
Jason Jeremy Iman | bb8787e | 2019-12-13 14:08:20 +0900 | [diff] [blame] | 88 | |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 89 | LOG(INFO) << "Crostini network service started for {id: " << vm_id << "}"; |
Garrick Evans | 6c7dcb8 | 2020-03-16 15:21:05 +0900 | [diff] [blame] | 90 | StartForwarding(shill_client_->default_interface(), tap->host_ifname()); |
Hugo Benichi | c6ae67c | 2020-08-14 15:02:13 +0900 | [diff] [blame] | 91 | auto source = is_termina ? TrafficSource::CROSVM : TrafficSource::PLUGINVM; |
| 92 | datapath_->StartRoutingDevice("", tap->host_ifname(), |
| 93 | tap->config().host_ipv4_addr(), source); |
Garrick Evans | 6c7dcb8 | 2020-03-16 15:21:05 +0900 | [diff] [blame] | 94 | |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 95 | if (adb_sideloading_enabled_) |
Garrick Evans | 6c7dcb8 | 2020-03-16 15:21:05 +0900 | [diff] [blame] | 96 | StartAdbPortForwarding(tap->phys_ifname()); |
| 97 | |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 98 | taps_.emplace(key, std::move(tap)); |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 99 | return true; |
| 100 | } |
| 101 | |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 102 | void CrostiniService::Stop(uint64_t vm_id, bool is_termina) { |
| 103 | const auto key = MakeKey(vm_id, is_termina); |
| 104 | const auto it = taps_.find(key); |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 105 | if (it == taps_.end()) { |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 106 | LOG(WARNING) << "Unknown {id: " << vm_id << "}"; |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 107 | return; |
| 108 | } |
| 109 | |
Garrick Evans | 6c7dcb8 | 2020-03-16 15:21:05 +0900 | [diff] [blame] | 110 | const auto& ifname = it->second->host_ifname(); |
Hugo Benichi | c6ae67c | 2020-08-14 15:02:13 +0900 | [diff] [blame] | 111 | auto source = is_termina ? TrafficSource::CROSVM : TrafficSource::PLUGINVM; |
| 112 | datapath_->StopRoutingDevice("", ifname, |
| 113 | it->second->config().host_ipv4_addr(), source); |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 114 | StopForwarding(shill_client_->default_interface(), ifname); |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 115 | if (adb_sideloading_enabled_) |
| 116 | StopAdbPortForwarding(ifname); |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 117 | datapath_->RemoveInterface(ifname); |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 118 | taps_.erase(key); |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 119 | |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 120 | LOG(INFO) << "Crostini network service stopped for {id: " << vm_id << "}"; |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 121 | } |
| 122 | |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 123 | const Device* const CrostiniService::TAP(uint64_t vm_id, |
| 124 | bool is_termina) const { |
| 125 | const auto it = taps_.find(MakeKey(vm_id, is_termina)); |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 126 | if (it == taps_.end()) { |
| 127 | return nullptr; |
| 128 | } |
| 129 | return it->second.get(); |
| 130 | } |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 131 | |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 132 | std::unique_ptr<Device> CrostiniService::AddTAP(bool is_termina, |
| 133 | int subnet_index) { |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 134 | auto ipv4_subnet = addr_mgr_->AllocateIPv4Subnet( |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 135 | is_termina ? AddressManager::Guest::VM_TERMINA |
Garrick Evans | 1fa4e64 | 2020-03-13 11:43:41 +0900 | [diff] [blame] | 136 | : AddressManager::Guest::VM_PLUGIN, |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 137 | subnet_index); |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 138 | if (!ipv4_subnet) { |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 139 | LOG(ERROR) << "Subnet already in use or unavailable."; |
| 140 | return nullptr; |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 141 | } |
| 142 | auto host_ipv4_addr = ipv4_subnet->AllocateAtOffset(0); |
| 143 | if (!host_ipv4_addr) { |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 144 | LOG(ERROR) << "Host address already in use or unavailable."; |
| 145 | return nullptr; |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 146 | } |
| 147 | auto guest_ipv4_addr = ipv4_subnet->AllocateAtOffset(1); |
| 148 | if (!guest_ipv4_addr) { |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 149 | LOG(ERROR) << "VM address already in use or unavailable."; |
| 150 | return nullptr; |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 151 | } |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 152 | std::unique_ptr<Subnet> lxd_subnet; |
| 153 | if (is_termina) { |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 154 | lxd_subnet = |
| 155 | addr_mgr_->AllocateIPv4Subnet(AddressManager::Guest::CONTAINER); |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 156 | if (!lxd_subnet) { |
| 157 | LOG(ERROR) << "lxd subnet already in use or unavailable."; |
| 158 | return nullptr; |
| 159 | } |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 160 | } |
| 161 | |
Garrick Evans | 7d9a232 | 2020-04-02 11:59:56 +0900 | [diff] [blame] | 162 | const auto mac_addr = addr_mgr_->GenerateMacAddress(subnet_index); |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 163 | const std::string tap = |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 164 | datapath_->AddTAP("" /* auto-generate name */, &mac_addr, |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 165 | host_ipv4_addr.get(), vm_tools::kCrosVmUser); |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 166 | if (tap.empty()) { |
Garrick Evans | 51d5b55 | 2020-01-30 10:42:06 +0900 | [diff] [blame] | 167 | LOG(ERROR) << "Failed to create TAP device."; |
| 168 | return nullptr; |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 169 | } |
| 170 | |
Garrick Evans | 3d97a39 | 2020-02-21 15:24:37 +0900 | [diff] [blame] | 171 | if (lxd_subnet) { |
| 172 | // Setup lxd route for the container using the VM as a gateway. |
| 173 | if (!datapath_->AddIPv4Route(ipv4_subnet->AddressAtOffset(1), |
| 174 | lxd_subnet->AddressAtOffset(0), |
| 175 | lxd_subnet->Netmask())) { |
| 176 | LOG(ERROR) << "Failed to setup lxd route"; |
| 177 | return nullptr; |
| 178 | } |
| 179 | } |
| 180 | |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 181 | auto config = std::make_unique<Device::Config>( |
Garrick Evans | 6c7dcb8 | 2020-03-16 15:21:05 +0900 | [diff] [blame] | 182 | mac_addr, std::move(ipv4_subnet), std::move(host_ipv4_addr), |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 183 | std::move(guest_ipv4_addr), std::move(lxd_subnet)); |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 184 | |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 185 | Device::Options opts{ |
| 186 | .fwd_multicast = true, |
| 187 | .ipv6_enabled = true, |
Garrick Evans | b1c9371 | 2020-01-22 09:28:25 +0900 | [diff] [blame] | 188 | }; |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 189 | |
Garrick Evans | 6c7dcb8 | 2020-03-16 15:21:05 +0900 | [diff] [blame] | 190 | return std::make_unique<Device>(tap, tap, "", std::move(config), opts); |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 191 | } |
| 192 | |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 193 | void CrostiniService::OnDefaultInterfaceChanged( |
| 194 | const std::string& new_ifname, const std::string& prev_ifname) { |
| 195 | for (const auto& t : taps_) { |
Garrick Evans | 6c7dcb8 | 2020-03-16 15:21:05 +0900 | [diff] [blame] | 196 | StopForwarding(prev_ifname, t.second->host_ifname()); |
| 197 | StartForwarding(new_ifname, t.second->host_ifname()); |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 198 | } |
| 199 | } |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 200 | |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 201 | void CrostiniService::StartForwarding(const std::string& phys_ifname, |
Jason Jeremy Iman | 0e9f826 | 2020-03-06 14:50:49 +0900 | [diff] [blame] | 202 | const std::string& virt_ifname) { |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 203 | if (!phys_ifname.empty()) |
Jason Jeremy Iman | 0e9f826 | 2020-03-06 14:50:49 +0900 | [diff] [blame] | 204 | forwarder_->StartForwarding(phys_ifname, virt_ifname, true /*ipv6*/, |
| 205 | true /*multicast*/); |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 206 | } |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 207 | |
Garrick Evans | 1b1f67c | 2020-02-04 16:21:25 +0900 | [diff] [blame] | 208 | void CrostiniService::StopForwarding(const std::string& phys_ifname, |
| 209 | const std::string& virt_ifname) { |
| 210 | if (!phys_ifname.empty()) |
| 211 | forwarder_->StopForwarding(phys_ifname, virt_ifname, true /*ipv6*/, |
| 212 | true /*multicast*/); |
Garrick Evans | 47c1927 | 2019-11-21 10:58:21 +0900 | [diff] [blame] | 213 | } |
| 214 | |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 215 | void CrostiniService::StartAdbPortForwarding(const std::string& ifname) { |
Jason Jeremy Iman | a7273a3 | 2020-08-04 11:25:31 +0900 | [diff] [blame] | 216 | if (!datapath_->AddAdbPortForwardRule(ifname)) { |
| 217 | LOG(ERROR) << "Error adding ADB port forwarding rule for " << ifname; |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 218 | return; |
| 219 | } |
| 220 | |
Jason Jeremy Iman | a7273a3 | 2020-08-04 11:25:31 +0900 | [diff] [blame] | 221 | if (!datapath_->AddAdbPortAccessRule(ifname)) { |
| 222 | datapath_->DeleteAdbPortForwardRule(ifname); |
| 223 | LOG(ERROR) << "Error adding ADB port access rule for " << ifname; |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 224 | return; |
| 225 | } |
| 226 | |
| 227 | if (datapath_->runner().sysctl_w( |
| 228 | "net.ipv4.conf." + ifname + ".route_localnet", "1") != 0) { |
| 229 | LOG(ERROR) << "Failed to set up route localnet for " << ifname; |
| 230 | return; |
| 231 | } |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 232 | } |
| 233 | |
| 234 | void CrostiniService::StopAdbPortForwarding(const std::string& ifname) { |
Jason Jeremy Iman | a7273a3 | 2020-08-04 11:25:31 +0900 | [diff] [blame] | 235 | datapath_->DeleteAdbPortForwardRule(ifname); |
| 236 | datapath_->DeleteAdbPortAccessRule(ifname); |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 237 | } |
| 238 | |
| 239 | void CrostiniService::CheckAdbSideloadingStatus() { |
Jason Jeremy Iman | 10314a8 | 2020-05-12 00:09:53 +0900 | [diff] [blame] | 240 | static int num_try = 0; |
Jason Jeremy Iman | dd372fd | 2020-08-11 00:07:58 +0900 | [diff] [blame] | 241 | if (num_try >= kAdbSideloadMaxTry) { |
| 242 | LOG(WARNING) << "Failed to get ADB sideloading status after " << num_try |
| 243 | << " tries. ADB sideloading will not work"; |
Jason Jeremy Iman | 10314a8 | 2020-05-12 00:09:53 +0900 | [diff] [blame] | 244 | return; |
Jason Jeremy Iman | dd372fd | 2020-08-11 00:07:58 +0900 | [diff] [blame] | 245 | } |
Jason Jeremy Iman | 10314a8 | 2020-05-12 00:09:53 +0900 | [diff] [blame] | 246 | |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 247 | dbus::ObjectProxy* proxy = bus_->GetObjectProxy( |
| 248 | login_manager::kSessionManagerServiceName, |
| 249 | dbus::ObjectPath(login_manager::kSessionManagerServicePath)); |
| 250 | dbus::MethodCall method_call(login_manager::kSessionManagerInterface, |
| 251 | login_manager::kSessionManagerQueryAdbSideload); |
| 252 | std::unique_ptr<dbus::Response> dbus_response = |
| 253 | proxy->CallMethodAndBlock(&method_call, kDbusTimeoutMs); |
| 254 | |
| 255 | if (!dbus_response) { |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 256 | base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( |
| 257 | FROM_HERE, |
| 258 | base::BindOnce(&CrostiniService::CheckAdbSideloadingStatus, |
| 259 | weak_factory_.GetWeakPtr()), |
| 260 | kAdbSideloadUpdateDelay); |
Jason Jeremy Iman | 10314a8 | 2020-05-12 00:09:53 +0900 | [diff] [blame] | 261 | num_try++; |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 262 | return; |
| 263 | } |
| 264 | |
| 265 | dbus::MessageReader reader(dbus_response.get()); |
| 266 | reader.PopBool(&adb_sideloading_enabled_); |
| 267 | if (!adb_sideloading_enabled_) |
| 268 | return; |
| 269 | |
| 270 | // If ADB sideloading is enabled, start ADB forwarding on all configured |
| 271 | // Crostini's TAP interfaces. |
| 272 | for (const auto& tap : taps_) { |
Garrick Evans | 6c7dcb8 | 2020-03-16 15:21:05 +0900 | [diff] [blame] | 273 | StartAdbPortForwarding(tap.second->phys_ifname()); |
Jason Jeremy Iman | fa8b6d2 | 2020-02-20 03:44:21 +0000 | [diff] [blame] | 274 | } |
| 275 | } |
| 276 | |
Garrick Evans | 3388a03 | 2020-03-24 11:25:55 +0900 | [diff] [blame] | 277 | } // namespace patchpanel |