endian: Introduced endian module

To allow the little endian functions, currently used in two of the NTLM
source files, to be used by other modules such as the SMB module.

sepheaders.c: Applied curl oding standards

sepheaders.c: Fixed resource leak on failure

vtls: Use '(void) arg' for unused parameters

Prefer void for unused parameters, rather than assigning an argument to
itself as a) unintelligent compilers won't optimize it out, b) it can't
be used for const parameters, c) it will cause compilation warnings for
clang with -Wself-assign and d) is inconsistent with other areas of the
curl source code.

smb.c: Fixed compilation warning

smb.c:586: warning: conversion to 'short unsigned int' from 'int' may
           alter its value

smb: Use the connection's upload buffer

Use the connection's upload buffer instead of allocating our own send
buffer.

RELEASE-NOTES: Synced with 1933f9d33c

schannel: Moved the ISC return flag definitions to the SSPI module

Moved our Initialize Security Context return attribute definitions to
the SSPI module, as a) these can be used by other SSPI based providers
and b) the ISC required attributes are defined there.

smb: Close the connection after a failed client write

darwinssl: Fixed compilation warning

vtls.c:683:43: warning: unused parameter 'data'

sockfilt.c: Fixed compilation warnings

sockfilt.c:288: warning: conversion to 'DWORD' from 'size_t' may alter
                its value
sockfilt.c:291: warning: conversion to 'DWORD' from 'size_t' may alter
                its value
sockfilt.c:323: warning: conversion to 'DWORD' from 'size_t' may alter
                its value
sockfilt.c:326: warning: conversion to 'DWORD' from 'size_t' may alter
                its value

test1509: Fixed compilation warning

lib1509.c:93:18: warning: conversion to 'long int' from 'size_t' may
                 alter its value

test556: Fixed compilation warning

lib556.c:90: warning: conversion to 'unsigned int' from 'size_t' may
             alter its value

sasl_gssapi: Fixed use of dummy username with real username

vtls: Fixed compilation warning and an ignored return code

curl_schannel.h:123: warning: right-hand operand of comma expression
                     has no effect

Some instances of the curlssl_close_all() function were declared with a
void return type whilst others as int. The schannel version returned
CURLE_NOT_BUILT_IN and others simply returned zero, but in all cases the
return code was ignored by the calling function Curl_ssl_close_all().

For the time being and to keep the internal API consistent, changed all
declarations to use a void return type.

To reduce code we might want to consider removing the unimplemented
versions and use a void #define like schannel does.

TODO: 2.3 Better support for same name resolves

test1520: Fixed initial teething problems

* Missing initialisation of upload status caused a seg fault
* Missing data termination caused corrupt data to be uploaded
* Data verification should be performed in <upload> element
* Added missing recipient list cleanup

test1520: Fixed compilation errors

tests: Added test for bug #1456

checksrc.bat: Fixed a problem opening files with spaces in the filename

openldap: Prefer use of 'CURLcode result'

openldap: Use 'LDAPMessage *msg' for messages

This frees up the 'result' variable for CURLcode based result codes.

nss: Don't ignore Curl_extract_certinfo() OOM failure

nss: Don't ignore Curl_ssl_init_certinfo() OOM failure

nss: Use 'CURLcode result' for curl result codes

...and don't use CURLE_OK in failure/success comparisons.

getinfo: Code style policing

getinfo: Use 'CURLcode result' for curl result codes

darwinssl: Use 'CURLcode result' for curl result codes

polarssl: Use 'CURLcode result' for curl result codes

docs: Updated following the addition of SASL GSSAPI via GSS-API libraries

As this feature has been implemented for 7.40.0.

asiohiper.cpp: No need to initialise members of ConnInfo

...as calloc() automatically clears the area of memory with zeros.

asiohiper.cpp: Updated for curl coding standards

...with the exception of the start of block statement curly brackets.

code/docs: Use correct case for IPv4 and IPv6

For consistency, as we seem to have a bit of a mixed bag, changed all
instances of ipv4 and ipv6 in comments and documentations to use the
correct case.

runtests: Fixed detection of Unix Sockets feature

...following change in curl --version output.

code/docs: Use Unix rather than UNIX to avoid use of the trademark

Use Unix when generically writing about Unix based systems as UNIX is
the trademark and should only be used in a particular product's name.

ip2ip.c: Fixed compilation warning when IPv6 Scope ID not supported

if2ip.c:119: warning: unused parameter 'remote_scope_id'

...and some minor code style policing in the same function.

vtls: Don't set cert info count until memory allocation is successful

Otherwise Curl_ssl_init_certinfo() can fail and set the num_of_certs
member variable to the requested count, which could then be used
incorrectly as libcurl closes down.

vtls: Use CURLcode for Curl_ssl_init_certinfo() return type

The return type for this function was 0 on success and 1 on error. This
was then examined by the calling functions and, in most cases, used to
return CURLE_OUT_OF_MEMORY.

Instead use CURLcode for the return type and return the out of memory
error directly, propagating it up the call stack.

configure: Use camel case for UNIX sockets feature output

To match the curl --version output.

sockfilt.c: Reduce the number of individual memory allocations

Merge multiple internal arrays into one, even if some variables
will not not be used. They are all created with the number of
file descriptors as their size.

Also fix possible thread handle leak in CloseHandle-loop.

sockfilt.c: Replace 100ms sleep with thread throttle

Improves performance of test cases 574 and 575 by 50%.

A value of zero causes the thread to relinquish the remainder
of its time slice to any other thread of equal priority that is
ready to run. If there are no other threads of equal priority
ready to run, the function returns immediately, and the thread
continues execution.

http://msdn.microsoft.com/library/windows/desktop/ms686307.aspx

tool_help: Use camel case for UNIX sockets feature output

In line with the other features listed in the --version output,
capitalise the UNIX socket feature.

vtls: Use bool for Curl_ssl_getsessionid() return type

The return type of this function is a boolean value, and even uses a
bool internally, so use bool in the function declaration as well as
the variables that store the return value, to avoid any confusion.

schannel: Minor code style policing for casts

schannel: Prefer 'CURLcode result' for curl result codes

cyassl: Prefer 'CURLcode result' for curl result codes

tool_xattr: Use 'CURLcode result' for curl result codes

curl_ntlm_core.c: Fixed compilation warnings

curl_ntlm_core.c:301: warning: pointer targets in passing argument 2 of
                      'CryptImportKey' differ in signedness
curl_ntlm_core.c:310: warning: passing argument 6 of 'CryptEncrypt' from
                      incompatible pointer type
curl_ntlm_core.c:540: warning: passing argument 4 of 'CryptGetHashParam'
                      from incompatible pointer type

RELEASE-NOTES: Synced with 8830df8b66

gtls: Use preferred 'CURLcode result'

openldap: Use standard naming for setup connection function

Renamed ldap_setup() to ldap_setup_connection() to follow more widely
used function naming.

rtmp: Use standard naming for setup connection function

Renamed rtmp_setup() to rtmp_setup_connection() to follow more widely
used function naming.

smb: Use standard naming for setup connection function

Renamed smb_setup() to smb_setup_connection() to follow more widely
used function naming.

config-win32.h: Fixed line length > 79 columns

openssl: Prefer we don't use NULL in comparisons

build: Removed WIN32 definition from the Visual Studio projects

As this pre-processor definition is defined in curl_setup.h there is no
need to include it in the Visual Studio project files.

build: Removed WIN64 definition from the libcurl Visual Studio projects

Removed the WIN64 pre-processor definition from the libcurl project
files as:

* WIN64 is not used in our source code
* The curl projects files don't define it
* It isn't required by or used in the platform SDK
* For backwards compatability curl_setup.h defines WIN32
* The compiler automatically defines _WIN64 for x64 builds

Historically Visual Studio projects have defined WIN32, in addition to
the compiler defined _WIN32 definition, and I had incorrectly changed
that to WIN64 for the x64 libcurl builds but not in the curl projects.

As such, it is questionable whether this should be defined or not. For
more information see the following cache of a discussion that took
place on the microsoft.public.vc.mfc newsgroup:

http://www.tech-archive.net/Archive/VC/microsoft.public.vc.mfc/2008-06/msg00074.html

openssl.c Fix for compilation errors with older versions of OpenSSL

openssl.c:1408: error: 'TLS1_1_VERSION' undeclared
openssl.c:1411: error: 'TLS1_2_VERSION' undeclared

Fix comment edit in vms/backup_gnv_curl_src.com

packages/vms/backup_gnv_curl_src.com: Originally copied from Bash port.

curl: show size of inhibited data when using -v

To offer some more info and yet it doesn't use more lines.

openssl: fix SSL/TLS versions in verbose output

openssl: make it compile against openssl 1.1.0-DEV master branch

sshserver.pl: clarify and streamline variable names

openssl: warn for SRP set if SSLv3 is used, not for TLS version

... as it requires TLS and it was was left to warn on the default from
when default was SSL...

smb: use memcpy() instead of strncpy()

... as it never copies the trailing zero anyway and always just the four
bytes so let's not mislead anyone into thinking it is actually treated
as a string.

Coverity CID: 1260214

VMS: Updates for 0740-0D1220

lib/setup-vms.h : VAX HP OpenSSL port is ancient, needs help.
                  More defines to set symbols to uppercase.

src/tool_main.c : Fix parameter to vms_special_exit() call.

packages/vms/ :
  backup_gnv_curl_src.com : Fix the error message to have the correct package.

  build_curl-config_script.com : Rewrite to be more accurate.

  build_libcurl_pc.com : Use tool_version.h now.

  build_vms.com : Fix to handle lib/vtls directory.

  curl_gnv_build_steps.txt : Updated build procedure documentation.

  generate_config_vms_h_curl.com :
       * VAX does not support 64 bit ints, so no NTLM support for now.
       * VAX HP SSL port is ancient, needs some help.
       * Disable NGHTTP2 for now, not ported to VMS.
       * Disable UNIX_SOCKETS, not available on VMS yet.
       * HP GSSAPI port does not have gss_nt_service_name.

  gnv_link_curl.com : Update for new curl structure.

  pcsi_product_gnv_curl.com : Set up to optionally do a complete build.

sockfilt.c: use non-Ex functions that are available before WinXP

It was initially reported by Guenter that GetFileSizeEx
requires (_WIN32_WINNT >= 0x0500) to be true.

tests: use Cygwin-style paths in SSH, SSHD and SFTP config files

Second patch to enable Windows support using Cygwin-based OpenSSH.

Tested with CopSSH 5.0.0 free edition using an msys shell on Windows 7.

tests: support spaces in paths to SSH, SSHD and SFTP binaries

First patch to enable Windows support using Cygwin-based OpenSSH.

non-ascii: Reduce variable usage

Removed 'next' variable in Curl_convert_form(). Rather than setting it
from 'form->next' and using that to set 'form' after the conversion
just use 'form = form->next' instead.

non-ascii: Prefer while loop rather than a do loop

This also removes the need to check that the 'form' argument is valid.

non-ascii: Reduce variable scope

As 'result' isn't used out side the conversion callback code and
previously caused variable shadowing in the libiconv based code.

non-ascii: We prefer 'CURLcode result'

This also fixes a variable shadowing issue when HAVE_ICONV is defined
as rc was declared for the result code of libiconv based functions.

secureserver.pl: clean up formatting of config and fix verbose output

Verbose output was not matching the actual configuration file,
because FIPS and Windows conditions were ignored.

secureserver.pl: update Windows detection and fix path conversion

secureserver.pl: make OpenSSL CApath and cert absolute path values

Recent stunnel versions (5.08) seem to have trouble with relative
paths on Windows. This turns the relative paths into absolute ones.

if2ip: dummy scope parameter for Curl_if2ip() call in SIOCGIFADDR-enabled code.

parseurlandfillconn(): fix improper non-numeric scope_id stripping.
Fixes SF bug 1149: http://sourceforge.net/p/curl/bugs/1449/

IPV6: address scope != scope id
There was a confusion between these: this commit tries to disambiguate them.
- Scope can be computed from the address itself.
- Scope id is scope dependent: it is currently defined as 1-based local
  interface index for link-local scoped addresses, and as a site index(?) for
  (obsolete) site-local addresses. Linux only supports it for link-local
  addresses.
The URL parser properly parses a scope id as an interface index, but stores it
in a field named "scope": confusion. The field has been renamed into "scope_id".
Curl_if2ip() used the scope id as it was a scope. This caused failures
to bind to an interface.
Scope is now computed from the addresses and Curl_if2ip() matches them.
If redundantly specified in the URL, scope id is check for mismatch with
the interface index.

This commit should fix SF bug #1451.

connect: singleipconnect(): properly try other address families after failure

SFTP: work-around servers that return zero size on STAT

Bug: http://curl.haxx.se/mail/lib-2014-12/0103.html
Pathed-by: Marc Renault

glob_next_url: make the loop count upwards

As the former contruct apparently caused a compiler warning, mentioned
in d8efde07e556c.

tool_operate: we prefer 'CURLcode result'

tool_urlglob: unify return codes to use CURLcode

There was a mix of GlobCode, CURLcode and ints and they were mostly
passing around CURLcode errors. This change makes the functions use only
CURLcode and removes the GlobCode type completely.

tool_urlglob.c: partly reverse dc19789444

The loop in glob_next_url() needs to be done backwards to maintain the
logic. dc19789444 caused test 1235 to fail.

KNOWN_BUGS: the SFTP code doesn't support CURLINFO_FILETIME

opts: Warn CURLOPT_TIMEOUT overrides when set after CURLOPT_TIMEOUT_MS

Change CURLOPT_TIMEOUT doc to warn that if CURLOPT_TIMEOUT and
CURLOPT_TIMEOUT_MS are both set whichever one is set last is the one
that will be used.

Prior to this change that behavior was only noted in the
CURLOPT_TIMEOUT_MS doc.

darwinssl: fix incorrect usage of aprintf()

Commit b13923f changed an snprintf() to use aprintf(), but the API usage
wasn't correct, and was causing a crash to occur. This fixes it.

copyright: Updated the copyright year following recent updates

tool_urlglob.c: reverse two loops

By counting from 0 and up instead of backwards like before, we remove
the need for the "funny" check of the unsigned variable when decreased
passed zero. Easier to read and less risk for compiler warnings.

tool_urlglob.c: Added braces to clarify the conditions

tool_urlglob.c: Silence warning C6293: Ill-defined for-loop

The >= 0 is actually not required, since i underflows and
the for-loop is stopped using the < condition, but this
makes the VS2012 compiler and code analysis happy.

tool_binmode.c: Explicitly ignore the return code of setmode

Fixes code analysis warning C6031:
return value ignored: <function> could return unexpected value

lib: Fixed multiple code analysis warnings if SAL are available

warning C28252: Inconsistent annotation for function:
parameter has another annotation on this instance

smb.c: Fixed code analysis warning

smb.c:320: warning C6297: Arithmetic overflow: 32-bit value is shifted,
           then cast to 64-bit value. Result may not be an expected
           value

tool_util.c: Use GetTickCount64 if it is available

smb: Use HAVE_PROCESS_H for process.h inclusion

Rather than testing against _WIN32 use the preferred HAVE_PROCESS_H
pre-processor define when including process.h.

darwinssl: aprintf() to allocate the session key

... to avoid using a fixed memory size that risks being too large or too
small.

curl_schannel: Improvements to memory re-allocation strategy

- do not grow memory by doubling its size
- do not leak previously allocated memory if reallocation fails
- replace while-loop with a single check to make sure
  that the requested amount of data fits into the buffer

Bug: http://curl.haxx.se/bug/view.cgi?id=1450
Reported-by: Warren Menzer

asyn-ares: We prefer use of 'CURLcode result'