commit ac5fcc75aad84adac0b4954ab462c725c40d2830
author Mike Frysinger <vapier@chromium.org> Wed May 14 20:07:15 2014
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Sat May 17 20:17:36 2014
tree 6e48b57556609f0bc89ca8c9c9c3285ec819d5ec
parent d4839c81a6e3e30ba362ff2c4444a190065d6dd2

do not use HANDLE_EINTR w/close

HANDLE_EINTR is both not safe and not useful on Linux systems.
This code is already ignoring the result, so don't need to ignore
it twice.

See http://crbug.com/269623 for details.

BUG=chromium:373154
TEST=`cbuildbot {arm,amd64,x86}-generic-full` passes

Change-Id: I7c7d025efd44f84e084b2ddb26de2da380d57d24
Reviewed-on: https://chromium-review.googlesource.com/199831
Reviewed-by: Darren Krahn <dkrahn@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>

platform.cc

diff --git a/platform.cc b/platform.cc
index e75d33b..ca25ade 100644
--- a/platform.cc
+++ b/platform.cc
@@ -799,7 +799,7 @@
   if (dir) {
     ignore_result(closedir(dir));
   } else {
-    ignore_result(HANDLE_EINTR(close(fd)));
+    ignore_result(close(fd));
   }
   if (result < 0) {
     PLOG(WARNING) << "Failed to sync " << path;