diff options
| author | Eran Messeri <eranm@google.com> | 2022-03-31 14:06:20 +0000 |
|---|---|---|
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2022-03-31 14:06:20 +0000 |
| commit | d8fdf0b80433dd6c7b5de0128195c84b26ac66dc (patch) | |
| tree | 5ace759f227e207da9b97c2185132e006e4a332f /security/keymint/aidl/vts/functional/KeyMintTest.cpp | |
| parent | 67fa92a1e596b848f18a05edde4feaba0dc49deb (diff) | |
| parent | 1b7abc43b633b47612d48c01cb995262fcc1107b (diff) | |
Merge "KeyMint: Device IDs attestation based on verion." am: 1b7abc43b6
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2028224
Change-Id: Ic98d604c43a6c8dec4eef7532fba17b7824813ce
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
Diffstat (limited to 'security/keymint/aidl/vts/functional/KeyMintTest.cpp')
| -rw-r--r-- | security/keymint/aidl/vts/functional/KeyMintTest.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/security/keymint/aidl/vts/functional/KeyMintTest.cpp b/security/keymint/aidl/vts/functional/KeyMintTest.cpp index d8502e43cc..3b75c50acd 100644 --- a/security/keymint/aidl/vts/functional/KeyMintTest.cpp +++ b/security/keymint/aidl/vts/functional/KeyMintTest.cpp @@ -1986,8 +1986,8 @@ TEST_P(NewKeyGenerationTest, EcdsaAttestationIdTags) { if (SecLevel() == SecurityLevel::STRONGBOX) { if (result == ErrorCode::ATTESTATION_KEYS_NOT_PROVISIONED) return; } - if (result == ErrorCode::CANNOT_ATTEST_IDS) { - // Device ID attestation is optional; KeyMint may not support it at all. + if (result == ErrorCode::CANNOT_ATTEST_IDS && !isDeviceIdAttestationRequired()) { + // ID attestation was optional till api level 32, from api level 33 it is mandatory. continue; } ASSERT_EQ(result, ErrorCode::OK); |