summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* nonblock: fix setting non-blocking mode for AmigaDaniel Stenberg2015-11-271-2/+2
| | | | | | | | | IoctlSocket() apparently wants a pointer to a long, passed as a char * in its third parameter. This bug was introduced already back in commit c5fdeef41d from October 1 2001! Bug: http://curl.haxx.se/mail/lib-2015-11/0088.html Reported-by: Norbert Kett
* lib: Only define curl_dofreeaddrinfo if struct addrinfo is availableDan Fandrich2015-11-271-1/+2
|
* timecond: do not add if-modified-since without timeconditionDaniel Stenberg2015-11-251-7/+12
| | | | | | | | | | | The RTSP code path didn't skip adding the if-modified-since for certain RTSP code paths, even if CURLOPT_TIMECONDITION was set to CURL_TIMECOND_NONE. Also, an unknown non-zero CURLOPT_TIMECONDITION value no longer equals CURL_TIMECOND_IFMODSINCE. Bug: http://stackoverflow.com/questions/33903982/curl-timecond-none-doesnt-work-how-to-remove-if-modified-since-header
* done: make sure the final progress update is madeDaniel Stenberg2015-11-241-2/+7
| | | | | | | | | | | | | | It would previously be skipped if an existing error was returned, but would lead to a previous value being left there and later used. CURLINFO_TOTAL_TIME for example. Still it avoids that final progress update if we reached DONE as the result of a callback abort to avoid another callback to be called after an abort-by-callback. Reported-by: Lukas Ruzicka Closes #538
* Revert "cleanup: general removal of TODO (and similar) comments"Daniel Stenberg2015-11-2417-23/+78
| | | | | | | This reverts commit 64e959ffe37c436503f9fed1ce2d6ee6ae50bd9a. Feedback-by: Dan Fandrich URL: http://curl.haxx.se/mail/lib-2015-11/0062.html
* smb.c: Fixed compilation warningsSteve Holme2015-11-211-3/+5
| | | | | | | | | smb.c:134:3: warning: conversion to 'short unsigned int' from 'int' may alter its value smb.c:146:42: warning: conversion to 'unsigned int' from 'long long unsigned int' may alter its value smb.c:146:65: warning: conversion to 'unsigned int' from 'long long unsigned int' may alter its value
* schannel: Corrected copy/paste error in commit 8d17117683Steve Holme2015-11-211-6/+5
|
* schannel: Use GetVersionEx() when VerifyVersionInfo() isn't availableSteve Holme2015-11-211-5/+22
| | | | Regression from commit 7a8e861a5 as highlighted in the msys autobuilds.
* pop3: Differentiate between success and continuation responsesSteve Holme2015-11-201-4/+10
|
* pop3: Added clarity on some server response codesSteve Holme2015-11-201-2/+4
|
* http2: http_done: don't free already-freed push headersDaniel Stenberg2015-11-171-4/+7
| | | | | | | | The push headers are freed after the push callback has been invoked, meaning this code should only free the headers if the callback was never invoked and thus the headers weren't freed at that time. Reported-by: Davey Shafik
* getconnectinfo: Don't call recv(2) if socket == -1Anders Bakken2015-11-161-2/+2
| | | | Closes #528
* http2: minor comment typoDaniel Stenberg2015-11-161-1/+1
|
* sasl; fix checksrc warningsDaniel Stenberg2015-11-151-4/+6
|
* oauth2: Support OAUTHBEARER failures sent as continuation responsesSteve Holme2015-11-152-2/+31
| | | | | | | According to RFC7628 a failure message may be sent by the server in a base64 encoded JSON string as a continuation response. Currently only implemented for OAUTHBEARER and not XAUTH2.
* oauth2: Added support for OAUTHBEARER SASL mechanism to IMAP, POP3 and SNMPSteve Holme2015-11-142-21/+44
| | | | | | OAUTHBEARER is now the official "registered" SASL mechanism name for OAuth 2.0. However, we don't want to drop support for XOAUTH2 as some servers won't support the new mechanism yet.
* openssl: Free modules on cleanupSebastian Pohlschmidt2015-11-131-0/+3
| | | | | | | Curl_ossl_init calls OPENSSL_load_builtin_modules() but Curl_ossl_cleanup doesn't make a call to free these modules. Bug: https://github.com/bagder/curl/issues/526
* cleanup: general removal of TODO (and similar) commentsDaniel Stenberg2015-11-1317-78/+23
| | | | | | They tend to never get updated anyway so they're frequently inaccurate and we never go back to revisit them anyway. We document issues to work on properly in KNOWN_BUGS and TODO instead.
* ftplistparser: remove empty functionDaniel Stenberg2015-11-131-23/+5
|
* openssl: remove #if check for 0.9.7 for ENGINE_load_private_keyDaniel Stenberg2015-11-131-1/+1
|
* openssl: all supported versions have X509_STORE_set_flagsDaniel Stenberg2015-11-131-7/+0
| | | | Simplify by removing #ifdefs and macros
* openssl: remove 0.9.3 checkDaniel Stenberg2015-11-131-3/+1
|
* openssl: remove #ifdefs for < 0.9.5 supportDaniel Stenberg2015-11-131-60/+29
| | | | We only support >= 0.9.7
* lib/vtls/openssl: remove unused traces of yassl ifdefsDaniel Stenberg2015-11-131-6/+0
|
* unit1603: Added unit tests for hash functionsdfandrich2015-11-121-2/+21
|
* sasl: Re-introduced XOAUTH2 in the default enabled authentication mechanismSteve Holme2015-11-122-4/+2
| | | | | | | | | Following the fix in commit d6d58dd558 it is necessary to re-introduce XOAUTH2 in the default enabled authentication mechanism, which was removed in commit 7b2012f262, otherwise users will have to specify AUTH=XOAUTH2 in the URL. Note: OAuth 2.0 will only be used when the bearer is specified.
* sasl_sspi: fix identity memory leak in digest authenticationStefan Bühler2015-11-121-0/+4
|
* sasl_sspi: fixed unicode build for digest authenticationStefan Bühler2015-11-121-1/+10
| | | | Closes #525
* oauth2: Re-factored OAuth 2.0 state variableSteve Holme2015-11-122-4/+4
|
* sasl: Don't choose OAuth 2.0 if mechanism not advertisedSteve Holme2015-11-121-1/+2
| | | | | | Regression from commit 9e8ced9890 which meant if --oauth2-bearer was specified but the SASL mechanism wasn't supported by the server then the mechanism would be chosen.
* oauth2: Introduced support for host and port detailsSteve Holme2015-11-111-5/+18
| | | | | | Added support to the OAuth 2.0 message function for host and port, in order to accommodate the official OAUTHBEARER SASL mechanism which is to be added shortly.
* curl_setup.h: Removed duplicate CURL_DISABLE_RTSP when HTTP_ONLY definedSteve Holme2015-11-111-3/+0
|
* cmake: Add missing feature macros in config header (Part 2)Steve Holme2015-11-111-0/+9
| | | | In addition to commit a215381c94 added the RTSP, RTMP and SMB protocols.
* cmake: Add missing feature macros in config headerDouglas Creager2015-11-101-0/+12
| | | | | | | | | | The curl_config.h file can be generated either from curl_config.h.cmake or curl_config.h.in, depending on whether you're building using CMake or the autotools. The CMake template header doesn't include entries for all of the protocols that you can disable, which (I think) means that you can't actually disable those protocols when building via CMake. Closes #523
* BoringSSL: Work with stricter BIO_get_mem_data()Douglas Creager2015-11-101-2/+2
| | | | | | | | BoringSSL implements `BIO_get_mem_data` as a function, instead of a macro, and expects the output pointer to be a `char **`. We have to add an explicit cast to grab the pointer as a `const char **`. Closes #524
* http2: rectify the http2 version #if checkDaniel Stenberg2015-11-101-1/+1
| | | | We need 1.0.0 or later. Also verified by configure.
* oauth2: Don't use XAUTH2 in OAuth 2.0 function nameSteve Holme2015-11-091-10/+10
|
* oauth2: Don't use XOAUTH2 in OAuth 2.0 variablesSteve Holme2015-11-093-7/+8
|
* oauth2: Use OAuth 2.0 rather than XOAUTH2 in commentsSteve Holme2015-11-092-2/+2
| | | | | When referring to OAuth 2.0 we should use the official name rather the SASL mechanism name.
* imap: avoid freeing constant stringDaniel Stenberg2015-11-091-1/+1
| | | | | | | The fix in 1a614c6c3 was wrong and would leed to free() of a fixed string. Pointed-out-by: Kamil Dudka
* imap: Don't check for continuation when executing a CUSTOMREQUESTJustin Ehlert2015-11-081-2/+2
| | | | | Bug: https://github.com/bagder/curl/issues/486 Closes https://github.com/bagder/curl/pull/487
* imap: checksrc: remove space after while before parenDaniel Stenberg2015-11-071-1/+1
|
* checksrc.whitelist: "missing space after close paren"Daniel Stenberg2015-11-071-0/+1
| | | | ... when it was within a string!
* imap: Quote other 'atom-specials' and not just the space characterSteve Holme2015-11-071-8/+18
| | | | Closes #517
* imap: Fixed double quote in LIST command when mailbox contains spacesSteve Holme2015-11-071-8/+8
|
* imap: fix compiler warningDaniel Stenberg2015-11-061-1/+1
| | | | | imap.c:657:13: error: assignment discards 'const' qualifier from pointer target type [-Werror=discarded-qualifiers]
* imap: Don't call imap_atom() when no mailbox specified in LIST commandSteve Holme2015-11-061-2/+2
|
* getinfo: CURLINFO_ACTIVESOCKET: fix bad socket valueJay Satiro2015-11-061-12/+1
| | | | | | | | | | | | | | - Set user info param to the socket returned by Curl_getconnectinfo, regardless of if the socket is bad. Effectively this means the user info param now will receive CURL_SOCKET_BAD instead of -1 on bad socket. - Remove incorrect comments. CURLINFO_ACTIVESOCKET is documented to write CURL_SOCKET_BAD to user info param but prior to this change it wrote -1. Bug: https://github.com/bagder/curl/pull/518 Reported-by: Marcel Raad
* curl_ntlm_core: fix 2 curl_off_t constant overflows.Patrick Monnerat2015-11-051-2/+2
|
* rawstr: Speed up Curl_raw_toupper by 40%Lauri Kasanen2015-11-021-1/+7
| | | | | | | | | | Rationale: when starting up a curl-using app, all cookies from the jar are checked against each other. This was causing a startup delay in the Fifth browser. All tests pass. Signed-off-by: Lauri Kasanen <cand@gmx.com>