Mercurial > vim
diff src/channel.c @ 7982:5c30ba57aaea v7.4.1286
commit https://github.com/vim/vim/commit/7a84dbe6be0ef0e1ffbb7148cfe4ab50b9ba8f41
Author: Bram Moolenaar <Bram@vim.org>
Date: Sun Feb 7 21:29:00 2016 +0100
patch 7.4.1286
Problem: ch_open() with a timeout doesn't work correctly.
Solution: Change how select() is used. Don't give an error on timeout.
Add a test for ch_open() failing.
| author | Christian Brabandt <cb@256bit.org> |
|---|---|
| date | Sun, 07 Feb 2016 21:30:05 +0100 |
| parents | 45ea5ebf3a98 |
| children | 8e0d4cd8157a |
line wrap: on
line diff
--- a/src/channel.c +++ b/src/channel.c @@ -431,18 +431,16 @@ channel_open(char *hostname, int port_in } } - if (waittime >= 0) + if (waittime >= 0 && ret < 0) { struct timeval tv; - fd_set rfds, wfds; + fd_set wfds; - FD_ZERO(&rfds); FD_ZERO(&wfds); - FD_SET(sd, &rfds); FD_SET(sd, &wfds); tv.tv_sec = waittime / 1000; tv.tv_usec = (waittime % 1000) * 1000; - ret = select((int)sd+1, &rfds, &wfds, NULL, &tv); + ret = select((int)sd + 1, NULL, &wfds, NULL, &tv); if (ret < 0) { SOCK_ERRNO; @@ -452,15 +450,16 @@ channel_open(char *hostname, int port_in sock_close(sd); return -1; } - if (!FD_ISSET(sd, &rfds) && !FD_ISSET(sd, &wfds)) + if (!FD_ISSET(sd, &wfds)) { - errno = ECONNREFUSED; - CHERROR("Cannot connect to port\n", ""); - PERROR(_("E902: Cannot connect to port")); + /* don't give an error, we just timed out. */ sock_close(sd); return -1; } + } + if (waittime >= 0) + { #ifdef _WIN32 val = 0; ioctlsocket(sd, FIONBIO, &val);