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

Commit 97272d8d authored by David Drysdale's avatar David Drysdale
Browse files

KeyMint: fix auth test HAT 

The test case for an auth-per-operation HAT with an invalid HMAC
is wrong -- it is re-using the previous HAT, which fails for a
different reason (has an old challenge).

Fix the test to use the HAT that's wrong in the intended way.

Bug: 297333975
Test: VtsAidlKeyMintTargetTest
Change-Id: I15fe9b0c1b53452df0f67dd44534fdb80a6c2a9c
parent bc2537e4

security/keymint/aidl/vts/functional/AuthTest.cpp

+1 −1
Original line number Diff line number Diff line
@@ -545,7 +545,7 @@ TEST_P(AuthTest, AuthPerOperation) { 
    ASSERT_GT(dodgy_hat->mac.size(), 0);

    dodgy_hat->mac[0] ^= 0x01;

    EXPECT_EQ(ErrorCode::KEY_USER_NOT_AUTHENTICATED,

              Finish(message, {} /* signature */, &ciphertext, hat.value()));

              Finish(message, {} /* signature */, &ciphertext, dodgy_hat.value()));

}



// Test use of a key that requires an auth token for each action on the operation, with