blob: 8d39b2233330f1828254bdc90561b9df37e0a2fa [file] [log] [blame]
Janis Danisevskisa75e2082020-10-07 16:44:26 -07001// Copyright 2020, 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//! This module holds global state of Keystore such as the thread local
16//! database connections and connections to services that Keystore needs
17//! to talk to.
18
Janis Danisevskis93927dd2020-12-23 12:23:08 -080019use crate::gc::Gc;
Hasini Gunasinghea020b532021-01-07 21:42:35 +000020use crate::legacy_blob::LegacyBlobLoader;
Hasini Gunasinghe3ed5da72021-02-04 15:18:54 +000021use crate::legacy_migrator::LegacyMigrator;
Janis Danisevskisb42fc182020-12-15 08:41:27 -080022use crate::super_key::SuperKeyManager;
Janis Danisevskis2ee014b2021-05-05 14:29:08 -070023use crate::utils::watchdog as wd;
Janis Danisevskisba998992020-12-29 16:08:40 -080024use crate::utils::Asp;
Janis Danisevskis5ed8c532021-01-11 14:19:42 -080025use crate::{async_task::AsyncTask, database::MonotonicRawTime};
Janis Danisevskisba998992020-12-29 16:08:40 -080026use crate::{
27 database::KeystoreDB,
Max Bires8e93d2b2021-01-14 13:17:59 -080028 database::Uuid,
Janis Danisevskis8c6378e2021-01-01 09:30:37 -080029 error::{map_binder_status, map_binder_status_code, Error, ErrorCode},
Janis Danisevskisba998992020-12-29 16:08:40 -080030};
Max Bires8e93d2b2021-01-14 13:17:59 -080031use crate::{enforcements::Enforcements, error::map_km_error};
32use android_hardware_security_keymint::aidl::android::hardware::security::keymint::{
Max Biresb2e1d032021-02-08 21:35:05 -080033 IKeyMintDevice::IKeyMintDevice, IRemotelyProvisionedComponent::IRemotelyProvisionedComponent,
34 KeyMintHardwareInfo::KeyMintHardwareInfo, SecurityLevel::SecurityLevel,
Max Bires8e93d2b2021-01-14 13:17:59 -080035};
Stephen Crane221bbb52020-12-16 15:52:10 -080036use android_hardware_security_keymint::binder::{StatusCode, Strong};
Janis Danisevskis8c6378e2021-01-01 09:30:37 -080037use android_security_compat::aidl::android::security::compat::IKeystoreCompatService::IKeystoreCompatService;
Janis Danisevskisba998992020-12-29 16:08:40 -080038use anyhow::{Context, Result};
David Drysdale0e45a612021-02-25 17:24:36 +000039use binder::FromIBinder;
Janis Danisevskisef14e1a2021-02-23 23:16:55 -080040use keystore2_vintf::get_aidl_instances;
Janis Danisevskisb42fc182020-12-15 08:41:27 -080041use lazy_static::lazy_static;
Janis Danisevskis7e8b4622021-02-13 10:01:59 -080042use std::sync::{Arc, Mutex};
Janis Danisevskis93927dd2020-12-23 12:23:08 -080043use std::{cell::RefCell, sync::Once};
Janis Danisevskis3f2955c2021-02-02 21:53:35 -080044use std::{collections::HashMap, path::Path, path::PathBuf};
Janis Danisevskis93927dd2020-12-23 12:23:08 -080045
46static DB_INIT: Once = Once::new();
47
48/// Open a connection to the Keystore 2.0 database. This is called during the initialization of
49/// the thread local DB field. It should never be called directly. The first time this is called
50/// we also call KeystoreDB::cleanup_leftovers to restore the key lifecycle invariant. See the
Janis Danisevskis7e8b4622021-02-13 10:01:59 -080051/// documentation of cleanup_leftovers for more details. The function also constructs a blob
52/// garbage collector. The initializing closure constructs another database connection without
53/// a gc. Although one GC is created for each thread local database connection, this closure
54/// is run only once, as long as the ASYNC_TASK instance is the same. So only one additional
55/// database connection is created for the garbage collector worker.
Hasini Gunasinghe3ed5da72021-02-04 15:18:54 +000056pub fn create_thread_local_db() -> KeystoreDB {
Janis Danisevskis3395f862021-05-06 10:54:17 -070057 let mut db = KeystoreDB::new(
58 &DB_PATH.lock().expect("Could not get the database directory."),
59 Some(GC.clone()),
60 )
61 .expect("Failed to open database.");
Janis Danisevskis93927dd2020-12-23 12:23:08 -080062 DB_INIT.call_once(|| {
63 log::info!("Touching Keystore 2.0 database for this first time since boot.");
Matthew Maurerd7815ca2021-05-06 21:58:45 -070064 db.insert_last_off_body(MonotonicRawTime::now());
Janis Danisevskis93927dd2020-12-23 12:23:08 -080065 log::info!("Calling cleanup leftovers.");
66 let n = db.cleanup_leftovers().expect("Failed to cleanup database on startup.");
67 if n != 0 {
68 log::info!(
69 concat!(
70 "Cleaned up {} failed entries. ",
71 "This indicates keystore crashed during key generation."
72 ),
73 n
74 );
75 }
Janis Danisevskis93927dd2020-12-23 12:23:08 -080076 });
77 db
78}
Janis Danisevskisa75e2082020-10-07 16:44:26 -070079
80thread_local! {
81 /// Database connections are not thread safe, but connecting to the
82 /// same database multiple times is safe as long as each connection is
83 /// used by only one thread. So we store one database connection per
84 /// thread in this thread local key.
85 pub static DB: RefCell<KeystoreDB> =
Janis Danisevskis93927dd2020-12-23 12:23:08 -080086 RefCell::new(create_thread_local_db());
Janis Danisevskisa75e2082020-10-07 16:44:26 -070087}
Janis Danisevskisb42fc182020-12-15 08:41:27 -080088
Max Bires8e93d2b2021-01-14 13:17:59 -080089#[derive(Default)]
90struct DevicesMap {
91 devices_by_uuid: HashMap<Uuid, (Asp, KeyMintHardwareInfo)>,
92 uuid_by_sec_level: HashMap<SecurityLevel, Uuid>,
93}
94
95impl DevicesMap {
96 fn dev_by_sec_level(
97 &self,
98 sec_level: &SecurityLevel,
99 ) -> Option<(Asp, KeyMintHardwareInfo, Uuid)> {
100 self.uuid_by_sec_level.get(sec_level).and_then(|uuid| self.dev_by_uuid(uuid))
101 }
102
103 fn dev_by_uuid(&self, uuid: &Uuid) -> Option<(Asp, KeyMintHardwareInfo, Uuid)> {
104 self.devices_by_uuid
105 .get(uuid)
106 .map(|(dev, hw_info)| ((*dev).clone(), (*hw_info).clone(), *uuid))
107 }
108
David Drysdale0e45a612021-02-25 17:24:36 +0000109 fn devices<T: FromIBinder + ?Sized>(&self) -> Vec<Strong<T>> {
110 self.devices_by_uuid.values().filter_map(|(asp, _)| asp.get_interface::<T>().ok()).collect()
111 }
112
Max Bires8e93d2b2021-01-14 13:17:59 -0800113 /// The requested security level and the security level of the actual implementation may
114 /// differ. So we map the requested security level to the uuid of the implementation
115 /// so that there cannot be any confusion as to which KeyMint instance is requested.
116 fn insert(&mut self, sec_level: SecurityLevel, dev: Asp, hw_info: KeyMintHardwareInfo) {
117 // For now we use the reported security level of the KM instance as UUID.
118 // TODO update this section once UUID was added to the KM hardware info.
119 let uuid: Uuid = sec_level.into();
120 self.devices_by_uuid.insert(uuid, (dev, hw_info));
121 self.uuid_by_sec_level.insert(sec_level, uuid);
122 }
123}
124
Max Biresb2e1d032021-02-08 21:35:05 -0800125#[derive(Default)]
126struct RemotelyProvisionedDevicesMap {
127 devices_by_sec_level: HashMap<SecurityLevel, Asp>,
128}
129
130impl RemotelyProvisionedDevicesMap {
131 fn dev_by_sec_level(&self, sec_level: &SecurityLevel) -> Option<Asp> {
132 self.devices_by_sec_level.get(sec_level).map(|dev| (*dev).clone())
133 }
134
135 fn insert(&mut self, sec_level: SecurityLevel, dev: Asp) {
136 self.devices_by_sec_level.insert(sec_level, dev);
137 }
138}
139
Janis Danisevskisb42fc182020-12-15 08:41:27 -0800140lazy_static! {
Janis Danisevskis3f2955c2021-02-02 21:53:35 -0800141 /// The path where keystore stores all its keys.
142 pub static ref DB_PATH: Mutex<PathBuf> = Mutex::new(
143 Path::new("/data/misc/keystore").to_path_buf());
Janis Danisevskisb42fc182020-12-15 08:41:27 -0800144 /// Runtime database of unwrapped super keys.
Janis Danisevskis7e8b4622021-02-13 10:01:59 -0800145 pub static ref SUPER_KEY: Arc<SuperKeyManager> = Default::default();
Janis Danisevskisba998992020-12-29 16:08:40 -0800146 /// Map of KeyMint devices.
Max Bires8e93d2b2021-01-14 13:17:59 -0800147 static ref KEY_MINT_DEVICES: Mutex<DevicesMap> = Default::default();
Janis Danisevskisc3a496b2021-01-05 10:37:22 -0800148 /// Timestamp service.
149 static ref TIME_STAMP_DEVICE: Mutex<Option<Asp>> = Default::default();
Max Biresb2e1d032021-02-08 21:35:05 -0800150 /// RemotelyProvisionedComponent HAL devices.
151 static ref REMOTELY_PROVISIONED_COMPONENT_DEVICES: Mutex<RemotelyProvisionedDevicesMap> = Default::default();
Janis Danisevskis93927dd2020-12-23 12:23:08 -0800152 /// A single on-demand worker thread that handles deferred tasks with two different
153 /// priorities.
Janis Danisevskis7e8b4622021-02-13 10:01:59 -0800154 pub static ref ASYNC_TASK: Arc<AsyncTask> = Default::default();
Janis Danisevskis5ed8c532021-01-11 14:19:42 -0800155 /// Singleton for enforcements.
Paul Crowley7c57bf12021-02-02 16:26:57 -0800156 pub static ref ENFORCEMENTS: Enforcements = Default::default();
Hasini Gunasinghea020b532021-01-07 21:42:35 +0000157 /// LegacyBlobLoader is initialized and exists globally.
158 /// The same directory used by the database is used by the LegacyBlobLoader as well.
Hasini Gunasinghe3ed5da72021-02-04 15:18:54 +0000159 pub static ref LEGACY_BLOB_LOADER: Arc<LegacyBlobLoader> = Arc::new(LegacyBlobLoader::new(
160 &DB_PATH.lock().expect("Could not get the database path for legacy blob loader.")));
161 /// Legacy migrator. Atomically migrates legacy blobs to the database.
162 pub static ref LEGACY_MIGRATOR: Arc<LegacyMigrator> =
Janis Danisevskisec6586d2021-04-30 10:54:07 -0700163 Arc::new(LegacyMigrator::new(Arc::new(Default::default())));
Pavel Grafov94243c22021-04-21 18:03:11 +0100164 /// Background thread which handles logging via statsd and logd
165 pub static ref LOGS_HANDLER: Arc<AsyncTask> = Default::default();
Janis Danisevskis3395f862021-05-06 10:54:17 -0700166
167 static ref GC: Arc<Gc> = Arc::new(Gc::new_init_with(ASYNC_TASK.clone(), || {
168 (
169 Box::new(|uuid, blob| {
170 let km_dev: Strong<dyn IKeyMintDevice> =
171 get_keymint_dev_by_uuid(uuid).map(|(dev, _)| dev)?.get_interface()?;
172 let _wp = wd::watch_millis("In invalidate key closure: calling deleteKey", 500);
173 map_km_error(km_dev.deleteKey(&*blob))
174 .context("In invalidate key closure: Trying to invalidate key blob.")
175 }),
176 KeystoreDB::new(&DB_PATH.lock().expect("Could not get the database directory."), None)
177 .expect("Failed to open database."),
178 SUPER_KEY.clone(),
179 )
180 }));
Janis Danisevskisba998992020-12-29 16:08:40 -0800181}
182
183static KEYMINT_SERVICE_NAME: &str = "android.hardware.security.keymint.IKeyMintDevice";
184
185/// Make a new connection to a KeyMint device of the given security level.
Janis Danisevskis8c6378e2021-01-01 09:30:37 -0800186/// If no native KeyMint device can be found this function also brings
187/// up the compatibility service and attempts to connect to the legacy wrapper.
Max Bires8e93d2b2021-01-14 13:17:59 -0800188fn connect_keymint(security_level: &SecurityLevel) -> Result<(Asp, KeyMintHardwareInfo)> {
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800189 let keymint_instances =
190 get_aidl_instances("android.hardware.security.keymint", 1, "IKeyMintDevice");
191
Max Bires8e93d2b2021-01-14 13:17:59 -0800192 let service_name = match *security_level {
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800193 SecurityLevel::TRUSTED_ENVIRONMENT => {
194 if keymint_instances.as_vec()?.iter().any(|instance| *instance == "default") {
195 Some(format!("{}/default", KEYMINT_SERVICE_NAME))
196 } else {
197 None
198 }
199 }
200 SecurityLevel::STRONGBOX => {
201 if keymint_instances.as_vec()?.iter().any(|instance| *instance == "strongbox") {
202 Some(format!("{}/strongbox", KEYMINT_SERVICE_NAME))
203 } else {
204 None
205 }
206 }
Janis Danisevskisba998992020-12-29 16:08:40 -0800207 _ => {
208 return Err(Error::Km(ErrorCode::HARDWARE_TYPE_UNAVAILABLE))
209 .context("In connect_keymint.")
210 }
211 };
212
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800213 let keymint = if let Some(service_name) = service_name {
214 map_binder_status_code(binder::get_interface(&service_name))
215 .context("In connect_keymint: Trying to connect to genuine KeyMint service.")
216 } else {
217 // This is a no-op if it was called before.
218 keystore2_km_compat::add_keymint_device_service();
Janis Danisevskis8c6378e2021-01-01 09:30:37 -0800219
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800220 let keystore_compat_service: Strong<dyn IKeystoreCompatService> =
221 map_binder_status_code(binder::get_interface("android.security.compat"))
222 .context("In connect_keymint: Trying to connect to compat service.")?;
223 map_binder_status(keystore_compat_service.getKeyMintDevice(*security_level))
224 .map_err(|e| match e {
225 Error::BinderTransaction(StatusCode::NAME_NOT_FOUND) => {
226 Error::Km(ErrorCode::HARDWARE_TYPE_UNAVAILABLE)
Janis Danisevskis8c6378e2021-01-01 09:30:37 -0800227 }
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800228 e => e,
229 })
230 .context("In connect_keymint: Trying to get Legacy wrapper.")
231 }?;
Janis Danisevskisba998992020-12-29 16:08:40 -0800232
Janis Danisevskis2ee014b2021-05-05 14:29:08 -0700233 let wp = wd::watch_millis("In connect_keymint: calling getHardwareInfo()", 500);
Max Bires8e93d2b2021-01-14 13:17:59 -0800234 let hw_info = map_km_error(keymint.getHardwareInfo())
235 .context("In connect_keymint: Failed to get hardware info.")?;
Janis Danisevskis2ee014b2021-05-05 14:29:08 -0700236 drop(wp);
Max Bires8e93d2b2021-01-14 13:17:59 -0800237
238 Ok((Asp::new(keymint.as_binder()), hw_info))
Janis Danisevskisba998992020-12-29 16:08:40 -0800239}
240
241/// Get a keymint device for the given security level either from our cache or
Max Bires8e93d2b2021-01-14 13:17:59 -0800242/// by making a new connection. Returns the device, the hardware info and the uuid.
243/// TODO the latter can be removed when the uuid is part of the hardware info.
244pub fn get_keymint_device(
245 security_level: &SecurityLevel,
246) -> Result<(Asp, KeyMintHardwareInfo, Uuid)> {
Janis Danisevskisba998992020-12-29 16:08:40 -0800247 let mut devices_map = KEY_MINT_DEVICES.lock().unwrap();
Max Bires8e93d2b2021-01-14 13:17:59 -0800248 if let Some((dev, hw_info, uuid)) = devices_map.dev_by_sec_level(&security_level) {
249 Ok((dev, hw_info, uuid))
Janis Danisevskisba998992020-12-29 16:08:40 -0800250 } else {
Max Bires8e93d2b2021-01-14 13:17:59 -0800251 let (dev, hw_info) = connect_keymint(security_level).context("In get_keymint_device.")?;
252 devices_map.insert(*security_level, dev, hw_info);
253 // Unwrap must succeed because we just inserted it.
254 Ok(devices_map.dev_by_sec_level(security_level).unwrap())
255 }
256}
257
258/// Get a keymint device for the given uuid. This will only access the cache, but will not
259/// attempt to establish a new connection. It is assumed that the cache is already populated
260/// when this is called. This is a fair assumption, because service.rs iterates through all
261/// security levels when it gets instantiated.
262pub fn get_keymint_dev_by_uuid(uuid: &Uuid) -> Result<(Asp, KeyMintHardwareInfo)> {
263 let devices_map = KEY_MINT_DEVICES.lock().unwrap();
264 if let Some((dev, hw_info, _)) = devices_map.dev_by_uuid(uuid) {
265 Ok((dev, hw_info))
266 } else {
267 Err(Error::sys()).context("In get_keymint_dev_by_uuid: No KeyMint instance found.")
Janis Danisevskisba998992020-12-29 16:08:40 -0800268 }
Janis Danisevskisb42fc182020-12-15 08:41:27 -0800269}
Janis Danisevskisc3a496b2021-01-05 10:37:22 -0800270
David Drysdale0e45a612021-02-25 17:24:36 +0000271/// Return all known keymint devices.
272pub fn get_keymint_devices() -> Vec<Strong<dyn IKeyMintDevice>> {
273 KEY_MINT_DEVICES.lock().unwrap().devices()
274}
275
Janis Danisevskisc3a496b2021-01-05 10:37:22 -0800276static TIME_STAMP_SERVICE_NAME: &str = "android.hardware.security.secureclock.ISecureClock";
277
278/// Make a new connection to a secure clock service.
279/// If no native SecureClock device can be found brings up the compatibility service and attempts
280/// to connect to the legacy wrapper.
281fn connect_secureclock() -> Result<Asp> {
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800282 let secureclock_instances =
283 get_aidl_instances("android.hardware.security.secureclock", 1, "ISecureClock");
Janis Danisevskisc3a496b2021-01-05 10:37:22 -0800284
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800285 let secure_clock_available =
286 secureclock_instances.as_vec()?.iter().any(|instance| *instance == "default");
Janis Danisevskisc3a496b2021-01-05 10:37:22 -0800287
Max Bires130e51b2021-04-05 14:07:20 -0700288 let default_time_stamp_service_name = format!("{}/default", TIME_STAMP_SERVICE_NAME);
289
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800290 let secureclock = if secure_clock_available {
Max Bires130e51b2021-04-05 14:07:20 -0700291 map_binder_status_code(binder::get_interface(&default_time_stamp_service_name))
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800292 .context("In connect_secureclock: Trying to connect to genuine secure clock service.")
293 } else {
294 // This is a no-op if it was called before.
295 keystore2_km_compat::add_keymint_device_service();
296
297 let keystore_compat_service: Strong<dyn IKeystoreCompatService> =
298 map_binder_status_code(binder::get_interface("android.security.compat"))
299 .context("In connect_secureclock: Trying to connect to compat service.")?;
300
301 // Legacy secure clock services were only implemented by TEE.
302 map_binder_status(keystore_compat_service.getSecureClock())
303 .map_err(|e| match e {
304 Error::BinderTransaction(StatusCode::NAME_NOT_FOUND) => {
305 Error::Km(ErrorCode::HARDWARE_TYPE_UNAVAILABLE)
Janis Danisevskisc3a496b2021-01-05 10:37:22 -0800306 }
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800307 e => e,
308 })
309 .context("In connect_secureclock: Trying to get Legacy wrapper.")
310 }?;
Janis Danisevskisc3a496b2021-01-05 10:37:22 -0800311
312 Ok(Asp::new(secureclock.as_binder()))
313}
314
315/// Get the timestamp service that verifies auth token timeliness towards security levels with
316/// different clocks.
317pub fn get_timestamp_service() -> Result<Asp> {
318 let mut ts_device = TIME_STAMP_DEVICE.lock().unwrap();
319 if let Some(dev) = &*ts_device {
320 Ok(dev.clone())
321 } else {
322 let dev = connect_secureclock().context("In get_timestamp_service.")?;
323 *ts_device = Some(dev.clone());
324 Ok(dev)
325 }
326}
Max Biresb2e1d032021-02-08 21:35:05 -0800327
328static REMOTE_PROVISIONING_HAL_SERVICE_NAME: &str =
329 "android.hardware.security.keymint.IRemotelyProvisionedComponent";
330
331fn connect_remotely_provisioned_component(security_level: &SecurityLevel) -> Result<Asp> {
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800332 let remotely_prov_instances =
333 get_aidl_instances("android.hardware.security.keymint", 1, "IRemotelyProvisionedComponent");
334
Max Biresb2e1d032021-02-08 21:35:05 -0800335 let service_name = match *security_level {
336 SecurityLevel::TRUSTED_ENVIRONMENT => {
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800337 if remotely_prov_instances.as_vec()?.iter().any(|instance| *instance == "default") {
338 Some(format!("{}/default", REMOTE_PROVISIONING_HAL_SERVICE_NAME))
339 } else {
340 None
341 }
Max Biresb2e1d032021-02-08 21:35:05 -0800342 }
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800343 SecurityLevel::STRONGBOX => {
344 if remotely_prov_instances.as_vec()?.iter().any(|instance| *instance == "strongbox") {
345 Some(format!("{}/strongbox", REMOTE_PROVISIONING_HAL_SERVICE_NAME))
346 } else {
347 None
348 }
Max Biresb2e1d032021-02-08 21:35:05 -0800349 }
Janis Danisevskisef14e1a2021-02-23 23:16:55 -0800350 _ => None,
351 }
352 .ok_or(Error::Km(ErrorCode::HARDWARE_TYPE_UNAVAILABLE))
353 .context("In connect_remotely_provisioned_component.")?;
Max Biresb2e1d032021-02-08 21:35:05 -0800354
355 let rem_prov_hal: Strong<dyn IRemotelyProvisionedComponent> =
356 map_binder_status_code(binder::get_interface(&service_name))
357 .context(concat!(
358 "In connect_remotely_provisioned_component: Trying to connect to",
359 " RemotelyProvisionedComponent service."
360 ))
361 .map_err(|e| e)?;
362 Ok(Asp::new(rem_prov_hal.as_binder()))
363}
364
365/// Get a remote provisiong component device for the given security level either from the cache or
366/// by making a new connection. Returns the device.
367pub fn get_remotely_provisioned_component(security_level: &SecurityLevel) -> Result<Asp> {
368 let mut devices_map = REMOTELY_PROVISIONED_COMPONENT_DEVICES.lock().unwrap();
369 if let Some(dev) = devices_map.dev_by_sec_level(&security_level) {
370 Ok(dev)
371 } else {
372 let dev = connect_remotely_provisioned_component(security_level)
373 .context("In get_remotely_provisioned_component.")?;
374 devices_map.insert(*security_level, dev);
375 // Unwrap must succeed because we just inserted it.
376 Ok(devices_map.dev_by_sec_level(security_level).unwrap())
377 }
378}