[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[gnurl] 40/222: ftp: Expression 'ftpc->wait_data_conn' is always false
From: |
gnunet |
Subject: |
[gnurl] 40/222: ftp: Expression 'ftpc->wait_data_conn' is always false |
Date: |
Thu, 07 Nov 2019 00:08:56 +0100 |
This is an automated email from the git hooks/post-receive script.
ng0 pushed a commit to branch master
in repository gnurl.
commit 49f3117a238b6eac0e22a32f50699a9eddcb66ab
Author: Daniel Stenberg <address@hidden>
AuthorDate: Thu Sep 19 09:18:26 2019 +0200
ftp: Expression 'ftpc->wait_data_conn' is always false
Fixes warning detected by PVS-Studio
Fixes #4374
---
lib/ftp.c | 9 ++-------
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/lib/ftp.c b/lib/ftp.c
index a4a37c9af..4cd9a1acf 100644
--- a/lib/ftp.c
+++ b/lib/ftp.c
@@ -3570,13 +3570,8 @@ static CURLcode ftp_do_more(struct connectdata *conn,
int *completep)
return result;
result = ftp_multi_statemach(conn, &complete);
- if(ftpc->wait_data_conn)
- /* if we reach the end of the FTP state machine here, *complete will be
- TRUE but so is ftpc->wait_data_conn, which says we need to wait for
- the data connection and therefore we're not actually complete */
- *completep = 0;
- else
- *completep = (int)complete;
+ /* ftpc->wait_data_conn is always false here */
+ *completep = (int)complete;
}
else {
/* download */
--
To stop receiving notification emails like this one, please contact
address@hidden.
- [gnurl] 46/222: easy: part of conditional expression is always true: !result, (continued)
- [gnurl] 46/222: easy: part of conditional expression is always true: !result, gnunet, 2019/11/06
- [gnurl] 30/222: http: merge two "case" statements, gnunet, 2019/11/06
- [gnurl] 36/222: doh: avoid truncating DNS QTYPE to lower octet, gnunet, 2019/11/06
- [gnurl] 29/222: FTP: remove trailing slash from path for LIST/MLSD, gnunet, 2019/11/06
- [gnurl] 43/222: url: remove dead code, gnunet, 2019/11/06
- [gnurl] 42/222: url: part of expression is always true: (bundle->multiuse == 0), gnunet, 2019/11/06
- [gnurl] 44/222: version: Expression 'left > 1' is always true, gnunet, 2019/11/06
- [gnurl] 34/222: version: next release will be 7.67.0, gnunet, 2019/11/06
- [gnurl] 33/222: RELEASE-NOTES: synced, gnunet, 2019/11/06
- [gnurl] 31/222: README: add OSS-Fuzz badge [skip ci], gnunet, 2019/11/06
- [gnurl] 40/222: ftp: Expression 'ftpc->wait_data_conn' is always false,
gnunet <=
- [gnurl] 45/222: netrc: part of conditional expression is always true: !done, gnunet, 2019/11/06
- [gnurl] 35/222: urlapi: CURLU_NO_AUTHORITY allows empty authority/host part, gnunet, 2019/11/06
- [gnurl] 38/222: ftp: part of conditional expression is always true: !result, gnunet, 2019/11/06
- [gnurl] 37/222: http: fix Expression 'http->postdata' is always false, gnunet, 2019/11/06
- [gnurl] 41/222: ftp: the conditional expression is always true, gnunet, 2019/11/06
- [gnurl] 54/222: urlapi: Expression 'storep' is always true, gnunet, 2019/11/06
- [gnurl] 47/222: multi: value '2L' is assigned to a boolean, gnunet, 2019/11/06
- [gnurl] 55/222: libssh2: part of conditional expression is always true: !result, gnunet, 2019/11/06
- [gnurl] 53/222: urlapi: 'scheme' is always true, gnunet, 2019/11/06
- [gnurl] 58/222: tool_operate: removed unused variable 'done', gnunet, 2019/11/06