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