Don't validate FD_SETSIZE on Win32
authorJeetu Golani <jeetu.golani@gmail.com>
Sun, 12 Dec 2010 21:48:41 +0000 (16:48 -0500)
committerPeter Harris <git@peter.is-a-geek.org>
Sun, 12 Dec 2010 21:48:41 +0000 (16:48 -0500)
Windows' file handles have never been small or consecutive, so Windows'
select has always been implemented the same way as everyone else's poll.

On Windows, FD_SETSIZE is the size of the poll array, not the maximum
SOCKET number.

Signed-off-by: Peter Harris <git@peter.is-a-geek.org>
src/xcb_conn.c

index 35b9996..a017c20 100644 (file)
@@ -257,6 +257,7 @@ xcb_connection_t *xcb_connect_to_fd(int fd, xcb_auth_info_t *auth_info)
 {
     xcb_connection_t* c;
 
+#ifndef _WIN32
 #ifndef USE_POLL
     if(fd >= FD_SETSIZE) /* would overflow in FD_SET */
     {
@@ -264,6 +265,7 @@ xcb_connection_t *xcb_connect_to_fd(int fd, xcb_auth_info_t *auth_info)
         return (xcb_connection_t *) &error_connection;
     }
 #endif
+#endif /* !_WIN32*/
 
     c = calloc(1, sizeof(xcb_connection_t));
     if(!c) {