Migrate to the librustutils cutils bindings.
Bug: 182498247
Test: Build
Change-Id: I0ab7fb092574c74b09c1b5b60e82ff776a214d53
diff --git a/keystore2/Android.bp b/keystore2/Android.bp
index fe0a11a..e340a6d 100644
--- a/keystore2/Android.bp
+++ b/keystore2/Android.bp
@@ -40,7 +40,6 @@
"android.system.keystore2-V1-rust",
"libanyhow",
"libbinder_rs",
- "libcutils_bindgen",
"libkeystore2_aaid-rust",
"libkeystore2_apc_compat-rust",
"libkeystore2_crypto_rust",
diff --git a/keystore2/legacykeystore/Android.bp b/keystore2/legacykeystore/Android.bp
index fb6f60f..da6aa8a 100644
--- a/keystore2/legacykeystore/Android.bp
+++ b/keystore2/legacykeystore/Android.bp
@@ -31,10 +31,10 @@
"android.security.legacykeystore-rust",
"libanyhow",
"libbinder_rs",
- "libcutils_bindgen",
"libkeystore2",
"liblog_rust",
"librusqlite",
+ "librustutils",
"libthiserror",
],
}
@@ -49,11 +49,11 @@
"android.security.legacykeystore-rust",
"libanyhow",
"libbinder_rs",
- "libcutils_bindgen",
"libkeystore2",
"libkeystore2_test_utils",
"liblog_rust",
"librusqlite",
+ "librustutils",
"libthiserror",
],
}
diff --git a/keystore2/legacykeystore/lib.rs b/keystore2/legacykeystore/lib.rs
index e57f159..da60297 100644
--- a/keystore2/legacykeystore/lib.rs
+++ b/keystore2/legacykeystore/lib.rs
@@ -161,7 +161,7 @@
self.with_transaction(TransactionBehavior::Immediate, |tx| {
tx.execute(
"DELETE FROM profiles WHERE cast ( ( owner/? ) as int) = ?;",
- params![cutils_bindgen::AID_USER_OFFSET, user_id],
+ params![rustutils::users::AID_USER_OFFSET, user_id],
)
.context("In remove_uid: Failed to delete.")
})?;
@@ -600,9 +600,9 @@
.expect("Failed to open database.");
// Insert three entries for owner 2.
- db.put(2 + 2 * cutils_bindgen::AID_USER_OFFSET, "test1", TEST_BLOB1)
+ db.put(2 + 2 * rustutils::users::AID_USER_OFFSET, "test1", TEST_BLOB1)
.expect("Failed to insert test1.");
- db.put(4 + 2 * cutils_bindgen::AID_USER_OFFSET, "test2", TEST_BLOB2)
+ db.put(4 + 2 * rustutils::users::AID_USER_OFFSET, "test2", TEST_BLOB2)
.expect("Failed to insert test2.");
db.put(3, "test3", TEST_BLOB3).expect("Failed to insert test3.");
@@ -610,12 +610,12 @@
assert_eq!(
Vec::<String>::new(),
- db.list(2 + 2 * cutils_bindgen::AID_USER_OFFSET).expect("Failed to list entries.")
+ db.list(2 + 2 * rustutils::users::AID_USER_OFFSET).expect("Failed to list entries.")
);
assert_eq!(
Vec::<String>::new(),
- db.list(4 + 2 * cutils_bindgen::AID_USER_OFFSET).expect("Failed to list entries.")
+ db.list(4 + 2 * rustutils::users::AID_USER_OFFSET).expect("Failed to list entries.")
);
assert_eq!(vec!["test3".to_string(),], db.list(3).expect("Failed to list entries."));
diff --git a/keystore2/src/utils.rs b/keystore2/src/utils.rs
index d71a4fc..f6d92ee 100644
--- a/keystore2/src/utils.rs
+++ b/keystore2/src/utils.rs
@@ -188,16 +188,15 @@
}
/// AID offset for uid space partitioning.
-pub const AID_USER_OFFSET: u32 = cutils_bindgen::AID_USER_OFFSET;
+pub const AID_USER_OFFSET: u32 = rustutils::users::AID_USER_OFFSET;
/// AID of the keystore process itself, used for keys that
/// keystore generates for its own use.
-pub const AID_KEYSTORE: u32 = cutils_bindgen::AID_KEYSTORE;
+pub const AID_KEYSTORE: u32 = rustutils::users::AID_KEYSTORE;
/// Extracts the android user from the given uid.
pub fn uid_to_android_user(uid: u32) -> u32 {
- // Safety: No memory access
- unsafe { cutils_bindgen::multiuser_get_user_id(uid) }
+ rustutils::users::multiuser_get_user_id(uid)
}
/// This module provides helpers for simplified use of the watchdog module.