use IGNORE_EINTR w/close HANDLE_EINTR is both not safe and not useful on Linux systems. Switch to IGNORE_EINTR like Chromium has done everywhere. See http://crbug.com/269623 for details. BUG=chromium:373154 TEST=`cbuildbot {arm,amd64,x86}-generic-full` passes Change-Id: Iaf55cde898a2b7eb699053974f6b3f41432e4f10 Reviewed-on: https://chromium-review.googlesource.com/199852 Reviewed-by: Christopher Wiley <wiley@chromium.org> Reviewed-by: Paul Stewart <pstew@chromium.org> Commit-Queue: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org>
diff --git a/file_io.cc b/file_io.cc index 5bab442..833a444 100644 --- a/file_io.cc +++ b/file_io.cc
@@ -36,7 +36,7 @@ } int FileIO::Close(int fd) { - return HANDLE_EINTR(close(fd)); + return IGNORE_EINTR(close(fd)); } int FileIO::SetFdNonBlocking(int fd) {
diff --git a/sockets.cc b/sockets.cc index bb7ddae..39c159e 100644 --- a/sockets.cc +++ b/sockets.cc
@@ -54,7 +54,7 @@ } int Sockets::Close(int fd) const { - return HANDLE_EINTR(close(fd)); + return IGNORE_EINTR(close(fd)); } int Sockets::Connect(int sockfd,