| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 1 | // 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 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 15 | //! Implements the android.security.legacykeystore interface. | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 16 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 17 | use android_security_legacykeystore::aidl::android::security::legacykeystore::{ | 
|  | 18 | ILegacyKeystore::BnLegacyKeystore, ILegacyKeystore::ILegacyKeystore, | 
|  | 19 | ILegacyKeystore::ERROR_ENTRY_NOT_FOUND, ILegacyKeystore::ERROR_PERMISSION_DENIED, | 
|  | 20 | ILegacyKeystore::ERROR_SYSTEM_ERROR, ILegacyKeystore::UID_SELF, | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 21 | }; | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 22 | use android_security_legacykeystore::binder::{ | 
| Andrew Walbran | de45c8b | 2021-04-13 14:42:38 +0000 | [diff] [blame] | 23 | BinderFeatures, ExceptionCode, Result as BinderResult, Status as BinderStatus, Strong, | 
|  | 24 | ThreadState, | 
|  | 25 | }; | 
| Seth Moore | fbe5cf5 | 2021-06-09 15:59:00 -0700 | [diff] [blame] | 26 | use anyhow::{Context, Result}; | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 27 | use keystore2::{ | 
| Janis Danisevskis | f84d0b0 | 2022-01-26 14:11:14 -0800 | [diff] [blame] | 28 | async_task::AsyncTask, error::anyhow_error_to_cstring, globals::SUPER_KEY, | 
|  | 29 | legacy_blob::LegacyBlobLoader, maintenance::DeleteListener, maintenance::Domain, | 
|  | 30 | utils::uid_to_android_user, utils::watchdog as wd, | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 31 | }; | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 32 | use rusqlite::{ | 
|  | 33 | params, Connection, OptionalExtension, Transaction, TransactionBehavior, NO_PARAMS, | 
|  | 34 | }; | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 35 | use std::sync::Arc; | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 36 | use std::{ | 
|  | 37 | collections::HashSet, | 
|  | 38 | path::{Path, PathBuf}, | 
|  | 39 | }; | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 40 |  | 
|  | 41 | struct DB { | 
|  | 42 | conn: Connection, | 
|  | 43 | } | 
|  | 44 |  | 
|  | 45 | impl DB { | 
|  | 46 | fn new(db_file: &Path) -> Result<Self> { | 
|  | 47 | let mut db = Self { | 
|  | 48 | conn: Connection::open(db_file).context("Failed to initialize SQLite connection.")?, | 
|  | 49 | }; | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 50 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 51 | db.init_tables().context("Trying to initialize legacy keystore db.")?; | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 52 | Ok(db) | 
|  | 53 | } | 
|  | 54 |  | 
|  | 55 | fn with_transaction<T, F>(&mut self, behavior: TransactionBehavior, f: F) -> Result<T> | 
|  | 56 | where | 
|  | 57 | F: Fn(&Transaction) -> Result<T>, | 
|  | 58 | { | 
|  | 59 | loop { | 
|  | 60 | match self | 
|  | 61 | .conn | 
|  | 62 | .transaction_with_behavior(behavior) | 
|  | 63 | .context("In with_transaction.") | 
|  | 64 | .and_then(|tx| f(&tx).map(|result| (result, tx))) | 
|  | 65 | .and_then(|(result, tx)| { | 
|  | 66 | tx.commit().context("In with_transaction: Failed to commit transaction.")?; | 
|  | 67 | Ok(result) | 
|  | 68 | }) { | 
|  | 69 | Ok(result) => break Ok(result), | 
|  | 70 | Err(e) => { | 
|  | 71 | if Self::is_locked_error(&e) { | 
|  | 72 | std::thread::sleep(std::time::Duration::from_micros(500)); | 
|  | 73 | continue; | 
|  | 74 | } else { | 
|  | 75 | return Err(e).context("In with_transaction."); | 
|  | 76 | } | 
|  | 77 | } | 
|  | 78 | } | 
|  | 79 | } | 
|  | 80 | } | 
|  | 81 |  | 
|  | 82 | fn is_locked_error(e: &anyhow::Error) -> bool { | 
| Janis Danisevskis | 13f0915 | 2021-04-19 09:55:15 -0700 | [diff] [blame] | 83 | matches!( | 
|  | 84 | e.root_cause().downcast_ref::<rusqlite::ffi::Error>(), | 
|  | 85 | Some(rusqlite::ffi::Error { code: rusqlite::ErrorCode::DatabaseBusy, .. }) | 
|  | 86 | | Some(rusqlite::ffi::Error { code: rusqlite::ErrorCode::DatabaseLocked, .. }) | 
|  | 87 | ) | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 88 | } | 
|  | 89 |  | 
|  | 90 | fn init_tables(&mut self) -> Result<()> { | 
|  | 91 | self.with_transaction(TransactionBehavior::Immediate, |tx| { | 
|  | 92 | tx.execute( | 
|  | 93 | "CREATE TABLE IF NOT EXISTS profiles ( | 
|  | 94 | owner INTEGER, | 
|  | 95 | alias BLOB, | 
|  | 96 | profile BLOB, | 
|  | 97 | UNIQUE(owner, alias));", | 
|  | 98 | NO_PARAMS, | 
|  | 99 | ) | 
|  | 100 | .context("Failed to initialize \"profiles\" table.")?; | 
|  | 101 | Ok(()) | 
|  | 102 | }) | 
|  | 103 | } | 
|  | 104 |  | 
|  | 105 | fn list(&mut self, caller_uid: u32) -> Result<Vec<String>> { | 
|  | 106 | self.with_transaction(TransactionBehavior::Deferred, |tx| { | 
|  | 107 | let mut stmt = tx | 
|  | 108 | .prepare("SELECT alias FROM profiles WHERE owner = ? ORDER BY alias ASC;") | 
|  | 109 | .context("In list: Failed to prepare statement.")?; | 
|  | 110 |  | 
| Chris Wailes | 263de9f | 2022-08-11 15:00:51 -0700 | [diff] [blame] | 111 | // This allow is necessary to avoid the following error: | 
|  | 112 | // | 
|  | 113 | // error[E0597]: `stmt` does not live long enough | 
|  | 114 | // | 
|  | 115 | // See: https://github.com/rust-lang/rust-clippy/issues/8114 | 
|  | 116 | #[allow(clippy::let_and_return)] | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 117 | let aliases = stmt | 
|  | 118 | .query_map(params![caller_uid], |row| row.get(0))? | 
|  | 119 | .collect::<rusqlite::Result<Vec<String>>>() | 
|  | 120 | .context("In list: query_map failed."); | 
|  | 121 | aliases | 
|  | 122 | }) | 
|  | 123 | } | 
|  | 124 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 125 | fn put(&mut self, caller_uid: u32, alias: &str, entry: &[u8]) -> Result<()> { | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 126 | self.with_transaction(TransactionBehavior::Immediate, |tx| { | 
|  | 127 | tx.execute( | 
|  | 128 | "INSERT OR REPLACE INTO profiles (owner, alias, profile) values (?, ?, ?)", | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 129 | params![caller_uid, alias, entry,], | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 130 | ) | 
|  | 131 | .context("In put: Failed to insert or replace.")?; | 
|  | 132 | Ok(()) | 
|  | 133 | }) | 
|  | 134 | } | 
|  | 135 |  | 
|  | 136 | fn get(&mut self, caller_uid: u32, alias: &str) -> Result<Option<Vec<u8>>> { | 
|  | 137 | self.with_transaction(TransactionBehavior::Deferred, |tx| { | 
|  | 138 | tx.query_row( | 
|  | 139 | "SELECT profile FROM profiles WHERE owner = ? AND alias = ?;", | 
|  | 140 | params![caller_uid, alias], | 
|  | 141 | |row| row.get(0), | 
|  | 142 | ) | 
|  | 143 | .optional() | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 144 | .context("In get: failed loading entry.") | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 145 | }) | 
|  | 146 | } | 
|  | 147 |  | 
|  | 148 | fn remove(&mut self, caller_uid: u32, alias: &str) -> Result<bool> { | 
|  | 149 | let removed = self.with_transaction(TransactionBehavior::Immediate, |tx| { | 
|  | 150 | tx.execute( | 
|  | 151 | "DELETE FROM profiles WHERE owner = ? AND alias = ?;", | 
|  | 152 | params![caller_uid, alias], | 
|  | 153 | ) | 
|  | 154 | .context("In remove: Failed to delete row.") | 
|  | 155 | })?; | 
|  | 156 | Ok(removed == 1) | 
|  | 157 | } | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 158 |  | 
|  | 159 | fn remove_uid(&mut self, uid: u32) -> Result<()> { | 
|  | 160 | self.with_transaction(TransactionBehavior::Immediate, |tx| { | 
|  | 161 | tx.execute("DELETE FROM profiles WHERE owner = ?;", params![uid]) | 
|  | 162 | .context("In remove_uid: Failed to delete.") | 
|  | 163 | })?; | 
|  | 164 | Ok(()) | 
|  | 165 | } | 
|  | 166 |  | 
|  | 167 | fn remove_user(&mut self, user_id: u32) -> Result<()> { | 
|  | 168 | self.with_transaction(TransactionBehavior::Immediate, |tx| { | 
|  | 169 | tx.execute( | 
|  | 170 | "DELETE FROM profiles WHERE cast ( ( owner/? ) as int) = ?;", | 
| Joel Galenson | 81a50f2 | 2021-07-29 15:39:10 -0700 | [diff] [blame] | 171 | params![rustutils::users::AID_USER_OFFSET, user_id], | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 172 | ) | 
|  | 173 | .context("In remove_uid: Failed to delete.") | 
|  | 174 | })?; | 
|  | 175 | Ok(()) | 
|  | 176 | } | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 177 | } | 
|  | 178 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 179 | /// This is the main LegacyKeystore error type, it wraps binder exceptions and the | 
|  | 180 | /// LegacyKeystore errors. | 
| Chris Wailes | 263de9f | 2022-08-11 15:00:51 -0700 | [diff] [blame] | 181 | #[derive(Debug, thiserror::Error, PartialEq, Eq)] | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 182 | pub enum Error { | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 183 | /// Wraps a LegacyKeystore error code. | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 184 | #[error("Error::Error({0:?})")] | 
|  | 185 | Error(i32), | 
|  | 186 | /// Wraps a Binder exception code other than a service specific exception. | 
|  | 187 | #[error("Binder exception code {0:?}, {1:?}")] | 
|  | 188 | Binder(ExceptionCode, i32), | 
|  | 189 | } | 
|  | 190 |  | 
|  | 191 | impl Error { | 
|  | 192 | /// Short hand for `Error::Error(ERROR_SYSTEM_ERROR)` | 
|  | 193 | pub fn sys() -> Self { | 
|  | 194 | Error::Error(ERROR_SYSTEM_ERROR) | 
|  | 195 | } | 
|  | 196 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 197 | /// Short hand for `Error::Error(ERROR_ENTRY_NOT_FOUND)` | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 198 | pub fn not_found() -> Self { | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 199 | Error::Error(ERROR_ENTRY_NOT_FOUND) | 
|  | 200 | } | 
|  | 201 |  | 
|  | 202 | /// Short hand for `Error::Error(ERROR_PERMISSION_DENIED)` | 
|  | 203 | pub fn perm() -> Self { | 
|  | 204 | Error::Error(ERROR_PERMISSION_DENIED) | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 205 | } | 
|  | 206 | } | 
|  | 207 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 208 | /// This function should be used by legacykeystore service calls to translate error conditions | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 209 | /// into service specific exceptions. | 
|  | 210 | /// | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 211 | /// All error conditions get logged by this function, except for ERROR_ENTRY_NOT_FOUND error. | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 212 | /// | 
|  | 213 | /// `Error::Error(x)` variants get mapped onto a service specific error code of `x`. | 
|  | 214 | /// | 
|  | 215 | /// All non `Error` error conditions get mapped onto `ERROR_SYSTEM_ERROR`. | 
|  | 216 | /// | 
|  | 217 | /// `handle_ok` will be called if `result` is `Ok(value)` where `value` will be passed | 
|  | 218 | /// as argument to `handle_ok`. `handle_ok` must generate a `BinderResult<T>`, but it | 
|  | 219 | /// typically returns Ok(value). | 
|  | 220 | fn map_or_log_err<T, U, F>(result: Result<U>, handle_ok: F) -> BinderResult<T> | 
|  | 221 | where | 
|  | 222 | F: FnOnce(U) -> BinderResult<T>, | 
|  | 223 | { | 
|  | 224 | result.map_or_else( | 
|  | 225 | |e| { | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 226 | let root_cause = e.root_cause(); | 
| Hasini Gunasinghe | e1d1bbd | 2021-04-20 18:13:25 +0000 | [diff] [blame] | 227 | let (rc, log_error) = match root_cause.downcast_ref::<Error>() { | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 228 | // Make the entry not found errors silent. | 
|  | 229 | Some(Error::Error(ERROR_ENTRY_NOT_FOUND)) => (ERROR_ENTRY_NOT_FOUND, false), | 
| Hasini Gunasinghe | e1d1bbd | 2021-04-20 18:13:25 +0000 | [diff] [blame] | 230 | Some(Error::Error(e)) => (*e, true), | 
|  | 231 | Some(Error::Binder(_, _)) | None => (ERROR_SYSTEM_ERROR, true), | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 232 | }; | 
| Hasini Gunasinghe | e1d1bbd | 2021-04-20 18:13:25 +0000 | [diff] [blame] | 233 | if log_error { | 
|  | 234 | log::error!("{:?}", e); | 
|  | 235 | } | 
| Janis Danisevskis | ea03cff | 2021-12-16 08:10:17 -0800 | [diff] [blame] | 236 | Err(BinderStatus::new_service_specific_error( | 
|  | 237 | rc, | 
|  | 238 | anyhow_error_to_cstring(&e).as_deref(), | 
|  | 239 | )) | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 240 | }, | 
|  | 241 | handle_ok, | 
|  | 242 | ) | 
|  | 243 | } | 
|  | 244 |  | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 245 | struct LegacyKeystoreDeleteListener { | 
|  | 246 | legacy_keystore: Arc<LegacyKeystore>, | 
|  | 247 | } | 
|  | 248 |  | 
|  | 249 | impl DeleteListener for LegacyKeystoreDeleteListener { | 
|  | 250 | fn delete_namespace(&self, domain: Domain, namespace: i64) -> Result<()> { | 
|  | 251 | self.legacy_keystore.delete_namespace(domain, namespace) | 
|  | 252 | } | 
|  | 253 | fn delete_user(&self, user_id: u32) -> Result<()> { | 
|  | 254 | self.legacy_keystore.delete_user(user_id) | 
|  | 255 | } | 
|  | 256 | } | 
|  | 257 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 258 | /// Implements ILegacyKeystore AIDL interface. | 
|  | 259 | pub struct LegacyKeystore { | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 260 | db_path: PathBuf, | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 261 | async_task: AsyncTask, | 
|  | 262 | } | 
|  | 263 |  | 
|  | 264 | struct AsyncState { | 
|  | 265 | recently_imported: HashSet<(u32, String)>, | 
|  | 266 | legacy_loader: LegacyBlobLoader, | 
|  | 267 | db_path: PathBuf, | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 268 | } | 
|  | 269 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 270 | impl LegacyKeystore { | 
|  | 271 | /// Note: The filename was chosen before the purpose of this module was extended. | 
|  | 272 | ///       It is kept for backward compatibility with early adopters. | 
|  | 273 | const LEGACY_KEYSTORE_FILE_NAME: &'static str = "vpnprofilestore.sqlite"; | 
|  | 274 |  | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 275 | const WIFI_NAMESPACE: i64 = 102; | 
|  | 276 | const AID_WIFI: u32 = 1010; | 
|  | 277 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 278 | /// Creates a new LegacyKeystore instance. | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 279 | pub fn new_native_binder( | 
|  | 280 | path: &Path, | 
|  | 281 | ) -> (Box<dyn DeleteListener + Send + Sync + 'static>, Strong<dyn ILegacyKeystore>) { | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 282 | let mut db_path = path.to_path_buf(); | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 283 | db_path.push(Self::LEGACY_KEYSTORE_FILE_NAME); | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 284 |  | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 285 | let legacy_keystore = Arc::new(Self { db_path, async_task: Default::default() }); | 
|  | 286 | legacy_keystore.init_shelf(path); | 
|  | 287 | let service = LegacyKeystoreService { legacy_keystore: legacy_keystore.clone() }; | 
|  | 288 | ( | 
|  | 289 | Box::new(LegacyKeystoreDeleteListener { legacy_keystore }), | 
|  | 290 | BnLegacyKeystore::new_binder(service, BinderFeatures::default()), | 
|  | 291 | ) | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 292 | } | 
|  | 293 |  | 
|  | 294 | fn open_db(&self) -> Result<DB> { | 
|  | 295 | DB::new(&self.db_path).context("In open_db: Failed to open db.") | 
|  | 296 | } | 
|  | 297 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 298 | fn get_effective_uid(uid: i32) -> Result<u32> { | 
|  | 299 | const AID_SYSTEM: u32 = 1000; | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 300 | let calling_uid = ThreadState::get_calling_uid(); | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 301 | let uid = uid as u32; | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 302 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 303 | if uid == UID_SELF as u32 || uid == calling_uid { | 
|  | 304 | Ok(calling_uid) | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 305 | } else if calling_uid == AID_SYSTEM && uid == Self::AID_WIFI { | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 306 | // The only exception for legacy reasons is allowing SYSTEM to access | 
|  | 307 | // the WIFI namespace. | 
|  | 308 | // IMPORTANT: If you attempt to add more exceptions, it means you are adding | 
|  | 309 | // more callers to this deprecated feature. DON'T! | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 310 | Ok(Self::AID_WIFI) | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 311 | } else { | 
|  | 312 | Err(Error::perm()).with_context(|| { | 
|  | 313 | format!("In get_effective_uid: caller: {}, requested uid: {}.", calling_uid, uid) | 
|  | 314 | }) | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 315 | } | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 316 | } | 
|  | 317 |  | 
|  | 318 | fn get(&self, alias: &str, uid: i32) -> Result<Vec<u8>> { | 
|  | 319 | let mut db = self.open_db().context("In get.")?; | 
|  | 320 | let uid = Self::get_effective_uid(uid).context("In get.")?; | 
|  | 321 |  | 
|  | 322 | if let Some(entry) = db.get(uid, alias).context("In get: Trying to load entry from DB.")? { | 
|  | 323 | return Ok(entry); | 
|  | 324 | } | 
| Janis Danisevskis | f84d0b0 | 2022-01-26 14:11:14 -0800 | [diff] [blame] | 325 | if self.get_legacy(uid, alias).context("In get: Trying to import legacy blob.")? { | 
|  | 326 | // If we were able to import a legacy blob try again. | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 327 | if let Some(entry) = | 
|  | 328 | db.get(uid, alias).context("In get: Trying to load entry from DB.")? | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 329 | { | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 330 | return Ok(entry); | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 331 | } | 
|  | 332 | } | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 333 | Err(Error::not_found()).context("In get: No such entry.") | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 334 | } | 
|  | 335 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 336 | fn put(&self, alias: &str, uid: i32, entry: &[u8]) -> Result<()> { | 
|  | 337 | let uid = Self::get_effective_uid(uid).context("In put.")?; | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 338 | let mut db = self.open_db().context("In put.")?; | 
| Janis Danisevskis | f84d0b0 | 2022-01-26 14:11:14 -0800 | [diff] [blame] | 339 | db.put(uid, alias, entry).context("In put: Trying to insert entry into DB.")?; | 
|  | 340 | // When replacing an entry, make sure that there is no stale legacy file entry. | 
|  | 341 | let _ = self.remove_legacy(uid, alias); | 
|  | 342 | Ok(()) | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 343 | } | 
|  | 344 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 345 | fn remove(&self, alias: &str, uid: i32) -> Result<()> { | 
|  | 346 | let uid = Self::get_effective_uid(uid).context("In remove.")?; | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 347 | let mut db = self.open_db().context("In remove.")?; | 
| Janis Danisevskis | f84d0b0 | 2022-01-26 14:11:14 -0800 | [diff] [blame] | 348 |  | 
|  | 349 | if self.remove_legacy(uid, alias).context("In remove: trying to remove legacy entry")? { | 
|  | 350 | return Ok(()); | 
|  | 351 | } | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 352 | let removed = | 
|  | 353 | db.remove(uid, alias).context("In remove: Trying to remove entry from DB.")?; | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 354 | if removed { | 
|  | 355 | Ok(()) | 
|  | 356 | } else { | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 357 | Err(Error::not_found()).context("In remove: No such entry.") | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 358 | } | 
|  | 359 | } | 
|  | 360 |  | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 361 | fn delete_namespace(&self, domain: Domain, namespace: i64) -> Result<()> { | 
|  | 362 | let uid = match domain { | 
|  | 363 | Domain::APP => namespace as u32, | 
|  | 364 | Domain::SELINUX => { | 
|  | 365 | if namespace == Self::WIFI_NAMESPACE { | 
|  | 366 | // Namespace WIFI gets mapped to AID_WIFI. | 
|  | 367 | Self::AID_WIFI | 
|  | 368 | } else { | 
|  | 369 | // Nothing to do for any other namespace. | 
|  | 370 | return Ok(()); | 
|  | 371 | } | 
|  | 372 | } | 
|  | 373 | _ => return Ok(()), | 
|  | 374 | }; | 
|  | 375 |  | 
|  | 376 | if let Err(e) = self.bulk_delete_uid(uid) { | 
|  | 377 | log::warn!("In LegacyKeystore::delete_namespace: {:?}", e); | 
|  | 378 | } | 
|  | 379 | let mut db = self.open_db().context("In LegacyKeystore::delete_namespace.")?; | 
|  | 380 | db.remove_uid(uid).context("In LegacyKeystore::delete_namespace.") | 
|  | 381 | } | 
|  | 382 |  | 
|  | 383 | fn delete_user(&self, user_id: u32) -> Result<()> { | 
|  | 384 | if let Err(e) = self.bulk_delete_user(user_id) { | 
|  | 385 | log::warn!("In LegacyKeystore::delete_user: {:?}", e); | 
|  | 386 | } | 
|  | 387 | let mut db = self.open_db().context("In LegacyKeystore::delete_user.")?; | 
|  | 388 | db.remove_user(user_id).context("In LegacyKeystore::delete_user.") | 
|  | 389 | } | 
|  | 390 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 391 | fn list(&self, prefix: &str, uid: i32) -> Result<Vec<String>> { | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 392 | let mut db = self.open_db().context("In list.")?; | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 393 | let uid = Self::get_effective_uid(uid).context("In list.")?; | 
|  | 394 | let mut result = self.list_legacy(uid).context("In list.")?; | 
|  | 395 | result.append(&mut db.list(uid).context("In list: Trying to get list of entries.")?); | 
| Charisee | 28e6f0b | 2022-09-15 01:07:46 +0000 | [diff] [blame] | 396 | result.retain(|s| s.starts_with(prefix)); | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 397 | result.sort_unstable(); | 
|  | 398 | result.dedup(); | 
|  | 399 | Ok(result) | 
|  | 400 | } | 
|  | 401 |  | 
|  | 402 | fn init_shelf(&self, path: &Path) { | 
|  | 403 | let mut db_path = path.to_path_buf(); | 
|  | 404 | self.async_task.queue_hi(move |shelf| { | 
|  | 405 | let legacy_loader = LegacyBlobLoader::new(&db_path); | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 406 | db_path.push(Self::LEGACY_KEYSTORE_FILE_NAME); | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 407 |  | 
|  | 408 | shelf.put(AsyncState { legacy_loader, db_path, recently_imported: Default::default() }); | 
|  | 409 | }) | 
|  | 410 | } | 
|  | 411 |  | 
|  | 412 | fn do_serialized<F, T: Send + 'static>(&self, f: F) -> Result<T> | 
|  | 413 | where | 
|  | 414 | F: FnOnce(&mut AsyncState) -> Result<T> + Send + 'static, | 
|  | 415 | { | 
|  | 416 | let (sender, receiver) = std::sync::mpsc::channel::<Result<T>>(); | 
|  | 417 | self.async_task.queue_hi(move |shelf| { | 
|  | 418 | let state = shelf.get_downcast_mut::<AsyncState>().expect("Failed to get shelf."); | 
|  | 419 | sender.send(f(state)).expect("Failed to send result."); | 
|  | 420 | }); | 
|  | 421 | receiver.recv().context("In do_serialized: Failed to receive result.")? | 
|  | 422 | } | 
|  | 423 |  | 
|  | 424 | fn list_legacy(&self, uid: u32) -> Result<Vec<String>> { | 
|  | 425 | self.do_serialized(move |state| { | 
|  | 426 | state | 
|  | 427 | .legacy_loader | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 428 | .list_legacy_keystore_entries_for_uid(uid) | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 429 | .context("Trying to list legacy keystore entries.") | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 430 | }) | 
|  | 431 | .context("In list_legacy.") | 
|  | 432 | } | 
|  | 433 |  | 
|  | 434 | fn get_legacy(&self, uid: u32, alias: &str) -> Result<bool> { | 
|  | 435 | let alias = alias.to_string(); | 
|  | 436 | self.do_serialized(move |state| { | 
|  | 437 | if state.recently_imported.contains(&(uid, alias.clone())) { | 
|  | 438 | return Ok(true); | 
|  | 439 | } | 
|  | 440 | let mut db = DB::new(&state.db_path).context("In open_db: Failed to open db.")?; | 
| Janis Danisevskis | f84d0b0 | 2022-01-26 14:11:14 -0800 | [diff] [blame] | 441 | let imported = | 
|  | 442 | Self::import_one_legacy_entry(uid, &alias, &state.legacy_loader, &mut db) | 
|  | 443 | .context("Trying to import legacy keystore entries.")?; | 
|  | 444 | if imported { | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 445 | state.recently_imported.insert((uid, alias)); | 
|  | 446 | } | 
| Janis Danisevskis | f84d0b0 | 2022-01-26 14:11:14 -0800 | [diff] [blame] | 447 | Ok(imported) | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 448 | }) | 
|  | 449 | .context("In get_legacy.") | 
|  | 450 | } | 
|  | 451 |  | 
| Janis Danisevskis | f84d0b0 | 2022-01-26 14:11:14 -0800 | [diff] [blame] | 452 | fn remove_legacy(&self, uid: u32, alias: &str) -> Result<bool> { | 
|  | 453 | let alias = alias.to_string(); | 
|  | 454 | self.do_serialized(move |state| { | 
|  | 455 | if state.recently_imported.contains(&(uid, alias.clone())) { | 
|  | 456 | return Ok(false); | 
|  | 457 | } | 
|  | 458 | state | 
|  | 459 | .legacy_loader | 
|  | 460 | .remove_legacy_keystore_entry(uid, &alias) | 
|  | 461 | .context("Trying to remove legacy entry.") | 
|  | 462 | }) | 
|  | 463 | } | 
|  | 464 |  | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 465 | fn bulk_delete_uid(&self, uid: u32) -> Result<()> { | 
|  | 466 | self.do_serialized(move |state| { | 
|  | 467 | let entries = state | 
|  | 468 | .legacy_loader | 
|  | 469 | .list_legacy_keystore_entries_for_uid(uid) | 
|  | 470 | .context("In bulk_delete_uid: Trying to list entries.")?; | 
|  | 471 | for alias in entries.iter() { | 
|  | 472 | if let Err(e) = state.legacy_loader.remove_legacy_keystore_entry(uid, alias) { | 
|  | 473 | log::warn!("In bulk_delete_uid: Failed to delete legacy entry. {:?}", e); | 
|  | 474 | } | 
|  | 475 | } | 
|  | 476 | Ok(()) | 
|  | 477 | }) | 
|  | 478 | } | 
|  | 479 |  | 
|  | 480 | fn bulk_delete_user(&self, user_id: u32) -> Result<()> { | 
|  | 481 | self.do_serialized(move |state| { | 
|  | 482 | let entries = state | 
|  | 483 | .legacy_loader | 
|  | 484 | .list_legacy_keystore_entries_for_user(user_id) | 
|  | 485 | .context("In bulk_delete_user: Trying to list entries.")?; | 
|  | 486 | for (uid, entries) in entries.iter() { | 
|  | 487 | for alias in entries.iter() { | 
|  | 488 | if let Err(e) = state.legacy_loader.remove_legacy_keystore_entry(*uid, alias) { | 
|  | 489 | log::warn!("In bulk_delete_user: Failed to delete legacy entry. {:?}", e); | 
|  | 490 | } | 
|  | 491 | } | 
|  | 492 | } | 
|  | 493 | Ok(()) | 
|  | 494 | }) | 
|  | 495 | } | 
|  | 496 |  | 
| Janis Danisevskis | f84d0b0 | 2022-01-26 14:11:14 -0800 | [diff] [blame] | 497 | fn import_one_legacy_entry( | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 498 | uid: u32, | 
|  | 499 | alias: &str, | 
|  | 500 | legacy_loader: &LegacyBlobLoader, | 
|  | 501 | db: &mut DB, | 
|  | 502 | ) -> Result<bool> { | 
|  | 503 | let blob = legacy_loader | 
| Janis Danisevskis | f84d0b0 | 2022-01-26 14:11:14 -0800 | [diff] [blame] | 504 | .read_legacy_keystore_entry(uid, alias, |ciphertext, iv, tag, _salt, _key_size| { | 
| Charisee | 03e0084 | 2023-01-25 01:41:23 +0000 | [diff] [blame] | 505 | if let Some(key) = | 
|  | 506 | SUPER_KEY.read().unwrap().get_per_boot_key_by_user_id(uid_to_android_user(uid)) | 
| Janis Danisevskis | f84d0b0 | 2022-01-26 14:11:14 -0800 | [diff] [blame] | 507 | { | 
|  | 508 | key.decrypt(ciphertext, iv, tag) | 
|  | 509 | } else { | 
|  | 510 | Err(Error::sys()).context("No key found for user. Device may be locked.") | 
|  | 511 | } | 
|  | 512 | }) | 
|  | 513 | .context("In import_one_legacy_entry: Trying to read legacy keystore entry.")?; | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 514 | if let Some(entry) = blob { | 
|  | 515 | db.put(uid, alias, &entry) | 
| Janis Danisevskis | f84d0b0 | 2022-01-26 14:11:14 -0800 | [diff] [blame] | 516 | .context("In import_one_legacy_entry: Trying to insert entry into DB.")?; | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 517 | legacy_loader | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 518 | .remove_legacy_keystore_entry(uid, alias) | 
| Janis Danisevskis | f84d0b0 | 2022-01-26 14:11:14 -0800 | [diff] [blame] | 519 | .context("In import_one_legacy_entry: Trying to delete legacy keystore entry.")?; | 
| Janis Danisevskis | 0689107 | 2021-02-11 10:28:17 -0800 | [diff] [blame] | 520 | Ok(true) | 
|  | 521 | } else { | 
|  | 522 | Ok(false) | 
|  | 523 | } | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 524 | } | 
|  | 525 | } | 
|  | 526 |  | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 527 | struct LegacyKeystoreService { | 
|  | 528 | legacy_keystore: Arc<LegacyKeystore>, | 
|  | 529 | } | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 530 |  | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 531 | impl binder::Interface for LegacyKeystoreService {} | 
|  | 532 |  | 
|  | 533 | impl ILegacyKeystore for LegacyKeystoreService { | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 534 | fn get(&self, alias: &str, uid: i32) -> BinderResult<Vec<u8>> { | 
|  | 535 | let _wp = wd::watch_millis("ILegacyKeystore::get", 500); | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 536 | map_or_log_err(self.legacy_keystore.get(alias, uid), Ok) | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 537 | } | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 538 | fn put(&self, alias: &str, uid: i32, entry: &[u8]) -> BinderResult<()> { | 
|  | 539 | let _wp = wd::watch_millis("ILegacyKeystore::put", 500); | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 540 | map_or_log_err(self.legacy_keystore.put(alias, uid, entry), Ok) | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 541 | } | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 542 | fn remove(&self, alias: &str, uid: i32) -> BinderResult<()> { | 
|  | 543 | let _wp = wd::watch_millis("ILegacyKeystore::remove", 500); | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 544 | map_or_log_err(self.legacy_keystore.remove(alias, uid), Ok) | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 545 | } | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 546 | fn list(&self, prefix: &str, uid: i32) -> BinderResult<Vec<String>> { | 
|  | 547 | let _wp = wd::watch_millis("ILegacyKeystore::list", 500); | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 548 | map_or_log_err(self.legacy_keystore.list(prefix, uid), Ok) | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 549 | } | 
|  | 550 | } | 
|  | 551 |  | 
|  | 552 | #[cfg(test)] | 
|  | 553 | mod db_test { | 
|  | 554 | use super::*; | 
|  | 555 | use keystore2_test_utils::TempDir; | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 556 | use std::sync::Arc; | 
|  | 557 | use std::thread; | 
|  | 558 | use std::time::Duration; | 
|  | 559 | use std::time::Instant; | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 560 |  | 
| Chris Wailes | d5aaaef | 2021-07-27 16:04:33 -0700 | [diff] [blame] | 561 | static TEST_ALIAS: &str = "test_alias"; | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 562 | static TEST_BLOB1: &[u8] = &[1, 2, 3, 4, 5, 6, 7, 8, 9, 0]; | 
|  | 563 | static TEST_BLOB2: &[u8] = &[2, 2, 3, 4, 5, 6, 7, 8, 9, 0]; | 
|  | 564 | static TEST_BLOB3: &[u8] = &[3, 2, 3, 4, 5, 6, 7, 8, 9, 0]; | 
|  | 565 | static TEST_BLOB4: &[u8] = &[3, 2, 3, 4, 5, 6, 7, 8, 9, 0]; | 
|  | 566 |  | 
|  | 567 | #[test] | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 568 | fn test_entry_db() { | 
|  | 569 | let test_dir = TempDir::new("entrydb_test_").expect("Failed to create temp dir."); | 
|  | 570 | let mut db = DB::new(&test_dir.build().push(LegacyKeystore::LEGACY_KEYSTORE_FILE_NAME)) | 
|  | 571 | .expect("Failed to open database."); | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 572 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 573 | // Insert three entries for owner 2. | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 574 | db.put(2, "test1", TEST_BLOB1).expect("Failed to insert test1."); | 
|  | 575 | db.put(2, "test2", TEST_BLOB2).expect("Failed to insert test2."); | 
|  | 576 | db.put(2, "test3", TEST_BLOB3).expect("Failed to insert test3."); | 
|  | 577 |  | 
|  | 578 | // Check list returns all inserted aliases. | 
|  | 579 | assert_eq!( | 
|  | 580 | vec!["test1".to_string(), "test2".to_string(), "test3".to_string(),], | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 581 | db.list(2).expect("Failed to list entries.") | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 582 | ); | 
|  | 583 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 584 | // There should be no entries for owner 1. | 
|  | 585 | assert_eq!(Vec::<String>::new(), db.list(1).expect("Failed to list entries.")); | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 586 |  | 
|  | 587 | // Check the content of the three entries. | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 588 | assert_eq!(Some(TEST_BLOB1), db.get(2, "test1").expect("Failed to get entry.").as_deref()); | 
|  | 589 | assert_eq!(Some(TEST_BLOB2), db.get(2, "test2").expect("Failed to get entry.").as_deref()); | 
|  | 590 | assert_eq!(Some(TEST_BLOB3), db.get(2, "test3").expect("Failed to get entry.").as_deref()); | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 591 |  | 
|  | 592 | // Remove test2 and check and check that it is no longer retrievable. | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 593 | assert!(db.remove(2, "test2").expect("Failed to remove entry.")); | 
|  | 594 | assert!(db.get(2, "test2").expect("Failed to get entry.").is_none()); | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 595 |  | 
|  | 596 | // test2 should now no longer be in the list. | 
|  | 597 | assert_eq!( | 
|  | 598 | vec!["test1".to_string(), "test3".to_string(),], | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 599 | db.list(2).expect("Failed to list entries.") | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 600 | ); | 
|  | 601 |  | 
|  | 602 | // Put on existing alias replaces it. | 
|  | 603 | // Verify test1 is TEST_BLOB1. | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 604 | assert_eq!(Some(TEST_BLOB1), db.get(2, "test1").expect("Failed to get entry.").as_deref()); | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 605 | db.put(2, "test1", TEST_BLOB4).expect("Failed to replace test1."); | 
|  | 606 | // Verify test1 is TEST_BLOB4. | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 607 | assert_eq!(Some(TEST_BLOB4), db.get(2, "test1").expect("Failed to get entry.").as_deref()); | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 608 | } | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 609 |  | 
|  | 610 | #[test] | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 611 | fn test_delete_uid() { | 
|  | 612 | let test_dir = TempDir::new("test_delete_uid_").expect("Failed to create temp dir."); | 
|  | 613 | let mut db = DB::new(&test_dir.build().push(LegacyKeystore::LEGACY_KEYSTORE_FILE_NAME)) | 
|  | 614 | .expect("Failed to open database."); | 
|  | 615 |  | 
|  | 616 | // Insert three entries for owner 2. | 
|  | 617 | db.put(2, "test1", TEST_BLOB1).expect("Failed to insert test1."); | 
|  | 618 | db.put(2, "test2", TEST_BLOB2).expect("Failed to insert test2."); | 
|  | 619 | db.put(3, "test3", TEST_BLOB3).expect("Failed to insert test3."); | 
|  | 620 |  | 
|  | 621 | db.remove_uid(2).expect("Failed to remove uid 2"); | 
|  | 622 |  | 
|  | 623 | assert_eq!(Vec::<String>::new(), db.list(2).expect("Failed to list entries.")); | 
|  | 624 |  | 
|  | 625 | assert_eq!(vec!["test3".to_string(),], db.list(3).expect("Failed to list entries.")); | 
|  | 626 | } | 
|  | 627 |  | 
|  | 628 | #[test] | 
|  | 629 | fn test_delete_user() { | 
|  | 630 | let test_dir = TempDir::new("test_delete_user_").expect("Failed to create temp dir."); | 
|  | 631 | let mut db = DB::new(&test_dir.build().push(LegacyKeystore::LEGACY_KEYSTORE_FILE_NAME)) | 
|  | 632 | .expect("Failed to open database."); | 
|  | 633 |  | 
|  | 634 | // Insert three entries for owner 2. | 
| Joel Galenson | 81a50f2 | 2021-07-29 15:39:10 -0700 | [diff] [blame] | 635 | db.put(2 + 2 * rustutils::users::AID_USER_OFFSET, "test1", TEST_BLOB1) | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 636 | .expect("Failed to insert test1."); | 
| Joel Galenson | 81a50f2 | 2021-07-29 15:39:10 -0700 | [diff] [blame] | 637 | db.put(4 + 2 * rustutils::users::AID_USER_OFFSET, "test2", TEST_BLOB2) | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 638 | .expect("Failed to insert test2."); | 
|  | 639 | db.put(3, "test3", TEST_BLOB3).expect("Failed to insert test3."); | 
|  | 640 |  | 
|  | 641 | db.remove_user(2).expect("Failed to remove user 2"); | 
|  | 642 |  | 
|  | 643 | assert_eq!( | 
|  | 644 | Vec::<String>::new(), | 
| Joel Galenson | 81a50f2 | 2021-07-29 15:39:10 -0700 | [diff] [blame] | 645 | db.list(2 + 2 * rustutils::users::AID_USER_OFFSET).expect("Failed to list entries.") | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 646 | ); | 
|  | 647 |  | 
|  | 648 | assert_eq!( | 
|  | 649 | Vec::<String>::new(), | 
| Joel Galenson | 81a50f2 | 2021-07-29 15:39:10 -0700 | [diff] [blame] | 650 | db.list(4 + 2 * rustutils::users::AID_USER_OFFSET).expect("Failed to list entries.") | 
| Janis Danisevskis | 5898d15 | 2021-06-15 08:23:46 -0700 | [diff] [blame] | 651 | ); | 
|  | 652 |  | 
|  | 653 | assert_eq!(vec!["test3".to_string(),], db.list(3).expect("Failed to list entries.")); | 
|  | 654 | } | 
|  | 655 |  | 
|  | 656 | #[test] | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 657 | fn concurrent_legacy_keystore_entry_test() -> Result<()> { | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 658 | let temp_dir = Arc::new( | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 659 | TempDir::new("concurrent_legacy_keystore_entry_test_") | 
|  | 660 | .expect("Failed to create temp dir."), | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 661 | ); | 
|  | 662 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 663 | let db_path = temp_dir.build().push(LegacyKeystore::LEGACY_KEYSTORE_FILE_NAME).to_owned(); | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 664 |  | 
|  | 665 | let test_begin = Instant::now(); | 
|  | 666 |  | 
|  | 667 | let mut db = DB::new(&db_path).expect("Failed to open database."); | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 668 | const ENTRY_COUNT: u32 = 5000u32; | 
|  | 669 | const ENTRY_DB_COUNT: u32 = 5000u32; | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 670 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 671 | let mut actual_entry_count = ENTRY_COUNT; | 
|  | 672 | // First insert ENTRY_COUNT entries. | 
|  | 673 | for count in 0..ENTRY_COUNT { | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 674 | if Instant::now().duration_since(test_begin) >= Duration::from_secs(15) { | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 675 | actual_entry_count = count; | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 676 | break; | 
|  | 677 | } | 
|  | 678 | let alias = format!("test_alias_{}", count); | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 679 | db.put(1, &alias, TEST_BLOB1).expect("Failed to add entry (1)."); | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 680 | } | 
|  | 681 |  | 
|  | 682 | // Insert more keys from a different thread and into a different namespace. | 
|  | 683 | let db_path1 = db_path.clone(); | 
|  | 684 | let handle1 = thread::spawn(move || { | 
|  | 685 | let mut db = DB::new(&db_path1).expect("Failed to open database."); | 
|  | 686 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 687 | for count in 0..actual_entry_count { | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 688 | if Instant::now().duration_since(test_begin) >= Duration::from_secs(40) { | 
|  | 689 | return; | 
|  | 690 | } | 
|  | 691 | let alias = format!("test_alias_{}", count); | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 692 | db.put(2, &alias, TEST_BLOB2).expect("Failed to add entry (2)."); | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 693 | } | 
|  | 694 |  | 
|  | 695 | // Then delete them again. | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 696 | for count in 0..actual_entry_count { | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 697 | if Instant::now().duration_since(test_begin) >= Duration::from_secs(40) { | 
|  | 698 | return; | 
|  | 699 | } | 
|  | 700 | let alias = format!("test_alias_{}", count); | 
|  | 701 | db.remove(2, &alias).expect("Remove Failed (2)."); | 
|  | 702 | } | 
|  | 703 | }); | 
|  | 704 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 705 | // And start deleting the first set of entries. | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 706 | let db_path2 = db_path.clone(); | 
|  | 707 | let handle2 = thread::spawn(move || { | 
|  | 708 | let mut db = DB::new(&db_path2).expect("Failed to open database."); | 
|  | 709 |  | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 710 | for count in 0..actual_entry_count { | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 711 | if Instant::now().duration_since(test_begin) >= Duration::from_secs(40) { | 
|  | 712 | return; | 
|  | 713 | } | 
|  | 714 | let alias = format!("test_alias_{}", count); | 
|  | 715 | db.remove(1, &alias).expect("Remove Failed (1))."); | 
|  | 716 | } | 
|  | 717 | }); | 
|  | 718 |  | 
|  | 719 | // While a lot of inserting and deleting is going on we have to open database connections | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 720 | // successfully and then insert and delete a specific entry. | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 721 | let db_path3 = db_path.clone(); | 
|  | 722 | let handle3 = thread::spawn(move || { | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 723 | for _count in 0..ENTRY_DB_COUNT { | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 724 | if Instant::now().duration_since(test_begin) >= Duration::from_secs(40) { | 
|  | 725 | return; | 
|  | 726 | } | 
|  | 727 | let mut db = DB::new(&db_path3).expect("Failed to open database."); | 
|  | 728 |  | 
| Chris Wailes | d5aaaef | 2021-07-27 16:04:33 -0700 | [diff] [blame] | 729 | db.put(3, TEST_ALIAS, TEST_BLOB3).expect("Failed to add entry (3)."); | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 730 |  | 
| Chris Wailes | d5aaaef | 2021-07-27 16:04:33 -0700 | [diff] [blame] | 731 | db.remove(3, TEST_ALIAS).expect("Remove failed (3)."); | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 732 | } | 
|  | 733 | }); | 
|  | 734 |  | 
|  | 735 | // While thread 3 is inserting and deleting TEST_ALIAS, we try to get the alias. | 
|  | 736 | // This may yield an entry or none, but it must not fail. | 
|  | 737 | let handle4 = thread::spawn(move || { | 
| Janis Danisevskis | 3eb829d | 2021-06-14 14:18:20 -0700 | [diff] [blame] | 738 | for _count in 0..ENTRY_DB_COUNT { | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 739 | if Instant::now().duration_since(test_begin) >= Duration::from_secs(40) { | 
|  | 740 | return; | 
|  | 741 | } | 
|  | 742 | let mut db = DB::new(&db_path).expect("Failed to open database."); | 
|  | 743 |  | 
|  | 744 | // This may return Some or None but it must not fail. | 
| Chris Wailes | d5aaaef | 2021-07-27 16:04:33 -0700 | [diff] [blame] | 745 | db.get(3, TEST_ALIAS).expect("Failed to get entry (4)."); | 
| Janis Danisevskis | 1be7e18 | 2021-04-12 14:31:12 -0700 | [diff] [blame] | 746 | } | 
|  | 747 | }); | 
|  | 748 |  | 
|  | 749 | handle1.join().expect("Thread 1 panicked."); | 
|  | 750 | handle2.join().expect("Thread 2 panicked."); | 
|  | 751 | handle3.join().expect("Thread 3 panicked."); | 
|  | 752 | handle4.join().expect("Thread 4 panicked."); | 
|  | 753 |  | 
|  | 754 | Ok(()) | 
|  | 755 | } | 
| Janis Danisevskis | 77d7204 | 2021-01-20 15:36:30 -0800 | [diff] [blame] | 756 | } |