blob: 144524fe34b4f313b497f4887e0c46a48dd0dc14 [file] [log] [blame]
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001// Copyright 2021, The Android Open Source Project
2//
3// Licensed under the Apache License, Version 2.0 (the "License");
4// you may not use this file except in compliance with the License.
5// You may obtain a copy of the License at
6//
7// http://www.apache.org/licenses/LICENSE-2.0
8//
9// Unless required by applicable law or agreed to in writing, software
10// distributed under the License is distributed on an "AS IS" BASIS,
11// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12// See the License for the specific language governing permissions and
13// limitations under the License.
14
Andrew Walbranf6bf6862021-05-21 12:41:13 +000015//! Implementation of the AIDL interface of the VirtualizationService.
Andrew Walbrand6dce6f2021-03-05 16:39:08 +000016
Seungjae Yoo13af0b62024-05-20 14:15:13 +090017use crate::{get_calling_pid, get_calling_uid, get_this_pid};
Alan Stokesfda70842023-12-20 17:50:14 +000018use crate::atom::{write_vm_booted_stats, write_vm_creation_stats};
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +000019use crate::composite::make_composite_image;
Mu-Le Leeb2d5a312024-06-05 10:52:47 +080020use crate::crosvm::{AudioConfig, CrosvmConfig, DiskFile, DisplayConfig, GpuConfig, InputDeviceOption, PayloadState, VmContext, VmInstance, VmState};
Jaewan Kim61f86142023-03-28 15:12:52 +090021use crate::debug_config::DebugConfig;
Shikha Panwar55e10ec2024-02-13 12:53:49 +000022use crate::dt_overlay::{create_device_tree_overlay, VM_DT_OVERLAY_MAX_SIZE, VM_DT_OVERLAY_PATH};
Nikita Ioffe5dfddf22023-06-29 16:11:26 +010023use crate::payload::{add_microdroid_payload_images, add_microdroid_system_images, add_microdroid_vendor_image};
Seungjae Yoofd9a0622022-10-14 10:01:29 +090024use crate::selinux::{getfilecon, SeContext};
Jiyong Park753553b2021-07-12 21:21:09 +090025use android_os_permissions_aidl::aidl::android::os::IPermissionController;
David Brazdil49f96f52022-12-16 21:29:13 +000026use android_system_virtualizationcommon::aidl::android::system::virtualizationcommon::{
Alice Wang4e3015d2023-10-10 09:35:37 +000027 Certificate::Certificate,
Andrew Walbranc92d35f2022-01-12 12:45:19 +000028 DeathReason::DeathReason,
David Brazdil49f96f52022-12-16 21:29:13 +000029 ErrorCode::ErrorCode,
30};
31use android_system_virtualizationservice::aidl::android::system::virtualizationservice::{
Inseob Kim53d0b212023-07-20 16:58:37 +090032 AssignableDevice::AssignableDevice,
David Brazdil7d1e5ec2023-02-06 17:56:29 +000033 CpuTopology::CpuTopology,
Andrew Walbran6b650662021-09-07 13:13:23 +000034 DiskImage::DiskImage,
Jeongik Chac181be72024-03-27 00:18:30 +090035 InputDevice::InputDevice,
Alan Stokes0cc59ee2021-09-24 11:20:34 +010036 IVirtualMachine::{BnVirtualMachine, IVirtualMachine},
Andrew Walbran6b650662021-09-07 13:13:23 +000037 IVirtualMachineCallback::IVirtualMachineCallback,
38 IVirtualizationService::IVirtualizationService,
Jiyong Park029977d2021-11-24 21:56:49 +090039 Partition::Partition,
Andrew Walbran6b650662021-09-07 13:13:23 +000040 PartitionType::PartitionType,
Inseob Kim0168b462022-12-27 14:54:35 +090041 VirtualMachineAppConfig::{DebugLevel::DebugLevel, Payload::Payload, VirtualMachineAppConfig},
Jooyung Han21e9b922021-06-26 04:14:16 +090042 VirtualMachineConfig::VirtualMachineConfig,
Andrew Walbran6b650662021-09-07 13:13:23 +000043 VirtualMachineDebugInfo::VirtualMachineDebugInfo,
Alan Stokes0d1ef782022-09-27 13:46:35 +010044 VirtualMachinePayloadConfig::VirtualMachinePayloadConfig,
Jooyung Han21e9b922021-06-26 04:14:16 +090045 VirtualMachineRawConfig::VirtualMachineRawConfig,
Andrew Walbran6b650662021-09-07 13:13:23 +000046 VirtualMachineState::VirtualMachineState,
Jooyung Han21e9b922021-06-26 04:14:16 +090047};
David Brazdilafc9a9e2023-01-12 16:08:10 +000048use android_system_virtualizationservice_internal::aidl::android::system::virtualizationservice_internal::IVirtualizationServiceInternal::IVirtualizationServiceInternal;
Seungjae Yoodd91f0f2022-11-09 15:25:21 +090049use android_system_virtualmachineservice::aidl::android::system::virtualmachineservice::IVirtualMachineService::{
David Brazdilafc9a9e2023-01-12 16:08:10 +000050 BnVirtualMachineService, IVirtualMachineService,
Seungjae Yoofd9a0622022-10-14 10:01:29 +090051};
Shikha Panware45e9422024-02-28 21:18:10 +000052use android_hardware_security_secretkeeper::aidl::android::hardware::security::secretkeeper::ISecretkeeper::{BnSecretkeeper, ISecretkeeper};
Shikha Panwar5d6a6752023-12-14 22:08:26 +000053use android_hardware_security_secretkeeper::aidl::android::hardware::security::secretkeeper::SecretId::SecretId;
54use android_hardware_security_authgraph::aidl::android::hardware::security::authgraph::{
55 Arc::Arc as AuthgraphArc, IAuthGraphKeyExchange::IAuthGraphKeyExchange,
56 IAuthGraphKeyExchange::BnAuthGraphKeyExchange, Identity::Identity, KeInitResult::KeInitResult,
57 Key::Key, PubKey::PubKey, SessionIdSignature::SessionIdSignature, SessionInfo::SessionInfo,
58 SessionInitiationInfo::SessionInitiationInfo,
59};
Alan Stokes25f69362023-03-06 16:51:54 +000060use anyhow::{anyhow, bail, Context, Result};
Seungjae Yoofd9a0622022-10-14 10:01:29 +090061use apkverify::{HashAlgorithm, V4Signature};
Jiyong Parkd7bd2f22023-08-10 20:41:19 +090062use avflog::LogResult;
Alan Stokes0e82b502022-08-08 14:44:48 +010063use binder::{
David Brazdilafc9a9e2023-01-12 16:08:10 +000064 self, wait_for_interface, BinderFeatures, ExceptionCode, Interface, ParcelFileDescriptor,
65 Status, StatusCode, Strong,
Jiyong Park2227eaa2023-08-04 11:59:18 +090066 IntoBinderResult,
Andrew Walbrana89fc132021-03-17 17:08:36 +000067};
Shikha Panwar55e10ec2024-02-13 12:53:49 +000068use cstr::cstr;
Inseob Kim46257382024-01-03 15:41:22 +090069use glob::glob;
David Brazdil49f96f52022-12-16 21:29:13 +000070use lazy_static::lazy_static;
Seungjae Yoo0a8c84c2022-07-11 08:19:15 +000071use log::{debug, error, info, warn};
Inseob Kim89b24592024-02-23 18:59:43 +090072use microdroid_payload_config::{ApkConfig, Task, TaskType, VmPayloadConfig};
Inseob Kim0168b462022-12-27 14:54:35 +090073use nix::unistd::pipe;
David Brazdil73988ea2022-11-11 15:10:32 +000074use rpcbinder::RpcServer;
Alan Stokes25f69362023-03-06 16:51:54 +000075use rustutils::system_properties;
Jiyong Parkdcf17412022-02-08 15:07:23 +090076use semver::VersionReq;
Inseob Kim6ef80972023-07-20 17:23:36 +090077use std::collections::HashSet;
Andrew Walbrandff3b942021-06-09 15:20:36 +000078use std::convert::TryInto;
Shikha Panwar55e10ec2024-02-13 12:53:49 +000079use std::fs;
Jaewan Kim39952072024-01-19 17:04:53 +090080use std::ffi::CStr;
Inseob Kim6ef80972023-07-20 17:23:36 +090081use std::fs::{canonicalize, read_dir, remove_file, File, OpenOptions};
Shikha Panwar9ae2e622024-01-30 12:22:30 +000082use std::io::{BufRead, BufReader, Error, ErrorKind, Seek, SeekFrom, Write};
Inseob Kim46257382024-01-03 15:41:22 +090083use std::iter;
Nikita Ioffe5776f082023-02-10 21:38:26 +000084use std::num::{NonZeroU16, NonZeroU32};
Inseob Kimff48a7c2024-02-26 22:07:21 +090085use std::os::unix::io::{AsRawFd, FromRawFd, IntoRawFd};
David Brazdilafc9a9e2023-01-12 16:08:10 +000086use std::os::unix::raw::pid_t;
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +000087use std::path::{Path, PathBuf};
Andrew Walbran320b5602021-03-04 16:11:12 +000088use std::sync::{Arc, Mutex, Weak};
Seungjae Yoo14e60182024-02-21 13:28:31 +090089use vbmeta::VbMetaImage;
Pierre-Clément Tosid3bbe1d2024-04-15 18:03:51 +010090use vmconfig::{VmConfig, get_debug_level};
David Brazdilafc9a9e2023-01-12 16:08:10 +000091use vsock::VsockStream;
Jooyung Han35edb8f2021-07-01 16:17:16 +090092use zip::ZipArchive;
Andrew Walbrand6dce6f2021-03-05 16:39:08 +000093
David Brazdil41d1a872022-10-05 14:44:19 +010094/// The unique ID of a VM used (together with a port number) for vsock communication.
95pub type Cid = u32;
96
David Brazdil4b4c5102022-12-19 22:56:20 +000097pub const BINDER_SERVICE_IDENTIFIER: &str = "android.system.virtualizationservice";
98
Jooyung Han95884632021-07-06 22:27:54 +090099/// The size of zero.img.
100/// Gaps in composite disk images are filled with a shared zero.img.
101const ZERO_FILLER_SIZE: u64 = 4096;
102
David Brazdilf50c7a62023-04-19 14:22:42 +0000103/// Magic string for the instance image
104const ANDROID_VM_INSTANCE_MAGIC: &str = "Android-VM-instance";
105
106/// Version of the instance image format
107const ANDROID_VM_INSTANCE_VERSION: u16 = 1;
108
Alan Stokes0d1ef782022-09-27 13:46:35 +0100109const MICRODROID_OS_NAME: &str = "microdroid";
110
Shikha Panwar5d6a6752023-12-14 22:08:26 +0000111const SECRETKEEPER_IDENTIFIER: &str =
Shikha Panwarbe5dee72024-02-21 19:06:20 +0000112 "android.hardware.security.secretkeeper.ISecretkeeper/default";
Shikha Panwar5d6a6752023-12-14 22:08:26 +0000113
David Brazdilf50c7a62023-04-19 14:22:42 +0000114const UNFORMATTED_STORAGE_MAGIC: &str = "UNFORMATTED-STORAGE";
115
116/// crosvm requires all partitions to be a multiple of 4KiB.
117const PARTITION_GRANULARITY_BYTES: u64 = 4096;
118
Shikha Panwar55e10ec2024-02-13 12:53:49 +0000119const VM_REFERENCE_DT_ON_HOST_PATH: &str = "/proc/device-tree/avf/reference";
120
David Brazdil49f96f52022-12-16 21:29:13 +0000121lazy_static! {
David Brazdil4b4c5102022-12-19 22:56:20 +0000122 pub static ref GLOBAL_SERVICE: Strong<dyn IVirtualizationServiceInternal> =
123 wait_for_interface(BINDER_SERVICE_IDENTIFIER)
124 .expect("Could not connect to VirtualizationServiceInternal");
Inseob Kim46257382024-01-03 15:41:22 +0900125 static ref SUPPORTED_OS_NAMES: HashSet<String> =
126 get_supported_os_names().expect("Failed to get list of supported os names");
David Brazdil49f96f52022-12-16 21:29:13 +0000127}
128
Nikita Ioffef1ce9872022-12-09 13:31:59 +0000129fn create_or_update_idsig_file(
130 input_fd: &ParcelFileDescriptor,
131 idsig_fd: &ParcelFileDescriptor,
132) -> Result<()> {
133 let mut input = clone_file(input_fd)?;
134 let metadata = input.metadata().context("failed to get input metadata")?;
135 if !metadata.is_file() {
136 bail!("input is not a regular file");
137 }
Alan Stokes25f69362023-03-06 16:51:54 +0000138 let mut sig =
139 V4Signature::create(&mut input, get_current_sdk()?, 4096, &[], HashAlgorithm::SHA256)
140 .context("failed to create idsig")?;
Nikita Ioffef1ce9872022-12-09 13:31:59 +0000141
142 let mut output = clone_file(idsig_fd)?;
Jiyong Park8d192952023-06-26 14:29:51 +0900143
144 // Optimization. We don't have to update idsig file whenever a VM is started. Don't update it,
145 // if the idsig file already has the same APK digest.
146 if output.metadata()?.len() > 0 {
147 if let Ok(out_sig) = V4Signature::from_idsig(&mut output) {
148 if out_sig.signing_info.apk_digest == sig.signing_info.apk_digest {
149 debug!("idsig {:?} is up-to-date with apk {:?}.", output, input);
150 return Ok(());
151 }
152 }
153 // if we fail to read v4signature from output, that's fine. User can pass a random file.
154 // We will anyway overwrite the file to the v4signature generated from input_fd.
155 }
156
Shikha Panwar9ae2e622024-01-30 12:22:30 +0000157 output
158 .seek(SeekFrom::Start(0))
159 .context("failed to move cursor to start on the idsig output")?;
Nikita Ioffec09b0492022-12-14 20:18:33 +0000160 output.set_len(0).context("failed to set_len on the idsig output")?;
161 sig.write_into(&mut output).context("failed to write idsig")?;
Nikita Ioffef1ce9872022-12-09 13:31:59 +0000162 Ok(())
163}
164
Alan Stokes25f69362023-03-06 16:51:54 +0000165fn get_current_sdk() -> Result<u32> {
166 let current_sdk = system_properties::read("ro.build.version.sdk")?;
167 let current_sdk = current_sdk.ok_or_else(|| anyhow!("SDK version missing"))?;
168 current_sdk.parse().context("Malformed SDK version")
169}
170
David Brazdil4b4c5102022-12-19 22:56:20 +0000171pub fn remove_temporary_files(path: &PathBuf) -> Result<()> {
172 for dir_entry in read_dir(path)? {
173 remove_file(dir_entry?.path())?;
174 }
175 Ok(())
David Brazdil528e0472022-10-10 15:06:02 +0100176}
177
David Brazdil528e0472022-10-10 15:06:02 +0100178/// Implementation of `IVirtualizationService`, the entry point of the AIDL service.
David Brazdil49f96f52022-12-16 21:29:13 +0000179#[derive(Debug, Default)]
Andrew Walbranf6bf6862021-05-21 12:41:13 +0000180pub struct VirtualizationService {
Jiyong Park8611a6c2021-07-09 18:17:44 +0900181 state: Arc<Mutex<State>>,
Andrew Walbrand6dce6f2021-03-05 16:39:08 +0000182}
183
Shikha Panward8e35422021-10-11 13:51:27 +0000184impl Interface for VirtualizationService {
Andrei Homescu0cf8e222023-11-09 04:27:55 +0000185 fn dump(&self, writer: &mut dyn Write, _args: &[&CStr]) -> Result<(), StatusCode> {
Shikha Panward8e35422021-10-11 13:51:27 +0000186 check_permission("android.permission.DUMP").or(Err(StatusCode::PERMISSION_DENIED))?;
187 let state = &mut *self.state.lock().unwrap();
188 let vms = state.vms();
Andrei Homescu0cf8e222023-11-09 04:27:55 +0000189 writeln!(writer, "Running {0} VMs:", vms.len()).or(Err(StatusCode::UNKNOWN_ERROR))?;
Shikha Panward8e35422021-10-11 13:51:27 +0000190 for vm in vms {
Andrei Homescu0cf8e222023-11-09 04:27:55 +0000191 writeln!(writer, "VM CID: {}", vm.cid).or(Err(StatusCode::UNKNOWN_ERROR))?;
192 writeln!(writer, "\tState: {:?}", vm.vm_state.lock().unwrap())
Shikha Panward8e35422021-10-11 13:51:27 +0000193 .or(Err(StatusCode::UNKNOWN_ERROR))?;
Andrei Homescu0cf8e222023-11-09 04:27:55 +0000194 writeln!(writer, "\tPayload state {:?}", vm.payload_state())
Shikha Panward8e35422021-10-11 13:51:27 +0000195 .or(Err(StatusCode::UNKNOWN_ERROR))?;
Andrei Homescu0cf8e222023-11-09 04:27:55 +0000196 writeln!(writer, "\tProtected: {}", vm.protected).or(Err(StatusCode::UNKNOWN_ERROR))?;
197 writeln!(writer, "\ttemporary_directory: {}", vm.temporary_directory.to_string_lossy())
Shikha Panward8e35422021-10-11 13:51:27 +0000198 .or(Err(StatusCode::UNKNOWN_ERROR))?;
Andrei Homescu0cf8e222023-11-09 04:27:55 +0000199 writeln!(writer, "\trequester_uid: {}", vm.requester_uid)
Shikha Panward8e35422021-10-11 13:51:27 +0000200 .or(Err(StatusCode::UNKNOWN_ERROR))?;
Andrei Homescu0cf8e222023-11-09 04:27:55 +0000201 writeln!(writer, "\trequester_debug_pid: {}", vm.requester_debug_pid)
Shikha Panward8e35422021-10-11 13:51:27 +0000202 .or(Err(StatusCode::UNKNOWN_ERROR))?;
203 }
204 Ok(())
205 }
206}
Andrew Walbranf6bf6862021-05-21 12:41:13 +0000207impl IVirtualizationService for VirtualizationService {
Andrew Walbranf8d94112021-09-07 11:45:36 +0000208 /// Creates (but does not start) a new VM with the given configuration, assigning it the next
209 /// available CID.
Andrew Walbrand6dce6f2021-03-05 16:39:08 +0000210 ///
211 /// Returns a binder `IVirtualMachine` object referring to it, as a handle for the client.
Andrew Walbranf8d94112021-09-07 11:45:36 +0000212 fn createVm(
Andrew Walbrana89fc132021-03-17 17:08:36 +0000213 &self,
Andrew Walbran3a5a9212021-05-04 17:09:08 +0000214 config: &VirtualMachineConfig,
Jiyong Parke6fb1672023-06-26 16:45:55 +0900215 console_out_fd: Option<&ParcelFileDescriptor>,
216 console_in_fd: Option<&ParcelFileDescriptor>,
Andrew Walbrana89fc132021-03-17 17:08:36 +0000217 log_fd: Option<&ParcelFileDescriptor>,
218 ) -> binder::Result<Strong<dyn IVirtualMachine>> {
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000219 let mut is_protected = false;
Jiyong Parke6fb1672023-06-26 16:45:55 +0900220 let ret = self.create_vm_internal(
221 config,
222 console_out_fd,
223 console_in_fd,
224 log_fd,
225 &mut is_protected,
226 );
Seungjae Yoo0a8c84c2022-07-11 08:19:15 +0000227 write_vm_creation_stats(config, is_protected, &ret);
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000228 ret
Andrew Walbrand6dce6f2021-03-05 16:39:08 +0000229 }
Andrew Walbran320b5602021-03-04 16:11:12 +0000230
Shikha Panwar61a74b52024-02-16 13:17:01 +0000231 /// Allocate a new instance_id to the VM
232 fn allocateInstanceId(&self) -> binder::Result<[u8; 64]> {
Shikha Panwarbd5eba92024-03-27 20:23:31 +0000233 check_manage_access()?;
Shikha Panwar61a74b52024-02-16 13:17:01 +0000234 GLOBAL_SERVICE.allocateInstanceId()
235 }
236
Andrew Walbrandff3b942021-06-09 15:20:36 +0000237 /// Initialise an empty partition image of the given size to be used as a writable partition.
238 fn initializeWritablePartition(
239 &self,
240 image_fd: &ParcelFileDescriptor,
Alan Stokesff0005f2023-01-30 09:53:00 +0000241 size_bytes: i64,
Jiyong Park9dd389e2021-08-23 20:42:59 +0900242 partition_type: PartitionType,
Andrew Walbrandff3b942021-06-09 15:20:36 +0000243 ) -> binder::Result<()> {
Jiyong Park753553b2021-07-12 21:21:09 +0900244 check_manage_access()?;
Jiyong Park2227eaa2023-08-04 11:59:18 +0900245 let size_bytes = size_bytes
246 .try_into()
247 .with_context(|| format!("Invalid size: {}", size_bytes))
248 .or_binder_exception(ExceptionCode::ILLEGAL_ARGUMENT)?;
David Brazdilf50c7a62023-04-19 14:22:42 +0000249 let size_bytes = round_up(size_bytes, PARTITION_GRANULARITY_BYTES);
Shikha Panwar9ae2e622024-01-30 12:22:30 +0000250 let mut image = clone_file(image_fd)?;
David Brazdilf50c7a62023-04-19 14:22:42 +0000251 // initialize the file. Any data in the file will be erased.
Shikha Panwar9ae2e622024-01-30 12:22:30 +0000252 image
253 .seek(SeekFrom::Start(0))
254 .context("failed to move cursor to start")
255 .or_service_specific_exception(-1)?;
Jiyong Park2227eaa2023-08-04 11:59:18 +0900256 image.set_len(0).context("Failed to reset a file").or_service_specific_exception(-1)?;
Frederick Mayle0d310572024-05-02 12:34:58 -0700257 // Set the file length. In most filesystems, this will not allocate any physical disk
258 // space, it will only change the logical size.
259 image
260 .set_len(size_bytes)
261 .context("Failed to extend file")
Jiyong Park2227eaa2023-08-04 11:59:18 +0900262 .or_service_specific_exception(-1)?;
David Brazdilf50c7a62023-04-19 14:22:42 +0000263
264 match partition_type {
265 PartitionType::RAW => Ok(()),
Frederick Mayle0d310572024-05-02 12:34:58 -0700266 PartitionType::ANDROID_VM_INSTANCE => format_as_android_vm_instance(&mut image),
267 PartitionType::ENCRYPTEDSTORE => format_as_encryptedstore(&mut image),
David Brazdilf50c7a62023-04-19 14:22:42 +0000268 _ => Err(Error::new(
269 ErrorKind::Unsupported,
270 format!("Unsupported partition type {:?}", partition_type),
271 )),
272 }
Jiyong Park2227eaa2023-08-04 11:59:18 +0900273 .with_context(|| format!("Failed to initialize partition as {:?}", partition_type))
274 .or_service_specific_exception(-1)?;
David Brazdilf50c7a62023-04-19 14:22:42 +0000275
276 Ok(())
Andrew Walbrandff3b942021-06-09 15:20:36 +0000277 }
278
Jiyong Park0a248432021-08-20 23:32:39 +0900279 /// Creates or update the idsig file by digesting the input APK file.
280 fn createOrUpdateIdsigFile(
281 &self,
282 input_fd: &ParcelFileDescriptor,
283 idsig_fd: &ParcelFileDescriptor,
284 ) -> binder::Result<()> {
Jiyong Parkc3ca24f2022-06-28 10:45:15 +0900285 check_manage_access()?;
286
Jiyong Park2227eaa2023-08-04 11:59:18 +0900287 create_or_update_idsig_file(input_fd, idsig_fd).or_service_specific_exception(-1)?;
Jiyong Park0a248432021-08-20 23:32:39 +0900288 Ok(())
289 }
290
Andrew Walbran320b5602021-03-04 16:11:12 +0000291 /// Get a list of all currently running VMs. This method is only intended for debug purposes,
292 /// and as such is only permitted from the shell user.
293 fn debugListVms(&self) -> binder::Result<Vec<VirtualMachineDebugInfo>> {
David Brazdil209074a2023-01-12 16:44:51 +0000294 // Delegate to the global service, including checking the debug permission.
David Brazdild4f51a52023-01-11 14:09:27 +0000295 GLOBAL_SERVICE.debugListVms()
Andrew Walbran320b5602021-03-04 16:11:12 +0000296 }
Inseob Kim53d0b212023-07-20 16:58:37 +0900297
298 /// Get a list of assignable device types.
299 fn getAssignableDevices(&self) -> binder::Result<Vec<AssignableDevice>> {
300 // Delegate to the global service, including checking the permission.
301 GLOBAL_SERVICE.getAssignableDevices()
302 }
Nikita Ioffef7c742a2023-09-04 16:57:59 +0100303
Inseob Kim46257382024-01-03 15:41:22 +0900304 /// Get a list of supported OSes.
305 fn getSupportedOSList(&self) -> binder::Result<Vec<String>> {
306 Ok(Vec::from_iter(SUPPORTED_OS_NAMES.iter().cloned()))
307 }
308
Nikita Ioffef7c742a2023-09-04 16:57:59 +0100309 /// Returns whether given feature is enabled
310 fn isFeatureEnabled(&self, feature: &str) -> binder::Result<bool> {
311 check_manage_access()?;
Shikha Panwar6d306412024-02-17 21:37:49 +0000312 Ok(avf_features::is_feature_enabled(feature))
Nikita Ioffef7c742a2023-09-04 16:57:59 +0100313 }
Alice Wange64dd182024-01-17 15:57:55 +0000314
315 fn enableTestAttestation(&self) -> binder::Result<()> {
316 GLOBAL_SERVICE.enableTestAttestation()
317 }
Alice Wang0362f7f2024-03-21 08:16:26 +0000318
319 fn isRemoteAttestationSupported(&self) -> binder::Result<bool> {
320 check_manage_access()?;
321 GLOBAL_SERVICE.isRemoteAttestationSupported()
322 }
Shikha Panwar7aef66d2024-03-13 00:28:41 +0000323
324 fn isUpdatableVmSupported(&self) -> binder::Result<bool> {
325 // The response is specific to Microdroid. Updatable VMs are only possible if device
326 // supports Secretkeeper. Guest OS needs to use Secretkeeper based secrets. Microdroid does
327 // this, however other guest OSes may do things differently.
328 check_manage_access()?;
329 Ok(is_secretkeeper_supported())
330 }
Shikha Panwarbd5eba92024-03-27 20:23:31 +0000331
Shikha Panwar07d701c2024-03-28 14:43:52 +0000332 fn removeVmInstance(&self, instance_id: &[u8; 64]) -> binder::Result<()> {
333 check_manage_access()?;
334 GLOBAL_SERVICE.removeVmInstance(instance_id)
335 }
336
Shikha Panwarbd5eba92024-03-27 20:23:31 +0000337 fn claimVmInstance(&self, instance_id: &[u8; 64]) -> binder::Result<()> {
338 check_manage_access()?;
339 GLOBAL_SERVICE.claimVmInstance(instance_id)
340 }
Andrew Walbran320b5602021-03-04 16:11:12 +0000341}
342
Jiyong Park8611a6c2021-07-09 18:17:44 +0900343impl VirtualizationService {
344 pub fn init() -> VirtualizationService {
David Brazdil49f96f52022-12-16 21:29:13 +0000345 VirtualizationService::default()
Jiyong Park8611a6c2021-07-09 18:17:44 +0900346 }
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000347
David Brazdil209074a2023-01-12 16:44:51 +0000348 fn create_vm_context(
349 &self,
350 requester_debug_pid: pid_t,
351 ) -> binder::Result<(VmContext, Cid, PathBuf)> {
David Brazdil8cf8f482022-11-23 14:21:26 +0000352 const NUM_ATTEMPTS: usize = 5;
353
354 for _ in 0..NUM_ATTEMPTS {
Charisee96113f32023-01-26 09:00:42 +0000355 let vm_context = GLOBAL_SERVICE.allocateGlobalVmContext(requester_debug_pid)?;
David Brazdild4f51a52023-01-11 14:09:27 +0000356 let cid = vm_context.getCid()? as Cid;
357 let temp_dir: PathBuf = vm_context.getTemporaryDirectory()?.into();
David Brazdil8cf8f482022-11-23 14:21:26 +0000358 let service = VirtualMachineService::new_binder(self.state.clone(), cid).as_binder();
359
360 // Start VM service listening for connections from the new CID on port=CID.
David Brazdil8cf8f482022-11-23 14:21:26 +0000361 let port = cid;
David Brazdil3238da42022-11-18 10:04:51 +0000362 match RpcServer::new_vsock(service, cid, port) {
David Brazdil8cf8f482022-11-23 14:21:26 +0000363 Ok(vm_server) => {
364 vm_server.start();
David Brazdild4f51a52023-01-11 14:09:27 +0000365 return Ok((VmContext::new(vm_context, vm_server), cid, temp_dir));
David Brazdil8cf8f482022-11-23 14:21:26 +0000366 }
367 Err(err) => {
368 warn!("Could not start RpcServer on port {}: {}", port, err);
369 }
370 }
371 }
Jiyong Park2227eaa2023-08-04 11:59:18 +0900372 Err(anyhow!("Too many attempts to create VM context failed"))
373 .or_service_specific_exception(-1)
David Brazdil8cf8f482022-11-23 14:21:26 +0000374 }
375
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000376 fn create_vm_internal(
377 &self,
378 config: &VirtualMachineConfig,
Jiyong Parke6fb1672023-06-26 16:45:55 +0900379 console_out_fd: Option<&ParcelFileDescriptor>,
380 console_in_fd: Option<&ParcelFileDescriptor>,
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000381 log_fd: Option<&ParcelFileDescriptor>,
382 is_protected: &mut bool,
383 ) -> binder::Result<Strong<dyn IVirtualMachine>> {
David Brazdil209074a2023-01-12 16:44:51 +0000384 let requester_uid = get_calling_uid();
385 let requester_debug_pid = get_calling_pid();
386
Nikita Ioffe631717e2023-09-05 13:38:07 +0100387 check_config_features(config)?;
388
David Brazdil209074a2023-01-12 16:44:51 +0000389 // Allocating VM context checks the MANAGE_VIRTUAL_MACHINE permission.
390 let (vm_context, cid, temporary_directory) = self.create_vm_context(requester_debug_pid)?;
Inseob Kim1119d702022-05-02 18:01:58 +0900391
Alan Stokesfda70842023-12-20 17:50:14 +0000392 if is_custom_config(config) {
Alan Stokes7bc146c2022-10-20 17:10:32 +0100393 check_use_custom_virtual_machine()?;
Inseob Kim1119d702022-05-02 18:01:58 +0900394 }
395
Nikita Ioffe5776f082023-02-10 21:38:26 +0000396 let gdb_port = extract_gdb_port(config);
397
398 // Additional permission checks if caller request gdb.
399 if gdb_port.is_some() {
400 check_gdb_allowed(config)?;
401 }
402
Nikita Ioffeb2c6f3a2024-06-06 12:51:53 +0000403 let device_tree_overlay = maybe_create_device_tree_overlay(config, &temporary_directory)?;
Seungjae Yooec3bc522023-11-09 10:14:30 +0900404
Jaewan Kimf3143242024-03-15 06:56:31 +0000405 let debug_config = DebugConfig::new(config);
Nikita Ioffe2cb75cf2024-06-04 16:43:56 +0000406 let ramdump = if !uses_gki_kernel(config) && debug_config.is_ramdump_needed() {
Jiyong Parked180932023-02-24 19:55:41 +0900407 Some(prepare_ramdump_file(&temporary_directory)?)
408 } else {
409 None
410 };
411
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000412 let state = &mut *self.state.lock().unwrap();
Jiyong Parke6fb1672023-06-26 16:45:55 +0900413 let console_out_fd =
Pierre-Clément Tosifc523c52024-08-20 10:34:06 +0100414 clone_or_prepare_logger_fd(console_out_fd, format!("Console({})", cid))?;
Jiyong Parke6fb1672023-06-26 16:45:55 +0900415 let console_in_fd = console_in_fd.map(clone_file).transpose()?;
Pierre-Clément Tosifc523c52024-08-20 10:34:06 +0100416 let log_fd = clone_or_prepare_logger_fd(log_fd, format!("Log({})", cid))?;
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000417
418 // Counter to generate unique IDs for temporary image files.
419 let mut next_temporary_image_id = 0;
420 // Files which are referred to from composite images. These must be mapped to the crosvm
421 // child process, and not closed before it is started.
422 let mut indirect_files = vec![];
423
Alan Stokes7bc146c2022-10-20 17:10:32 +0100424 let (is_app_config, config) = match config {
425 VirtualMachineConfig::RawConfig(config) => (false, BorrowedOrOwned::Borrowed(config)),
426 VirtualMachineConfig::AppConfig(config) => {
Jiyong Park2227eaa2023-08-04 11:59:18 +0900427 let config = load_app_config(config, &debug_config, &temporary_directory)
428 .or_service_specific_exception_with(-1, |e| {
Jaewan Kim61f86142023-03-28 15:12:52 +0900429 *is_protected = config.protectedVm;
430 let message = format!("Failed to load app config: {:?}", e);
431 error!("{}", message);
Jiyong Park2227eaa2023-08-04 11:59:18 +0900432 message
Jaewan Kim61f86142023-03-28 15:12:52 +0900433 })?;
Alan Stokes7bc146c2022-10-20 17:10:32 +0100434 (true, BorrowedOrOwned::Owned(config))
435 }
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000436 };
437 let config = config.as_ref();
438 *is_protected = config.protectedVm;
439
440 // Check if partition images are labeled incorrectly. This is to prevent random images
441 // which are not protected by the Android Verified Boot (e.g. bits downloaded by apps) from
Alan Stokes3e5eec12023-09-07 12:10:00 +0100442 // being loaded in a pVM. This applies to everything but the instance image in the raw
443 // config, and everything but the non-executable, generated partitions in the app
444 // config.
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000445 config
446 .disks
447 .iter()
448 .flat_map(|disk| disk.partitions.iter())
449 .filter(|partition| {
450 if is_app_config {
Alan Stokes53cc5ca2022-08-30 14:28:19 +0100451 !is_safe_app_partition(&partition.label)
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000452 } else {
Alice Wangc206b9b2023-08-28 14:13:51 +0000453 !is_safe_raw_partition(&partition.label)
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000454 }
455 })
456 .try_for_each(check_label_for_partition)
Jiyong Park2227eaa2023-08-04 11:59:18 +0900457 .or_service_specific_exception(-1)?;
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000458
Inseob Kimff48a7c2024-02-26 22:07:21 +0900459 // Check if files for payloads and bases are NOT coming from /vendor and /odm, as they may
460 // have unstable interfaces.
461 // TODO(b/316431494): remove once Treble interfaces are stabilized.
462 check_partitions_for_files(config).or_service_specific_exception(-1)?;
463
Alan Stokes185fe112023-01-10 16:20:55 +0000464 let kernel = maybe_clone_file(&config.kernel)?;
465 let initrd = maybe_clone_file(&config.initrd)?;
466
Alan Stokes185fe112023-01-10 16:20:55 +0000467 if config.protectedVm {
Nikita Ioffef934e812024-07-05 15:44:41 +0000468 // In a protected VM, we require custom kernels to come from a trusted source
469 // (b/237054515).
Jiyong Park2227eaa2023-08-04 11:59:18 +0900470 check_label_for_kernel_files(&kernel, &initrd).or_service_specific_exception(-1)?;
Nikita Ioffef934e812024-07-05 15:44:41 +0000471 // Fail fast with a meaningful error message in case device doesn't support pVMs.
472 check_protected_vm_is_supported()?;
Alan Stokes185fe112023-01-10 16:20:55 +0000473 }
474
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000475 let zero_filler_path = temporary_directory.join("zero.img");
Jiyong Park2227eaa2023-08-04 11:59:18 +0900476 write_zero_filler(&zero_filler_path)
477 .context("Failed to make composite image")
478 .with_log()
479 .or_service_specific_exception(-1)?;
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000480
481 // Assemble disk images if needed.
482 let disks = config
483 .disks
484 .iter()
485 .map(|disk| {
486 assemble_disk_image(
487 disk,
488 &zero_filler_path,
489 &temporary_directory,
490 &mut next_temporary_image_id,
491 &mut indirect_files,
492 )
493 })
494 .collect::<Result<Vec<DiskFile>, _>>()?;
495
David Brazdil7d1e5ec2023-02-06 17:56:29 +0000496 let (cpus, host_cpu_topology) = match config.cpuTopology {
497 CpuTopology::MATCH_HOST => (None, true),
498 CpuTopology::ONE_CPU => (NonZeroU32::new(1), false),
499 val => {
Jiyong Park2227eaa2023-08-04 11:59:18 +0900500 return Err(anyhow!("Failed to parse CPU topology value {:?}", val))
501 .with_log()
502 .or_service_specific_exception(-1);
David Brazdil7d1e5ec2023-02-06 17:56:29 +0000503 }
504 };
505
David Brazdil2dfefd12023-11-17 14:07:36 +0000506 let (vfio_devices, dtbo) = if !config.devices.is_empty() {
Inseob Kim6ef80972023-07-20 17:23:36 +0900507 let mut set = HashSet::new();
508 for device in config.devices.iter() {
Jiyong Park2227eaa2023-08-04 11:59:18 +0900509 let path = canonicalize(device)
510 .with_context(|| format!("can't canonicalize {device}"))
511 .or_service_specific_exception(-1)?;
Inseob Kim6ef80972023-07-20 17:23:36 +0900512 if !set.insert(path) {
Jiyong Park2227eaa2023-08-04 11:59:18 +0900513 return Err(anyhow!("duplicated device {device}"))
514 .or_binder_exception(ExceptionCode::ILLEGAL_ARGUMENT);
Inseob Kim6ef80972023-07-20 17:23:36 +0900515 }
516 }
Jakob Vukalovicd42aa2c2023-11-09 16:04:00 +0000517 let devices = GLOBAL_SERVICE.bindDevicesToVfioDriver(&config.devices)?;
David Brazdil2dfefd12023-11-17 14:07:36 +0000518 let dtbo_file = File::from(
519 GLOBAL_SERVICE
520 .getDtboFile()?
521 .as_ref()
522 .try_clone()
Jaewan Kim39952072024-01-19 17:04:53 +0900523 .context("Failed to create VM DTBO from ParcelFileDescriptor")
David Brazdil2dfefd12023-11-17 14:07:36 +0000524 .or_binder_exception(ExceptionCode::BAD_PARCELABLE)?,
525 );
526 (devices, Some(dtbo_file))
Inseob Kim7307a892023-09-14 13:37:58 +0900527 } else {
David Brazdil2dfefd12023-11-17 14:07:36 +0000528 (vec![], None)
Inseob Kim7307a892023-09-14 13:37:58 +0900529 };
Jeongik Cha798401c2024-04-11 21:54:49 +0900530 let display_config = if cfg!(paravirtualized_devices) {
531 config
532 .displayConfig
533 .as_ref()
534 .map(DisplayConfig::new)
535 .transpose()
536 .or_binder_exception(ExceptionCode::ILLEGAL_ARGUMENT)?
537 } else {
538 None
539 };
Jason Macnakea7b0fa2024-05-08 17:02:44 -0700540 let gpu_config = if cfg!(paravirtualized_devices) {
541 config
542 .gpuConfig
543 .as_ref()
544 .map(GpuConfig::new)
545 .transpose()
546 .or_binder_exception(ExceptionCode::ILLEGAL_ARGUMENT)?
547 } else {
548 None
549 };
Jeongik Cha1df1c032024-04-03 16:03:12 +0900550
Jeongik Chac181be72024-03-27 00:18:30 +0900551 let input_device_options = if cfg!(paravirtualized_devices) {
552 config
553 .inputDevices
554 .iter()
555 .map(to_input_device_option_from)
556 .collect::<Result<Vec<InputDeviceOption>, _>>()
557 .or_binder_exception(ExceptionCode::ILLEGAL_ARGUMENT)?
558 } else {
559 vec![]
560 };
561
Seungjae Yoo13af0b62024-05-20 14:15:13 +0900562 // Create TAP network interface if the VM supports network.
Seungjae Yoo986d7a42024-05-28 14:06:58 +0900563 let tap = if cfg!(network) && config.networkSupported {
Seungjae Yoo13af0b62024-05-20 14:15:13 +0900564 if *is_protected {
565 return Err(anyhow!("Network feature is not supported for pVM yet"))
566 .with_log()
567 .or_binder_exception(ExceptionCode::UNSUPPORTED_OPERATION)?;
568 }
Seungjae Yoo986d7a42024-05-28 14:06:58 +0900569 Some(File::from(
570 GLOBAL_SERVICE
571 .createTapInterface(&get_this_pid().to_string())?
572 .as_ref()
573 .try_clone()
574 .context("Failed to get TAP interface from ParcelFileDescriptor")
575 .or_binder_exception(ExceptionCode::BAD_PARCELABLE)?,
576 ))
Seungjae Yoo13af0b62024-05-20 14:15:13 +0900577 } else {
578 None
579 };
Mu-Le Leeb2d5a312024-06-05 10:52:47 +0800580
581 let audio_config = if cfg!(paravirtualized_devices) {
582 config.audioConfig.as_ref().map(AudioConfig::new)
583 } else {
584 None
585 };
Seungjae Yoo13af0b62024-05-20 14:15:13 +0900586
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000587 // Actually start the VM.
588 let crosvm_config = CrosvmConfig {
589 cid,
Seungjae Yoo62085c02022-08-12 04:44:52 +0000590 name: config.name.clone(),
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000591 bootloader: maybe_clone_file(&config.bootloader)?,
Alan Stokes185fe112023-01-10 16:20:55 +0000592 kernel,
593 initrd,
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000594 disks,
595 params: config.params.to_owned(),
596 protected: *is_protected,
Jaewan Kim61f86142023-03-28 15:12:52 +0900597 debug_config,
Frederick Mayle68d06e72024-07-12 17:18:11 -0700598 memory_mib: config
599 .memoryMib
600 .try_into()
601 .ok()
602 .and_then(NonZeroU32::new)
603 .unwrap_or(NonZeroU32::new(256).unwrap()),
David Brazdil7d1e5ec2023-02-06 17:56:29 +0000604 cpus,
605 host_cpu_topology,
Jiyong Parke6fb1672023-06-26 16:45:55 +0900606 console_out_fd,
607 console_in_fd,
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000608 log_fd,
Jiyong Parked180932023-02-24 19:55:41 +0900609 ramdump,
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000610 indirect_files,
611 platform_version: parse_platform_version_req(&config.platformVersion)?,
Jiyong Parke6ed0f92022-06-22 00:13:00 +0900612 detect_hangup: is_app_config,
Nikita Ioffe5776f082023-02-10 21:38:26 +0000613 gdb_port,
Inseob Kim7307a892023-09-14 13:37:58 +0900614 vfio_devices,
David Brazdil2dfefd12023-11-17 14:07:36 +0000615 dtbo,
Shikha Panwar55e10ec2024-02-13 12:53:49 +0000616 device_tree_overlay,
Jeongik Cha1df1c032024-04-03 16:03:12 +0900617 display_config,
Jeongik Chac181be72024-03-27 00:18:30 +0900618 input_device_options,
Vincent Donnefort538a2c62024-03-20 16:01:10 +0000619 hugepages: config.hugePages,
Seungjae Yoo986d7a42024-05-28 14:06:58 +0900620 tap,
Yi-Yo Chiang8dd32552024-05-22 19:38:16 +0800621 console_input_device: config.consoleInputDevice.clone(),
David Dai23cff712024-06-13 19:23:45 +0000622 boost_uclamp: config.boostUclamp,
Jason Macnakea7b0fa2024-05-08 17:02:44 -0700623 gpu_config,
Mu-Le Leeb2d5a312024-06-05 10:52:47 +0800624 audio_config,
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000625 };
626 let instance = Arc::new(
David Brazdil528e0472022-10-10 15:06:02 +0100627 VmInstance::new(
628 crosvm_config,
629 temporary_directory,
630 requester_uid,
631 requester_debug_pid,
632 vm_context,
633 )
Jiyong Park2227eaa2023-08-04 11:59:18 +0900634 .with_context(|| format!("Failed to create VM with config {:?}", config))
635 .with_log()
636 .or_service_specific_exception(-1)?,
Shikha Panwar061aa2c2022-04-05 12:52:56 +0000637 );
638 state.add_vm(Arc::downgrade(&instance));
639 Ok(VirtualMachine::create(instance))
640 }
Jiyong Park8611a6c2021-07-09 18:17:44 +0900641}
642
Alan Stokesfda70842023-12-20 17:50:14 +0000643/// Returns whether a VM config represents a "custom" virtual machine, which requires the
644/// USE_CUSTOM_VIRTUAL_MACHINE.
645fn is_custom_config(config: &VirtualMachineConfig) -> bool {
646 match config {
647 // Any raw (non-Microdroid) VM is considered custom.
648 VirtualMachineConfig::RawConfig(_) => true,
649 VirtualMachineConfig::AppConfig(config) => {
650 // Some features are reserved for platform apps only, even when using
651 // VirtualMachineAppConfig. Almost all of these features are grouped in the
652 // CustomConfig struct:
653 // - controlling CPUs;
654 // - gdbPort is set, meaning that crosvm will start a gdb server;
655 // - using anything other than the default kernel;
656 // - specifying devices to be assigned.
657 if config.customConfig.is_some() {
658 true
659 } else {
660 // Additional custom features not included in CustomConfig:
661 // - specifying a config file;
Alan Stokes736f6822024-02-16 16:08:00 +0000662 // - specifying extra APKs;
663 // - specifying an OS other than Microdroid.
Inseob Kim89b24592024-02-23 18:59:43 +0900664 (match &config.payload {
Alan Stokesfda70842023-12-20 17:50:14 +0000665 Payload::ConfigPath(_) => true,
Inseob Kim89b24592024-02-23 18:59:43 +0900666 Payload::PayloadConfig(payload_config) => !payload_config.extraApks.is_empty(),
667 }) || config.osName != MICRODROID_OS_NAME
Alan Stokesfda70842023-12-20 17:50:14 +0000668 }
669 }
670 }
671}
672
Seungjae Yoo14e60182024-02-21 13:28:31 +0900673fn extract_vendor_hashtree_digest(config: &VirtualMachineConfig) -> Result<Option<Vec<u8>>> {
674 let VirtualMachineConfig::AppConfig(config) = config else {
675 return Ok(None);
676 };
677 let Some(custom_config) = &config.customConfig else {
678 return Ok(None);
679 };
680 let Some(file) = custom_config.vendorImage.as_ref() else {
681 return Ok(None);
682 };
683
684 let file = clone_file(file)?;
685 let size =
686 file.metadata().context("Failed to get metadata from microdroid vendor image")?.len();
687 let vbmeta = VbMetaImage::verify_reader_region(&file, 0, size)
688 .context("Failed to get vbmeta from microdroid-vendor.img")?;
689
690 for descriptor in vbmeta.descriptors()?.iter() {
691 if let vbmeta::Descriptor::Hashtree(_) = descriptor {
692 let root_digest = hex::encode(descriptor.to_hashtree()?.root_digest());
693 return Ok(Some(root_digest.as_bytes().to_vec()));
694 }
695 }
696 Err(anyhow!("No hashtree digest is extracted from microdroid vendor image"))
697}
698
Nikita Ioffeb2c6f3a2024-06-06 12:51:53 +0000699fn maybe_create_device_tree_overlay(
700 config: &VirtualMachineConfig,
701 temporary_directory: &Path,
702) -> binder::Result<Option<File>> {
703 // Currently, VirtMgr adds the host copy of reference DT & untrusted properties
704 // (e.g. instance-id)
705 let host_ref_dt = Path::new(VM_REFERENCE_DT_ON_HOST_PATH);
706 let host_ref_dt = if host_ref_dt.exists()
707 && read_dir(host_ref_dt).or_service_specific_exception(-1)?.next().is_some()
708 {
709 Some(host_ref_dt)
710 } else {
711 warn!("VM reference DT doesn't exist in host DT");
712 None
713 };
714
715 let vendor_hashtree_digest = extract_vendor_hashtree_digest(config)
716 .context("Failed to extract vendor hashtree digest")
717 .or_service_specific_exception(-1)?;
718
719 let trusted_props = if let Some(ref vendor_hashtree_digest) = vendor_hashtree_digest {
720 info!(
721 "Passing vendor hashtree digest to pvmfw. This will be rejected if it doesn't \
722 match the trusted digest in the pvmfw config, causing the VM to fail to start."
723 );
724 vec![(cstr!("vendor_hashtree_descriptor_root_digest"), vendor_hashtree_digest.as_slice())]
725 } else {
726 vec![]
727 };
728
729 let instance_id;
730 let mut untrusted_props = Vec::with_capacity(2);
731 if cfg!(llpvm_changes) {
732 instance_id = extract_instance_id(config);
733 untrusted_props.push((cstr!("instance-id"), &instance_id[..]));
734 let want_updatable = extract_want_updatable(config);
735 if want_updatable && is_secretkeeper_supported() {
736 // Let guest know that it can defer rollback protection to Secretkeeper by setting
737 // an empty property in untrusted node in DT. This enables Updatable VMs.
738 untrusted_props.push((cstr!("defer-rollback-protection"), &[]))
739 }
740 }
741
742 let device_tree_overlay = if host_ref_dt.is_some()
743 || !untrusted_props.is_empty()
744 || !trusted_props.is_empty()
745 {
746 let dt_output = temporary_directory.join(VM_DT_OVERLAY_PATH);
747 let mut data = [0_u8; VM_DT_OVERLAY_MAX_SIZE];
748 let fdt =
749 create_device_tree_overlay(&mut data, host_ref_dt, &untrusted_props, &trusted_props)
750 .map_err(|e| anyhow!("Failed to create DT overlay, {e:?}"))
751 .or_service_specific_exception(-1)?;
752 fs::write(&dt_output, fdt.as_slice()).or_service_specific_exception(-1)?;
753 Some(File::open(dt_output).or_service_specific_exception(-1)?)
754 } else {
755 None
756 };
757 Ok(device_tree_overlay)
758}
759
Andrew Walbranfbb39d22021-07-28 17:01:25 +0000760fn write_zero_filler(zero_filler_path: &Path) -> Result<()> {
Jooyung Han95884632021-07-06 22:27:54 +0900761 let file = OpenOptions::new()
762 .create_new(true)
763 .read(true)
764 .write(true)
765 .open(zero_filler_path)
766 .with_context(|| "Failed to create zero.img")?;
767 file.set_len(ZERO_FILLER_SIZE)?;
Andrew Walbranfbb39d22021-07-28 17:01:25 +0000768 Ok(())
Jooyung Han95884632021-07-06 22:27:54 +0900769}
770
David Brazdilf50c7a62023-04-19 14:22:42 +0000771fn format_as_android_vm_instance(part: &mut dyn Write) -> std::io::Result<()> {
772 part.write_all(ANDROID_VM_INSTANCE_MAGIC.as_bytes())?;
773 part.write_all(&ANDROID_VM_INSTANCE_VERSION.to_le_bytes())?;
774 part.flush()
775}
776
777fn format_as_encryptedstore(part: &mut dyn Write) -> std::io::Result<()> {
778 part.write_all(UNFORMATTED_STORAGE_MAGIC.as_bytes())?;
779 part.flush()
780}
781
782fn round_up(input: u64, granularity: u64) -> u64 {
783 if granularity == 0 {
784 return input;
785 }
786 // If the input is absurdly large we round down instead of up; it's going to fail anyway.
787 let result = input.checked_add(granularity - 1).unwrap_or(input);
788 (result / granularity) * granularity
789}
790
Jeongik Chac181be72024-03-27 00:18:30 +0900791fn to_input_device_option_from(input_device: &InputDevice) -> Result<InputDeviceOption> {
792 Ok(match input_device {
793 InputDevice::SingleTouch(single_touch) => InputDeviceOption::SingleTouch {
794 file: clone_file(single_touch.pfd.as_ref().ok_or(anyhow!("pfd should have value"))?)?,
795 height: u32::try_from(single_touch.height)?,
796 width: u32::try_from(single_touch.width)?,
797 name: if !single_touch.name.is_empty() {
798 Some(single_touch.name.clone())
799 } else {
800 None
801 },
802 },
803 InputDevice::EvDev(evdev) => InputDeviceOption::EvDev(clone_file(
804 evdev.pfd.as_ref().ok_or(anyhow!("pfd should have value"))?,
805 )?),
Jeongik Cha985b6402024-04-15 18:13:00 +0900806 InputDevice::Keyboard(keyboard) => InputDeviceOption::Keyboard(clone_file(
807 keyboard.pfd.as_ref().ok_or(anyhow!("pfd should have value"))?,
808 )?),
Jeongik Cha343894e2024-05-17 20:39:31 +0900809 InputDevice::Mouse(mouse) => InputDeviceOption::Mouse(clone_file(
810 mouse.pfd.as_ref().ok_or(anyhow!("pfd should have value"))?,
811 )?),
Jiyong Park6c1b9f02024-07-05 16:21:10 +0900812 InputDevice::Switches(switches) => InputDeviceOption::Switches(clone_file(
813 switches.pfd.as_ref().ok_or(anyhow!("pfd should have value"))?,
814 )?),
Jeongik Cha112a2682024-07-09 12:28:45 +0900815 InputDevice::Trackpad(trackpad) => InputDeviceOption::MultiTouchTrackpad {
816 file: clone_file(trackpad.pfd.as_ref().ok_or(anyhow!("pfd should have value"))?)?,
817 height: u32::try_from(trackpad.height)?,
818 width: u32::try_from(trackpad.width)?,
819 name: if !trackpad.name.is_empty() { Some(trackpad.name.clone()) } else { None },
820 },
Jeongik Cha10494912024-07-16 11:19:50 +0900821 InputDevice::MultiTouch(multi_touch) => InputDeviceOption::MultiTouch {
822 file: clone_file(multi_touch.pfd.as_ref().ok_or(anyhow!("pfd should have value"))?)?,
823 height: u32::try_from(multi_touch.height)?,
824 width: u32::try_from(multi_touch.width)?,
825 name: if !multi_touch.name.is_empty() { Some(multi_touch.name.clone()) } else { None },
826 },
Jeongik Chac181be72024-03-27 00:18:30 +0900827 })
828}
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000829/// Given the configuration for a disk image, assembles the `DiskFile` to pass to crosvm.
830///
831/// This may involve assembling a composite disk from a set of partition images.
832fn assemble_disk_image(
833 disk: &DiskImage,
Jooyung Han95884632021-07-06 22:27:54 +0900834 zero_filler_path: &Path,
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000835 temporary_directory: &Path,
836 next_temporary_image_id: &mut u64,
837 indirect_files: &mut Vec<File>,
Andrew Walbran806f1542021-06-10 14:07:12 +0000838) -> Result<DiskFile, Status> {
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000839 let image = if !disk.partitions.is_empty() {
840 if disk.image.is_some() {
841 warn!("DiskImage {:?} contains both image and partitions.", disk);
Jiyong Park2227eaa2023-08-04 11:59:18 +0900842 return Err(anyhow!("DiskImage contains both image and partitions"))
843 .or_binder_exception(ExceptionCode::ILLEGAL_ARGUMENT);
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000844 }
845
Andrew Walbran3eca16c2021-06-14 11:15:14 +0000846 let composite_image_filenames =
847 make_composite_image_filenames(temporary_directory, next_temporary_image_id);
848 let (image, partition_files) = make_composite_image(
849 &disk.partitions,
Jooyung Han95884632021-07-06 22:27:54 +0900850 zero_filler_path,
Andrew Walbran3eca16c2021-06-14 11:15:14 +0000851 &composite_image_filenames.composite,
852 &composite_image_filenames.header,
853 &composite_image_filenames.footer,
854 )
Jiyong Park2227eaa2023-08-04 11:59:18 +0900855 .with_context(|| format!("Failed to make composite disk image with config {:?}", disk))
856 .with_log()
857 .or_service_specific_exception(-1)?;
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000858
859 // Pass the file descriptors for the various partition files to crosvm when it
860 // is run.
861 indirect_files.extend(partition_files);
862
863 image
864 } else if let Some(image) = &disk.image {
865 clone_file(image)?
866 } else {
867 warn!("DiskImage {:?} didn't contain image or partitions.", disk);
Jiyong Park2227eaa2023-08-04 11:59:18 +0900868 return Err(anyhow!("DiskImage didn't contain image or partitions."))
869 .or_binder_exception(ExceptionCode::ILLEGAL_ARGUMENT);
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000870 };
871
872 Ok(DiskFile { image, writable: disk.writable })
873}
874
Nikita Ioffeaa6858c2023-07-04 01:37:41 +0100875fn append_kernel_param(param: &str, vm_config: &mut VirtualMachineRawConfig) {
876 if let Some(ref mut params) = vm_config.params {
877 params.push(' ');
878 params.push_str(param)
879 } else {
880 vm_config.params = Some(param.to_owned())
881 }
882}
883
Inseob Kim46257382024-01-03 15:41:22 +0900884fn extract_os_name_from_config_path(config: &Path) -> Option<String> {
885 if config.extension()?.to_str()? != "json" {
886 return None;
Inseob Kim172f9eb2023-11-06 17:02:08 +0900887 }
Inseob Kim46257382024-01-03 15:41:22 +0900888
889 Some(config.with_extension("").file_name()?.to_str()?.to_owned())
890}
891
892fn extract_os_names_from_configs(config_glob_pattern: &str) -> Result<HashSet<String>> {
893 let configs = glob(config_glob_pattern)?.collect::<Result<Vec<_>, _>>()?;
894 let os_names =
895 configs.iter().filter_map(|x| extract_os_name_from_config_path(x)).collect::<HashSet<_>>();
896
897 Ok(os_names)
898}
899
900fn get_supported_os_names() -> Result<HashSet<String>> {
901 if !cfg!(vendor_modules) {
902 return Ok(iter::once(MICRODROID_OS_NAME.to_owned()).collect());
903 }
904
905 extract_os_names_from_configs("/apex/com.android.virt/etc/microdroid*.json")
906}
907
908fn is_valid_os(os_name: &str) -> bool {
909 SUPPORTED_OS_NAMES.contains(os_name)
Inseob Kim172f9eb2023-11-06 17:02:08 +0900910}
911
Nikita Ioffe2cb75cf2024-06-04 16:43:56 +0000912fn uses_gki_kernel(config: &VirtualMachineConfig) -> bool {
913 if !cfg!(vendor_modules) {
914 return false;
915 }
916 match config {
917 VirtualMachineConfig::RawConfig(_) => false,
918 VirtualMachineConfig::AppConfig(config) => config.osName.starts_with("microdroid_gki-"),
919 }
920}
921
Jooyung Han21e9b922021-06-26 04:14:16 +0900922fn load_app_config(
923 config: &VirtualMachineAppConfig,
Jaewan Kim61f86142023-03-28 15:12:52 +0900924 debug_config: &DebugConfig,
Jooyung Han21e9b922021-06-26 04:14:16 +0900925 temporary_directory: &Path,
Jooyung Hanadfb76c2021-06-28 17:29:30 +0900926) -> Result<VirtualMachineRawConfig> {
Andrew Walbrancc0db522021-07-12 17:03:42 +0000927 let apk_file = clone_file(config.apk.as_ref().unwrap())?;
928 let idsig_file = clone_file(config.idsig.as_ref().unwrap())?;
Jiyong Park8d081812021-07-23 17:45:04 +0900929 let instance_file = clone_file(config.instanceImage.as_ref().unwrap())?;
Jooyung Han21e9b922021-06-26 04:14:16 +0900930
Shikha Panwar22e70452022-10-10 18:32:55 +0000931 let storage_image = if let Some(file) = config.encryptedStorageImage.as_ref() {
932 Some(clone_file(file)?)
933 } else {
934 None
935 };
936
Alan Stokesfda70842023-12-20 17:50:14 +0000937 let vm_payload_config;
938 let extra_apk_files: Vec<_>;
939 match &config.payload {
Alan Stokes0d1ef782022-09-27 13:46:35 +0100940 Payload::ConfigPath(config_path) => {
Alan Stokesfda70842023-12-20 17:50:14 +0000941 vm_payload_config =
942 load_vm_payload_config_from_file(&apk_file, config_path.as_str())
943 .with_context(|| format!("Couldn't read config from {}", config_path))?;
944 extra_apk_files = vm_payload_config
945 .extra_apks
946 .iter()
947 .enumerate()
948 .map(|(i, apk)| {
949 File::open(PathBuf::from(&apk.path))
950 .with_context(|| format!("Failed to open extra apk #{i} {}", apk.path))
951 })
952 .collect::<Result<_>>()?;
Alan Stokes0d1ef782022-09-27 13:46:35 +0100953 }
Alan Stokesfda70842023-12-20 17:50:14 +0000954 Payload::PayloadConfig(payload_config) => {
955 vm_payload_config = create_vm_payload_config(payload_config)?;
956 extra_apk_files =
957 payload_config.extraApks.iter().map(clone_file).collect::<binder::Result<_>>()?;
958 }
Alan Stokes0d1ef782022-09-27 13:46:35 +0100959 };
Jooyung Han21e9b922021-06-26 04:14:16 +0900960
Inseob Kim89b24592024-02-23 18:59:43 +0900961 let payload_config_os = vm_payload_config.os.name.as_str();
962 if !payload_config_os.is_empty() && payload_config_os != "microdroid" {
963 bail!("'os' in payload config is deprecated");
964 }
965
Inseob Kim172f9eb2023-11-06 17:02:08 +0900966 // For now, the only supported OS is Microdroid and Microdroid GKI
Inseob Kim89b24592024-02-23 18:59:43 +0900967 let os_name = config.osName.as_str();
Inseob Kim172f9eb2023-11-06 17:02:08 +0900968 if !is_valid_os(os_name) {
Andrew Walbrancc0db522021-07-12 17:03:42 +0000969 bail!("Unknown OS \"{}\"", os_name);
Jooyung Han35edb8f2021-07-01 16:17:16 +0900970 }
Andrew Walbrancc0db522021-07-12 17:03:42 +0000971
972 // It is safe to construct a filename based on the os_name because we've already checked that it
973 // is one of the allowed values.
Jooyung Han21e9b922021-06-26 04:14:16 +0900974 let vm_config_path = PathBuf::from(format!("/apex/com.android.virt/etc/{}.json", os_name));
975 let vm_config_file = File::open(vm_config_path)?;
Andrew Walbrancc0db522021-07-12 17:03:42 +0000976 let mut vm_config = VmConfig::load(&vm_config_file)?.to_parcelable()?;
Jooyung Han21e9b922021-06-26 04:14:16 +0900977
Nikita Ioffea0eb5ee2023-06-26 18:18:21 +0100978 if let Some(custom_config) = &config.customConfig {
979 if let Some(file) = custom_config.customKernelImage.as_ref() {
980 vm_config.kernel = Some(ParcelFileDescriptor::new(clone_file(file)?))
981 }
Nikita Ioffea0eb5ee2023-06-26 18:18:21 +0100982 vm_config.gdbPort = custom_config.gdbPort;
Nikita Ioffe5dfddf22023-06-29 16:11:26 +0100983
984 if let Some(file) = custom_config.vendorImage.as_ref() {
Nikita Ioffeaa6858c2023-07-04 01:37:41 +0100985 add_microdroid_vendor_image(clone_file(file)?, &mut vm_config);
986 append_kernel_param("androidboot.microdroid.mount_vendor=1", &mut vm_config)
Nikita Ioffe5dfddf22023-06-29 16:11:26 +0100987 }
Inseob Kim6ef80972023-07-20 17:23:36 +0900988
Chris Wailes6177c662024-05-09 14:37:44 -0700989 vm_config.devices.clone_from(&custom_config.devices);
Seungjae Yoo13af0b62024-05-20 14:15:13 +0900990 vm_config.networkSupported = custom_config.networkSupported;
Nikita Ioffe26c35ed2023-06-05 17:49:08 +0100991 }
992
Andrew Walbrancc045902021-07-27 16:06:17 +0000993 if config.memoryMib > 0 {
994 vm_config.memoryMib = config.memoryMib;
Andrew Walbran45bcb0c2021-07-14 15:02:06 +0000995 }
996
Chris Wailes6177c662024-05-09 14:37:44 -0700997 vm_config.name.clone_from(&config.name);
Andrew Walbran3994f002022-01-27 17:33:45 +0000998 vm_config.protectedVm = config.protectedVm;
David Brazdil7d1e5ec2023-02-06 17:56:29 +0000999 vm_config.cpuTopology = config.cpuTopology;
Vincent Donnefort538a2c62024-03-20 16:01:10 +00001000 vm_config.hugePages = config.hugePages || vm_payload_config.hugepages;
David Dai23cff712024-06-13 19:23:45 +00001001 vm_config.boostUclamp = config.boostUclamp;
Jiyong Park032615f2022-01-10 13:55:34 +09001002
Shikha Panwar22e70452022-10-10 18:32:55 +00001003 // Microdroid takes additional init ramdisk & (optionally) storage image
Inseob Kim172f9eb2023-11-06 17:02:08 +09001004 add_microdroid_system_images(config, instance_file, storage_image, os_name, &mut vm_config)?;
Shikha Panwar22e70452022-10-10 18:32:55 +00001005
1006 // Include Microdroid payload disk (contains apks, idsigs) in vm config
1007 add_microdroid_payload_images(
Alan Stokes0d1ef782022-09-27 13:46:35 +01001008 config,
Jaewan Kim61f86142023-03-28 15:12:52 +09001009 debug_config,
Alan Stokes0d1ef782022-09-27 13:46:35 +01001010 temporary_directory,
1011 apk_file,
1012 idsig_file,
Alan Stokesfda70842023-12-20 17:50:14 +00001013 extra_apk_files,
Alan Stokes0d1ef782022-09-27 13:46:35 +01001014 &vm_payload_config,
1015 &mut vm_config,
1016 )?;
Jooyung Han21e9b922021-06-26 04:14:16 +09001017
Andrew Walbrancc0db522021-07-12 17:03:42 +00001018 Ok(vm_config)
Jooyung Han21e9b922021-06-26 04:14:16 +09001019}
1020
Inseob Kimff48a7c2024-02-26 22:07:21 +09001021fn check_partition_for_file(fd: &ParcelFileDescriptor) -> Result<()> {
1022 let path = format!("/proc/self/fd/{}", fd.as_raw_fd());
1023 let link = fs::read_link(&path).context(format!("can't read_link {path}"))?;
1024
1025 // microdroid vendor image is OK
1026 if cfg!(vendor_modules) && link == Path::new("/vendor/etc/avf/microdroid/microdroid_vendor.img")
1027 {
1028 return Ok(());
1029 }
1030
1031 if link.starts_with("/vendor") || link.starts_with("/odm") {
1032 bail!("vendor or odm file {} can't be used for VM", link.display());
1033 }
1034
1035 Ok(())
1036}
1037
1038fn check_partitions_for_files(config: &VirtualMachineRawConfig) -> Result<()> {
1039 config
1040 .disks
1041 .iter()
1042 .flat_map(|disk| disk.partitions.iter())
1043 .filter_map(|partition| partition.image.as_ref())
1044 .try_for_each(check_partition_for_file)?;
1045
1046 config.kernel.as_ref().map_or(Ok(()), check_partition_for_file)?;
1047 config.initrd.as_ref().map_or(Ok(()), check_partition_for_file)?;
1048 config.bootloader.as_ref().map_or(Ok(()), check_partition_for_file)?;
1049
1050 Ok(())
1051}
1052
Alan Stokes0d1ef782022-09-27 13:46:35 +01001053fn load_vm_payload_config_from_file(apk_file: &File, config_path: &str) -> Result<VmPayloadConfig> {
1054 let mut apk_zip = ZipArchive::new(apk_file)?;
1055 let config_file = apk_zip.by_name(config_path)?;
1056 Ok(serde_json::from_reader(config_file)?)
1057}
1058
Alan Stokes8f12f2b2023-01-09 09:19:20 +00001059fn create_vm_payload_config(
1060 payload_config: &VirtualMachinePayloadConfig,
1061) -> Result<VmPayloadConfig> {
Alan Stokes0d1ef782022-09-27 13:46:35 +01001062 // There isn't an actual config file. Construct a synthetic VmPayloadConfig from the explicit
1063 // parameters we've been given. Microdroid will do something equivalent inside the VM using the
1064 // payload config that we send it via the metadata file.
Alan Stokes8f12f2b2023-01-09 09:19:20 +00001065
1066 let payload_binary_name = &payload_config.payloadBinaryName;
1067 if payload_binary_name.contains('/') {
1068 bail!("Payload binary name must not specify a path: {payload_binary_name}");
1069 }
1070
1071 let task = Task { type_: TaskType::MicrodroidLauncher, command: payload_binary_name.clone() };
Alan Stokesfda70842023-12-20 17:50:14 +00001072
1073 // The VM only cares about how many there are, these names are actually ignored.
1074 let extra_apk_count = payload_config.extraApks.len();
1075 let extra_apks =
1076 (0..extra_apk_count).map(|i| ApkConfig { path: format!("extra-apk-{i}") }).collect();
1077
Inseob Kim89b24592024-02-23 18:59:43 +09001078 Ok(VmPayloadConfig { task: Some(task), extra_apks, ..Default::default() })
Alan Stokes0d1ef782022-09-27 13:46:35 +01001079}
1080
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +00001081/// Generates a unique filename to use for a composite disk image.
Andrew Walbran3eca16c2021-06-14 11:15:14 +00001082fn make_composite_image_filenames(
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +00001083 temporary_directory: &Path,
1084 next_temporary_image_id: &mut u64,
Andrew Walbran3eca16c2021-06-14 11:15:14 +00001085) -> CompositeImageFilenames {
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +00001086 let id = *next_temporary_image_id;
1087 *next_temporary_image_id += 1;
Andrew Walbran3eca16c2021-06-14 11:15:14 +00001088 CompositeImageFilenames {
1089 composite: temporary_directory.join(format!("composite-{}.img", id)),
1090 header: temporary_directory.join(format!("composite-{}-header.img", id)),
1091 footer: temporary_directory.join(format!("composite-{}-footer.img", id)),
1092 }
1093}
1094
1095/// Filenames for a composite disk image, including header and footer partitions.
1096#[derive(Clone, Debug, Eq, PartialEq)]
1097struct CompositeImageFilenames {
1098 /// The composite disk image itself.
1099 composite: PathBuf,
1100 /// The header partition image.
1101 header: PathBuf,
1102 /// The footer partition image.
1103 footer: PathBuf,
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +00001104}
1105
Jiyong Park753553b2021-07-12 21:21:09 +09001106/// Checks whether the caller has a specific permission
1107fn check_permission(perm: &str) -> binder::Result<()> {
David Brazdil1f530702022-10-03 12:18:10 +01001108 let calling_pid = get_calling_pid();
1109 let calling_uid = get_calling_uid();
Jiyong Park753553b2021-07-12 21:21:09 +09001110 // Root can do anything
1111 if calling_uid == 0 {
1112 return Ok(());
1113 }
1114 let perm_svc: Strong<dyn IPermissionController::IPermissionController> =
Frederick Mayleb2a02872024-05-08 13:35:12 -07001115 binder::wait_for_interface("permission")?;
Jiyong Park753553b2021-07-12 21:21:09 +09001116 if perm_svc.checkPermission(perm, calling_pid, calling_uid as i32)? {
Andrew Walbran806f1542021-06-10 14:07:12 +00001117 Ok(())
1118 } else {
Jiyong Park2227eaa2023-08-04 11:59:18 +09001119 Err(anyhow!("does not have the {} permission", perm))
1120 .or_binder_exception(ExceptionCode::SECURITY)
Andrew Walbran806f1542021-06-10 14:07:12 +00001121 }
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001122}
1123
Jiyong Park753553b2021-07-12 21:21:09 +09001124/// Check whether the caller of the current Binder method is allowed to manage VMs
1125fn check_manage_access() -> binder::Result<()> {
1126 check_permission("android.permission.MANAGE_VIRTUAL_MACHINE")
1127}
1128
Inseob Kim1119d702022-05-02 18:01:58 +09001129/// Check whether the caller of the current Binder method is allowed to create custom VMs
1130fn check_use_custom_virtual_machine() -> binder::Result<()> {
1131 check_permission("android.permission.USE_CUSTOM_VIRTUAL_MACHINE")
1132}
1133
Alan Stokes185fe112023-01-10 16:20:55 +00001134/// Return whether a partition is exempt from selinux label checks, because we know that it does
1135/// not contain code and is likely to be generated in an app-writable directory.
Alan Stokes53cc5ca2022-08-30 14:28:19 +01001136fn is_safe_app_partition(label: &str) -> bool {
Shikha Panwara2ff8c52022-11-30 19:25:46 +00001137 // See add_microdroid_system_images & add_microdroid_payload_images in payload.rs.
Alan Stokes53cc5ca2022-08-30 14:28:19 +01001138 label == "vm-instance"
Shikha Panwara2ff8c52022-11-30 19:25:46 +00001139 || label == "encryptedstore"
Alan Stokes53cc5ca2022-08-30 14:28:19 +01001140 || label == "microdroid-apk-idsig"
1141 || label == "payload-metadata"
1142 || label.starts_with("extra-idsig-")
1143}
1144
Alice Wangc206b9b2023-08-28 14:13:51 +00001145/// Returns whether a partition with the given label is safe for a raw config VM.
1146fn is_safe_raw_partition(label: &str) -> bool {
1147 label == "vm-instance"
1148}
1149
Alan Stokes185fe112023-01-10 16:20:55 +00001150/// Check that a file SELinux label is acceptable.
1151///
1152/// We only want to allow code in a VM to be sourced from places that apps, and the
Seungjae Yoo2b74c442023-11-15 18:05:07 +09001153/// system or vendor, do not have write access to.
Alan Stokes185fe112023-01-10 16:20:55 +00001154///
1155/// Note that sepolicy must also grant read access for these types to both virtualization
1156/// service and crosvm.
1157///
1158/// App private data files are deliberately excluded, to avoid arbitrary payloads being run on
1159/// user devices (W^X).
1160fn check_label_is_allowed(context: &SeContext) -> Result<()> {
1161 match context.selinux_type()? {
Alan Stokes53cc5ca2022-08-30 14:28:19 +01001162 | "apk_data_file" // APKs of an installed app
Alan Stokes53cc5ca2022-08-30 14:28:19 +01001163 | "shell_data_file" // test files created via adb shell
Alan Stokesfe4bb0c2023-03-20 14:15:36 +00001164 | "staging_data_file" // updated/staged APEX images
1165 | "system_file" // immutable dm-verity protected partition
1166 | "virtualizationservice_data_file" // files created by VS / VirtMgr
Seungjae Yoo2b74c442023-11-15 18:05:07 +09001167 | "vendor_microdroid_file" // immutable dm-verity protected partition (/vendor/etc/avf/microdroid/.*)
Alan Stokes53cc5ca2022-08-30 14:28:19 +01001168 => Ok(()),
Alan Stokes185fe112023-01-10 16:20:55 +00001169 _ => bail!("Label {} is not allowed", context),
Jiyong Park029977d2021-11-24 21:56:49 +09001170 }
1171}
1172
Alan Stokes185fe112023-01-10 16:20:55 +00001173fn check_label_for_partition(partition: &Partition) -> Result<()> {
1174 let file = partition.image.as_ref().unwrap().as_ref();
1175 check_label_is_allowed(&getfilecon(file)?)
1176 .with_context(|| format!("Partition {} invalid", &partition.label))
1177}
1178
1179fn check_label_for_kernel_files(kernel: &Option<File>, initrd: &Option<File>) -> Result<()> {
1180 if let Some(f) = kernel {
1181 check_label_for_file(f, "kernel")?;
1182 }
1183 if let Some(f) = initrd {
1184 check_label_for_file(f, "initrd")?;
1185 }
1186 Ok(())
1187}
1188fn check_label_for_file(file: &File, name: &str) -> Result<()> {
1189 check_label_is_allowed(&getfilecon(file)?).with_context(|| format!("{} file invalid", name))
1190}
1191
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001192/// Implementation of the AIDL `IVirtualMachine` interface. Used as a handle to a VM.
1193#[derive(Debug)]
1194struct VirtualMachine {
1195 instance: Arc<VmInstance>,
1196}
1197
1198impl VirtualMachine {
1199 fn create(instance: Arc<VmInstance>) -> Strong<dyn IVirtualMachine> {
David Brazdil4b4c5102022-12-19 22:56:20 +00001200 BnVirtualMachine::new_binder(VirtualMachine { instance }, BinderFeatures::default())
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001201 }
1202}
1203
1204impl Interface for VirtualMachine {}
1205
1206impl IVirtualMachine for VirtualMachine {
1207 fn getCid(&self) -> binder::Result<i32> {
Jiyong Park753553b2021-07-12 21:21:09 +09001208 // Don't check permission. The owner of the VM might have passed this binder object to
1209 // others.
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001210 Ok(self.instance.cid as i32)
1211 }
Andrew Walbrandae07162021-03-12 17:05:20 +00001212
Andrew Walbran6b650662021-09-07 13:13:23 +00001213 fn getState(&self) -> binder::Result<VirtualMachineState> {
Jiyong Park753553b2021-07-12 21:21:09 +09001214 // Don't check permission. The owner of the VM might have passed this binder object to
1215 // others.
Andrew Walbran6b650662021-09-07 13:13:23 +00001216 Ok(get_state(&self.instance))
Andrew Walbrandae07162021-03-12 17:05:20 +00001217 }
1218
1219 fn registerCallback(
1220 &self,
1221 callback: &Strong<dyn IVirtualMachineCallback>,
1222 ) -> binder::Result<()> {
Jiyong Park753553b2021-07-12 21:21:09 +09001223 // Don't check permission. The owner of the VM might have passed this binder object to
1224 // others.
1225 //
Andrew Walbrandae07162021-03-12 17:05:20 +00001226 // TODO: Should this give an error if the VM is already dead?
1227 self.instance.callbacks.add(callback.clone());
1228 Ok(())
1229 }
Andrew Walbrancbe8b082021-08-06 15:42:11 +00001230
Andrew Walbranf8d94112021-09-07 11:45:36 +00001231 fn start(&self) -> binder::Result<()> {
Jiyong Park2227eaa2023-08-04 11:59:18 +09001232 self.instance
1233 .start()
1234 .with_context(|| format!("Error starting VM with CID {}", self.instance.cid))
1235 .with_log()
1236 .or_service_specific_exception(-1)
Andrew Walbranf8d94112021-09-07 11:45:36 +00001237 }
1238
Inseob Kima446f802022-07-11 19:46:37 +09001239 fn stop(&self) -> binder::Result<()> {
Jiyong Park2227eaa2023-08-04 11:59:18 +09001240 self.instance
1241 .kill()
1242 .with_context(|| format!("Error stopping VM with CID {}", self.instance.cid))
1243 .with_log()
1244 .or_service_specific_exception(-1)
Inseob Kima446f802022-07-11 19:46:37 +09001245 }
1246
Keir Fraser48221ba2024-07-12 14:05:02 +00001247 fn getMemoryBalloon(&self) -> binder::Result<i64> {
1248 let balloon = self
1249 .instance
1250 .get_memory_balloon()
1251 .with_context(|| format!("Error getting balloon for VM with CID {}", self.instance.cid))
1252 .with_log()
1253 .or_service_specific_exception(-1)?;
1254 Ok(balloon.try_into().unwrap())
1255 }
1256
1257 fn setMemoryBalloon(&self, num_bytes: i64) -> binder::Result<()> {
Jiyong Park2227eaa2023-08-04 11:59:18 +09001258 self.instance
Keir Fraser48221ba2024-07-12 14:05:02 +00001259 .set_memory_balloon(num_bytes.try_into().unwrap())
1260 .with_context(|| format!("Error setting balloon for VM with CID {}", self.instance.cid))
Jiyong Park2227eaa2023-08-04 11:59:18 +09001261 .with_log()
1262 .or_service_specific_exception(-1)
Keir Frasercdd4b112022-11-24 14:02:25 +00001263 }
1264
Andrew Walbrancbe8b082021-08-06 15:42:11 +00001265 fn connectVsock(&self, port: i32) -> binder::Result<ParcelFileDescriptor> {
Andrew Walbranf8d94112021-09-07 11:45:36 +00001266 if !matches!(&*self.instance.vm_state.lock().unwrap(), VmState::Running { .. }) {
Jiyong Park2227eaa2023-08-04 11:59:18 +09001267 return Err(anyhow!("VM is not running")).or_service_specific_exception(-1);
Andrew Walbrancbe8b082021-08-06 15:42:11 +00001268 }
Alan Stokes10c47672022-12-13 17:17:08 +00001269 let port = port as u32;
1270 if port < 1024 {
Jiyong Park2227eaa2023-08-04 11:59:18 +09001271 return Err(anyhow!("Can't connect to privileged port {port}"))
1272 .or_service_specific_exception(-1);
Alan Stokes10c47672022-12-13 17:17:08 +00001273 }
Jiyong Park2227eaa2023-08-04 11:59:18 +09001274 let stream = VsockStream::connect_with_cid_port(self.instance.cid, port)
1275 .context("Failed to connect")
1276 .or_service_specific_exception(-1)?;
Andrew Walbrancbe8b082021-08-06 15:42:11 +00001277 Ok(vsock_stream_to_pfd(stream))
1278 }
Yi-Yo Chiang2fbf0da2024-06-14 22:56:56 +08001279
1280 fn setHostConsoleName(&self, ptsname: &str) -> binder::Result<()> {
1281 self.instance.vm_context.global_context.setHostConsoleName(ptsname)
1282 }
Jiyong Park23b67392024-07-04 13:51:42 +09001283
1284 fn suspend(&self) -> binder::Result<()> {
1285 self.instance
1286 .suspend()
1287 .with_context(|| format!("Error suspending VM with CID {}", self.instance.cid))
1288 .with_log()
1289 .or_service_specific_exception(-1)
1290 }
1291
1292 fn resume(&self) -> binder::Result<()> {
1293 self.instance
1294 .resume()
1295 .with_context(|| format!("Error resuming VM with CID {}", self.instance.cid))
1296 .with_log()
1297 .or_service_specific_exception(-1)
1298 }
Andrew Walbrandae07162021-03-12 17:05:20 +00001299}
1300
1301impl Drop for VirtualMachine {
1302 fn drop(&mut self) {
1303 debug!("Dropping {:?}", self);
Inseob Kima446f802022-07-11 19:46:37 +09001304 if let Err(e) = self.instance.kill() {
1305 debug!("Error stopping dropped VM with CID {}: {:?}", self.instance.cid, e);
1306 }
Andrew Walbrandae07162021-03-12 17:05:20 +00001307 }
1308}
1309
1310/// A set of Binders to be called back in response to various events on the VM, such as when it
1311/// dies.
1312#[derive(Debug, Default)]
1313pub struct VirtualMachineCallbacks(Mutex<Vec<Strong<dyn IVirtualMachineCallback>>>);
1314
1315impl VirtualMachineCallbacks {
Jiyong Park8611a6c2021-07-09 18:17:44 +09001316 /// Call all registered callbacks to notify that the payload has started.
David Brazdil451cc962022-10-14 14:08:12 +01001317 pub fn notify_payload_started(&self, cid: Cid) {
Jiyong Park8611a6c2021-07-09 18:17:44 +09001318 let callbacks = &*self.0.lock().unwrap();
Jiyong Park8611a6c2021-07-09 18:17:44 +09001319 for callback in callbacks {
David Brazdil451cc962022-10-14 14:08:12 +01001320 if let Err(e) = callback.onPayloadStarted(cid as i32) {
Alan Stokes70ccf162022-07-08 11:05:03 +01001321 error!("Error notifying payload start event from VM CID {}: {:?}", cid, e);
Jiyong Park8611a6c2021-07-09 18:17:44 +09001322 }
1323 }
1324 }
1325
Inseob Kim14cb8692021-08-31 21:50:39 +09001326 /// Call all registered callbacks to notify that the payload is ready to serve.
1327 pub fn notify_payload_ready(&self, cid: Cid) {
1328 let callbacks = &*self.0.lock().unwrap();
1329 for callback in callbacks {
1330 if let Err(e) = callback.onPayloadReady(cid as i32) {
Alan Stokes70ccf162022-07-08 11:05:03 +01001331 error!("Error notifying payload ready event from VM CID {}: {:?}", cid, e);
Inseob Kim14cb8692021-08-31 21:50:39 +09001332 }
1333 }
1334 }
1335
Inseob Kim2444af92021-08-31 01:22:50 +09001336 /// Call all registered callbacks to notify that the payload has finished.
1337 pub fn notify_payload_finished(&self, cid: Cid, exit_code: i32) {
1338 let callbacks = &*self.0.lock().unwrap();
1339 for callback in callbacks {
1340 if let Err(e) = callback.onPayloadFinished(cid as i32, exit_code) {
Alan Stokes70ccf162022-07-08 11:05:03 +01001341 error!("Error notifying payload finish event from VM CID {}: {:?}", cid, e);
Inseob Kim2444af92021-08-31 01:22:50 +09001342 }
1343 }
1344 }
1345
Jooyung Handd0a1732021-11-23 15:26:20 +09001346 /// Call all registered callbacks to say that the VM encountered an error.
Alan Stokes2bead0d2022-09-05 16:58:34 +01001347 pub fn notify_error(&self, cid: Cid, error_code: ErrorCode, message: &str) {
Jooyung Handd0a1732021-11-23 15:26:20 +09001348 let callbacks = &*self.0.lock().unwrap();
1349 for callback in callbacks {
1350 if let Err(e) = callback.onError(cid as i32, error_code, message) {
Alan Stokes70ccf162022-07-08 11:05:03 +01001351 error!("Error notifying error event from VM CID {}: {:?}", cid, e);
Jooyung Handd0a1732021-11-23 15:26:20 +09001352 }
1353 }
1354 }
1355
Andrew Walbrandae07162021-03-12 17:05:20 +00001356 /// Call all registered callbacks to say that the VM has died.
Andrew Walbranc92d35f2022-01-12 12:45:19 +00001357 pub fn callback_on_died(&self, cid: Cid, reason: DeathReason) {
Andrew Walbrandae07162021-03-12 17:05:20 +00001358 let callbacks = &*self.0.lock().unwrap();
1359 for callback in callbacks {
Andrew Walbranc92d35f2022-01-12 12:45:19 +00001360 if let Err(e) = callback.onDied(cid as i32, reason) {
Alan Stokes70ccf162022-07-08 11:05:03 +01001361 error!("Error notifying exit of VM CID {}: {:?}", cid, e);
Andrew Walbrandae07162021-03-12 17:05:20 +00001362 }
1363 }
1364 }
1365
1366 /// Add a new callback to the set.
1367 fn add(&self, callback: Strong<dyn IVirtualMachineCallback>) {
1368 self.0.lock().unwrap().push(callback);
1369 }
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001370}
1371
Andrew Walbranf6bf6862021-05-21 12:41:13 +00001372/// The mutable state of the VirtualizationService. There should only be one instance of this
1373/// struct.
Chris Wailes641fc4a2021-12-01 15:03:21 -08001374#[derive(Debug, Default)]
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001375struct State {
Alan Stokes3e5eec12023-09-07 12:10:00 +01001376 /// The VMs which have been started. When VMs are started a weak reference is added to this
1377 /// list while a strong reference is returned to the caller over Binder. Once all copies of
1378 /// the Binder client are dropped the weak reference here will become invalid, and will be
1379 /// removed from the list opportunistically the next time `add_vm` is called.
Andrew Walbran320b5602021-03-04 16:11:12 +00001380 vms: Vec<Weak<VmInstance>>,
1381}
1382
1383impl State {
Andrew Walbrandae07162021-03-12 17:05:20 +00001384 /// Get a list of VMs which still have Binder references to them.
Andrew Walbran320b5602021-03-04 16:11:12 +00001385 fn vms(&self) -> Vec<Arc<VmInstance>> {
1386 // Attempt to upgrade the weak pointers to strong pointers.
1387 self.vms.iter().filter_map(Weak::upgrade).collect()
1388 }
1389
1390 /// Add a new VM to the list.
1391 fn add_vm(&mut self, vm: Weak<VmInstance>) {
1392 // Garbage collect any entries from the stored list which no longer exist.
1393 self.vms.retain(|vm| vm.strong_count() > 0);
1394
1395 // Actually add the new VM.
1396 self.vms.push(vm);
1397 }
David Brazdil3c2ddef2021-03-18 13:09:57 +00001398
Jiyong Park8611a6c2021-07-09 18:17:44 +09001399 /// Get a VM that corresponds to the given cid
1400 fn get_vm(&self, cid: Cid) -> Option<Arc<VmInstance>> {
1401 self.vms().into_iter().find(|vm| vm.cid == cid)
1402 }
Jiyong Parkd50a0242021-09-16 21:00:14 +09001403}
1404
Andrew Walbran6b650662021-09-07 13:13:23 +00001405/// Gets the `VirtualMachineState` of the given `VmInstance`.
1406fn get_state(instance: &VmInstance) -> VirtualMachineState {
Andrew Walbranf8d94112021-09-07 11:45:36 +00001407 match &*instance.vm_state.lock().unwrap() {
1408 VmState::NotStarted { .. } => VirtualMachineState::NOT_STARTED,
1409 VmState::Running { .. } => match instance.payload_state() {
Andrew Walbran6b650662021-09-07 13:13:23 +00001410 PayloadState::Starting => VirtualMachineState::STARTING,
1411 PayloadState::Started => VirtualMachineState::STARTED,
1412 PayloadState::Ready => VirtualMachineState::READY,
1413 PayloadState::Finished => VirtualMachineState::FINISHED,
Jiyong Parka4eebde2022-07-12 18:01:12 +09001414 PayloadState::Hangup => VirtualMachineState::DEAD,
Andrew Walbranf8d94112021-09-07 11:45:36 +00001415 },
1416 VmState::Dead => VirtualMachineState::DEAD,
1417 VmState::Failed => VirtualMachineState::DEAD,
Andrew Walbran6b650662021-09-07 13:13:23 +00001418 }
1419}
1420
David Brazdilf50c7a62023-04-19 14:22:42 +00001421/// Converts a `&ParcelFileDescriptor` to a `File` by cloning the file.
Jiyong Park2227eaa2023-08-04 11:59:18 +09001422pub fn clone_file(file: &ParcelFileDescriptor) -> binder::Result<File> {
1423 file.as_ref()
1424 .try_clone()
1425 .context("Failed to clone File from ParcelFileDescriptor")
1426 .or_binder_exception(ExceptionCode::BAD_PARCELABLE)
Andrei Homescu11333c62023-11-09 04:26:39 +00001427 .map(File::from)
David Brazdilf50c7a62023-04-19 14:22:42 +00001428}
1429
Andrew Walbrand3a84182021-09-07 14:48:52 +00001430/// Converts an `&Option<ParcelFileDescriptor>` to an `Option<File>` by cloning the file.
Jiyong Park2227eaa2023-08-04 11:59:18 +09001431fn maybe_clone_file(file: &Option<ParcelFileDescriptor>) -> binder::Result<Option<File>> {
Andrew Walbrand3a84182021-09-07 14:48:52 +00001432 file.as_ref().map(clone_file).transpose()
1433}
1434
Andrew Walbrancbe8b082021-08-06 15:42:11 +00001435/// Converts a `VsockStream` to a `ParcelFileDescriptor`.
1436fn vsock_stream_to_pfd(stream: VsockStream) -> ParcelFileDescriptor {
1437 // SAFETY: ownership is transferred from stream to f
1438 let f = unsafe { File::from_raw_fd(stream.into_raw_fd()) };
1439 ParcelFileDescriptor::new(f)
1440}
1441
Jiyong Parkdcf17412022-02-08 15:07:23 +09001442/// Parses the platform version requirement string.
Jiyong Park2227eaa2023-08-04 11:59:18 +09001443fn parse_platform_version_req(s: &str) -> binder::Result<VersionReq> {
1444 VersionReq::parse(s)
1445 .with_context(|| format!("Invalid platform version requirement {}", s))
1446 .or_binder_exception(ExceptionCode::BAD_PARCELABLE)
Jiyong Parkdcf17412022-02-08 15:07:23 +09001447}
1448
Jiyong Parked180932023-02-24 19:55:41 +09001449/// Create the empty ramdump file
1450fn prepare_ramdump_file(temporary_directory: &Path) -> binder::Result<File> {
1451 // `ramdump_write` is sent to crosvm and will be the backing store for the /dev/hvc1 where
1452 // VM will emit ramdump to. `ramdump_read` will be sent back to the client (i.e. the VM
1453 // owner) for readout.
1454 let ramdump_path = temporary_directory.join("ramdump");
Jiyong Park2227eaa2023-08-04 11:59:18 +09001455 let ramdump = File::create(ramdump_path)
1456 .context("Failed to prepare ramdump file")
1457 .with_log()
1458 .or_service_specific_exception(-1)?;
Jiyong Parked180932023-02-24 19:55:41 +09001459 Ok(ramdump)
1460}
1461
Nikita Ioffe5776f082023-02-10 21:38:26 +00001462fn is_protected(config: &VirtualMachineConfig) -> bool {
1463 match config {
1464 VirtualMachineConfig::RawConfig(config) => config.protectedVm,
1465 VirtualMachineConfig::AppConfig(config) => config.protectedVm,
1466 }
1467}
1468
1469fn check_gdb_allowed(config: &VirtualMachineConfig) -> binder::Result<()> {
1470 if is_protected(config) {
Jiyong Park2227eaa2023-08-04 11:59:18 +09001471 return Err(anyhow!("Can't use gdb with protected VMs"))
1472 .or_binder_exception(ExceptionCode::SECURITY);
Nikita Ioffe5776f082023-02-10 21:38:26 +00001473 }
1474
Pierre-Clément Tosid3bbe1d2024-04-15 18:03:51 +01001475 if get_debug_level(config) == Some(DebugLevel::NONE) {
1476 return Err(anyhow!("Can't use gdb with non-debuggable VMs"))
1477 .or_binder_exception(ExceptionCode::SECURITY);
Nikita Ioffe5776f082023-02-10 21:38:26 +00001478 }
Pierre-Clément Tosid3bbe1d2024-04-15 18:03:51 +01001479
1480 Ok(())
Nikita Ioffe5776f082023-02-10 21:38:26 +00001481}
1482
Shikha Panwar61a74b52024-02-16 13:17:01 +00001483fn extract_instance_id(config: &VirtualMachineConfig) -> [u8; 64] {
1484 match config {
1485 VirtualMachineConfig::RawConfig(config) => config.instanceId,
1486 VirtualMachineConfig::AppConfig(config) => config.instanceId,
1487 }
1488}
1489
Alan Stokesc96b35e2024-05-03 13:21:20 +01001490fn extract_want_updatable(config: &VirtualMachineConfig) -> bool {
1491 match config {
1492 VirtualMachineConfig::RawConfig(_) => true,
1493 VirtualMachineConfig::AppConfig(config) => {
1494 let Some(custom) = &config.customConfig else { return true };
1495 custom.wantUpdatable
1496 }
1497 }
1498}
1499
Nikita Ioffe5776f082023-02-10 21:38:26 +00001500fn extract_gdb_port(config: &VirtualMachineConfig) -> Option<NonZeroU16> {
1501 match config {
1502 VirtualMachineConfig::RawConfig(config) => NonZeroU16::new(config.gdbPort as u16),
Nikita Ioffea0eb5ee2023-06-26 18:18:21 +01001503 VirtualMachineConfig::AppConfig(config) => {
1504 NonZeroU16::new(config.customConfig.as_ref().map(|c| c.gdbPort).unwrap_or(0) as u16)
1505 }
Nikita Ioffe5776f082023-02-10 21:38:26 +00001506 }
1507}
1508
Nikita Ioffe631717e2023-09-05 13:38:07 +01001509fn check_no_vendor_modules(config: &VirtualMachineConfig) -> binder::Result<()> {
1510 let VirtualMachineConfig::AppConfig(config) = config else { return Ok(()) };
1511 if let Some(custom_config) = &config.customConfig {
1512 if custom_config.vendorImage.is_some() || custom_config.customKernelImage.is_some() {
1513 return Err(anyhow!("vendor modules feature is disabled"))
1514 .or_binder_exception(ExceptionCode::UNSUPPORTED_OPERATION);
1515 }
1516 }
1517 Ok(())
1518}
1519
Nikita Ioffe94a8a182023-11-16 16:37:48 +00001520fn check_no_devices(config: &VirtualMachineConfig) -> binder::Result<()> {
1521 let VirtualMachineConfig::AppConfig(config) = config else { return Ok(()) };
1522 if let Some(custom_config) = &config.customConfig {
1523 if !custom_config.devices.is_empty() {
1524 return Err(anyhow!("device assignment feature is disabled"))
1525 .or_binder_exception(ExceptionCode::UNSUPPORTED_OPERATION);
1526 }
1527 }
1528 Ok(())
1529}
1530
Alan Stokesfda70842023-12-20 17:50:14 +00001531fn check_no_extra_apks(config: &VirtualMachineConfig) -> binder::Result<()> {
1532 let VirtualMachineConfig::AppConfig(config) = config else { return Ok(()) };
1533 let Payload::PayloadConfig(payload_config) = &config.payload else { return Ok(()) };
1534 if !payload_config.extraApks.is_empty() {
1535 return Err(anyhow!("multi-tenant feature is disabled"))
1536 .or_binder_exception(ExceptionCode::UNSUPPORTED_OPERATION);
1537 }
1538 Ok(())
1539}
1540
Nikita Ioffef934e812024-07-05 15:44:41 +00001541fn check_protected_vm_is_supported() -> binder::Result<()> {
1542 let is_pvm_supported =
1543 hypervisor_props::is_protected_vm_supported().or_service_specific_exception(-1)?;
1544 if is_pvm_supported {
1545 Ok(())
1546 } else {
1547 Err(anyhow!("pVM is not supported"))
1548 .or_binder_exception(ExceptionCode::UNSUPPORTED_OPERATION)
1549 }
1550}
1551
Nikita Ioffe631717e2023-09-05 13:38:07 +01001552fn check_config_features(config: &VirtualMachineConfig) -> binder::Result<()> {
1553 if !cfg!(vendor_modules) {
1554 check_no_vendor_modules(config)?;
1555 }
Nikita Ioffe94a8a182023-11-16 16:37:48 +00001556 if !cfg!(device_assignment) {
1557 check_no_devices(config)?;
1558 }
Alan Stokesfda70842023-12-20 17:50:14 +00001559 if !cfg!(multi_tenant) {
1560 check_no_extra_apks(config)?;
1561 }
Nikita Ioffe631717e2023-09-05 13:38:07 +01001562 Ok(())
1563}
1564
Inseob Kim0168b462022-12-27 14:54:35 +09001565fn clone_or_prepare_logger_fd(
Inseob Kim0168b462022-12-27 14:54:35 +09001566 fd: Option<&ParcelFileDescriptor>,
1567 tag: String,
1568) -> Result<Option<File>, Status> {
1569 if let Some(fd) = fd {
1570 return Ok(Some(clone_file(fd)?));
1571 }
1572
Frederick Maylefbbcfcd2024-04-08 16:31:54 -07001573 let (read_fd, write_fd) =
Jiyong Park2227eaa2023-08-04 11:59:18 +09001574 pipe().context("Failed to create pipe").or_service_specific_exception(-1)?;
Inseob Kim0168b462022-12-27 14:54:35 +09001575
Frederick Maylefbbcfcd2024-04-08 16:31:54 -07001576 let mut reader = BufReader::new(File::from(read_fd));
1577 let write_fd = File::from(write_fd);
Inseob Kim0168b462022-12-27 14:54:35 +09001578
1579 std::thread::spawn(move || loop {
1580 let mut buf = vec![];
1581 match reader.read_until(b'\n', &mut buf) {
1582 Ok(0) => {
1583 // EOF
1584 return;
1585 }
1586 Ok(size) => {
1587 if buf[size - 1] == b'\n' {
1588 buf.pop();
1589 }
1590 info!("{}: {}", &tag, &String::from_utf8_lossy(&buf));
1591 }
1592 Err(e) => {
1593 error!("Could not read console pipe: {:?}", e);
1594 return;
1595 }
1596 };
1597 });
1598
1599 Ok(Some(write_fd))
1600}
1601
Jooyung Han35edb8f2021-07-01 16:17:16 +09001602/// Simple utility for referencing Borrowed or Owned. Similar to std::borrow::Cow, but
1603/// it doesn't require that T implements Clone.
1604enum BorrowedOrOwned<'a, T> {
1605 Borrowed(&'a T),
1606 Owned(T),
1607}
1608
1609impl<'a, T> AsRef<T> for BorrowedOrOwned<'a, T> {
1610 fn as_ref(&self) -> &T {
1611 match self {
1612 Self::Borrowed(b) => b,
Chris Wailes68c39f82021-07-27 16:03:44 -07001613 Self::Owned(o) => o,
Jooyung Han35edb8f2021-07-01 16:17:16 +09001614 }
1615 }
1616}
Inseob Kim1b95f2e2021-08-19 13:17:40 +09001617
1618/// Implementation of `IVirtualMachineService`, the entry point of the AIDL service.
1619#[derive(Debug, Default)]
1620struct VirtualMachineService {
1621 state: Arc<Mutex<State>>,
Inseob Kimc7d28c72021-10-25 14:28:10 +00001622 cid: Cid,
Inseob Kim1b95f2e2021-08-19 13:17:40 +09001623}
1624
1625impl Interface for VirtualMachineService {}
1626
1627impl IVirtualMachineService for VirtualMachineService {
Inseob Kimc7d28c72021-10-25 14:28:10 +00001628 fn notifyPayloadStarted(&self) -> binder::Result<()> {
1629 let cid = self.cid;
Inseob Kim7f61fe72021-08-20 20:50:47 +09001630 if let Some(vm) = self.state.lock().unwrap().get_vm(cid) {
David Brazdil415097c2022-10-21 14:17:05 +01001631 info!("VM with CID {} started payload", cid);
Jiyong Park2227eaa2023-08-04 11:59:18 +09001632 vm.update_payload_state(PayloadState::Started)
1633 .or_binder_exception(ExceptionCode::ILLEGAL_STATE)?;
David Brazdil451cc962022-10-14 14:08:12 +01001634 vm.callbacks.notify_payload_started(cid);
Seungjae Yoo62085c02022-08-12 04:44:52 +00001635
Seungjae Yoo6d265d92022-11-15 10:51:33 +09001636 let vm_start_timestamp = vm.vm_metric.lock().unwrap().start_timestamp;
1637 write_vm_booted_stats(vm.requester_uid as i32, &vm.name, vm_start_timestamp);
Inseob Kim7f61fe72021-08-20 20:50:47 +09001638 Ok(())
1639 } else {
Jooyung Handd0a1732021-11-23 15:26:20 +09001640 error!("notifyPayloadStarted is called from an unknown CID {}", cid);
Jiyong Park2227eaa2023-08-04 11:59:18 +09001641 Err(anyhow!("cannot find a VM with CID {}", cid)).or_service_specific_exception(-1)
Inseob Kim1b95f2e2021-08-19 13:17:40 +09001642 }
Inseob Kim1b95f2e2021-08-19 13:17:40 +09001643 }
Inseob Kim2444af92021-08-31 01:22:50 +09001644
Inseob Kimc7d28c72021-10-25 14:28:10 +00001645 fn notifyPayloadReady(&self) -> binder::Result<()> {
1646 let cid = self.cid;
Inseob Kim14cb8692021-08-31 21:50:39 +09001647 if let Some(vm) = self.state.lock().unwrap().get_vm(cid) {
David Brazdil415097c2022-10-21 14:17:05 +01001648 info!("VM with CID {} reported payload is ready", cid);
Jiyong Park2227eaa2023-08-04 11:59:18 +09001649 vm.update_payload_state(PayloadState::Ready)
1650 .or_binder_exception(ExceptionCode::ILLEGAL_STATE)?;
Inseob Kim14cb8692021-08-31 21:50:39 +09001651 vm.callbacks.notify_payload_ready(cid);
1652 Ok(())
1653 } else {
Jooyung Handd0a1732021-11-23 15:26:20 +09001654 error!("notifyPayloadReady is called from an unknown CID {}", cid);
Jiyong Park2227eaa2023-08-04 11:59:18 +09001655 Err(anyhow!("cannot find a VM with CID {}", cid)).or_service_specific_exception(-1)
Inseob Kim14cb8692021-08-31 21:50:39 +09001656 }
1657 }
1658
Inseob Kimc7d28c72021-10-25 14:28:10 +00001659 fn notifyPayloadFinished(&self, exit_code: i32) -> binder::Result<()> {
1660 let cid = self.cid;
Inseob Kim2444af92021-08-31 01:22:50 +09001661 if let Some(vm) = self.state.lock().unwrap().get_vm(cid) {
David Brazdil415097c2022-10-21 14:17:05 +01001662 info!("VM with CID {} finished payload", cid);
Jiyong Park2227eaa2023-08-04 11:59:18 +09001663 vm.update_payload_state(PayloadState::Finished)
1664 .or_binder_exception(ExceptionCode::ILLEGAL_STATE)?;
Inseob Kim2444af92021-08-31 01:22:50 +09001665 vm.callbacks.notify_payload_finished(cid, exit_code);
1666 Ok(())
1667 } else {
Jooyung Handd0a1732021-11-23 15:26:20 +09001668 error!("notifyPayloadFinished is called from an unknown CID {}", cid);
Jiyong Park2227eaa2023-08-04 11:59:18 +09001669 Err(anyhow!("cannot find a VM with CID {}", cid)).or_service_specific_exception(-1)
Jooyung Handd0a1732021-11-23 15:26:20 +09001670 }
1671 }
1672
Alan Stokes2bead0d2022-09-05 16:58:34 +01001673 fn notifyError(&self, error_code: ErrorCode, message: &str) -> binder::Result<()> {
Jooyung Handd0a1732021-11-23 15:26:20 +09001674 let cid = self.cid;
1675 if let Some(vm) = self.state.lock().unwrap().get_vm(cid) {
David Brazdil415097c2022-10-21 14:17:05 +01001676 info!("VM with CID {} encountered an error", cid);
Jiyong Park2227eaa2023-08-04 11:59:18 +09001677 vm.update_payload_state(PayloadState::Finished)
1678 .or_binder_exception(ExceptionCode::ILLEGAL_STATE)?;
Jooyung Handd0a1732021-11-23 15:26:20 +09001679 vm.callbacks.notify_error(cid, error_code, message);
1680 Ok(())
1681 } else {
Seungjae Yooec8c1602022-06-20 05:28:00 +00001682 error!("notifyError is called from an unknown CID {}", cid);
Jiyong Park2227eaa2023-08-04 11:59:18 +09001683 Err(anyhow!("cannot find a VM with CID {}", cid)).or_service_specific_exception(-1)
Inseob Kim2444af92021-08-31 01:22:50 +09001684 }
1685 }
Alice Wangc2fec932023-02-23 16:24:02 +00001686
Shikha Panware45e9422024-02-28 21:18:10 +00001687 fn getSecretkeeper(&self) -> binder::Result<Strong<dyn ISecretkeeper>> {
1688 if !is_secretkeeper_supported() {
1689 return Err(StatusCode::NAME_NOT_FOUND)?;
1690 }
1691 let sk = binder::wait_for_interface(SECRETKEEPER_IDENTIFIER)?;
1692 Ok(BnSecretkeeper::new_binder(SecretkeeperProxy(sk), BinderFeatures::default()))
Shikha Panwar5d6a6752023-12-14 22:08:26 +00001693 }
1694
Alice Wange64dd182024-01-17 15:57:55 +00001695 fn requestAttestation(&self, csr: &[u8], test_mode: bool) -> binder::Result<Vec<Certificate>> {
1696 GLOBAL_SERVICE.requestAttestation(csr, get_calling_uid() as i32, test_mode)
Alice Wangc2fec932023-02-23 16:24:02 +00001697 }
Inseob Kim1b95f2e2021-08-19 13:17:40 +09001698}
1699
Shikha Panwar8707f0f2024-02-28 20:40:42 +00001700fn is_secretkeeper_supported() -> bool {
Shikha Panwar81d13d32024-03-10 19:39:23 +00001701 binder::is_declared(SECRETKEEPER_IDENTIFIER)
1702 .expect("Could not check for declared Secretkeeper interface")
Shikha Panwar8187ca22024-01-04 08:33:08 +00001703}
1704
Inseob Kim1b95f2e2021-08-19 13:17:40 +09001705impl VirtualMachineService {
Inseob Kimc7d28c72021-10-25 14:28:10 +00001706 fn new_binder(state: Arc<Mutex<State>>, cid: Cid) -> Strong<dyn IVirtualMachineService> {
Inseob Kim1b95f2e2021-08-19 13:17:40 +09001707 BnVirtualMachineService::new_binder(
Inseob Kimc7d28c72021-10-25 14:28:10 +00001708 VirtualMachineService { state, cid },
Inseob Kim1b95f2e2021-08-19 13:17:40 +09001709 BinderFeatures::default(),
1710 )
1711 }
1712}
Alan Stokes53cc5ca2022-08-30 14:28:19 +01001713
Stephen Hines4c1e2fc2024-02-09 16:34:47 -08001714struct SecretkeeperProxy(Strong<dyn ISecretkeeper>);
1715
1716impl Interface for SecretkeeperProxy {}
1717
1718impl ISecretkeeper for SecretkeeperProxy {
1719 fn processSecretManagementRequest(&self, req: &[u8]) -> binder::Result<Vec<u8>> {
1720 // Pass the request to the channel, and read the response.
1721 self.0.processSecretManagementRequest(req)
1722 }
1723
1724 fn getAuthGraphKe(&self) -> binder::Result<Strong<dyn IAuthGraphKeyExchange>> {
1725 let ag = AuthGraphKeyExchangeProxy(self.0.getAuthGraphKe()?);
1726 Ok(BnAuthGraphKeyExchange::new_binder(ag, BinderFeatures::default()))
1727 }
1728
1729 fn deleteIds(&self, ids: &[SecretId]) -> binder::Result<()> {
1730 self.0.deleteIds(ids)
1731 }
1732
1733 fn deleteAll(&self) -> binder::Result<()> {
1734 self.0.deleteAll()
1735 }
1736}
1737
1738struct AuthGraphKeyExchangeProxy(Strong<dyn IAuthGraphKeyExchange>);
1739
1740impl Interface for AuthGraphKeyExchangeProxy {}
1741
1742impl IAuthGraphKeyExchange for AuthGraphKeyExchangeProxy {
1743 fn create(&self) -> binder::Result<SessionInitiationInfo> {
1744 self.0.create()
1745 }
1746
1747 fn init(
1748 &self,
1749 peer_pub_key: &PubKey,
1750 peer_id: &Identity,
1751 peer_nonce: &[u8],
1752 peer_version: i32,
1753 ) -> binder::Result<KeInitResult> {
1754 self.0.init(peer_pub_key, peer_id, peer_nonce, peer_version)
1755 }
1756
1757 fn finish(
1758 &self,
1759 peer_pub_key: &PubKey,
1760 peer_id: &Identity,
1761 peer_signature: &SessionIdSignature,
1762 peer_nonce: &[u8],
1763 peer_version: i32,
1764 own_key: &Key,
1765 ) -> binder::Result<SessionInfo> {
1766 self.0.finish(peer_pub_key, peer_id, peer_signature, peer_nonce, peer_version, own_key)
1767 }
1768
1769 fn authenticationComplete(
1770 &self,
1771 peer_signature: &SessionIdSignature,
1772 shared_keys: &[AuthgraphArc; 2],
1773 ) -> binder::Result<[AuthgraphArc; 2]> {
1774 self.0.authenticationComplete(peer_signature, shared_keys)
1775 }
1776}
1777
Alan Stokes53cc5ca2022-08-30 14:28:19 +01001778#[cfg(test)]
1779mod tests {
1780 use super::*;
1781
1782 #[test]
1783 fn test_is_allowed_label_for_partition() -> Result<()> {
1784 let expected_results = vec![
1785 ("u:object_r:system_file:s0", true),
1786 ("u:object_r:apk_data_file:s0", true),
1787 ("u:object_r:app_data_file:s0", false),
1788 ("u:object_r:app_data_file:s0:c512,c768", false),
1789 ("u:object_r:privapp_data_file:s0:c512,c768", false),
1790 ("invalid", false),
1791 ("user:role:apk_data_file:severity:categories", true),
1792 ("user:role:apk_data_file:severity:categories:extraneous", false),
1793 ];
1794
1795 for (label, expected_valid) in expected_results {
1796 let context = SeContext::new(label)?;
1797 let result = check_label_is_allowed(&context);
1798 if expected_valid {
1799 assert!(result.is_ok(), "Expected label {} to be allowed, got {:?}", label, result);
1800 } else if result.is_ok() {
1801 bail!("Expected label {} to be disallowed", label);
1802 }
1803 }
1804 Ok(())
1805 }
Nikita Ioffef1ce9872022-12-09 13:31:59 +00001806
1807 #[test]
1808 fn test_create_or_update_idsig_file_empty_apk() -> Result<()> {
1809 let apk = tempfile::tempfile().unwrap();
1810 let idsig = tempfile::tempfile().unwrap();
1811
1812 let ret = create_or_update_idsig_file(
1813 &ParcelFileDescriptor::new(apk),
1814 &ParcelFileDescriptor::new(idsig),
1815 );
1816 assert!(ret.is_err(), "should fail");
1817 Ok(())
1818 }
1819
1820 #[test]
1821 fn test_create_or_update_idsig_dir_instead_of_file_for_apk() -> Result<()> {
1822 let tmp_dir = tempfile::TempDir::new().unwrap();
1823 let apk = File::open(tmp_dir.path()).unwrap();
1824 let idsig = tempfile::tempfile().unwrap();
1825
1826 let ret = create_or_update_idsig_file(
1827 &ParcelFileDescriptor::new(apk),
1828 &ParcelFileDescriptor::new(idsig),
1829 );
1830 assert!(ret.is_err(), "should fail");
1831 Ok(())
1832 }
1833
1834 /// Verifies that create_or_update_idsig_file won't oom if a fd that corresponds to a directory
1835 /// on ext4 filesystem is passed.
1836 /// On ext4 lseek on a directory fd will return (off_t)-1 (see:
1837 /// https://bugzilla.kernel.org/show_bug.cgi?id=200043), which will result in
1838 /// create_or_update_idsig_file ooming while attempting to allocate petabytes of memory.
1839 #[test]
1840 fn test_create_or_update_idsig_does_not_crash_dir_on_ext4() -> Result<()> {
1841 // APEXes are backed by the ext4.
1842 let apk = File::open("/apex/com.android.virt/").unwrap();
1843 let idsig = tempfile::tempfile().unwrap();
1844
1845 let ret = create_or_update_idsig_file(
1846 &ParcelFileDescriptor::new(apk),
1847 &ParcelFileDescriptor::new(idsig),
1848 );
1849 assert!(ret.is_err(), "should fail");
1850 Ok(())
1851 }
Jiyong Park8d192952023-06-26 14:29:51 +09001852
1853 #[test]
1854 fn test_create_or_update_idsig_does_not_update_if_already_valid() -> Result<()> {
1855 use std::io::Seek;
1856
1857 // Pick any APK
1858 let mut apk = File::open("/system/priv-app/Shell/Shell.apk").unwrap();
1859 let mut idsig = tempfile::tempfile().unwrap();
1860
1861 create_or_update_idsig_file(
1862 &ParcelFileDescriptor::new(apk.try_clone()?),
1863 &ParcelFileDescriptor::new(idsig.try_clone()?),
1864 )?;
1865 let modified_orig = idsig.metadata()?.modified()?;
1866 apk.rewind()?;
1867 idsig.rewind()?;
1868
1869 // Call the function again
1870 create_or_update_idsig_file(
1871 &ParcelFileDescriptor::new(apk.try_clone()?),
1872 &ParcelFileDescriptor::new(idsig.try_clone()?),
1873 )?;
1874 let modified_new = idsig.metadata()?.modified()?;
1875 assert!(modified_orig == modified_new, "idsig file was updated unnecessarily");
1876 Ok(())
1877 }
Nikita Ioffeaa6858c2023-07-04 01:37:41 +01001878
1879 #[test]
Shikha Panwar9ae2e622024-01-30 12:22:30 +00001880 fn test_create_or_update_idsig_on_non_empty_file() -> Result<()> {
1881 use std::io::Read;
1882
1883 // Pick any APK
1884 let mut apk = File::open("/system/priv-app/Shell/Shell.apk").unwrap();
1885 let idsig_empty = tempfile::tempfile().unwrap();
1886 let mut idsig_invalid = tempfile::tempfile().unwrap();
1887 idsig_invalid.write_all(b"Oops")?;
1888
1889 // Create new idsig
1890 create_or_update_idsig_file(
1891 &ParcelFileDescriptor::new(apk.try_clone()?),
1892 &ParcelFileDescriptor::new(idsig_empty.try_clone()?),
1893 )?;
1894 apk.rewind()?;
1895
1896 // Update idsig_invalid
1897 create_or_update_idsig_file(
1898 &ParcelFileDescriptor::new(apk.try_clone()?),
1899 &ParcelFileDescriptor::new(idsig_invalid.try_clone()?),
1900 )?;
1901
1902 // Ensure the 2 idsig files have same size!
1903 assert!(
1904 idsig_empty.metadata()?.len() == idsig_invalid.metadata()?.len(),
1905 "idsig files differ in size"
1906 );
1907 // Ensure the 2 idsig files have same content!
1908 for (b1, b2) in idsig_empty.bytes().zip(idsig_invalid.bytes()) {
1909 assert!(b1.unwrap() == b2.unwrap(), "idsig files differ")
1910 }
1911 Ok(())
1912 }
1913 #[test]
Nikita Ioffeaa6858c2023-07-04 01:37:41 +01001914 fn test_append_kernel_param_first_param() {
1915 let mut vm_config = VirtualMachineRawConfig { ..Default::default() };
1916 append_kernel_param("foo=1", &mut vm_config);
1917 assert_eq!(vm_config.params, Some("foo=1".to_owned()))
1918 }
1919
1920 #[test]
1921 fn test_append_kernel_param() {
1922 let mut vm_config =
1923 VirtualMachineRawConfig { params: Some("foo=5".to_owned()), ..Default::default() };
1924 append_kernel_param("bar=42", &mut vm_config);
1925 assert_eq!(vm_config.params, Some("foo=5 bar=42".to_owned()))
1926 }
Seungjae Yooec3bc522023-11-09 10:14:30 +09001927
Inseob Kim46257382024-01-03 15:41:22 +09001928 fn test_extract_os_name_from_config_path(
1929 path: &Path,
1930 expected_result: Option<&str>,
1931 ) -> Result<()> {
1932 let result = extract_os_name_from_config_path(path);
1933 if result.as_deref() != expected_result {
1934 bail!("Expected {:?} but was {:?}", expected_result, &result)
1935 }
1936 Ok(())
1937 }
1938
1939 #[test]
1940 fn test_extract_os_name_from_microdroid_config() -> Result<()> {
1941 test_extract_os_name_from_config_path(
1942 Path::new("/apex/com.android.virt/etc/microdroid.json"),
1943 Some("microdroid"),
1944 )
1945 }
1946
1947 #[test]
1948 fn test_extract_os_name_from_microdroid_gki_config() -> Result<()> {
1949 test_extract_os_name_from_config_path(
1950 Path::new("/apex/com.android.virt/etc/microdroid_gki-android14-6.1.json"),
1951 Some("microdroid_gki-android14-6.1"),
1952 )
1953 }
1954
1955 #[test]
1956 fn test_extract_os_name_from_invalid_path() -> Result<()> {
1957 test_extract_os_name_from_config_path(
1958 Path::new("/apex/com.android.virt/etc/microdroid.img"),
1959 None,
1960 )
1961 }
1962
1963 #[test]
1964 fn test_extract_os_name_from_configs() -> Result<()> {
1965 let tmp_dir = tempfile::TempDir::new()?;
1966 let tmp_dir_path = tmp_dir.path().to_owned();
1967
1968 let mut os_names: HashSet<String> = HashSet::new();
1969 os_names.insert("microdroid".to_owned());
1970 os_names.insert("microdroid_gki-android14-6.1".to_owned());
1971 os_names.insert("microdroid_gki-android15-6.1".to_owned());
1972
1973 // config files
1974 for os_name in &os_names {
1975 std::fs::write(tmp_dir_path.join(os_name.to_owned() + ".json"), b"")?;
1976 }
1977
1978 // fake files not related to configs
1979 std::fs::write(tmp_dir_path.join("microdroid_super.img"), b"")?;
1980 std::fs::write(tmp_dir_path.join("microdroid_foobar.apk"), b"")?;
1981
1982 let glob_pattern = match tmp_dir_path.join("microdroid*.json").to_str() {
1983 Some(s) => s.to_owned(),
1984 None => bail!("tmp_dir_path {:?} is not UTF-8", tmp_dir_path),
1985 };
1986
1987 let result = extract_os_names_from_configs(&glob_pattern)?;
1988 if result != os_names {
1989 bail!("Expected {:?} but was {:?}", os_names, result);
1990 }
1991 Ok(())
1992 }
Alan Stokes53cc5ca2022-08-30 14:28:19 +01001993}