Evolution-x/hardware_interfaces
1
0
Fork 0
mirror of https://github.com/Evolution-X/hardware_interfaces synced 2026-02-01 11:36:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

KeyMint VTS: allow for stricter SharedSecret impls

Browse Source 
Bug: 192223752
Test: VtsAidlSharedSecretTargetTest
Change-Id: Iccf2d0fe2a2d10ad12269dfecf78ea1d831c3ad4
This commit is contained in:
David Drysdale
2021-06-28 13:18:23 +01:00
parent 8701205c04
commit d46d61efec
1 changed files with 20 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
security/sharedsecret/aidl/vts/functional/SharedSecretAidlTest.cpp
View File

@@ -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";
EXPECT_NE(correct_response, responses[i].sharing_check)
<< "Others should calculate a different shared secret, due to the tweaked "
"nonce.";
// 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";
EXPECT_NE(correct_response, responses[i].sharing_check)
<< "Others should calculate a different shared secret, due to the tweaked "
"nonce.";
// 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 "
"seed.";
} else {
EXPECT_EQ(ErrorCode::INVALID_ARGUMENT, responses[i].error);
}
}
}
}