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

Commit 4ce26385 authored by David Drysdale's avatar David Drysdale Committed by Automerger Merge Worker
Browse files

Merge "KeyMint VTS: allow for stricter SharedSecret impls" into sc-dev am: 8e869891 

Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/15128553

Change-Id: Ie02582f752b43c9fbb8dd147f90cca67c5b71747
parents 033eb052 8e869891

security/sharedsecret/aidl/vts/functional/SharedSecretAidlTest.cpp

+20 −8
Original line number Diff line number Diff line
@@ -268,10 +268,16 @@ TEST_F(SharedSecretAidlTest, ComputeSharedSecretShortNonce) { 
                    << "Shared secret service that provided tweaked param should fail to compute "

                       "shared secret";

        } else {

            EXPECT_EQ(ErrorCode::OK, responses[i].error) << "Others should succeed";

            // Other services *may* succeed, or may notice the invalid size for the nonce.

            // However, if another service completes the computation, it should get the 'wrong'

            // answer.

            if (responses[i].error == ErrorCode::OK) {

                EXPECT_NE(correct_response, responses[i].sharing_check)

                        << "Others should calculate a different shared secret, due to the tweaked "

                           "nonce.";

            } else {

                EXPECT_EQ(ErrorCode::INVALID_ARGUMENT, responses[i].error);

            }

        }

    }

}
@@ -348,10 +354,16 @@ TEST_F(SharedSecretAidlTest, ComputeSharedSecretShortSeed) { 
                    << "Shared secret service that provided tweaked param should fail to compute "

                       "shared secret";

        } else {

            EXPECT_EQ(ErrorCode::OK, responses[i].error) << "Others should succeed";

            // Other services *may* succeed, or may notice the invalid size for the seed.

            // However, if another service completes the computation, it should get the 'wrong'

            // answer.

            if (responses[i].error == ErrorCode::OK) {

                EXPECT_NE(correct_response, responses[i].sharing_check)

                        << "Others should calculate a different shared secret, due to the tweaked "

                       "nonce.";

                           "seed.";

            } else {

                EXPECT_EQ(ErrorCode::INVALID_ARGUMENT, responses[i].error);

            }

        }

    }

}