Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6008f37a authored by Wenhao Wang's avatar Wenhao Wang
Browse files

Add TAG_MAC_LENGTH to EncryptionOperationsTest.AesWrongPurpose 

Without the setting of TAG_MAC_LENGTH, the test fails due to
MISSING_MAC_PURPOSE.

Bug: 145626599
Test: VtsHalKeymasterV4_0TargetTest
Change-Id: Ic58411b86e07dfeeb78211abf9271ee995beabc9
parent 7f6c8d03

keymaster/4.0/vts/functional/keymaster_hidl_hal_test.cpp

+8 −4
Original line number Diff line number Diff line
@@ -2645,8 +2645,10 @@ TEST_P(EncryptionOperationsTest, AesWrongPurpose) { 
                                   .Padding(PaddingMode::NONE));

    ASSERT_EQ(ErrorCode::OK, err) << "Got " << err;



    err = Begin(KeyPurpose::DECRYPT,

                AuthorizationSetBuilder().BlockMode(BlockMode::GCM).Padding(PaddingMode::NONE));

    err = Begin(KeyPurpose::DECRYPT, AuthorizationSetBuilder()

                                             .BlockMode(BlockMode::GCM)

                                             .Padding(PaddingMode::NONE)

                                             .Authorization(TAG_MAC_LENGTH, 128));

    EXPECT_EQ(ErrorCode::INCOMPATIBLE_PURPOSE, err) << "Got " << err;



    CheckedDeleteKey();
@@ -2659,8 +2661,10 @@ TEST_P(EncryptionOperationsTest, AesWrongPurpose) { 
                                                 .Authorization(TAG_MIN_MAC_LENGTH, 128)

                                                 .Padding(PaddingMode::NONE)));



    err = Begin(KeyPurpose::ENCRYPT,

                AuthorizationSetBuilder().BlockMode(BlockMode::GCM).Padding(PaddingMode::NONE));

    err = Begin(KeyPurpose::ENCRYPT, AuthorizationSetBuilder()

                                             .BlockMode(BlockMode::GCM)

                                             .Padding(PaddingMode::NONE)

                                             .Authorization(TAG_MAC_LENGTH, 128));

    EXPECT_EQ(ErrorCode::INCOMPATIBLE_PURPOSE, err) << "Got " << err;

}