diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2017-10-31 15:01:23 +0100 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2017-11-01 11:56:19 +0100 |
| commit | 25cb41d35daad7544518dfd43c522355b13d9d79 (patch) | |
| tree | e3e3542ad7a0303fcf9ee03450047d1cdaa4e03c | |
| parent | cda89c8b58bc6421911e6455120b86d39bf4c2b5 (diff) | |
CURLOPT_MAXREDIRS: allow -1 as a value
... which is valid according to documentation. Regression since
f121575c0b5f.
Verified now in test 501.
Reported-by: cbartl on github
Fixes #2038
Closes #2039
| -rw-r--r-- | lib/url.c | 2 | ||||
| -rw-r--r-- | tests/libtest/lib501.c | 5 |
2 files changed, 5 insertions, 2 deletions
@@ -1043,7 +1043,7 @@ CURLcode Curl_setopt(struct Curl_easy *data, CURLoption option, * headers. This should mostly be used to detect never-ending loops. */ arg = va_arg(param, long); - if(arg < 0) + if(arg < -1) return CURLE_BAD_FUNCTION_ARGUMENT; data->set.maxredirs = arg; break; diff --git a/tests/libtest/lib501.c b/tests/libtest/lib501.c index 7a2341246..af65bd4d7 100644 --- a/tests/libtest/lib501.c +++ b/tests/libtest/lib501.c @@ -5,7 +5,7 @@ * | (__| |_| | _ <| |___ * \___|\___/|_| \_\_____| * - * Copyright (C) 1998 - 2016, Daniel Stenberg, <daniel@haxx.se>, et al. + * Copyright (C) 1998 - 2017, 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 @@ -44,6 +44,9 @@ int test(char *URL) test_setopt(curl, CURLOPT_HEADER, 1L); + /* just verify that setting this to -1 is fine */ + test_setopt(curl, CURLOPT_MAXREDIRS, -1L); + res = curl_easy_perform(curl); test_cleanup: |