From ff819285328ff6678791e666d92172fd0d999995 Mon Sep 17 00:00:00 2001 From: David Drysdale Date: Wed, 18 Aug 2021 16:45:50 +0100 Subject: [PATCH] KeyMint VTS: add missing purpose/algo Test was producing an invalid set of parameters in a different way than intended. Bug: 197222749 Test: VtsAidlKeyMintTargetTest Change-Id: I07f706fec81d91e8eee9c0561428142559c54f12 --- security/keymint/aidl/vts/functional/KeyMintTest.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/security/keymint/aidl/vts/functional/KeyMintTest.cpp b/security/keymint/aidl/vts/functional/KeyMintTest.cpp index 931317fad2..651b21f19e 100644 --- a/security/keymint/aidl/vts/functional/KeyMintTest.cpp +++ b/security/keymint/aidl/vts/functional/KeyMintTest.cpp @@ -1843,12 +1843,13 @@ TEST_P(NewKeyGenerationTest, EcdsaMismatchKeySize) { if (SecLevel() == SecurityLevel::STRONGBOX) return; auto result = GenerateKey(AuthorizationSetBuilder() + .Authorization(TAG_ALGORITHM, Algorithm::EC) .Authorization(TAG_KEY_SIZE, 224) .Authorization(TAG_EC_CURVE, EcCurve::P_256) + .SigningKey() .Digest(Digest::NONE) .SetDefaultValidity()); - ASSERT_TRUE(result == ErrorCode::INVALID_ARGUMENT || - result == ErrorCode::UNSUPPORTED_ALGORITHM); + ASSERT_TRUE(result == ErrorCode::INVALID_ARGUMENT); } /*