summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* fixup disable tests for --disable-verbose buildsbagder/travis-disable-verboseDaniel Stenberg2018-10-181-2/+4
|
* fixup sendf.h #errror if no variadic macro define is setDaniel Stenberg2018-10-181-1/+1
|
* fixup more disable-verbose warningsDaniel Stenberg2018-10-183-11/+26
|
* fixup doh.c disable verbose warningsDaniel Stenberg2018-10-181-0/+8
|
* fixup http2 warnings on disabled verbose stringsDaniel Stenberg2018-10-171-1/+4
|
* fixup -Wno-variadic-macrosDaniel Stenberg2018-10-171-1/+1
|
* travis: add build for "configure --disable-verbose"Daniel Stenberg2018-10-161-0/+4
|
* x509asn1: Fix SAN IP address verificationMatthew Whitehead2018-10-161-2/+2
| | | | | | | | | | | | | | | For IP addresses in the subject alternative name field, the length of the IP address (and hence the number of bytes to perform a memcmp on) is incorrectly calculated to be zero. The code previously subtracted q from name.end. where in a successful case q = name.end and therefore addrlen equalled 0. The change modifies the code to subtract name.beg from name.end to calculate the length correctly. The issue only affects libcurl with GSKit SSL, not other SSL backends. The issue is not a security issue as IP verification would always fail. Fixes #3102 Closes #3141
* INSTALL: mention mesalink in TLS sectionDaniel Gustafsson2018-10-151-0/+1
| | | | | | | | | Commit 57348eb97d1b8fc3742e02c6587d2d02ff592da5 added support for the MesaLink vtls backend, but missed updating the TLS section containing supported backends in the docs. Closes #3134 Reviewed-by: Daniel Stenberg <daniel@haxx.se>
* nonblock: fix unused parameter warningMarcel Raad2018-10-141-1/+2
| | | | | If USE_BLOCKING_SOCKETS is defined, curlx_nonblock's arguments are not used.
* Curl_follow: Always free the passed new URLMichael Kaufmann2018-10-132-1/+4
| | | | Closes #3124
* replace rawgit links [ci skip]Viktor Szakats2018-10-122-2/+2
| | | | | | Ref: https://rawgit.com/ "RawGit has reached the end of its useful life" Ref: https://news.ycombinator.com/item?id=18202481 Closes https://github.com/curl/curl/pull/3131
* docs/BUG-BOUNTY.md: for vulns published since Aug 1st 2018Daniel Stenberg2018-10-121-0/+4
| | | | [ci skip]
* travis: make distcheck scan for BOM markersDaniel Stenberg2018-10-122-1/+3
| | | | | | and remove BOM from projects/wolfssl_override.props Closes #3126
* CMake: remove BOMMarcel Raad2018-10-111-1/+1
| | | | | | | Accidentally aded in commit 1bb86057ff07083deeb0b00f8ad35879ec4d03ea. Reported-by: Viktor Szakats Ref: https://github.com/curl/curl/pull/3120#issuecomment-428673136
* transfer: fix typo in commentDaniel Gustafsson2018-10-101-1/+1
|
* docs: add "see also" links for SSL optionsMichael Kaufmann2018-10-106-6/+14
| | | | | | | - link TLS 1.2 and TLS 1.3 options - link proxy and non-proxy options Closes #3121
* AppVeyor: remove BDIR variable that sneaked in againMarcel Raad2018-10-101-1/+0
| | | | | Removed in ae762e1abebe3a5fe75658583c85059a0957ef6e, accidentally added again in 9f3be5672dc4dda30ab43e0152e13d714a84d762.
* CMake: disable -Wpedantic-ms-formatMarcel Raad2018-10-101-2/+2
| | | | | | | As done in the autotools build. This is required for MinGW, which supports only %I64 for printing 64-bit values, but warns about it. Closes https://github.com/curl/curl/pull/3120
* ldap: show precise LDAP call in error message on WindowsViktor Szakats2018-10-091-1/+7
| | | | | | | | | | Also add a unique but common text ('bind via') to make it easy to grep this specific failure regardless of platform. Ref: https://github.com/curl/curl/pull/878/files#diff-7a636f08047c4edb53a240f540b4ecf6R468 Closes https://github.com/curl/curl/pull/3118 Reviewed-by: Daniel Stenberg <daniel@haxx.se> Reviewed-by: Marcel Raad <Marcel.Raad@teamviewer.com>
* docs/DEPRECATE: minor reformat to render nicer on webDaniel Stenberg2018-10-091-1/+1
|
* CURLOPT_SSL_VERIFYSTATUS: Fix typoDaniel Gustafsson2018-10-091-2/+2
| | | | Changes s/OSCP/OCSP/ and bumps the copyright year due to the change.
* curl_setup: define NOGDI on WindowsMarcel Raad2018-10-091-0/+3
| | | | | | | This avoids an ERROR macro clash between <wingdi.h> and <arpa/tftp.h> on MinGW. Closes https://github.com/curl/curl/pull/3113
* Windows: fixes for MinGW targeting Windows VistaMarcel Raad2018-10-094-4/+8
| | | | | | | Classic MinGW has neither InitializeCriticalSectionEx nor GetTickCount64, independent of the target Windows version. Closes https://github.com/curl/curl/pull/3113
* TODO: fixed 'API for URL parsing/splitting'Daniel Stenberg2018-10-081-8/+0
|
* KNOWN_BUGS: Fix various typosDaniel Gustafsson2018-10-081-6/+6
| | | | | Closes #3112 Reviewed-by: Daniel Stenberg <daniel@haxx.se>
* spelling fixes [ci skip]Viktor Szakats2018-10-0811-11/+11
| | | | | | | as detected by codespell 1.14.0 Closes https://github.com/curl/curl/pull/3114 Reviewed-by: Marcel Raad <Marcel.Raad@teamviewer.com>
* RELEASE-NOTES: syncedDaniel Stenberg2018-10-081-22/+37
|
* curl_ntlm_wb: check aprintf() return codesDaniel Stenberg2018-10-081-0/+6
| | | | | | ... when they return NULL we're out of memory and MUST return failure. closes #3111
* docs/BUG-BOUNTY: proposed additional docsDaniel Stenberg2018-10-081-0/+78
| | | | | | Bug bounty explainer. See https://bountygraph.com/programs/curl Closes #3067
* hostip: fix check on Curl_shuffle_addr return valueRick Deist2018-10-081-1/+1
| | | | Closes #3110
* FILE: fix CURLOPT_NOBODY and CURLOPT_HEADER outputDaniel Stenberg2018-10-0833-86/+82
| | | | | | | | | | | | | | | Now FILE transfers send headers to the header callback like HTTP and other protocols. Also made curl_easy_getinfo(...CURLINFO_PROTOCOL...) work for FILE in the callbacks. Makes "curl -i file://.." and "curl -I file://.." work like before again. Applied the bold header logic to them too. Regression from c1c2762 (7.61.0) Reported-by: Shaun Jackman Fixes #3083 Closes #3101
* gskit: make sure to terminate version stringDaniel Gustafsson2018-10-071-2/+1
| | | | | | | | | | | | In case a very small buffer was passed to the version function, it could result in the buffer not being NULL-terminated since strncpy() doesn't guarantee a terminator on an overflowed buffer. Rather than adding code to terminate (and handle zero-sized buffers), move to using snprintf() instead like all the other vtls backends. Closes #3105 Reviewed-by: Daniel Stenberg <daniel@haxx.se> Reviewed-by: Viktor Szakats <commit@vszakats.net>
* TODO: add LD_PRELOAD support on macOSDaniel Gustafsson2018-10-071-0/+6
| | | | Add DYLD_INSERT_LIBRARIES support to the TODO list. Reported in #2394.
* runtests: skip ld_preload tests on macOSDaniel Gustafsson2018-10-071-1/+1
| | | | | | | | | | The LD_PRELOAD functionality doesn't exist on macOS, so skip any tests requiring it. Fixes #2394 Closes #3106 Reported-by: Github user @jakirkham Reviewed-by: Daniel Stenberg <daniel@haxx.se>
* AppVeyor: use Debug builds to run testsMarcel Raad2018-10-071-5/+6
| | | | | | This enables more tests. Closes https://github.com/curl/curl/pull/3104
* AppVeyor: add HTTP_ONLY buildMarcel Raad2018-10-071-0/+16
| | | | Closes https://github.com/curl/curl/pull/3104
* AppVeyor: add WinSSL buildsMarcel Raad2018-10-071-8/+11
| | | | | | | Use the oldest and latest Windows SDKs for them. Also, remove all but one OpenSSL build. Closes https://github.com/curl/curl/pull/3104
* AppVeyor: add remaining Visual Studio versionsMarcel Raad2018-10-071-4/+9
| | | | | | | | This adds Visual Studio 9 and 10 builds. There's no 64-bit VC9 compiler on AppVeyor, so use it as the Win32 build. Also, VC9 cannot be used for running the test suite. Closes https://github.com/curl/curl/pull/3104
* AppVeyor: break long lineMarcel Raad2018-10-071-1/+9
| | | | Closes https://github.com/curl/curl/pull/3104
* AppVeyor: remove unused BDIR variableMarcel Raad2018-10-071-7/+0
| | | | Closes https://github.com/curl/curl/pull/3104
* test2100: test DoH using IPv4-onlyDaniel Stenberg2018-10-061-0/+0
| | | | | | | | | To make it only send one DoH request and avoid the race condition that could lead to the requests getting sent in reversed order and thus making it hard to compare in the test case. Fixes #3107 Closes #3108
* tests/FILEFORMAT: mention how to use <fileN> and <stripfileN> tooDaniel Stenberg2018-10-061-0/+19
| | | | [ci skip]
* RELEASE-NOTES: syncedDaniel Stenberg2018-10-051-9/+39
|
* timeval: fix use of weak symbol clock_gettime() on Apple platformsdmitrykos2018-10-051-2/+18
| | | | Closes #3048
* doh: keep the IPv4 address in (original) network byte orderDaniel Stenberg2018-10-053-24/+19
| | | | | | | | Ideally this will fix the reversed order shown in SPARC tests: resp 8: Expected 127.0.0.1 got 1.0.0.127 Closes #3091
* INTERNALS.md: wrap lines longer than 79Jay Satiro2018-10-051-2/+5
|
* INTERNALS: escape reference to parameterDaniel Gustafsson2018-10-051-2/+2
| | | | | | | | | | The parameter reference <string> was causing rendering issues in the generated HTML page, as <string> isn't a valid HTML tag. Fix by back- tick escaping it. Closes #3099 Reviewed-by: Jay Satiro <raysatiro@yahoo.com> Reviewed-by: Daniel Stenberg <daniel@haxx.se>
* checksrc: handle zero scoped ignore commandsDaniel Gustafsson2018-10-051-1/+10
| | | | | | | | | | | | | | | | If a !checksrc! disable command specified to ignore zero errors, it was still added to the ignore block even though nothing was ignored. While there were no blocks ignored that shouldn't be ignored, the processing ended with with a warning: <filename>:<line>:<col>: warning: Unused ignore: LONGLINE (UNUSEDIGNORE) /* !checksrc! disable LONGLINE 0 */ ^ Fix by instead treating a zero ignore as a a badcommand and throw a warning for that one. Closes #3096 Reviewed-by: Daniel Stenberg <daniel@haxx.se>
* checksrc: enable strict mode and warningsDaniel Gustafsson2018-10-051-8/+18
| | | | | | | | | | | | | | | | | | Enable strict and warnings mode for checksrc to ensure we aren't missing anything due to bugs in the checking code. This uncovered a few things which are all fixed in this commit: * several variables were used uninitialized * several variables were not defined in the correct scope * the whitelist filehandle was read even if the file didn't exist * the enable_warn() call when a disable counter had expired was passing incorrect variables, but since the checkwarn() call is unlikely to hit (the counter is only decremented to zero on actual ignores) it didn't manifest a problem. Closes #3090 Reviewed-by: Daniel Stenberg <daniel@haxx.se> Reviewed-by: Marcel Raad <Marcel.Raad@teamviewer.com>