From b5cc022359f37474ab36eca630e1cbe133f5b6b0 Mon Sep 17 00:00:00 2001 From: Alisher Alikhodjaev Date: Wed, 20 Sep 2023 15:15:38 -0700 Subject: [PATCH] Temporary disable one of the transmit checks The transmit without openning a channel passes on some platforms, because a basic channel can be opened by external applications and the state is maintained in the HAL. Bug: 300502872 Test: run vts -m VtsHalSecureElementTargetTest (cherry picked from https://android-review.googlesource.com/q/commit:d0ed43bb7eeb8c2cdc23d23d0d926e083322b7f3) Merged-In: If727c613e8575802b99604f7242e16cf85fc97a0 Change-Id: If727c613e8575802b99604f7242e16cf85fc97a0 --- secure_element/aidl/vts/VtsHalSecureElementTargetTest.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/secure_element/aidl/vts/VtsHalSecureElementTargetTest.cpp b/secure_element/aidl/vts/VtsHalSecureElementTargetTest.cpp index 97b4e27629..9678da4a5b 100644 --- a/secure_element/aidl/vts/VtsHalSecureElementTargetTest.cpp +++ b/secure_element/aidl/vts/VtsHalSecureElementTargetTest.cpp @@ -293,11 +293,13 @@ TEST_P(SecureElementAidl, transmit) { std::vector response; LogicalChannelResponse logical_channel_response; + /* Temporaly disable this check to clarify Basic Channel behavior (b/300502872) // Note: no channel is opened for this test // transmit() will return an empty response with the error // code CHANNEL_NOT_AVAILABLE when the SE cannot be // communicated with. EXPECT_ERR(secure_element_->transmit(kDataApdu, &response)); + */ EXPECT_OK(secure_element_->openLogicalChannel(kSelectableAid, 0x00, &logical_channel_response)); EXPECT_GE(logical_channel_response.selectResponse.size(), 2u);