Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 1 | // Copyright 2023, 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 | //! Class for encapsulating & managing represent VM secrets. |
| 16 | |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 17 | use anyhow::{anyhow, ensure, Result}; |
| 18 | use android_system_virtualmachineservice::aidl::android::system::virtualmachineservice::IVirtualMachineService::IVirtualMachineService; |
| 19 | use android_hardware_security_secretkeeper::aidl::android::hardware::security::secretkeeper::ISecretkeeper::ISecretkeeper; |
| 20 | use secretkeeper_comm::data_types::request::Request; |
| 21 | use binder::{Strong}; |
| 22 | use coset::CborSerializable; |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 23 | use dice_policy::{ConstraintSpec, ConstraintType, DicePolicy}; |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 24 | use diced_open_dice::{DiceArtifacts, OwnedDiceArtifacts}; |
| 25 | use keystore2_crypto::ZVec; |
| 26 | use openssl::hkdf::hkdf; |
| 27 | use openssl::md::Md; |
| 28 | use openssl::sha; |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 29 | use secretkeeper_client::dice::OwnedDiceArtifactsWithExplicitKey; |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 30 | use secretkeeper_client::SkSession; |
| 31 | use secretkeeper_comm::data_types::{Id, ID_SIZE, Secret, SECRET_SIZE}; |
| 32 | use secretkeeper_comm::data_types::response::Response; |
| 33 | use secretkeeper_comm::data_types::packet::{ResponsePacket, ResponseType}; |
| 34 | use secretkeeper_comm::data_types::request_response_impl::{ |
| 35 | StoreSecretRequest, GetSecretResponse, GetSecretRequest}; |
| 36 | use secretkeeper_comm::data_types::error::SecretkeeperError; |
| 37 | use zeroize::Zeroizing; |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 38 | |
Shikha Panwar | 3d3a70a | 2023-08-21 20:02:08 +0000 | [diff] [blame] | 39 | const ENCRYPTEDSTORE_KEY_IDENTIFIER: &str = "encryptedstore_key"; |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 40 | const AUTHORITY_HASH: i64 = -4670549; |
| 41 | const MODE: i64 = -4670551; |
| 42 | const CONFIG_DESC: i64 = -4670548; |
| 43 | const SECURITY_VERSION: i64 = -70005; |
Shikha Panwar | 3d3a70a | 2023-08-21 20:02:08 +0000 | [diff] [blame] | 44 | |
Shikha Panwar | 3d3a70a | 2023-08-21 20:02:08 +0000 | [diff] [blame] | 45 | // Generated using hexdump -vn32 -e'14/1 "0x%02X, " 1 "\n"' /dev/urandom |
| 46 | const SALT_ENCRYPTED_STORE: &[u8] = &[ |
| 47 | 0xFC, 0x1D, 0x35, 0x7B, 0x96, 0xF3, 0xEF, 0x17, 0x78, 0x7D, 0x70, 0xED, 0xEA, 0xFE, 0x1D, 0x6F, |
| 48 | 0xB3, 0xF9, 0x40, 0xCE, 0xDD, 0x99, 0x40, 0xAA, 0xA7, 0x0E, 0x92, 0x73, 0x90, 0x86, 0x4A, 0x75, |
| 49 | ]; |
| 50 | const SALT_PAYLOAD_SERVICE: &[u8] = &[ |
| 51 | 0x8B, 0x0F, 0xF0, 0xD3, 0xB1, 0x69, 0x2B, 0x95, 0x84, 0x2C, 0x9E, 0x3C, 0x99, 0x56, 0x7A, 0x22, |
| 52 | 0x55, 0xF8, 0x08, 0x23, 0x81, 0x5F, 0xF5, 0x16, 0x20, 0x3E, 0xBE, 0xBA, 0xB7, 0xA8, 0x43, 0x92, |
| 53 | ]; |
| 54 | |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 55 | // TODO(b/291213394): Differentiate the Id of nPVM based on 'salt' |
| 56 | const ID_NP_VM: [u8; ID_SIZE] = [ |
| 57 | 0xF1, 0xB2, 0xED, 0x3B, 0xD1, 0xBD, 0xF0, 0x7D, 0xE1, 0xF0, 0x01, 0xFC, 0x61, 0x71, 0xD3, 0x42, |
| 58 | 0xE5, 0x8A, 0xAF, 0x33, 0x6C, 0x11, 0xDC, 0xC8, 0x6F, 0xAE, 0x12, 0x5C, 0x26, 0x44, 0x6B, 0x86, |
| 59 | 0xCC, 0x24, 0xFD, 0xBF, 0x91, 0x4A, 0x54, 0x84, 0xF9, 0x01, 0x59, 0x25, 0x70, 0x89, 0x38, 0x8D, |
| 60 | 0x5E, 0xE6, 0x91, 0xDF, 0x68, 0x60, 0x69, 0x26, 0xBE, 0xFE, 0x79, 0x58, 0xF7, 0xEA, 0x81, 0x7D, |
| 61 | ]; |
| 62 | const SKP_SECRET_NP_VM: [u8; SECRET_SIZE] = [ |
| 63 | 0xA9, 0x89, 0x97, 0xFE, 0xAE, 0x97, 0x55, 0x4B, 0x32, 0x35, 0xF0, 0xE8, 0x93, 0xDA, 0xEA, 0x24, |
| 64 | 0x06, 0xAC, 0x36, 0x8B, 0x3C, 0x95, 0x50, 0x16, 0x67, 0x71, 0x65, 0x26, 0xEB, 0xD0, 0xC3, 0x98, |
| 65 | ]; |
| 66 | |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 67 | pub enum VmSecret { |
| 68 | // V2 secrets are derived from 2 independently secured secrets: |
| 69 | // 1. Secretkeeper protected secrets (skp secret). |
| 70 | // 2. Dice Sealing CDIs (Similar to V1). |
| 71 | // |
| 72 | // These are protected against rollback of boot images i.e. VM instance rebooted |
| 73 | // with downgraded images will not have access to VM's secret. |
| 74 | // V2 secrets require hardware support - Secretkeeper HAL, which (among other things) |
| 75 | // is backed by tamper-evident storage, providing rollback protection to these secrets. |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 76 | V2 { dice_artifacts: OwnedDiceArtifactsWithExplicitKey, skp_secret: ZVec }, |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 77 | // V1 secrets are not protected against rollback of boot images. |
| 78 | // They are reliable only if rollback of images was prevented by verified boot ie, |
| 79 | // each stage (including pvmfw/Microdroid/Microdroid Manager) prevents downgrade of next |
| 80 | // stage. These are now legacy secrets & used only when Secretkeeper HAL is not supported |
| 81 | // by device. |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 82 | V1 { dice_artifacts: OwnedDiceArtifacts }, |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 83 | } |
| 84 | |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 85 | fn get_id() -> [u8; ID_SIZE] { |
| 86 | if super::is_strict_boot() { |
| 87 | todo!("Id for protected VM is not implemented"); |
| 88 | } else { |
| 89 | ID_NP_VM |
| 90 | } |
| 91 | } |
| 92 | |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 93 | impl VmSecret { |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 94 | pub fn new( |
| 95 | dice_artifacts: OwnedDiceArtifacts, |
| 96 | vm_service: &Strong<dyn IVirtualMachineService>, |
| 97 | ) -> Result<VmSecret> { |
| 98 | ensure!(dice_artifacts.bcc().is_some(), "Dice chain missing"); |
| 99 | |
| 100 | if let Some(sk_service) = is_sk_supported(vm_service)? { |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 101 | let explicit_dice = |
| 102 | OwnedDiceArtifactsWithExplicitKey::from_owned_artifacts(dice_artifacts)?; |
| 103 | let explicit_dice_chain = explicit_dice |
| 104 | .explicit_key_dice_chain() |
| 105 | .ok_or(anyhow!("Missing explicit dice chain, this is unusual"))?; |
Shikha Panwar | ab958c4 | 2024-01-03 18:24:38 +0000 | [diff] [blame] | 106 | let policy = sealing_policy(explicit_dice_chain).map_err(anyhow_err)?; |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 107 | |
| 108 | // Start a new session with Secretkeeper! |
Shikha Panwar | ab958c4 | 2024-01-03 18:24:38 +0000 | [diff] [blame] | 109 | let mut session = SkSession::new(sk_service, &explicit_dice)?; |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 110 | let id = get_id(); |
| 111 | let mut skp_secret = Zeroizing::new([0u8; SECRET_SIZE]); |
| 112 | if super::is_strict_boot() { |
| 113 | if super::is_new_instance() { |
| 114 | *skp_secret = rand::random(); |
Shikha Panwar | ab958c4 | 2024-01-03 18:24:38 +0000 | [diff] [blame] | 115 | store_secret(&mut session, id, skp_secret.clone(), policy)?; |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 116 | } else { |
| 117 | // Subsequent run of the pVM -> get the secret stored in Secretkeeper. |
Shikha Panwar | ab958c4 | 2024-01-03 18:24:38 +0000 | [diff] [blame] | 118 | *skp_secret = get_secret(&mut session, id, Some(policy))?; |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 119 | } |
| 120 | } else { |
| 121 | // TODO(b/291213394): Non protected VM don't need to use Secretkeeper, remove this |
| 122 | // once we have sufficient testing on protected VM. |
Shikha Panwar | ab958c4 | 2024-01-03 18:24:38 +0000 | [diff] [blame] | 123 | store_secret(&mut session, id, SKP_SECRET_NP_VM.into(), policy)?; |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 124 | *skp_secret = get_secret(&mut session, id, None)?; |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 125 | } |
| 126 | return Ok(Self::V2 { |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 127 | dice_artifacts: explicit_dice, |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 128 | skp_secret: ZVec::try_from(skp_secret.to_vec())?, |
| 129 | }); |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 130 | } |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 131 | // Use V1 secrets if Secretkeeper is not supported. |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 132 | Ok(Self::V1 { dice_artifacts }) |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 133 | } |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 134 | |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 135 | pub fn dice_artifacts(&self) -> &dyn DiceArtifacts { |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 136 | match self { |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 137 | Self::V2 { dice_artifacts, .. } => dice_artifacts, |
| 138 | Self::V1 { dice_artifacts } => dice_artifacts, |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 139 | } |
| 140 | } |
| 141 | |
| 142 | fn get_vm_secret(&self, salt: &[u8], identifier: &[u8], key: &mut [u8]) -> Result<()> { |
| 143 | match self { |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 144 | Self::V2 { dice_artifacts, skp_secret } => { |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 145 | let mut hasher = sha::Sha256::new(); |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 146 | hasher.update(dice_artifacts.cdi_seal()); |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 147 | hasher.update(skp_secret); |
| 148 | hkdf(key, Md::sha256(), &hasher.finish(), salt, identifier)? |
| 149 | } |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 150 | Self::V1 { dice_artifacts } => { |
| 151 | hkdf(key, Md::sha256(), dice_artifacts.cdi_seal(), salt, identifier)? |
| 152 | } |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 153 | } |
| 154 | Ok(()) |
| 155 | } |
| 156 | |
Shikha Panwar | 3d3a70a | 2023-08-21 20:02:08 +0000 | [diff] [blame] | 157 | /// Derive sealing key for payload with following identifier. |
| 158 | pub fn derive_payload_sealing_key(&self, identifier: &[u8], key: &mut [u8]) -> Result<()> { |
| 159 | self.get_vm_secret(SALT_PAYLOAD_SERVICE, identifier, key) |
| 160 | } |
| 161 | |
| 162 | /// Derive encryptedstore key. This uses hardcoded random salt & fixed identifier. |
| 163 | pub fn derive_encryptedstore_key(&self, key: &mut [u8]) -> Result<()> { |
| 164 | self.get_vm_secret(SALT_ENCRYPTED_STORE, ENCRYPTEDSTORE_KEY_IDENTIFIER.as_bytes(), key) |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 165 | } |
| 166 | } |
| 167 | |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 168 | // Construct a sealing policy on the dice chain. VMs uses the following set of constraint for |
| 169 | // protecting secrets against rollback of boot images. |
| 170 | // 1. ExactMatch on AUTHORITY_HASH (Required ie, each DiceChainEntry must have it). |
| 171 | // 2. ExactMatch on MODE (Required) - Secret should be inaccessible if any of the runtime |
| 172 | // configuration changes. For ex, the secrets stored with a boot stage being in Normal mode |
| 173 | // should be inaccessible when the same stage is booted in Debug mode. |
| 174 | // 3. GreaterOrEqual on SECURITY_VERSION (Optional): The secrets will be accessible if version of |
| 175 | // any image is greater or equal to the set version. This is an optional field, certain |
| 176 | // components may chose to prevent booting of rollback images for ex, ABL is expected to provide |
| 177 | // rollback protection of pvmfw. Such components may chose to not put SECURITY_VERSION in the |
| 178 | // corresponding DiceChainEntry. |
| 179 | // TODO(b/291219197) : Add constraints on Extra apks as well! |
| 180 | fn sealing_policy(dice: &[u8]) -> Result<Vec<u8>, String> { |
| 181 | let constraint_spec = [ |
| 182 | ConstraintSpec::new( |
| 183 | ConstraintType::ExactMatch, |
| 184 | vec![AUTHORITY_HASH], |
| 185 | /* Optional */ false, |
| 186 | ), |
| 187 | ConstraintSpec::new(ConstraintType::ExactMatch, vec![MODE], /* Optional */ false), |
| 188 | ConstraintSpec::new( |
| 189 | ConstraintType::GreaterOrEqual, |
| 190 | vec![CONFIG_DESC, SECURITY_VERSION], |
| 191 | /* Optional */ true, |
| 192 | ), |
| 193 | ]; |
| 194 | |
| 195 | DicePolicy::from_dice_chain(dice, &constraint_spec)? |
| 196 | .to_vec() |
| 197 | .map_err(|e| format!("DicePolicy construction failed {e:?}")) |
| 198 | } |
| 199 | |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 200 | fn store_secret( |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 201 | session: &mut SkSession, |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 202 | id: [u8; ID_SIZE], |
| 203 | secret: Zeroizing<[u8; SECRET_SIZE]>, |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 204 | sealing_policy: Vec<u8>, |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 205 | ) -> Result<()> { |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 206 | let store_request = StoreSecretRequest { id: Id(id), secret: Secret(*secret), sealing_policy }; |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 207 | log::info!("Secretkeeper operation: {:?}", store_request); |
| 208 | |
| 209 | let store_request = store_request.serialize_to_packet().to_vec().map_err(anyhow_err)?; |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 210 | let store_response = session.secret_management_request(&store_request)?; |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 211 | let store_response = ResponsePacket::from_slice(&store_response).map_err(anyhow_err)?; |
| 212 | let response_type = store_response.response_type().map_err(anyhow_err)?; |
| 213 | ensure!( |
| 214 | response_type == ResponseType::Success, |
| 215 | "Secretkeeper store failed with error: {:?}", |
| 216 | *SecretkeeperError::deserialize_from_packet(store_response).map_err(anyhow_err)? |
| 217 | ); |
| 218 | Ok(()) |
| 219 | } |
| 220 | |
| 221 | fn get_secret( |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 222 | session: &mut SkSession, |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 223 | id: [u8; ID_SIZE], |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 224 | updated_sealing_policy: Option<Vec<u8>>, |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 225 | ) -> Result<[u8; SECRET_SIZE]> { |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 226 | let get_request = GetSecretRequest { id: Id(id), updated_sealing_policy }; |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 227 | log::info!("Secretkeeper operation: {:?}", get_request); |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 228 | let get_request = get_request.serialize_to_packet().to_vec().map_err(anyhow_err)?; |
Shikha Panwar | 6b17832 | 2023-12-23 00:05:17 +0000 | [diff] [blame] | 229 | let get_response = session.secret_management_request(&get_request)?; |
Shikha Panwar | 5d6a675 | 2023-12-14 22:08:26 +0000 | [diff] [blame] | 230 | let get_response = ResponsePacket::from_slice(&get_response).map_err(anyhow_err)?; |
| 231 | let response_type = get_response.response_type().map_err(anyhow_err)?; |
| 232 | ensure!( |
| 233 | response_type == ResponseType::Success, |
| 234 | "Secretkeeper get failed with error: {:?}", |
| 235 | *SecretkeeperError::deserialize_from_packet(get_response).map_err(anyhow_err)? |
| 236 | ); |
| 237 | let get_response = |
| 238 | *GetSecretResponse::deserialize_from_packet(get_response).map_err(anyhow_err)?; |
| 239 | Ok(get_response.secret.0) |
| 240 | } |
| 241 | |
| 242 | #[inline] |
| 243 | fn anyhow_err<E: core::fmt::Debug>(err: E) -> anyhow::Error { |
| 244 | anyhow!("{:?}", err) |
| 245 | } |
| 246 | |
| 247 | // Get the secretkeeper connection if supported. Host can be consulted whether the device supports |
| 248 | // secretkeeper but that should be used with caution for protected VM. |
| 249 | fn is_sk_supported( |
| 250 | host: &Strong<dyn IVirtualMachineService>, |
| 251 | ) -> Result<Option<Strong<dyn ISecretkeeper>>> { |
| 252 | let sk = if cfg!(llpvm_changes) { |
| 253 | if super::is_strict_boot() { |
| 254 | // TODO: For protected VM check for Secretkeeper authentication data in device tree. |
| 255 | None |
| 256 | } else { |
| 257 | // For non-protected VM, believe what host claims. |
| 258 | host.getSecretkeeper() |
| 259 | // TODO rename this error! |
| 260 | .map_err(|e| { |
| 261 | super::MicrodroidError::FailedToConnectToVirtualizationService(e.to_string()) |
| 262 | })? |
| 263 | } |
| 264 | } else { |
| 265 | // LLPVM flag is disabled |
| 266 | None |
| 267 | }; |
| 268 | Ok(sk) |
Shikha Panwar | 95084df | 2023-07-22 11:47:45 +0000 | [diff] [blame] | 269 | } |