commit 4c39f61ac6bd12d6118697d5d0e4d4c42ebcb4fd
author Steven Moreland <smoreland@google.com> Tue Jul 16 12:21:59 2019 -0700
committer android-build-merger <android-build-merger@google.com> Tue Jul 16 12:21:59 2019 -0700
tree 97185d3b44596e5420d03aee492d105ee8a006a0
parent 9a8f64c91172af8c5571c9c8631380e02b5ba844
parent d1756e2a41f32d1924dcde17c99ac20fe2e4e44e

Tests for Return crashes. am: 9cafed562a am: f13d1985d4 am: bcfdb3fbdf
am: d1756e2a41

Change-Id: I2ec5442eb75db0e1a409e88c6fe0e08d7ba395ff

test_main.cpp

diff --git a/test_main.cpp b/test_main.cpp
index 5defe8c..66c784b 100644
--- a/test_main.cpp
+++ b/test_main.cpp
@@ -489,6 +489,22 @@
     EXPECT_EQ(three, ret.withDefault(three));
 }
 
+TEST_F(LibHidlTest, ReturnDies) {
+    using ::android::hardware::Return;
+    using ::android::hardware::Status;
+
+    EXPECT_DEATH({ Return<void>(Status::fromStatusT(-EBUSY)); }, "");
+    EXPECT_DEATH({ Return<void>(Status::fromStatusT(-EBUSY)).isDeadObject(); }, "");
+    EXPECT_DEATH(
+            {
+                Return<int> ret = Return<int>(Status::fromStatusT(-EBUSY));
+                int foo = ret;  // should crash here
+                (void)foo;
+                ret.isOk();
+            },
+            "");
+}
+
 std::string toString(const ::android::hardware::Status &s) {
     using ::android::hardware::operator<<;
     std::ostringstream oss;