summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEmil Engler <me@emilengler.com>2020-11-25 18:26:56 +0100
committerDaniel Stenberg <daniel@haxx.se>2020-11-25 23:39:28 +0100
commit12cb7a1fe0282cc0b1ef4faa6013a2de03028aac (patch)
tree8f1bb2431df1913a1b699d86f95392612ae18f54
parent920f49a20be51170ec1804f2736ea3e2593cdbe5 (diff)
downloadcurl-12cb7a1fe0282cc0b1ef4faa6013a2de03028aac.tar.gz
file: avoid duplicated code sequence
file_disconnect() is identical with file_do() except the function header but as the arguments are unused anyway so why not just return file_do() directly! Reviewed-by: Daniel Stenberg Closes #6249
-rw-r--r--lib/file.c12
1 files changed, 1 insertions, 11 deletions
diff --git a/lib/file.c b/lib/file.c
index 5f06c4cca..a65eb7798 100644
--- a/lib/file.c
+++ b/lib/file.c
@@ -227,18 +227,8 @@ static CURLcode file_done(struct connectdata *conn,
static CURLcode file_disconnect(struct connectdata *conn,
bool dead_connection)
{
- struct FILEPROTO *file = conn->data->req.p.file;
(void)dead_connection; /* not used */
-
- if(file) {
- Curl_safefree(file->freepath);
- file->path = NULL;
- if(file->fd != -1)
- close(file->fd);
- file->fd = -1;
- }
-
- return CURLE_OK;
+ return file_done(conn, 0, 0);
}
#ifdef DOS_FILESYSTEM