[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[gnurl] 71/222: curl: exit the create_transfers loop on errors
From: |
gnunet |
Subject: |
[gnurl] 71/222: curl: exit the create_transfers loop on errors |
Date: |
Thu, 07 Nov 2019 00:09:27 +0100 |
This is an automated email from the git hooks/post-receive script.
ng0 pushed a commit to branch master
in repository gnurl.
commit f8a205853381650ea1b7e05abec93cd0ae2b77b5
Author: Daniel Stenberg <address@hidden>
AuthorDate: Sat Sep 21 23:46:41 2019 +0200
curl: exit the create_transfers loop on errors
When looping around the ranges and given URLs to create transfers, all
errors should exit the loop and return. Previously it would keep
looping.
Reported-by: SumatraPeter on github
Bug: #4393
Closes #4396
---
src/tool_operate.c | 13 +++++--------
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/src/tool_operate.c b/src/tool_operate.c
index 316951869..ede816d34 100644
--- a/src/tool_operate.c
+++ b/src/tool_operate.c
@@ -1885,15 +1885,9 @@ static CURLcode create_transfers(struct GlobalConfig
*global,
urls = NULL;
}
- if(infilenum > 1) {
- /* when file globbing, exit loop upon critical error */
- if(is_fatal_error(result))
- break;
- }
- else if(result)
- /* when not file globbing, exit loop upon any error */
+ if(result)
+ /* exit loop upon error */
break;
-
} /* loop to the next globbed upload file */
/* Free loop-local allocated memory */
@@ -1913,6 +1907,9 @@ static CURLcode create_transfers(struct GlobalConfig
*global,
Curl_safefree(urlnode->infile);
urlnode->flags = 0;
+ if(result)
+ /* exit loop upon error */
+ break;
} /* for-loop through all URLs */
quit_curl:
--
To stop receiving notification emails like this one, please contact
address@hidden.
- [gnurl] 52/222: urlapi: part of conditional expression is always true: (relurl[0] == '/'), (continued)
- [gnurl] 52/222: urlapi: part of conditional expression is always true: (relurl[0] == '/'), gnunet, 2019/11/06
- [gnurl] 56/222: tool_getparam: remove duplicate switch case, gnunet, 2019/11/06
- [gnurl] 65/222: appveyor: upgrade VS2017 to VS2019, gnunet, 2019/11/06
- [gnurl] 62/222: urlapi: avoid index underflow for short ipv6 hostnames, gnunet, 2019/11/06
- [gnurl] 63/222: cookie: pass in the correct cookie amount to qsort(), gnunet, 2019/11/06
- [gnurl] 66/222: urldata: use 'bool' for the bit type on MSVC compilers, gnunet, 2019/11/06
- [gnurl] 73/222: RELEASE-NOTES: synced, gnunet, 2019/11/06
- [gnurl] 72/222: openssl: fix compiler warning with LibreSSL, gnunet, 2019/11/06
- [gnurl] 79/222: vauth: The parameter 'status' must be surrounded by parentheses, gnunet, 2019/11/06
- [gnurl] 78/222: doh: allow only http and https in debug mode, gnunet, 2019/11/06
- [gnurl] 71/222: curl: exit the create_transfers loop on errors,
gnunet <=
- [gnurl] 80/222: quiche: The expression must be surrounded by parentheses, gnunet, 2019/11/06
- [gnurl] 69/222: travis: enable ngtcp2 h3-23 builds, gnunet, 2019/11/06
- [gnurl] 86/222: strcase: fix raw lowercasing the letter X, gnunet, 2019/11/06
- [gnurl] 85/222: http2: Expression 'stream->stream_id != - 1' is always true, gnunet, 2019/11/06
- [gnurl] 83/222: libssh: part of conditional expression is always true: !result, gnunet, 2019/11/06
- [gnurl] 76/222: http: lowercase headernames for HTTP/2 and HTTP/3, gnunet, 2019/11/06
- [gnurl] 84/222: http2: A value is being subtracted from the unsigned variable, gnunet, 2019/11/06
- [gnurl] 70/222: socks: Fix destination host shown on SOCKS5 error, gnunet, 2019/11/06
- [gnurl] 74/222: winbuild: Add manifest to curl.exe for proper OS version detection, gnunet, 2019/11/06
- [gnurl] 82/222: libssh: part of conditional expression is always true, gnunet, 2019/11/06