| commit | 41ae0f4527bbc6fbea249a4ac0ff6ffe34479c44 | [log] [tgz] |
|---|---|---|
| author | Ping Fan <ping.fan@mediatek.com> | Tue Aug 06 21:44:43 2024 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Tue Aug 06 21:44:43 2024 +0000 |
| tree | 2efe3199f9d5ed8590469b35c1e1d48cd9fcc667 | |
| parent | 4b4569e391bdbc15d6d043172c35f51f5c6ad7ed [diff] | |
| parent | 73b1d1aa5cec508e994483f8575f2906691df947 [diff] |
Fix validation logic in TvInputAidlTest am: 73b1d1aa5c Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3207431 Change-Id: I86970ff24c87b1f4577d6a050830b0369480cd30 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
diff --git a/tv/input/aidl/vts/functional/VtsHalTvInputTargetTest.cpp b/tv/input/aidl/vts/functional/VtsHalTvInputTargetTest.cpp index 8d3395b..90404f6 100644 --- a/tv/input/aidl/vts/functional/VtsHalTvInputTargetTest.cpp +++ b/tv/input/aidl/vts/functional/VtsHalTvInputTargetTest.cpp
@@ -135,13 +135,15 @@ } bool TvInputAidlTest::isValidHandle(NativeHandle& handle) { - if (handle.fds.empty()) { + if (handle.fds.empty() && handle.ints.empty()) { return false; } - for (size_t i = 0; i < handle.fds.size(); i++) { - int fd = handle.fds[i].get(); - if (fcntl(fd, F_GETFL) < 0) { - return false; + if (!(handle.fds.empty())) { + for (size_t i = 0; i < handle.fds.size(); i++) { + int fd = handle.fds[i].get(); + if (fcntl(fd, F_GETFL) < 0) { + return false; + } } } return true;