mirror of
https://github.com/Evolution-X/hardware_interfaces
synced 2026-02-01 22:04:26 +00:00
Merge "KeyMint VTS: enable patchlevel checks" am: 8fb4a1c82e am: 22a5ccd132
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1840175 Change-Id: I559ffd0182bf84090613a9ac8f4132f30420a9e0
This commit is contained in:
@@ -69,8 +69,6 @@ namespace aidl::android::hardware::security::keymint::test {
|
||||
|
||||
namespace {
|
||||
|
||||
bool check_patchLevels = false;
|
||||
|
||||
// The maximum number of times we'll attempt to verify that corruption
|
||||
// of an ecrypted blob results in an error. Retries are necessary as there
|
||||
// is a small (roughly 1/256) chance that corrupting ciphertext still results
|
||||
@@ -529,14 +527,12 @@ class NewKeyGenerationTest : public KeyMintAidlTestBase {
|
||||
EXPECT_TRUE(os_pl);
|
||||
EXPECT_EQ(*os_pl, os_patch_level());
|
||||
|
||||
if (check_patchLevels) {
|
||||
// Should include vendor and boot patchlevels.
|
||||
auto vendor_pl = auths.GetTagValue(TAG_VENDOR_PATCHLEVEL);
|
||||
EXPECT_TRUE(vendor_pl);
|
||||
EXPECT_EQ(*vendor_pl, vendor_patch_level());
|
||||
auto boot_pl = auths.GetTagValue(TAG_BOOT_PATCHLEVEL);
|
||||
EXPECT_TRUE(boot_pl);
|
||||
}
|
||||
// Should include vendor and boot patchlevels.
|
||||
auto vendor_pl = auths.GetTagValue(TAG_VENDOR_PATCHLEVEL);
|
||||
EXPECT_TRUE(vendor_pl);
|
||||
EXPECT_EQ(*vendor_pl, vendor_patch_level());
|
||||
auto boot_pl = auths.GetTagValue(TAG_BOOT_PATCHLEVEL);
|
||||
EXPECT_TRUE(boot_pl);
|
||||
|
||||
return auths;
|
||||
}
|
||||
@@ -6677,10 +6673,6 @@ int main(int argc, char** argv) {
|
||||
} else {
|
||||
std::cout << "NOT dumping attestations" << std::endl;
|
||||
}
|
||||
// TODO(drysdale): Remove this flag when available KeyMint devices comply with spec
|
||||
if (std::string(argv[i]) == "--check_patchLevels") {
|
||||
aidl::android::hardware::security::keymint::test::check_patchLevels = true;
|
||||
}
|
||||
}
|
||||
}
|
||||
return RUN_ALL_TESTS();
|
||||
|
||||
Reference in New Issue
Block a user