From: Daniel Stenberg <daniel@haxx.se>
Date: Thu, 19 Sep 2019 07:14:15 +0000 (+0200)
Subject: ftp: part of conditional expression is always true: !result
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=97c17e9fcbb4d27eec3cf51db8599ea151c16042;p=curl

ftp: part of conditional expression is always true: !result

Fixes warning detected by PVS-Studio
Fixes #4374
---

diff --git a/lib/ftp.c b/lib/ftp.c
index 068a1a024..59ca13ef1 100644
--- a/lib/ftp.c
+++ b/lib/ftp.c
@@ -523,7 +523,7 @@ static CURLcode AllowServerConnect(struct connectdata *conn, bool *connected)
   }
   else {
     /* Add timeout to multi handle and break out of the loop */
-    if(!result && *connected == FALSE) {
+    if(*connected == FALSE) {
       Curl_expire(data, data->set.accepttimeout > 0 ?
                   data->set.accepttimeout: DEFAULT_ACCEPT_TIMEOUT, 0);
     }