diff options
author | Mattias Andrée <maandree@operamail.com> | 2015-10-11 04:13:13 +0200 |
---|---|---|
committer | Mattias Andrée <maandree@operamail.com> | 2015-10-11 04:13:13 +0200 |
commit | 0a85228ff762b77de2d47119d1379e7ca6f48eb8 (patch) | |
tree | e2e936730368062cbafb42e6be6e8b57d5b192ce /src/libmdsclient | |
parent | whoops (diff) | |
download | mds-0a85228ff762b77de2d47119d1379e7ca6f48eb8.tar.gz mds-0a85228ff762b77de2d47119d1379e7ca6f48eb8.tar.bz2 mds-0a85228ff762b77de2d47119d1379e7ca6f48eb8.tar.xz |
Whilst POSIX leaves it explicitly unspecify whether close(2) closes the fildes on interruption, Linux (and possibly some other kernels) specify that it does close.
Signed-off-by: Mattias Andrée <maandree@operamail.com>
Diffstat (limited to 'src/libmdsclient')
-rw-r--r-- | src/libmdsclient/comm.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/libmdsclient/comm.c b/src/libmdsclient/comm.c index 6c72e9f..7a3ba20 100644 --- a/src/libmdsclient/comm.c +++ b/src/libmdsclient/comm.c @@ -86,12 +86,7 @@ void libmds_connection_destroy(libmds_connection_t* restrict this) if (this->socket_fd >= 0) { - while (close(this->socket_fd)) - { - if (errno == EINTR) - continue; - break; /* errno may be EBADF or EIO. */ - } + close(this->socket_fd); /* TODO Linux closes the filedescriptor on EINTR, but POSIX does not require that. */ this->socket_fd = -1; } |