commit d1756e2a41f32d1924dcde17c99ac20fe2e4e44e
author Steven Moreland <smoreland@google.com> Tue Jul 16 12:10:47 2019 -0700
committer android-build-merger <android-build-merger@google.com> Tue Jul 16 12:10:47 2019 -0700
tree 64c026043ea4fb52dc92a6017a329ee65c8af6ba
parent 04b6878cdfbb802e2f4bcbfc7b9196fbb62650b7
parent bcfdb3fbdfcec3e9433f50cb2b1f9c63b61b7116

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

Change-Id: Ib3873e4b822af4a8f8501bae261449aa8dd3eb7a

test_main.cpp

diff --git a/test_main.cpp b/test_main.cpp
index 6e96a42..af29f84 100644
--- a/test_main.cpp
+++ b/test_main.cpp
@@ -484,6 +484,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;