cmake: clean OtherTests, fixing -Werror

There were several -Wunused warnings and one duplicate macro definition.
The EXTRA_DEFINES variable of the CurlCheckCSources macro was being
abused ("__unused1\n#undef inline\n#define __unused2", seriously?) to
insert extra C code. Avoid this broken abstraction and use cmake's
check_c_source_compiles directly (works fine with CMake 2.8, maybe
even cmake 2.6).

After cleaning up all related variables (EXTRA_DEFINES,
HEADER_INCLUDES, auxiliary headers_hack), also remove a duplicate
add_headers_include macro and remove duplicate header additions before
the struct timeval check.

Oh, and now the code is converted to use CheckCSourceRuns and
CheckCSourceCompiles, the two curl-specific helpers can be removed.
Unfortunately, the cmake output is now slightly more verbose. Before:

    Performing Test int send(int, const void *, size_t, int) (curl_cv_func_send_test)
    Performing Test int send(int, const void *, size_t, int) (curl_cv_func_send_test) - Failed

Since check_c_source_compiles prints the varname, now you see:

    Performing Test curl_cv_func_send_test
    Performing Test curl_cv_func_send_test - Failed
    Tested: int send(int, const void *, size_t, int)

Compared cmake output with each other using vimdiff, no functional
differences were found. Tested with GCC 4.9.1 and Clang 3.5.0.

Signed-off-by: Peter Wu <peter@lekensteyn.nl>

cmake: fix gethostby{addr,name}_r in CurlTests

This patch cleans up the automatically-generated (?) code and fixes one
case that will always fail due to syntax error.

HAVE_GETHOSTBYADDR_R_5_REENTRANT always failed because of a trailing
character ("int length;q"). Several parameter type and unused variable
warnings popped up. This causes a detection failure with -Werror.

Observe that the REENTRANT cases are exactly the same as their
non-REENTRANT cases except for a `_REENTRANT` macro definition.
Merge all these pieces and build one big main function with different
cases, but reusing variables where logical.

For the cases where the parameters where NULL, I looked at
lib/hostip4.c to get an idea of the parameters types.

void-cast variables such as 'rc' to avoid -Wuninitialized errors.

Signed-off-by: Peter Wu <peter@lekensteyn.nl>

cmake: drop _BSD_SOURCE macro usage

autotools does not use features.h nor _BSD_SOURCE. As this macro
triggers warnings since glibc 2.20, remove it. It should not have
functional differences.

Signed-off-by: Peter Wu <peter@lekensteyn.nl>

RELEASE-NOTES: Synced with d71ea7c01e

Additionally, updated "GSSAPI" to "GSS-API" for a Cmake related change
as GSSAPI can be confused with the authentication mechanism rather than
a GSS-API implementation library such as MIT or Heimdal.

build: Added WinIDN build configuration options

Added support for WinIDN build configurations to the VC6 project files.

build: Added WinIDN build configuration options

Added support for WinIDN build configurations to the VC7 and VC7.1
project files.

build: Fixed the pre-processor separator in Visual Studio project files

A left over from the VC6 project files, so mainly cosmetic in Visual
Studio .NET as it can handle both comma and semi-colon characters for
separating multiple pre-processor definitions.

However, the IDE uses semi-colons if the value is edited, and as such,
this may cause problems in future for anyone updating the files or
merging patches.

Used the Visual Studio IDE to correct the separator character.

build: Added optional specific version generation of VC project files

..when working from the git repository. This is particularly useful
for single development environments where the project files for all
supported versions of Visual Studio may not be required.

build-openssl.bat: Fix x64 release build

Prior to this change if x64 release was specified a failed attempt was
made to build x86 release instead.

CURLOPT_XOAUTH2_BEARER.3: Corrected the OAuth version number

CURLOPT_SASL_IR.3: Added supported mechanism information

...and removed duplication of what protocols are supported from the
description text.

opts: Use common wording for MAIL related names

opts: Use common wording for TLS user/password option names

...and revised the proxy wording a little as well.

CURLOPT_MAXCONNECTS.3: Reworked the description to be less confusing

...and corrected a related typo in curl_easy_setopt.3.

RELEASE-NOTES: removed obsolete entry; fixed entry.

RELEASE-NOTES: Synced with e7da67f5d3

docs: Added mention of Kerberos for CURL_VERSION_SSPI

As this has been present for SOCKSv5 proxy since v7.19.4 and for IMAP,
POP3 and SMTP authentication since v7.38.0.

CURL_VERSION_KERBEROS4: Mark as deprecated

Support for Kerberos V4 was removed in v7.33.0.

sasl: Fixed Kerberos V5 inclusion when CURL_DISABLE_CRYPTO_AUTH is used

Typically the USE_WINDOWS_SSPI definition would not be used when the
CURL_DISABLE_CRYPTO_AUTH define is, however, it is still a valid build
configuration and, as such, the SASL Kerberos V5 (GSSAPI) authentication
data structures and functions would incorrectly be used when they
shouldn't be.

Introduced a new USE_KRB5 definition that takes into account the use of
CURL_DISABLE_CRYPTO_AUTH like USE_SPNEGO and USE_NTLM do.

openssl: Use 'CURLcode result'

More CURLcode fixes.

resume: consider a resume from [content-length] to be OK

Basically since servers often then don't respond well to this and
instead send the full contents and then libcurl would instead error out
with the assumption that the server doesn't support resume. As the data
is then already transfered, this is now considered fine.

Test case 1434 added to verify this. Test case 1042 slightly modified.

Reported-by: hugo
Bug: http://curl.haxx.se/bug/view.cgi?id=1443

openssl: Use 'CURLcode result'

More standardisation of CURLcode usage and coding style.

openssl: Use 'CURLcode result'

...and some minor code style changes.

ftplistparser: We prefer 'CURLcode result'

opts: Use common wording for user/password option names

CURLOPT_CONNECT_ONLY.3: Removed "This option is implemented for..." text

As this is covered by the PROTOCOLS section and saves having to update
two parts of the document with the same information in future.

CURLOPT_GSSAPI_DELEGATION.3: Use GSS-API rather than GSSAPI

As implementations are refereed to GSS-API libraries as per the RFC and
GSSAPI typically refers to an authentication mechanism.

CURLOPT_CONNECT_ONLY.3: Fixed incomplete protocol list

Added missing IMAP to the protocol list.

code cleanup: Use 'CURLcode result'

curl_easy_setopt.3: Fixed lots of typos

curl_easy_setopt.3: Moved CURLOPT_DIRLISTONLY into PROTOCOL OPTIONS

...as this option affects more that just FTP.

build: added Watcom support to build with WinSSL.

CURLOPT_PINNEDPUBLICKEY.3: added details

CURLOPT_CUSTOMREQUEST.3: Fixed incomplete protocol list

Whilst the description included information about SMTP, the protocol
list only showed "TTP, FTP, IMAP, POP3".

CURLOPT_DIRLISTONLY.3: Added information about the usage in POP3

openssl: enable NPN separately from ALPN

... and allow building with nghttp2 but completely without NPN and ALPN,
as nghttp2 can still be used for plain-text HTTP.

Reported-by: Lucas Pardue

configure.ac: remove checks for OpenSSL NPN/ALPN funcs again

... since the conditional in the code are now based on OpenSSL versions
instead to better support non-configure builds.

opts: added some "SEE ALSO" references

RELEASE-NOTES: Synced with 32913182dc

vtls.c: Fixed compilation warning

conversion from 'size_t' to 'unsigned int', possible loss of data

sspi: Return CURLE_LOGIN_DENIED on AcquireCredentialsHandle() failure

Return a more appropriate error, rather than CURLE_OUT_OF_MEMORY when
acquiring the credentials handle fails. This is then consistent with
the code prior to commit f7e24683c4 when log-in credentials were empty.

sasl_sspi: Allow DIGEST-MD5 to use current windows credentials

Fixed the ability to use the current log-in credentials with DIGEST-MD5.
I had previously disabled this functionality in commit 607883f13c as I
couldn't get this to work under Windows 8, however, from testing HTTP
Digest authentication through Windows SSPI and then further testing of
this code I have found it works in Windows 7.

Some further investigation is required to see what the differences are
between Windows 7 and 8, but for now enable this functionality as the
code will return an error when AcquireCredentialsHandle() fails.

transfer: drop the code handling the ssl_connect_retry flag

Its last use has been removed by the previous commit.

nss: drop the code for libcurl-level downgrade to SSLv3

This code was already deactivated by commit
ec783dc142129d3860e542b443caaa78a6172d56.

openssl: fix a line length warning

Added NetWare support to build with nghttp2.

Fixed error message since we require ALPN support.

Check for ALPN via OpenSSL version number.

This check works also with to non-configure platforms.

sasl_sspi: Fixed typo in comment

code cleanup: We prefer 'CURLcode result'

TODO: consider supporting STAT

mk-ca-bundle: spell fix "version"

HTTP: return larger than 3 digit response codes too

HTTP 1.1 is clearly specified to only allow three digit response codes,
and libcurl used sscanf("%3d") for that purpose. This made libcurl
support smaller numbers but not larger. It does now, but we will not
make any specific promises nor document this further since it is going
outside of what HTTP is.

Bug: http://curl.haxx.se/bug/view.cgi?id=1441
Reported-by: Balaji

src/: remove version.h.dist from gitignore

It has not been used since commit f7bfdbab in 2011

ntlm: We prefer 'CURLcode result'

Continuing commit 0eb3d15ccb more return code variable name changes.

Cosmetics: lowercase non-special subroutine names.

RELEASE-NOTES: Synced with 07ac29a058

http_negotiate: We prefer 'CURLcode result'

Continuing commit 0eb3d15ccb more return code variable name changes.

http_negotiate: Fixed missing check for USE_SPNEGO

sspi: Synchronization of cleanup code between auth mechanisms

sspi: Renamed max token length variables

Code cleanup to try and synchronise code between the different SSPI
based authentication mechanisms.

sspi: Renamed expiry time stamp variables

Code cleanup to try and synchronise code between the different SSPI
based authentication mechanisms.

sspi: Only call CompleteAuthToken() when complete is needed

Don't call CompleteAuthToken() after InitializeSecurityContext() has
returned SEC_I_CONTINUE_NEEDED as this return code only indicates the
function should be called again after receiving a response back from
the server.

This only affected the Digest and NTLM authentication code.

Added the "flaky" keyword to a number of tests

Each shows evidence of flakiness on at least one platform on
the autobuilds. Users can use this keyword to skip these tests
if desired.

ntlm: Return all errors from Curl_ntlm_core_mk_nt_hash()

For consistency with other areas of the NTLM code propagate all errors
from Curl_ntlm_core_mk_nt_hash() up the call stack rather than just
CURLE_OUT_OF_MEMORY.

ntlm: Return CURLcode from Curl_ntlm_core_mk_lm_hash()

ntlm: Use 'CURLcode result'

Continuing commit 0eb3d15ccb more return code variable name changes.

ntlm: Only define ntlm data structure when USE_NTLM is defined

ntlm: Changed handles to be dynamic like other SSPI handles

Code cleanup to try and synchronise code between the different SSPI
based authentication mechanisms.

ntlm: Renamed handle variables to match other SSPI structures

Code cleanup to try and synchronise code between the different SSPI
based authentication mechanisms.

ntlm: Renamed SSPI based input token variables

Code cleanup to try and synchronise code between the different SSPI
based authentication mechanisms.

ntlm: We prefer 'CURLcode result'

Continuing commit 0eb3d15ccb more return code variable name changes.

build: Added WinIDN build configuration options

Added support for WinIDN build configurations to the VC8 and VC9
project files.

darwinssl: detect possible future removal of SSLv3 from the framework

If Apple ever drops SSLv3 support from the Security framework, we'll fail with an error if the user insists on using SSLv3.

gskit.c: remove SSLv3 from SSL default.

gskit.c: use 'CURLcode result'

SSL: Remove SSLv3 from SSL default due to POODLE attack

- Remove SSLv3 from SSL default in darwinssl, schannel, cyassl, nss,
openssl effectively making the default TLS 1.x. axTLS is not affected
since it supports only TLS, and gnutls is not affected since it already
defaults to TLS 1.x.

- Update CURLOPT_SSLVERSION doc

pipelining: only output "is not blacklisted" in debug builds

*.3: add/extend "SEE ALSO" sections

curl_easy_pause.3: minor wording edit

curl_getdate.3: provide a "SEE ALSO" section

curl_global_init.3: minor formatting fix, add version info

url.c: use 'CURLcode result'

code cleanup: we prefer 'CURLcode result'

... for the local variable name in functions holding the return
code. Using the same name universally makes code easier to read and
follow.

Also, unify code for checking for CURLcode errors with:

 if(result) or if(!result)

instead of

 if(result == CURLE_OK), if(CURLE_OK == result) or if(result != CURLE_OK)

Curl_add_timecondition: skip superfluous varible assignment

Detected by cppcheck.

Curl_pp_flushsend: skip superfluous assignment

Detected by cppcheck.

Curl_pp_readresp: remove superfluous assignment

Variable already assigned a few lines up.

Detected by cppcheck.

Curl_proxyCONNECT: remove superfluous statement

The variable is already assigned, skip the duplicate assignment.

Pointed out by cppcheck.

Added MinGW support to build with nghttp2.

Added VC ssh2 target to main Makefile.

Some cosmetics and simplifies.

Remove dependency on openssl and cut.

Prefer usage of Perl modules for sha1 calculation since there
might be systems where openssl is not installed or not in path.
If openssl is used for sha1 calculation then dont rely on cut
since it is usually not available on other systems than Linux.

RELEASE-NOTES: synced with e116d0a62

CURLOPT_RESOLVE.3: add an example

gnutls: removed dead code

Bug: http://curl.haxx.se/bug/view.cgi?id=1437
Reported-by: Julien

Curl_rand: Uninitialized variable: r

This is not actually used uninitialized but we silence warnings.

Bug: http://curl.haxx.se/bug/view.cgi?id=1437
Reported-by: Julien

opts: provide more and updated examples

CURLOPT_RANGE.3: works for SFTP as well

... and added a small example

curl.1: edited for clarity

CURLOPT_SSLVERSION.3: provide an example