commit | ca99ef3f0d68b1dd9281606664108a78eb7e8811 | [log] [tgz] |
---|---|---|
author | David Drysdale <drysdale@google.com> | Wed Feb 28 07:49:59 2024 +0000 |
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Wed Feb 28 07:49:59 2024 +0000 |
tree | 08d22ecddc5ac2ebb44068575ac6e8fc6934edf9 | |
parent | 297d2743a58bef47bad2ddce030def6af233d240 [diff] | |
parent | 321ece1e9c67d9e604b0bce58295ebc32846cc34 [diff] |
KeyMint: use a smaller invalid IMEI value am: 8351f33b2b am: 3a9d425ec6 am: 321ece1e9c Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2977372 Change-Id: I747db154f674f3eb95f8b8fe15a3707c4a267c07 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
diff --git a/security/keymint/aidl/vts/functional/AttestKeyTest.cpp b/security/keymint/aidl/vts/functional/AttestKeyTest.cpp index 6d289ec..4ae0383 100644 --- a/security/keymint/aidl/vts/functional/AttestKeyTest.cpp +++ b/security/keymint/aidl/vts/functional/AttestKeyTest.cpp
@@ -939,7 +939,9 @@ .Authorization(TAG_ATTESTATION_ID_MODEL, "malicious-model"); if (isSecondImeiIdAttestationRequired()) { - attestation_id_tags.Authorization(TAG_ATTESTATION_ID_SECOND_IMEI, "invalid-second-imei"); + // Note: the invalid value here is < 16 bytes long to avoid triggering any implementation + // checks on valid IMEI lengths. + attestation_id_tags.Authorization(TAG_ATTESTATION_ID_SECOND_IMEI, "invalid-imei2"); } vector<uint8_t> key_blob; vector<KeyCharacteristics> key_characteristics;