Don't HANDLE_EINTR(close). Either IGNORE_EINTR(close) or just close.

sandbox/linux/tests/unit_tests.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <fcntl.h>
 #include <poll.h>
 #include <signal.h>
 #include <stdio.h>
 #include <sys/resource.h>
 #include <sys/time.h>
@@ skipping 132 matching lines @@
 
     // Disable core files. They are not very useful for our individual test
     // cases.
     struct rlimit no_core = {0};
     setrlimit(RLIMIT_CORE, &no_core);
 
     test(arg);
     _exit(kExpectedValue);
   }
 
-  (void)HANDLE_EINTR(close(fds[1]));
+  close(fds[1]);
   std::vector<char> msg_buf;
   ssize_t rc;
 
   // Make sure read() will never block as we'll use poll() to
   // block with a timeout instead.
   const int fcntl_ret = fcntl(fds[0], F_SETFL, O_NONBLOCK);
   ASSERT_EQ(fcntl_ret, 0);
   struct pollfd poll_fd = {fds[0], POLLIN | POLLRDHUP, 0};
 
   int poll_ret;
   // We prefer the SIGALRM timeout to trigger in the child than this timeout
   // so we double the common value here.
   int poll_timeout = GetSubProcessTimeoutTimeInSeconds() * 2 * 1000;
   while ((poll_ret = poll(&poll_fd, 1, poll_timeout) > 0)) {
     const size_t kCapacity = 256;
     const size_t len = msg_buf.size();
     msg_buf.resize(len + kCapacity);
     rc = HANDLE_EINTR(read(fds[0], &msg_buf[len], kCapacity));
     msg_buf.resize(len + std::max(rc, static_cast<ssize_t>(0)));
     if (rc <= 0)
       break;
   }
   ASSERT_NE(poll_ret, -1) << "poll() failed";
   ASSERT_NE(poll_ret, 0) << "Timeout while reading child state";
-  (void)HANDLE_EINTR(close(fds[0]));
+  close(fds[0]);
   std::string msg(msg_buf.begin(), msg_buf.end());
 
   int status = 0;
   int waitpid_returned = HANDLE_EINTR(waitpid(pid, &status, 0));
   ASSERT_EQ(pid, waitpid_returned) << TestFailedMessage(msg);
 
   // At run-time, we sometimes decide that a test shouldn't actually
   // run (e.g. when testing sandbox features on a kernel that doesn't
   // have sandboxing support). When that happens, don't attempt to
   // call the "death" function, as it might be looking for a
@@ skipping 63 matching lines @@
   fflush(stderr);
   _exit(kExitWithAssertionFailure);
 }
 
 void UnitTests::IgnoreThisTest() {
   fflush(stderr);
   _exit(kIgnoreThisTest);
 }
 
 }  // namespace