map interruption of close by signal to success rather than EINPROGRESS
authorRich Felker <dalias@aerifal.cx>
Fri, 20 Feb 2015 23:35:05 +0000 (18:35 -0500)
committerRich Felker <dalias@aerifal.cx>
Fri, 20 Feb 2015 23:35:05 +0000 (18:35 -0500)
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.

src/unistd/close.c

index 2f1eabd..fa3c6ca 100644 (file)
@@ -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);
 }