[PR] dtraev - FAIL, remove unneeded return statements

Author: cjappl

compiler-rt/lib/rtsan/tests/rtsan_test_interceptors_posix.cpp

@@ -248,6 +248,7 @@ TEST(TestRtsanInterceptors, SchedYieldDiesWhenRealtime) {
 /*
     Filesystem
 */
+
 TEST_F(RtsanFileTest, OpenDiesWhenRealtime) {
   auto Func = [this]() { open(GetTemporaryFilePath(), O_RDONLY); };
   ExpectRealtimeDeath(Func, MAYBE_APPEND_64("open"));
@@ -396,15 +397,13 @@ TEST(TestRtsanInterceptors, IoctlBehavesWithOutputPointer) {
   if (sock == -1) {
     perror("socket");
     GTEST_SKIP();
-    return;
   }
 
   struct ifaddrs *ifaddr = nullptr;
   if (getifaddrs(&ifaddr) == -1 || ifaddr == nullptr) {
     perror("getifaddrs");
     close(sock);
     GTEST_SKIP();
-    return;
   }
 
   struct ifreq ifr {};
@@ -415,8 +414,7 @@ TEST(TestRtsanInterceptors, IoctlBehavesWithOutputPointer) {
     perror("ioctl");
     close(sock);
     freeifaddrs(ifaddr);
-    ASSERT_TRUE(false) << "ioctl failed";
-    return;
+    FAIL();
   }
 
   freeifaddrs(ifaddr);