David Brazdil | 66fc120 | 2022-07-04 21:48:45 +0100 | [diff] [blame] | 1 | // Copyright 2022, 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 | |
| 15 | //! Integration test for Rialto. |
| 16 | |
| 17 | use android_system_virtualizationservice::{ |
| 18 | aidl::android::system::virtualizationservice::{ |
Alice Wang | a635769 | 2023-09-07 14:59:37 +0000 | [diff] [blame] | 19 | VirtualMachineConfig::VirtualMachineConfig, |
David Brazdil | 66fc120 | 2022-07-04 21:48:45 +0100 | [diff] [blame] | 20 | VirtualMachineRawConfig::VirtualMachineRawConfig, |
| 21 | }, |
| 22 | binder::{ParcelFileDescriptor, ProcessState}, |
| 23 | }; |
Alice Wang | 9646fb3 | 2023-09-08 10:01:31 +0000 | [diff] [blame] | 24 | use anyhow::{bail, Context, Result}; |
Alice Wang | b76b66f | 2024-03-26 08:16:23 +0000 | [diff] [blame] | 25 | use bssl_avf::{rand_bytes, sha256, EcKey, PKey}; |
Alice Wang | de6bee5 | 2023-11-10 09:58:40 +0000 | [diff] [blame] | 26 | use client_vm_csr::generate_attestation_key_and_csr; |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 27 | use coset::{CborSerializable, CoseMac0, CoseSign}; |
Alice Wang | 7eddbf4 | 2024-10-16 09:50:54 +0000 | [diff] [blame] | 28 | use hwtrust::{ |
| 29 | rkp, |
| 30 | session::{RkpInstance, Session}, |
| 31 | }; |
Nikita Ioffe | bd2e2e4 | 2024-07-05 15:04:49 +0000 | [diff] [blame] | 32 | use log::{info, warn}; |
Alice Wang | 9646fb3 | 2023-09-08 10:01:31 +0000 | [diff] [blame] | 33 | use service_vm_comm::{ |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 34 | ClientVmAttestationParams, Csr, CsrPayload, EcdsaP256KeyPair, GenerateCertificateRequestParams, |
Alice Wang | d3a9640 | 2023-11-24 15:37:39 +0000 | [diff] [blame] | 35 | Request, RequestProcessingError, Response, VmType, |
Alice Wang | 9646fb3 | 2023-09-08 10:01:31 +0000 | [diff] [blame] | 36 | }; |
Alice Wang | 1cc1350 | 2023-12-05 11:05:34 +0000 | [diff] [blame] | 37 | use service_vm_fake_chain::client_vm::{ |
| 38 | fake_client_vm_dice_artifacts, fake_sub_components, SubComponent, |
| 39 | }; |
Pierre-Clément Tosi | 311de8e | 2024-08-14 10:10:49 +0100 | [diff] [blame] | 40 | use service_vm_manager::{ServiceVm, VM_MEMORY_MB}; |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 41 | use std::fs; |
David Brazdil | 66fc120 | 2022-07-04 21:48:45 +0100 | [diff] [blame] | 42 | use std::fs::File; |
Alice Wang | 17dc76e | 2023-09-06 09:43:52 +0000 | [diff] [blame] | 43 | use std::path::PathBuf; |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 44 | use std::str::FromStr; |
Alice Wang | 17dc76e | 2023-09-06 09:43:52 +0000 | [diff] [blame] | 45 | use vmclient::VmInstance; |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 46 | use x509_cert::{ |
| 47 | certificate::{Certificate, Version}, |
| 48 | der::{self, asn1, Decode, Encode}, |
| 49 | name::Name, |
| 50 | spki::{AlgorithmIdentifier, ObjectIdentifier, SubjectPublicKeyInfo}, |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 51 | }; |
Alice Wang | 4e082c3 | 2023-07-11 07:41:50 +0000 | [diff] [blame] | 52 | |
Alice Wang | 9a8b39f | 2023-04-12 15:31:48 +0000 | [diff] [blame] | 53 | const UNSIGNED_RIALTO_PATH: &str = "/data/local/tmp/rialto_test/arm64/rialto_unsigned.bin"; |
| 54 | const INSTANCE_IMG_PATH: &str = "/data/local/tmp/rialto_test/arm64/instance.img"; |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 55 | const TEST_CERT_CHAIN_PATH: &str = "testdata/rkp_cert_chain.der"; |
David Brazdil | 66fc120 | 2022-07-04 21:48:45 +0100 | [diff] [blame] | 56 | |
Alice Wang | 0472f46 | 2024-02-06 08:53:19 +0000 | [diff] [blame] | 57 | #[cfg(dice_changes)] |
Alice Wang | 9a8b39f | 2023-04-12 15:31:48 +0000 | [diff] [blame] | 58 | #[test] |
Alice Wang | e910b90 | 2023-09-07 10:35:12 +0000 | [diff] [blame] | 59 | fn process_requests_in_protected_vm() -> Result<()> { |
Nikita Ioffe | bd2e2e4 | 2024-07-05 15:04:49 +0000 | [diff] [blame] | 60 | if hypervisor_props::is_protected_vm_supported()? { |
| 61 | // The test is skipped if the feature flag |dice_changes| is not enabled, because when |
| 62 | // the flag is off, the DICE chain is truncated in the pvmfw, and the service VM cannot |
| 63 | // verify the chain due to the missing entries in the chain. |
Pierre-Clément Tosi | 311de8e | 2024-08-14 10:10:49 +0100 | [diff] [blame] | 64 | check_processing_requests(VmType::ProtectedVm, None) |
Nikita Ioffe | bd2e2e4 | 2024-07-05 15:04:49 +0000 | [diff] [blame] | 65 | } else { |
| 66 | warn!("pVMs are not supported on device, skipping test"); |
| 67 | Ok(()) |
| 68 | } |
Alice Wang | 9a8b39f | 2023-04-12 15:31:48 +0000 | [diff] [blame] | 69 | } |
| 70 | |
Alice Wang | e910b90 | 2023-09-07 10:35:12 +0000 | [diff] [blame] | 71 | #[test] |
| 72 | fn process_requests_in_non_protected_vm() -> Result<()> { |
Pierre-Clément Tosi | 311de8e | 2024-08-14 10:10:49 +0100 | [diff] [blame] | 73 | const MEMORY_MB: i32 = 300; |
Pierre-Clément Tosi | d60a1d1 | 2024-08-28 20:04:24 +0100 | [diff] [blame] | 74 | check_processing_requests(VmType::NonProtectedVm, Some(MEMORY_MB))?; |
| 75 | check_processing_requests(VmType::NonProtectedVm, None) |
Pierre-Clément Tosi | 311de8e | 2024-08-14 10:10:49 +0100 | [diff] [blame] | 76 | } |
| 77 | |
| 78 | fn check_processing_requests(vm_type: VmType, vm_memory_mb: Option<i32>) -> Result<()> { |
| 79 | let mut vm = start_service_vm(vm_type, vm_memory_mb)?; |
Alice Wang | e910b90 | 2023-09-07 10:35:12 +0000 | [diff] [blame] | 80 | |
| 81 | check_processing_reverse_request(&mut vm)?; |
Alice Wang | 74eb78b | 2023-11-09 16:13:10 +0000 | [diff] [blame] | 82 | let key_pair = check_processing_generating_key_pair_request(&mut vm)?; |
Karuna Wadhera | c8fb3bf | 2024-07-01 12:54:25 +0000 | [diff] [blame] | 83 | check_processing_generating_certificate_request(&mut vm, &key_pair.maced_public_key)?; |
Alice Wang | d3a9640 | 2023-11-24 15:37:39 +0000 | [diff] [blame] | 84 | check_attestation_request(&mut vm, &key_pair, vm_type)?; |
Alice Wang | e910b90 | 2023-09-07 10:35:12 +0000 | [diff] [blame] | 85 | Ok(()) |
| 86 | } |
| 87 | |
| 88 | fn check_processing_reverse_request(vm: &mut ServiceVm) -> Result<()> { |
Alice Wang | 0486e25 | 2023-10-06 14:30:49 +0000 | [diff] [blame] | 89 | let message = "abc".repeat(500); |
Alice Wang | e910b90 | 2023-09-07 10:35:12 +0000 | [diff] [blame] | 90 | let request = Request::Reverse(message.as_bytes().to_vec()); |
| 91 | |
Alice Wang | fbdc85b | 2023-09-07 12:56:46 +0000 | [diff] [blame] | 92 | let response = vm.process_request(request)?; |
| 93 | info!("Received response: {response:?}."); |
Alice Wang | e910b90 | 2023-09-07 10:35:12 +0000 | [diff] [blame] | 94 | |
| 95 | let expected_response: Vec<u8> = message.as_bytes().iter().rev().cloned().collect(); |
| 96 | assert_eq!(Response::Reverse(expected_response), response); |
| 97 | Ok(()) |
| 98 | } |
| 99 | |
Alice Wang | 74eb78b | 2023-11-09 16:13:10 +0000 | [diff] [blame] | 100 | fn check_processing_generating_key_pair_request(vm: &mut ServiceVm) -> Result<EcdsaP256KeyPair> { |
Alice Wang | 9646fb3 | 2023-09-08 10:01:31 +0000 | [diff] [blame] | 101 | let request = Request::GenerateEcdsaP256KeyPair; |
| 102 | |
| 103 | let response = vm.process_request(request)?; |
| 104 | info!("Received response: {response:?}."); |
| 105 | |
| 106 | match response { |
Alice Wang | 74eb78b | 2023-11-09 16:13:10 +0000 | [diff] [blame] | 107 | Response::GenerateEcdsaP256KeyPair(key_pair) => { |
| 108 | assert_array_has_nonzero(&key_pair.maced_public_key); |
| 109 | assert_array_has_nonzero(&key_pair.key_blob); |
| 110 | Ok(key_pair) |
Alice Wang | a78d3f0 | 2023-09-13 12:39:16 +0000 | [diff] [blame] | 111 | } |
Alice Wang | ff5592d | 2023-09-13 15:27:39 +0000 | [diff] [blame] | 112 | _ => bail!("Incorrect response type: {response:?}"), |
Alice Wang | 9646fb3 | 2023-09-08 10:01:31 +0000 | [diff] [blame] | 113 | } |
| 114 | } |
| 115 | |
Alice Wang | a78d3f0 | 2023-09-13 12:39:16 +0000 | [diff] [blame] | 116 | fn assert_array_has_nonzero(v: &[u8]) { |
| 117 | assert!(v.iter().any(|&x| x != 0)) |
| 118 | } |
| 119 | |
Alice Wang | ff5592d | 2023-09-13 15:27:39 +0000 | [diff] [blame] | 120 | fn check_processing_generating_certificate_request( |
| 121 | vm: &mut ServiceVm, |
Alice Wang | 74eb78b | 2023-11-09 16:13:10 +0000 | [diff] [blame] | 122 | maced_public_key: &[u8], |
Alice Wang | ff5592d | 2023-09-13 15:27:39 +0000 | [diff] [blame] | 123 | ) -> Result<()> { |
| 124 | let params = GenerateCertificateRequestParams { |
Alice Wang | 74eb78b | 2023-11-09 16:13:10 +0000 | [diff] [blame] | 125 | keys_to_sign: vec![maced_public_key.to_vec()], |
Alice Wang | ff5592d | 2023-09-13 15:27:39 +0000 | [diff] [blame] | 126 | challenge: vec![], |
| 127 | }; |
Alice Wang | 9646fb3 | 2023-09-08 10:01:31 +0000 | [diff] [blame] | 128 | let request = Request::GenerateCertificateRequest(params); |
| 129 | |
| 130 | let response = vm.process_request(request)?; |
| 131 | info!("Received response: {response:?}."); |
| 132 | |
| 133 | match response { |
Karuna Wadhera | c8fb3bf | 2024-07-01 12:54:25 +0000 | [diff] [blame] | 134 | Response::GenerateCertificateRequest(csr) => check_csr(csr), |
Alice Wang | ff5592d | 2023-09-13 15:27:39 +0000 | [diff] [blame] | 135 | _ => bail!("Incorrect response type: {response:?}"), |
Alice Wang | 9646fb3 | 2023-09-08 10:01:31 +0000 | [diff] [blame] | 136 | } |
| 137 | } |
| 138 | |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 139 | fn check_attestation_request( |
| 140 | vm: &mut ServiceVm, |
| 141 | remotely_provisioned_key_pair: &EcdsaP256KeyPair, |
Alice Wang | d3a9640 | 2023-11-24 15:37:39 +0000 | [diff] [blame] | 142 | vm_type: VmType, |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 143 | ) -> Result<()> { |
Alice Wang | de6bee5 | 2023-11-10 09:58:40 +0000 | [diff] [blame] | 144 | /// The following data was generated randomly with urandom. |
| 145 | const CHALLENGE: [u8; 16] = [ |
| 146 | 0x7d, 0x86, 0x58, 0x79, 0x3a, 0x09, 0xdf, 0x1c, 0xa5, 0x80, 0x80, 0x15, 0x2b, 0x13, 0x17, |
| 147 | 0x5c, |
| 148 | ]; |
Alice Wang | 4ac9c8b | 2023-12-05 16:23:14 +0000 | [diff] [blame] | 149 | let dice_artifacts = fake_client_vm_dice_artifacts()?; |
Alice Wang | de6bee5 | 2023-11-10 09:58:40 +0000 | [diff] [blame] | 150 | let attestation_data = generate_attestation_key_and_csr(&CHALLENGE, &dice_artifacts)?; |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 151 | let cert_chain = fs::read(TEST_CERT_CHAIN_PATH)?; |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 152 | // The certificate chain contains several certificates, but we only need the first one. |
| 153 | // Parsing the data with trailing data always fails with a `TrailingData` error. |
| 154 | let cert_len: usize = match Certificate::from_der(&cert_chain).unwrap_err().kind() { |
| 155 | der::ErrorKind::TrailingData { decoded, .. } => decoded.try_into().unwrap(), |
| 156 | e => bail!("Unexpected error: {e}"), |
| 157 | }; |
Alice Wang | de6bee5 | 2023-11-10 09:58:40 +0000 | [diff] [blame] | 158 | |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 159 | // Builds the mock parameters for the client VM attestation. |
| 160 | // The `csr` and `remotely_provisioned_key_blob` parameters are extracted from the same |
| 161 | // libraries as in production. |
| 162 | // The `remotely_provisioned_cert` parameter is an RKP certificate extracted from a test |
| 163 | // certificate chain retrieved from RKPD. |
Alice Wang | de6bee5 | 2023-11-10 09:58:40 +0000 | [diff] [blame] | 164 | let params = ClientVmAttestationParams { |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 165 | csr: attestation_data.csr.clone().into_cbor_vec()?, |
| 166 | remotely_provisioned_key_blob: remotely_provisioned_key_pair.key_blob.to_vec(), |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 167 | remotely_provisioned_cert: cert_chain[..cert_len].to_vec(), |
Alice Wang | de6bee5 | 2023-11-10 09:58:40 +0000 | [diff] [blame] | 168 | }; |
Alice Wang | 74eb78b | 2023-11-09 16:13:10 +0000 | [diff] [blame] | 169 | let request = Request::RequestClientVmAttestation(params); |
| 170 | |
| 171 | let response = vm.process_request(request)?; |
| 172 | info!("Received response: {response:?}."); |
| 173 | |
| 174 | match response { |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 175 | Response::RequestClientVmAttestation(certificate) => { |
Alice Wang | d3a9640 | 2023-11-24 15:37:39 +0000 | [diff] [blame] | 176 | // The end-to-end test for non-protected VM attestation works because both the service |
| 177 | // VM and the client VM use the same fake DICE chain. |
| 178 | assert_eq!(vm_type, VmType::NonProtectedVm); |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 179 | check_certificate_for_client_vm( |
| 180 | &certificate, |
| 181 | &remotely_provisioned_key_pair.maced_public_key, |
| 182 | &attestation_data.csr, |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 183 | &Certificate::from_der(&cert_chain[..cert_len]).unwrap(), |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 184 | )?; |
| 185 | Ok(()) |
| 186 | } |
Alice Wang | d3a9640 | 2023-11-24 15:37:39 +0000 | [diff] [blame] | 187 | Response::Err(RequestProcessingError::InvalidDiceChain) => { |
| 188 | // The end-to-end test for protected VM attestation doesn't work because the service VM |
| 189 | // compares the fake DICE chain in the CSR with the real DICE chain. |
| 190 | // We cannot generate a valid DICE chain with the same payloads up to pvmfw. |
| 191 | assert_eq!(vm_type, VmType::ProtectedVm); |
| 192 | Ok(()) |
| 193 | } |
Alice Wang | 74eb78b | 2023-11-09 16:13:10 +0000 | [diff] [blame] | 194 | _ => bail!("Incorrect response type: {response:?}"), |
| 195 | } |
| 196 | } |
| 197 | |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 198 | fn check_vm_components(vm_components: &asn1::SequenceOf<asn1::Any, 4>) -> Result<()> { |
Alice Wang | 1cc1350 | 2023-12-05 11:05:34 +0000 | [diff] [blame] | 199 | let expected_components = fake_sub_components(); |
| 200 | assert_eq!(expected_components.len(), vm_components.len()); |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 201 | for (i, expected_component) in expected_components.iter().enumerate() { |
| 202 | check_vm_component(vm_components.get(i).unwrap(), expected_component)?; |
Alice Wang | 1cc1350 | 2023-12-05 11:05:34 +0000 | [diff] [blame] | 203 | } |
| 204 | Ok(()) |
| 205 | } |
| 206 | |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 207 | fn check_vm_component(vm_component: &asn1::Any, expected_component: &SubComponent) -> Result<()> { |
| 208 | let vm_component = vm_component.decode_as::<asn1::SequenceOf<asn1::Any, 4>>().unwrap(); |
Alice Wang | 1cc1350 | 2023-12-05 11:05:34 +0000 | [diff] [blame] | 209 | assert_eq!(4, vm_component.len()); |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 210 | let name = vm_component.get(0).unwrap().decode_as::<asn1::Utf8StringRef>().unwrap(); |
Alice Wang | 7eddbf4 | 2024-10-16 09:50:54 +0000 | [diff] [blame] | 211 | let name_str: &str = name.as_ref(); |
| 212 | assert_eq!(expected_component.name, name_str); |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 213 | let version = vm_component.get(1).unwrap().decode_as::<u64>().unwrap(); |
| 214 | assert_eq!(expected_component.version, version); |
| 215 | let code_hash = vm_component.get(2).unwrap().decode_as::<asn1::OctetString>().unwrap(); |
| 216 | assert_eq!(expected_component.code_hash, code_hash.as_bytes()); |
| 217 | let authority_hash = vm_component.get(3).unwrap().decode_as::<asn1::OctetString>().unwrap(); |
| 218 | assert_eq!(expected_component.authority_hash, authority_hash.as_bytes()); |
Alice Wang | 1cc1350 | 2023-12-05 11:05:34 +0000 | [diff] [blame] | 219 | Ok(()) |
| 220 | } |
| 221 | |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 222 | fn check_certificate_for_client_vm( |
| 223 | certificate: &[u8], |
| 224 | maced_public_key: &[u8], |
| 225 | csr: &Csr, |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 226 | parent_certificate: &Certificate, |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 227 | ) -> Result<()> { |
| 228 | let cose_mac = CoseMac0::from_slice(maced_public_key)?; |
Alice Wang | be7a4b1 | 2023-12-01 11:53:36 +0000 | [diff] [blame] | 229 | let authority_public_key = |
| 230 | EcKey::from_cose_public_key_slice(&cose_mac.payload.unwrap()).unwrap(); |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 231 | let cert = Certificate::from_der(certificate).unwrap(); |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 232 | |
| 233 | // Checks the certificate signature against the authority public key. |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 234 | const ECDSA_WITH_SHA_256: ObjectIdentifier = |
| 235 | ObjectIdentifier::new_unwrap("1.2.840.10045.4.3.2"); |
| 236 | let expected_algorithm = AlgorithmIdentifier { oid: ECDSA_WITH_SHA_256, parameters: None }; |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 237 | assert_eq!(expected_algorithm, cert.signature_algorithm); |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 238 | let tbs_cert = cert.tbs_certificate; |
| 239 | let digest = sha256(&tbs_cert.to_der().unwrap()).unwrap(); |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 240 | authority_public_key |
Alan Stokes | b2f52fb | 2024-05-09 10:12:55 +0100 | [diff] [blame] | 241 | .ecdsa_verify_der(cert.signature.raw_bytes(), &digest) |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 242 | .expect("Failed to verify the certificate signature with the authority public key"); |
| 243 | |
| 244 | // Checks that the certificate's subject public key is equal to the key in the CSR. |
| 245 | let cose_sign = CoseSign::from_slice(&csr.signed_csr_payload)?; |
| 246 | let csr_payload = |
| 247 | cose_sign.payload.as_ref().and_then(|v| CsrPayload::from_cbor_slice(v).ok()).unwrap(); |
Alice Wang | be7a4b1 | 2023-12-01 11:53:36 +0000 | [diff] [blame] | 248 | let subject_public_key = EcKey::from_cose_public_key_slice(&csr_payload.public_key).unwrap(); |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 249 | let expected_spki_data = |
Alice Wang | eb77f7d | 2023-12-01 09:13:58 +0000 | [diff] [blame] | 250 | PKey::try_from(subject_public_key).unwrap().subject_public_key_info().unwrap(); |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 251 | let expected_spki = SubjectPublicKeyInfo::from_der(&expected_spki_data).unwrap(); |
| 252 | assert_eq!(expected_spki, tbs_cert.subject_public_key_info); |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 253 | |
| 254 | // Checks the certificate extension. |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 255 | const ATTESTATION_EXTENSION_OID: ObjectIdentifier = |
| 256 | ObjectIdentifier::new_unwrap("1.3.6.1.4.1.11129.2.1.29.1"); |
| 257 | let extensions = tbs_cert.extensions.unwrap(); |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 258 | assert_eq!(1, extensions.len()); |
| 259 | let extension = &extensions[0]; |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 260 | assert_eq!(ATTESTATION_EXTENSION_OID, extension.extn_id); |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 261 | assert!(!extension.critical); |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 262 | let attestation_ext = |
| 263 | asn1::SequenceOf::<asn1::Any, 3>::from_der(extension.extn_value.as_bytes()).unwrap(); |
Alice Wang | 1cc1350 | 2023-12-05 11:05:34 +0000 | [diff] [blame] | 264 | assert_eq!(3, attestation_ext.len()); |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 265 | let challenge = attestation_ext.get(0).unwrap().decode_as::<asn1::OctetString>().unwrap(); |
| 266 | assert_eq!(csr_payload.challenge, challenge.as_bytes()); |
| 267 | let is_vm_secure = attestation_ext.get(1).unwrap().decode_as::<bool>().unwrap(); |
Alice Wang | d3a9640 | 2023-11-24 15:37:39 +0000 | [diff] [blame] | 268 | assert!( |
| 269 | !is_vm_secure, |
| 270 | "The VM shouldn't be secure as the last payload added in the test is in Debug mode" |
| 271 | ); |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 272 | let vm_components = |
| 273 | attestation_ext.get(2).unwrap().decode_as::<asn1::SequenceOf<asn1::Any, 4>>().unwrap(); |
| 274 | check_vm_components(&vm_components)?; |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 275 | |
| 276 | // Checks other fields on the certificate |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 277 | assert_eq!(Version::V3, tbs_cert.version); |
| 278 | assert_eq!(parent_certificate.tbs_certificate.validity, tbs_cert.validity); |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 279 | assert_eq!( |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 280 | Name::from_str("CN=Android Protected Virtual Machine Key").unwrap(), |
| 281 | tbs_cert.subject |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 282 | ); |
Alice Wang | 6a504ef | 2023-12-21 15:37:55 +0000 | [diff] [blame] | 283 | assert_eq!(parent_certificate.tbs_certificate.subject, tbs_cert.issuer); |
Alice Wang | 20b8ebc | 2023-11-17 09:54:47 +0000 | [diff] [blame] | 284 | |
| 285 | Ok(()) |
| 286 | } |
| 287 | |
Karuna Wadhera | c8fb3bf | 2024-07-01 12:54:25 +0000 | [diff] [blame] | 288 | fn check_csr(csr: Vec<u8>) -> Result<()> { |
Karuna Wadhera | d0792e8 | 2024-07-08 17:22:04 +0000 | [diff] [blame] | 289 | let mut session = Session::default(); |
| 290 | session.set_allow_any_mode(true); |
Alice Wang | 7eddbf4 | 2024-10-16 09:50:54 +0000 | [diff] [blame] | 291 | session.set_rkp_instance(RkpInstance::Avf); |
Karuna Wadhera | d0792e8 | 2024-07-08 17:22:04 +0000 | [diff] [blame] | 292 | let _csr = rkp::Csr::from_cbor(&session, &csr[..]).context("Failed to parse CSR")?; |
Alice Wang | f7c0f94 | 2023-09-14 09:33:04 +0000 | [diff] [blame] | 293 | Ok(()) |
| 294 | } |
| 295 | |
Pierre-Clément Tosi | 311de8e | 2024-08-14 10:10:49 +0100 | [diff] [blame] | 296 | fn start_service_vm(vm_type: VmType, vm_memory_mb: Option<i32>) -> Result<ServiceVm> { |
David Brazdil | 66fc120 | 2022-07-04 21:48:45 +0100 | [diff] [blame] | 297 | android_logger::init_once( |
Jeff Vander Stoep | d9dda0c | 2024-02-07 14:27:06 +0100 | [diff] [blame] | 298 | android_logger::Config::default() |
| 299 | .with_tag("rialto") |
| 300 | .with_max_level(log::LevelFilter::Debug), |
David Brazdil | 66fc120 | 2022-07-04 21:48:45 +0100 | [diff] [blame] | 301 | ); |
David Brazdil | 66fc120 | 2022-07-04 21:48:45 +0100 | [diff] [blame] | 302 | // We need to start the thread pool for Binder to work properly, especially link_to_death. |
| 303 | ProcessState::start_thread_pool(); |
Pierre-Clément Tosi | 311de8e | 2024-08-14 10:10:49 +0100 | [diff] [blame] | 304 | ServiceVm::start_vm(vm_instance(vm_type, vm_memory_mb)?, vm_type) |
Alice Wang | 17dc76e | 2023-09-06 09:43:52 +0000 | [diff] [blame] | 305 | } |
| 306 | |
Pierre-Clément Tosi | 311de8e | 2024-08-14 10:10:49 +0100 | [diff] [blame] | 307 | fn vm_instance(vm_type: VmType, vm_memory_mb: Option<i32>) -> Result<VmInstance> { |
Alice Wang | a635769 | 2023-09-07 14:59:37 +0000 | [diff] [blame] | 308 | match vm_type { |
Alice Wang | 1d9a587 | 2023-09-06 14:32:36 +0000 | [diff] [blame] | 309 | VmType::ProtectedVm => { |
Pierre-Clément Tosi | 311de8e | 2024-08-14 10:10:49 +0100 | [diff] [blame] | 310 | assert!(vm_memory_mb.is_none()); |
Alice Wang | a635769 | 2023-09-07 14:59:37 +0000 | [diff] [blame] | 311 | service_vm_manager::protected_vm_instance(PathBuf::from(INSTANCE_IMG_PATH)) |
Alice Wang | 1d9a587 | 2023-09-06 14:32:36 +0000 | [diff] [blame] | 312 | } |
Pierre-Clément Tosi | 311de8e | 2024-08-14 10:10:49 +0100 | [diff] [blame] | 313 | VmType::NonProtectedVm => nonprotected_vm_instance(vm_memory_mb.unwrap_or(VM_MEMORY_MB)), |
Alice Wang | a635769 | 2023-09-07 14:59:37 +0000 | [diff] [blame] | 314 | } |
| 315 | } |
| 316 | |
Pierre-Clément Tosi | 311de8e | 2024-08-14 10:10:49 +0100 | [diff] [blame] | 317 | fn nonprotected_vm_instance(memory_mib: i32) -> Result<VmInstance> { |
Alice Wang | a635769 | 2023-09-07 14:59:37 +0000 | [diff] [blame] | 318 | let rialto = File::open(UNSIGNED_RIALTO_PATH).context("Failed to open Rialto kernel binary")?; |
Alice Wang | b76b66f | 2024-03-26 08:16:23 +0000 | [diff] [blame] | 319 | // Do not use `#allocateInstanceId` to generate the instance ID because the method |
| 320 | // also adds an instance ID to the database it manages. |
| 321 | // This is not necessary for this test. |
| 322 | let mut instance_id = [0u8; 64]; |
| 323 | rand_bytes(&mut instance_id).unwrap(); |
David Brazdil | 66fc120 | 2022-07-04 21:48:45 +0100 | [diff] [blame] | 324 | let config = VirtualMachineConfig::RawConfig(VirtualMachineRawConfig { |
Pierre-Clément Tosi | 311de8e | 2024-08-14 10:10:49 +0100 | [diff] [blame] | 325 | name: format!("Non protected rialto ({memory_mib}MiB)"), |
Frederick Mayle | 7584240 | 2024-08-05 19:32:08 -0700 | [diff] [blame] | 326 | kernel: Some(ParcelFileDescriptor::new(rialto)), |
Alice Wang | a635769 | 2023-09-07 14:59:37 +0000 | [diff] [blame] | 327 | protectedVm: false, |
Pierre-Clément Tosi | 311de8e | 2024-08-14 10:10:49 +0100 | [diff] [blame] | 328 | memoryMib: memory_mib, |
David Brazdil | 66fc120 | 2022-07-04 21:48:45 +0100 | [diff] [blame] | 329 | platformVersion: "~1.0".to_string(), |
Alice Wang | b76b66f | 2024-03-26 08:16:23 +0000 | [diff] [blame] | 330 | instanceId: instance_id, |
Inseob Kim | 6ef8097 | 2023-07-20 17:23:36 +0900 | [diff] [blame] | 331 | ..Default::default() |
David Brazdil | 66fc120 | 2022-07-04 21:48:45 +0100 | [diff] [blame] | 332 | }); |
Alice Wang | a635769 | 2023-09-07 14:59:37 +0000 | [diff] [blame] | 333 | let console = Some(service_vm_manager::android_log_fd()?); |
| 334 | let log = Some(service_vm_manager::android_log_fd()?); |
| 335 | let virtmgr = vmclient::VirtualizationService::new().context("Failed to spawn VirtMgr")?; |
| 336 | let service = virtmgr.connect().context("Failed to connect to VirtMgr")?; |
| 337 | info!("Connected to VirtMgr for service VM"); |
Elie Kheirallah | 5c807a2 | 2024-09-23 20:40:42 +0000 | [diff] [blame] | 338 | VmInstance::create( |
| 339 | service.as_ref(), |
| 340 | &config, |
| 341 | console, |
| 342 | /* consoleIn */ None, |
| 343 | log, |
| 344 | /* dump_dt */ None, |
| 345 | None, |
| 346 | ) |
| 347 | .context("Failed to create VM") |
David Brazdil | 66fc120 | 2022-07-04 21:48:45 +0100 | [diff] [blame] | 348 | } |