- Nov 09, 2017
-
-
Daniel Stenberg authored
... previously it would store it already in the happy eyeballs stage which could lead to the IPv6 bit being set for an IPv4 connection, leading to curl not wanting to do EPSV=>PASV for FTP transfers. Closes #2053
-
Jay Satiro authored
- Don't call zlib's inflate() when avail_in stream bytes is 0. This is a follow up to the parent commit 19e66e53. Prior to that change libcurl's inflate_stream could call zlib's inflate even when no bytes were available, causing inflate to return Z_BUF_ERROR, and then inflate_stream would treat that as a hard error and return CURLE_BAD_CONTENT_ENCODING. According to the zlib FAQ, Z_BUF_ERROR is not fatal. This bug would happen randomly since packet sizes are arbitrary. A test of 10,000 transfers had 55 fail (ie 0.55%). Ref: https://zlib.net/zlib_faq.html#faq05 Closes https://github.com/curl/curl/pull/2060
-
- Nov 07, 2017
-
-
Patrick Monnerat authored
-
- Nov 06, 2017
-
-
Daniel Stenberg authored
There was a duplicate check for backslashes in the setcharset() function. Coverity CID 1420611
-
Daniel Stenberg authored
Since 'conn' won't be NULL in there and we also access the pointer in there without the check. Coverity CID 1420610
-
- Nov 05, 2017
-
-
Viktor Szakats authored
It adds the ability to link against static brotli libs. Also fix brotli include path.
-
Viktor Szakats authored
-
Patrick Monnerat authored
This uses the brotli external library (https://github.com/google/brotli). Brotli becomes a feature: additional curl_version_info() bit and structure fields are provided for it and CURLVERSION_NOW bumped. Tests 314 and 315 check Brotli content unencoding with correct and erroneous data. Some tests are updated to accomodate with the now configuration dependent parameters of the Accept-Encoding header.
-
Patrick Monnerat authored
This is implemented as an output streaming stack of unencoders, the last calling the client write procedure. New test 230 checks this feature. Bug: https://github.com/curl/curl/pull/2002 Reported-By: Daniel Bankhead
-
- Nov 04, 2017
-
-
Jay Satiro authored
Since CURLSSH_AUTH_ANY (aka CURLSSH_AUTH_DEFAULT) is ~0 an arg value check on this option is incorrect; we have to accept any value. Prior to this change since f121575c (7.56.1+) CURLOPT_SSH_AUTH_TYPES erroneously rejected CURLSSH_AUTH_ANY with CURLE_BAD_FUNCTION_ARGUMENT. Bug: https://github.com/curl/curl/commit/f121575#commitcomment-25347120
-
Daniel Stenberg authored
It triggers an assert() when built with memdebug since malloc(0) may return NULL *or* a valid pointer. Detected by OSS-Fuzz: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=4054 Assisted-by: Max Dymond Closes #2054
-
- Nov 03, 2017
-
-
Daniel Stenberg authored
Regression since f121575c Reported-by: Petr Voytsik Fixes #2047
-
- Nov 02, 2017
-
-
Jay Satiro authored
Prior to this change since f121575c (7.56.1+) CURLOPT_DNS_CACHE_TIMEOUT erroneously rejected -1 with CURLE_BAD_FUNCTION_ARGUMENT.
-
- Nov 01, 2017
-
-
Dan Fandrich authored
This caused the torture tests on test 1800 to fail.
-
Daniel Stenberg authored
... which is valid according to documentation. Regression since f121575c. Verified now in test 501. Reported-by: cbartl on github Fixes #2038 Closes #2039
-
Daniel Stenberg authored
-
Jay Satiro authored
.. also add same arg value check to CURLOPT_POSTFIELDSIZE_LARGE. Prior to this change since f121575c (7.56.1+) CURLOPT_POSTFIELDSIZE erroneously rejected -1 value with CURLE_BAD_FUNCTION_ARGUMENT. Bug: https://curl.haxx.se/mail/lib-2017-11/0000.html Reported-by: Andrew Lambert
-
- Oct 31, 2017
-
-
Daniel Stenberg authored
... when expiring old cookies. Reported-by: Pavel Gushchin Fixes #2032 Closes #2035
-
- Oct 30, 2017
-
-
Marcel Raad authored
This avoids build errors and warnings caused by implicit casts. Closes https://github.com/curl/curl/pull/2031
-
Daniel Stenberg authored
Pointed-out-by: Dmitri Tikhonov Bug: #2034
-
Daniel Stenberg authored
s/curlx_tvnow/Curl_now
-
Dmitri Tikhonov authored
If clock_gettime() is not supported, use mach_absolute_time() on MacOS. closes #2033
-
- Oct 29, 2017
-
-
Daniel Stenberg authored
-
- Oct 28, 2017
-
-
Jakub Zakrzewski authored
The config files define curl and libcurl targets as imported targets CURL::curl and CURL::libcurl. For backward compatibility with CMake- provided find-module the CURL_INCLUDE_DIRS and CURL_LIBRARIES are also set. Closes #1879
-
Florin Petriuc authored
Signed-off-by: Florin <petriuc.florin@gmail.com>
-
Daniel Stenberg authored
returning 'time_t' is problematic when that type is unsigned and we return values less than zero to signal "already expired", used in several places in the code. Closes #2021
-
- Oct 27, 2017
-
-
Daniel Stenberg authored
Regression since f121575c Reported-by: Rob Cotrone
-
Marcel Raad authored
This avoids warnings about unused stuff. Closes https://github.com/curl/curl/pull/2023
-
Daniel Stenberg authored
-
Martin Storsjo authored
If WINAPI_FAMILY is defined, it should be safe to try to include winapifamily.h to check what the define evaluates to. This should fix detection of CURL_WINDOWS_APP if building with _WIN32_WINNT set to 0x0600. Closes #2025
-
- Oct 26, 2017
-
-
Jay Satiro authored
- When uploading via chunked-encoding don't compare file size to bytes sent to determine whether the upload has finished. Chunked-encoding adds its own overhead which why the bytes sent is not equal to the file size. Prior to this change if a file was uploaded in chunked-encoding and its size was known it was possible that the upload could end prematurely without sending the final few chunks. That would result in a server hang waiting for the remaining data, likely followed by a disconnect. The scope of this bug is limited to some arbitrary file sizes which have not been determined. One size that triggers the bug is 475020. Bug: https://github.com/curl/curl/issues/2001 Reported-by: <moohoorama@users.noreply.github.com> Closes https://github.com/curl/curl/pull/2010
-
Daniel Stenberg authored
... by using curl_off_t for the typedef if time_t is larger than 4 bytes. Reported-by: Gisle Vanem Bug: https://github.com/curl/curl/commit/b9d25f9a6b3ca791385b80a6a3c3fa5ae113e1e0#co mmitcomment-25205058 Closes #2019
-
Daniel Stenberg authored
... instead of doing an infinite loop! Added test 1162 to verify. Reported-by: Max Dymond Fixes #2015 Closes #2017
-
Max Dymond authored
Fixes timeouts in the fuzzing tests for non-FTP protocols. Closes #2016
-
- Oct 25, 2017
-
-
Max Dymond authored
Allow users to specify their own hash define for CURL_CONNECTION_HASH_SIZE so that both values can be overridden. Closes #1982
-
Daniel Stenberg authored
... since the 'tv' stood for timeval and this function does not return a timeval struct anymore. Also, cleaned up the Curl_timediff*() functions to avoid typecasts and clean up the descriptive comments. Closes #2011
-
Daniel Stenberg authored
-
Max Dymond authored
When using the FTP list parser, ensure that the memory that's allocated is always freed. Detected by OSS-fuzz: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=3682 Closes #2013
-
Daniel Stenberg authored
... to cater for systems with unsigned time_t variables. - Renamed the functions to curlx_timediff and Curl_timediff_us. - Added overflow protection for both of them in either direction for both 32 bit and 64 bit time_ts - Reprefixed the curlx_time functions to use Curl_* Reported-by: Peter Piekarski Fixes #2004 Closes #2005
-
- Oct 22, 2017
-
-
Jon DeVree authored
Aurora is no longer used by Mozilla https://hacks.mozilla.org/2017/04/simplifying-firefox-release-channels/
-