diff options
author | Daniel Stenberg <daniel@haxx.se> | 2020-02-28 23:55:05 +0100 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2020-02-29 10:53:09 +0100 |
commit | fa0216b294af4c7113a9040ca65eefc7fc18ac1c (patch) | |
tree | fa845a386251732a7dcf6891d9f0c0e843faa033 | |
parent | 0e06c1637b30800d41636dcd02f5becf3f6664c1 (diff) |
pause: force-drain the transfer on unpause
... since the socket might not actually be readable anymore when for
example the data is already buffered in the TLS layer.
Fixes #4966
Reported-by: Anders Berg
Closes #5000
-rw-r--r-- | lib/easy.c | 1 | ||||
-rw-r--r-- | lib/transfer.c | 5 |
2 files changed, 4 insertions, 2 deletions
diff --git a/lib/easy.c b/lib/easy.c index 454621076..1a6912748 100644 --- a/lib/easy.c +++ b/lib/easy.c @@ -1033,6 +1033,7 @@ CURLcode curl_easy_pause(struct Curl_easy *data, int action) to have this handle checked soon */ if((newstate & (KEEP_RECV_PAUSE|KEEP_SEND_PAUSE)) != (KEEP_RECV_PAUSE|KEEP_SEND_PAUSE)) { + data->state.drain++; Curl_expire(data, 0, EXPIRE_RUN_NOW); /* get this handle going again */ if(data->multi) Curl_update_timer(data->multi); diff --git a/lib/transfer.c b/lib/transfer.c index ead8b36db..827076183 100644 --- a/lib/transfer.c +++ b/lib/transfer.c @@ -5,7 +5,7 @@ * | (__| |_| | _ <| |___ * \___|\___/|_| \_\_____| * - * Copyright (C) 1998 - 2019, Daniel Stenberg, <daniel@haxx.se>, et al. + * Copyright (C) 1998 - 2020, Daniel Stenberg, <daniel@haxx.se>, et al. * * This software is licensed as described in the file COPYING, which * you should have received as part of this distribution. The terms @@ -1217,7 +1217,8 @@ CURLcode Curl_readwrite(struct connectdata *conn, else fd_write = CURL_SOCKET_BAD; - if(conn->data->state.drain) { + if(data->state.drain) { + data->state.drain--; select_res |= CURL_CSELECT_IN; DEBUGF(infof(data, "Curl_readwrite: forcibly told to drain data\n")); } |