summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* darwinssl: fix compiler warningsbagder/darwinssl-warningsDaniel Stenberg2017-07-272-14/+16
|
* darwinssl: fix variable type mistake (regression)Daniel Stenberg2017-07-271-1/+2
| | | | | | | | ... which made --tlsv1.2 not work because it would blank the max tls version variable. Reported-by: Nick Miyake Bug: #1703
* multi: mention integer overflow risk if using > 500 million socketsDaniel Stenberg2017-07-271-0/+4
| | | | | | | Reported-by: ovidiu-benea@users.noreply.github.com Closes #1675 Closes #1683
* checksrc: escape open brace in regexDaniel Stenberg2017-07-271-2/+2
| | | | ... to silence warning.
* nss: fix a possible use-after-free in SelectClientCert()Kamil Dudka2017-07-201-0/+8
| | | | | | | | | | ... causing a SIGSEGV in showit() in case the handle used to initiate the connection has already been freed. This commit fixes a bug introduced in curl-7_19_5-204-g5f0cae803. Reported-by: Rob Sanders Bug: https://bugzilla.redhat.com/1436158
* nss: unify the coding style of nss_send() and nss_recv()Kamil Dudka2017-07-201-6/+11
| | | | No changes in behavior intended by this commit.
* tests/server/resolve.c: fix deprecation warningMarcel Raad2017-07-181-26/+28
| | | | | | | | MSVC warns that gethostbyname is deprecated. Always use getaddrinfo instead to fix this when IPv6 is enabled, also for IPv4 resolves. This is also consistent with what libcurl does. Closes https://github.com/curl/curl/pull/1682
* darwinssl: fix pinnedpubkey build errorJay Satiro2017-07-171-1/+1
| | | | | | | - s/SessionHandle/Curl_easy/ Bug: https://github.com/curl/curl/commit/eb16305#commitcomment-23035670 Reported-by: Gisle Vanem
* rtspd: fix GCC warning after MSVC warning fixMarcel Raad2017-07-161-1/+3
| | | | | | | | Older GCC warns: /tests/server/rtspd.c:1194:10: warning: missing braces around initializer [-Wmissing-braces] Fix this by using memset instead of an initializer.
* libtest: fix MSVC warning C4706Marcel Raad2017-07-163-9/+12
| | | | | | With warning level 4, MSVC warns about assignments within conditional expressions. Change the while loop to a do-while loop to fix this. This change is also consistent with CODE_STYLE.md.
* sockfilt: suppress conversion warning with explicit castMarcel Raad2017-07-161-1/+1
| | | | MSVC warns when implicitly casting -1 to unsigned long.
* rtspd: fix MSVC level 4 warningMarcel Raad2017-07-161-1/+1
| | | | warning C4701: potentially uninitialized local variable 'req' used
* winbuild: re-enable warning C4127 for curl toolMarcel Raad2017-07-141-1/+1
| | | | | Disabled in cda19a345f6970e22fe8b7a808aeb8f086a21eac. It only needs to be disabled for libcurl.
* winbuild: build with warning level 4Marcel Raad2017-07-131-4/+4
| | | | | | | | | | This is consistent with 7bc64561a2e63ca93e4b0b31d350773ba80955c2, which changed the warning level from 3 to 4 for the Visual Studio project files. But disable the level 4 warning C4127 "conditional expression is constant", as that one is issued by older versions of the Windows SDK as well as curl itself under some circumstances. Closes https://github.com/curl/curl/pull/1667
* travis: install libidn2Max Dymond2017-07-121-0/+1
| | | | | | Install libidn2 to increase test coverage (IDN tests) Closes https://github.com/curl/curl/pull/1673
* travis: enable warnings also in release modeMarcel Raad2017-07-121-1/+1
| | | | | | ... to get warnings also on Linux/GCC and OSX/clang. Closes https://github.com/curl/curl/pull/1666
* travis: install libssh2Max Dymond2017-07-121-0/+1
| | | | Install libssh2 to increase test coverage (SFTP, SCP)
* system.h: include winsock2.h before windows.hMarcel Raad2017-07-121-1/+1
| | | | | ... to avoid compiler warnings if the user doesn't want WIN32_LEAN_AND_MEAN.
* build: remove WIN32_LEAN_AND_MEAN from individual build systemsMarcel Raad2017-07-112-5/+0
| | | | | | | | It's defined for all build systems in curl_setup.h since commit beb08481d01a07a8b10938b1078a5e298b1c2912. This caused macro redefinition warnings in the configure builds. Closes https://github.com/curl/curl/pull/1677
* ISSUE_TEMPLATE: Add a comment not to file security issues on githubJay Satiro2017-07-111-2/+2
|
* curl_setup: always define WIN32_LEAN_AND_MEAN on WindowsMarcel Raad2017-07-113-7/+12
| | | | | | | | | Make sure to always define WIN32_LEAN_AND_MEAN before including any Windows headers to avoid pulling in unnecessary headers. This avoids unnecessary macro clashes and compiler warnings. Ref: https://github.com/curl/curl/issues/1562 Closes https://github.com/curl/curl/pull/1672
* strerror: Preserve Windows error code in some functionsJay Satiro2017-07-116-48/+51
| | | | | | | | | | | This is a follow-up to af02162 which removed (SET_)ERRNO macros. That commit was an earlier draft that I committed by mistake, which was then remedied by a5834e5 and e909de6, and now this commit. With this commit there is now no difference between the current code and the changes that were approved in the final draft. Thanks-to: Max Dymond, Marcel Raad, Daniel Stenberg, Gisle Vanem Ref: https://github.com/curl/curl/pull/1589
* tests: Fix up issues with errno in test filesMax Dymond2017-07-107-25/+25
| | | | Closes https://github.com/curl/curl/pull/1671
* errno: fix non-windows builds after af0216251b94e7Daniel Stenberg2017-07-102-6/+8
|
* make: fix docs build on OpenBSDRyan Winograd2017-07-101-0/+3
| | | | Ref: #1591
* ldap: fix MinGW compiler warningMarcel Raad2017-07-101-2/+1
| | | | | | | | | | ldap_bind_s is marked as deprecated in w32api's winldap.h shipping with the latest original MinGW, resulting in compiler warnings since commit f0fe66f13c93d3d0af45d9fb1231c9164e0f9dc8. Fix this for the non-SSPI case by using ldap_simple_bind_s again instead of ldap_bind_s with LDAP_AUTH_SIMPLE. Closes https://github.com/curl/curl/pull/1664
* curl-compilers.m4: disable warning spam with Cygwin's clangMarcel Raad2017-07-101-1/+9
| | | | | | | | | | | | When building with Cygwin or MinGW, libtool uses a wrapper executable instead of a wrapper script [1], which is written in C and throws missing-variable-declarations warnings. Don't enable these warnings on Cygwin and MinGW in order to avoid warnings for every executable built, which spams the test suite output when using Cygwin's clang. [1] https://www.gnu.org/software/libtool/manual/html_node/Wrapper-executables.html Closes https://github.com/curl/curl/pull/1665
* curl_setup_once: Remove ERRNO/SET_ERRNO macrosJay Satiro2017-07-1014-93/+76
| | | | | | | | | | | | Prior to this change (SET_)ERRNO mapped to GetLastError/SetLastError for Win32 and regular errno otherwise. I reviewed the code and found no justifiable reason for conflating errno on WIN32 with GetLastError/SetLastError. All Win32 CRTs support errno, and any Win32 multithreaded CRT supports thread-local errno. Fixes https://github.com/curl/curl/issues/895 Closes https://github.com/curl/curl/pull/1589
* tool_getparam: fix potentially uninitialized errJay Satiro2017-07-091-1/+2
|
* smb: rename variable to fix shadowing warningMarcel Raad2017-07-091-5/+5
| | | | | | | | | GCC 4.6.3 on travis complains: smb.c: In function ‘get_posix_time’: smb.c:725:13: error: declaration of ‘time’ shadows a global declaration [-Werror=shadow] Fix this by renaming the variable.
* tool_cb_wrt: fix variable shadowing warningMarcel Raad2017-07-091-3/+3
| | | | | | | | | | | GCC 4.4 complains: tool_cb_wrt.c:81: error: declaration of ‘isatty’ shadows a global declaration /usr/include/unistd.h:782: error: shadowed declaration is here Fix this by renaming the variable. Closes https://github.com/curl/curl/pull/1661
* RELEASE-NOTES: synced with be2c999b8Daniel Stenberg2017-07-081-6/+29
|
* travis: install stunnelDaniel Stenberg2017-07-071-0/+1
|
* valgrind.supp: supress OpenSSL false positive seen on travisDaniel Stenberg2017-07-071-0/+21
|
* travis: detect and use valgrind for normal buildsDaniel Stenberg2017-07-071-1/+1
| | | | Closes #1653
* travis: add SMB, DICT, TELNET torture to coverage testDaniel Stenberg2017-07-071-1/+1
|
* cmake: offer CMAKE_DEBUG_POSTFIX when building with MSVCPaul Harris2017-07-072-5/+10
| | | | | | Removes BUILD_RELEASE_DEBUG_DIRS since it wasn't used anywhere. Closes #1649
* CURLOPT_POSTFIELDS.3: explain the 100-continue magic betterDaniel Stenberg2017-07-071-3/+5
|
* test1452: add telnet negotiationMax Dymond2017-07-076-4/+552
| | | | | | | Add a basic telnet server for negotiating some telnet options before echoing back any data that's sent to it. Closes #1645
* travis: do more tests in the coverage runDaniel Stenberg2017-07-071-2/+3
| | | | I added a selection of torture and event tests that run "fast enough"
* curl_easy_escape.3: mention the (lack of) encodingDaniel Stenberg2017-07-071-1/+10
| | | | | Fixes #1612 Reported-by: Jeroen Ooms
* memdebug: don't setbuf() if the file open failedGisle Vanem2017-07-061-1/+2
| | | | Bug: https://github.com/curl/curl/issues/828#issuecomment-313475151
* appveyor: enable CURL_WERROR on all buildsDaniel Stenberg2017-07-061-1/+1
|
* cmake: add CURL_WERROR for enabling "warning as errors"Daniel Stenberg2017-07-061-0/+11
|
* cmake: remove spurious "-l" from linker flagsHannes Magnusson2017-07-061-1/+1
| | | | Fixes #1552
* test506: skip if threaded-resolverDaniel Stenberg2017-07-061-0/+5
|
* runtests: support "threaded-resolver" as a featureDaniel Stenberg2017-07-062-0/+11
| | | | ... to let tests require it or skip if present
* asyn-thread.c: fix unused variable warnings on macOSDaniel Stenberg2017-07-061-13/+14
|
* http: s/TINY_INITIAL_POST_SIZE/EXPECT_100_THRESHOLDDaniel Stenberg2017-07-062-4/+9
| | | | | Make the name reflect its use better, and add a short comment describing what it's for.
* cmake: if inet_pton is used, bump _WIN32_WINNTDaniel Stenberg2017-07-051-7/+11
| | | | | | | ... and make sure inet_pton is always checked for when *not* using Windows, which is a regression from 4fc6ebe18. Idea-by: Sergei Nikulov
View Lorry Controller Status