| commit | 2a63e73a04cbf91a1dc04b924ace52a0c5b983bf | [log] [tgz] |
|---|---|---|
| author | David Drysdale <drysdale@google.com> | Wed Feb 28 07:05:43 2024 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Wed Feb 28 07:05:43 2024 +0000 |
| tree | 8f28aa9bbee261df0934351bccfc4fa28fd49699 | |
| parent | eef4bc0a45432de62af7ef664b75298827b108eb [diff] | |
| parent | 3a9d425ec6db298deef9fc2e9a5eb6ac6182201e [diff] |
KeyMint: use a smaller invalid IMEI value am: 8351f33b2b am: 3a9d425ec6 Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2977372 Change-Id: I5bc89bccf047b507485f0f4436f62f6a53117766 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;