Merge "keymaster: spec does not require that update produce output" into pi-dev (c7d39cf9) · Commits · e / os / android_hardware_interfaces

keymaster/4.0/vts/functional/keymaster_hidl_hal_test.cpp

+4 −3

Original line number	Diff line number	Diff line
		@@ -3005,6 +3005,7 @@ TEST_F(EncryptionOperationsTest, AesGcmAadNoData) {
		* Verifies that AES GCM mode works when provided additional authenticated data in multiple chunks.
		*/
		TEST_F(EncryptionOperationsTest, AesGcmMultiPartAad) {
		const size_t tag_bits = 128;
		ASSERT_EQ(ErrorCode::OK, GenerateKey(AuthorizationSetBuilder()
		.Authorization(TAG_NO_AUTH_REQUIRED)
		.AesEncryptionKey(128)
		@@ -3016,7 +3017,7 @@ TEST_F(EncryptionOperationsTest, AesGcmMultiPartAad) {
		auto begin_params = AuthorizationSetBuilder()
		.BlockMode(BlockMode::GCM)
		.Padding(PaddingMode::NONE)
		.Authorization(TAG_MAC_LENGTH, 128);
		.Authorization(TAG_MAC_LENGTH, tag_bits);
		AuthorizationSet begin_out_params;

		auto update_params =
		@@ -3038,10 +3039,11 @@ TEST_F(EncryptionOperationsTest, AesGcmMultiPartAad) {
		EXPECT_EQ(ErrorCode::OK, Update(op_handle_, update_params, message, &update_out_params,
		&ciphertext, &input_consumed));
		EXPECT_EQ(message.size(), input_consumed);
		EXPECT_EQ(message.size(), ciphertext.size());
		EXPECT_TRUE(update_out_params.empty());

		EXPECT_EQ(ErrorCode::OK, Finish("" /* input */, &ciphertext));
		// Expect 128-bit (16-byte) tag appended to ciphertext.
		EXPECT_EQ(message.size() + (tag_bits >> 3), ciphertext.size());

		// Grab nonce.
		begin_params.push_back(begin_out_params);
		@@ -3097,7 +3099,6 @@ TEST_F(EncryptionOperationsTest, AesGcmAadOutOfOrder) {
		EXPECT_EQ(ErrorCode::OK, Update(op_handle_, update_params, message, &update_out_params,
		&ciphertext, &input_consumed));
		EXPECT_EQ(message.size(), input_consumed);
		EXPECT_EQ(message.size(), ciphertext.size());
		EXPECT_TRUE(update_out_params.empty());

		// More AAD