From 56962e42a509dc4d0d9541e46b93689dac61c4fd Mon Sep 17 00:00:00 2001 From: Jeetu Golani Date: Thu, 22 Apr 2010 23:23:27 +0530 Subject: Set errno=0 in read_block. On Win32 there is no errno and this makes the do..while loop execute only once. Also set the return value to -1 in _xcb_open if control reaches the end - if all goes well it shouldn't reach there. --- src/xcb_in.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) (limited to 'src/xcb_in.c') diff --git a/src/xcb_in.c b/src/xcb_in.c index 8689344..b481295 100644 --- a/src/xcb_in.c +++ b/src/xcb_in.c @@ -297,15 +297,12 @@ static int read_block(const int fd, void *buf, const ssize_t len) fd_set fds; FD_ZERO(&fds); FD_SET(fd, &fds); -#ifndef _WIN32 + + /* Initializing errno here makes sure that for Win32 this loop will execute only once */ + errno = 0; do { ret = select(fd + 1, &fds, 0, 0, 0); } while (ret == -1 && errno == EINTR); -#else - /* the do while loop used for the non-windows version isn't required*/ - /* for windows since there are no signals in Windows hence no EINTR*/ - ret = select(fd + 1, &fds, 0, 0, 0); -#endif /* !_WIN32 */ #endif /* USE_POLL */ } if(ret <= 0) -- cgit v1.2.3