osaf/services/saf/logsv/lgs/lgs_stream.c | 4 ---- 1 files changed, 0 insertions(+), 4 deletions(-)
In case of close(), the first thing Linux does is deallocate the file descriptor; then it starts flushing pending written data. If this process is interrupted, it will return EINTR, but the file descriptor itself is already deallocated, and may have been reused long before close() returned. Hence it is not required to retry close(). diff --git a/osaf/services/saf/logsv/lgs/lgs_stream.c b/osaf/services/saf/logsv/lgs/lgs_stream.c --- a/osaf/services/saf/logsv/lgs/lgs_stream.c +++ b/osaf/services/saf/logsv/lgs/lgs_stream.c @@ -46,12 +46,8 @@ static int fileclose(int fd) TRACE_ENTER2("fd=%d", fd); -close_retry: rc = close(fd); - if (rc == -1 && errno == EINTR) - goto close_retry; - if (rc == -1) { LOG_ER("fileclose() %s",strerror(errno)); } ------------------------------------------------------------------------------ "Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE Instantly run your Selenium tests across 300+ browser/OS combos. Get unparalleled scalability from the best Selenium testing platform available. Simple to use. Nothing to install. Get started now for free." http://p.sf.net/sfu/SauceLabs _______________________________________________ Opensaf-devel mailing list Opensaf-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-devel