| commit | 046d443f5ffd7b18fd05a7d7cdaf6345d65dd5f1 | [log] [tgz] |
|---|---|---|
| author | Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> | Tue Aug 27 22:22:17 2024 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Tue Aug 27 22:22:17 2024 +0000 |
| tree | 6196b677247a100938971ba99938c2b2faf81140 | |
| parent | 5b43d645748c8d52bfaabd6c112c3e1d60ff4b76 [diff] | |
| parent | 9648fd64d8db2fe9a08f07b1a1e1724d2b3a54c1 [diff] |
Merge "Fixing isDeviceIdAttestationRequired to require both cases instead of just one" into main am: dd98cc1189 am: 9648fd64d8 Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3243582 Change-Id: I25b99f2fd9243feea66220c9b4542d9fe1e9753a Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
diff --git a/security/keymint/aidl/vts/functional/KeyMintAidlTestBase.cpp b/security/keymint/aidl/vts/functional/KeyMintAidlTestBase.cpp index 2ba75a3..c19ab11 100644 --- a/security/keymint/aidl/vts/functional/KeyMintAidlTestBase.cpp +++ b/security/keymint/aidl/vts/functional/KeyMintAidlTestBase.cpp
@@ -236,10 +236,10 @@ /** * An API to determine device IDs attestation is required or not, - * which is mandatory for KeyMint version 2 or first_api_level 33 or greater. + * which is mandatory for KeyMint version 2 and first_api_level 33 or greater. */ bool KeyMintAidlTestBase::isDeviceIdAttestationRequired() { - return AidlVersion() >= 2 || property_get_int32("ro.vendor.api_level", 0) >= __ANDROID_API_T__; + return AidlVersion() >= 2 && property_get_int32("ro.vendor.api_level", 0) >= __ANDROID_API_T__; } /**