summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRam Mohan M2017-10-11 05:08:25 -0500
committerPawin Vongmasa2017-10-17 06:34:30 -0500
commita0698429dd47e2f029866765c6980bc8720a7acb (patch)
tree148aca952162336840b6e7d1441c49440cc4e94e /media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp
parent3021f1fafbfb1a4b464effa543aee381f42d8080 (diff)
downloadplatform-hardware-interfaces-a0698429dd47e2f029866765c6980bc8720a7acb.tar.gz
platform-hardware-interfaces-a0698429dd47e2f029866765c6980bc8720a7acb.tar.xz
platform-hardware-interfaces-a0698429dd47e2f029866765c6980bc8720a7acb.zip
bug fix: during fatal fails bypass freeNode() calls
If test application undergoes a fatal failure bypass all the omx api calls till the application safely returns. Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \ && vts-tradefed run commandAndExit vts \ --skip-all-system-status-check --primary-abi-only \ --skip-preconditions --module VtsHalMediaOmxV1_0Host \ -l INFO Bug: 63796949 Change-Id: If5a33672a31665db913e53c6f8ba8bea6cfd3ebd
Diffstat (limited to 'media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp')
-rw-r--r--media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp4
1 files changed, 4 insertions, 0 deletions
diff --git a/media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp b/media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp
index 9e185f27..38cdcd65 100644
--- a/media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp
+++ b/media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp
@@ -230,6 +230,10 @@ class AudioDecHidlTest : public ::testing::VtsHalHidlTargetTestBase {
230 230
231 virtual void TearDown() override { 231 virtual void TearDown() override {
232 if (omxNode != nullptr) { 232 if (omxNode != nullptr) {
233 // If you have encountered a fatal failure, it is possible that
234 // freeNode() will not go through. Instead of hanging the app.
235 // let it pass through and report errors
236 if (::testing::Test::HasFatalFailure()) return;
233 EXPECT_TRUE((omxNode->freeNode()).isOk()); 237 EXPECT_TRUE((omxNode->freeNode()).isOk());
234 omxNode = nullptr; 238 omxNode = nullptr;
235 } 239 }