map interruption of close by signal to success rather than EINPROGRESS

commit 82dc1e2e783815e00a90cd3f681436a80d54a314 addressed the
resolution of Austin Group issue 529, which requires close to leave
the fd open when failing with EINTR, by returning the newly defined
error code EINPROGRESS. this turns out to be a bad idea, though, since
legacy applications not aware of the new specification are likely to
interpret any error from close except EINTR as a hard failure.

diff --git a/src/unistd/close.c b/src/unistd/close.c
index 2f1eabd7debe8acdd0e85c825a881aa79b009651..fa3c6cab7e9813ad280329bfd7ebf45cb738c321 100644 (file)
--- a/src/unistd/close.c
+++ b/src/unistd/close.c
@@ -14,6 +14,6 @@ int close(int fd)
 {
        fd = __aio_close(fd);
        int r = __syscall_cp(SYS_close, fd);
-       if (r == -EINTR) r = -EINPROGRESS;
+       if (r == -EINTR) r = 0;
        return __syscall_ret(r);
 }