| commit | b5cc022359f37474ab36eca630e1cbe133f5b6b0 | [log] [tgz] |
|---|---|---|
| author | Alisher Alikhodjaev <alisher@google.com> | Wed Sep 20 15:15:38 2023 -0700 |
| committer | Cherrypicker Worker <android-build-cherrypicker-worker@google.com> | Wed Sep 20 23:41:50 2023 +0000 |
| tree | 60161014bb6578b16803cfefd7d202d462272e79 | |
| parent | 1c91d9ccdc4b2517f6afdcbba7a08fb3c3f6a2b1 [diff] |
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
diff --git a/secure_element/aidl/vts/VtsHalSecureElementTargetTest.cpp b/secure_element/aidl/vts/VtsHalSecureElementTargetTest.cpp index 97b4e27..9678da4 100644 --- a/secure_element/aidl/vts/VtsHalSecureElementTargetTest.cpp +++ b/secure_element/aidl/vts/VtsHalSecureElementTargetTest.cpp
@@ -293,11 +293,13 @@ std::vector<uint8_t> 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);