diff options
Diffstat (limited to 'src/wsocket.c')
| -rw-r--r-- | src/wsocket.c | 37 |
1 files changed, 23 insertions, 14 deletions
diff --git a/src/wsocket.c b/src/wsocket.c index d34724b..b4a4384 100644 --- a/src/wsocket.c +++ b/src/wsocket.c | |||
| @@ -238,8 +238,10 @@ int socket_sendto(p_socket ps, const char *data, size_t count, size_t *sent, | |||
| 238 | /*-------------------------------------------------------------------------*\ | 238 | /*-------------------------------------------------------------------------*\ |
| 239 | * Receive with timeout | 239 | * Receive with timeout |
| 240 | \*-------------------------------------------------------------------------*/ | 240 | \*-------------------------------------------------------------------------*/ |
| 241 | int socket_recv(p_socket ps, char *data, size_t count, size_t *got, p_timeout tm) { | 241 | int socket_recv(p_socket ps, char *data, size_t count, size_t *got, |
| 242 | int err; | 242 | p_timeout tm) |
| 243 | { | ||
| 244 | int err, prev = IO_DONE; | ||
| 243 | *got = 0; | 245 | *got = 0; |
| 244 | if (*ps == SOCKET_INVALID) return IO_CLOSED; | 246 | if (*ps == SOCKET_INVALID) return IO_CLOSED; |
| 245 | for ( ;; ) { | 247 | for ( ;; ) { |
| @@ -250,11 +252,14 @@ int socket_recv(p_socket ps, char *data, size_t count, size_t *got, p_timeout tm | |||
| 250 | } | 252 | } |
| 251 | if (taken == 0) return IO_CLOSED; | 253 | if (taken == 0) return IO_CLOSED; |
| 252 | err = WSAGetLastError(); | 254 | err = WSAGetLastError(); |
| 253 | /* On Windows, and on UDP, a connreset simply means the | 255 | /* On UDP, a connreset simply means the previous send failed. |
| 254 | * previous send failed. On TCP, it means our socket | 256 | * So we try again. |
| 255 | * is now useless, so the error must pass. I am | 257 | * On TCP, it means our socket is now useless, so the error passes. |
| 256 | * hoping waitfd will still get the error. */ | 258 | * (We will loop again, exiting because the same error will happen) */ |
| 257 | if (err != WSAEWOULDBLOCK && err != WSAECONNRESET) return err; | 259 | if (err != WSAEWOULDBLOCK) { |
| 260 | if (err != WSAECONNRESET || prev == WSAECONNRESET) return err; | ||
| 261 | prev = err; | ||
| 262 | } | ||
| 258 | if ((err = socket_waitfd(ps, WAITFD_R, tm)) != IO_DONE) return err; | 263 | if ((err = socket_waitfd(ps, WAITFD_R, tm)) != IO_DONE) return err; |
| 259 | } | 264 | } |
| 260 | } | 265 | } |
| @@ -263,8 +268,9 @@ int socket_recv(p_socket ps, char *data, size_t count, size_t *got, p_timeout tm | |||
| 263 | * Recvfrom with timeout | 268 | * Recvfrom with timeout |
| 264 | \*-------------------------------------------------------------------------*/ | 269 | \*-------------------------------------------------------------------------*/ |
| 265 | int socket_recvfrom(p_socket ps, char *data, size_t count, size_t *got, | 270 | int socket_recvfrom(p_socket ps, char *data, size_t count, size_t *got, |
| 266 | SA *addr, socklen_t *len, p_timeout tm) { | 271 | SA *addr, socklen_t *len, p_timeout tm) |
| 267 | int err; | 272 | { |
| 273 | int err, prev = IO_DONE; | ||
| 268 | *got = 0; | 274 | *got = 0; |
| 269 | if (*ps == SOCKET_INVALID) return IO_CLOSED; | 275 | if (*ps == SOCKET_INVALID) return IO_CLOSED; |
| 270 | for ( ;; ) { | 276 | for ( ;; ) { |
| @@ -275,11 +281,14 @@ int socket_recvfrom(p_socket ps, char *data, size_t count, size_t *got, | |||
| 275 | } | 281 | } |
| 276 | if (taken == 0) return IO_CLOSED; | 282 | if (taken == 0) return IO_CLOSED; |
| 277 | err = WSAGetLastError(); | 283 | err = WSAGetLastError(); |
| 278 | /* On Windows, and on UDP, a connreset simply means the | 284 | /* On UDP, a connreset simply means the previous send failed. |
| 279 | * previous send failed. On TCP, it means our socket | 285 | * So we try again. |
| 280 | * is now useless, so the error must pass. I am | 286 | * On TCP, it means our socket is now useless, so the error passes. |
| 281 | * hoping waitfd will still get the error. */ | 287 | * (We will loop again, exiting because the same error will happen) */ |
| 282 | if (err != WSAEWOULDBLOCK && err != WSAECONNRESET) return err; | 288 | if (err != WSAEWOULDBLOCK) { |
| 289 | if (err != WSAECONNRESET || prev == WSAECONNRESET) return err; | ||
| 290 | prev = err; | ||
| 291 | } | ||
| 283 | if ((err = socket_waitfd(ps, WAITFD_R, tm)) != IO_DONE) return err; | 292 | if ((err = socket_waitfd(ps, WAITFD_R, tm)) != IO_DONE) return err; |
| 284 | } | 293 | } |
| 285 | } | 294 | } |
