tests: verify OPEN/CLOSESOCKETFUNCTION

Test 585: Fix opensocket return type, and avoid function name clash.

version: linkage fix

Fix linkage on c-ares enabled Windows static builds

curl_easy_setopt.3: document CLOSESOCKET* options

tests: verify OPEN/CLOSESOCKETFUNCTION

Test 585 and 586 were added. Using a modified lib500.c

symbols-in-versions: add CLOSESOCKET*

CLOSESOCKETFUNCTION: use the callback

Fix the return type of the callback to match close() and make use of it.

CLOSESOCKETFUNCTION: added

Introduced the initial setup to allow closesocket callbacks by making
sure sclose() is only ever called from one place in the libcurl source
and still run all test cases fine.

GnuTLS handshake: fix timeout

Commit cbf4961bf3e garbled the timeout handling while doing SSL
handshaking (in an attempt at fixing another bug). This puts sanity
back.

Bug: http://curl.haxx.se/mail/lib-2011-05/0167.html
Reported by: Ethan Glasser Camp

checksrc: trailing whitespace detection fix

negotiate sspi: fix sequential requests

tests: added HTTP If-Modified-Since tests

Added test 1126 and 1127 to verify curl's behaviour when If-Modified-Since
is used and a 200 is returned.

The list of test cases in Makefile.am is now sorted numerically.

include: cleanup

Made the public headers checksrc compliant

Removed types.h (it's been unused since April 2004)

Made the root makefile do make in include by default as well, so that
TAGS and the checksrc will work better.

Fixed compilation when RTSP is disabled

Fixed LDAP after RTSP readwrite change

RELEASE-NOTES: synced with 32001ac4149b206

set_userpass: convert from protocol-specific to generic

The protocol handler's flags field now can set that the protocol
requires a password, so that the set_userpass function doesn't have to
have the specific knowledge of which protocols that do.

RTSP: cleanups

Made several functions static

Made one function defined to nothing when RTSP is disabled to avoid
the #ifdefs in code.

Removed explicit rtsp.h includes

RTSP: convert protocol-specific checks to generic

Add a 'readwrite' function to the protocol handler struct and use that
for the extra readwrite functionality RTSP needs.

SSL: check for SSL, not specific protocols

Code cleanup to check less for protocols and more for the specific
relevant feature. Like if SSL is required.

http_perhapsrewind: remove HTTP check

No need to check for HTTP as this is now a HTTP-specific function

http_perhapsrewind:

make it static, remove Curl_ prefix

remove FILE protocol-specific checks

Also, convert the BANPROXY flag into NONETWORK for the protocols
(file:// only atm) that don't work over networks.

curl.1: --socks* options no longer needed

As we now can specify all the socks proxy types with the regular --proxy
option using protocol prefix.

socks proxy: allow socks5h:// prefix too

Using 'socks5h' as proxy protocol will make it a
CURLPROXY_SOCKS5_HOSTNAME proxy which is SOCKS5 and asking the proxy to
resolve host names. I found no "standard" protocol name for this.

curl.1: minor edit of --ftp-ssl*

curl.1: use GNU style and sort options

Follow style of GNU layout (cp, mv ...) where options are separated with
comma: -o, --option

Order item alphabetically (by length also): -o, -O, --option

Follow style of GNU layout by moving help related options to the end:
--help, -M, --version

Corrected comments

closepolicy has been deprecated and unused for years

ConnectionStore: remove unused return code

indent correctly

curl_easy_getinfo.3: clarify some timing info

curl_easy_setopt.3: clarify the SSH KEYFILE options usage

The internal defaults are important info

docs: mention the protocol:// support in proxy strings

--data-ascii: add mention

As it is a separate option it should have a .IP title

url encode docs: mention '-', '.', '_' and '~'

Clarify that the '-', '.', '_' or '~' letters are also not escaped since
they shouldn't according to RFC3986 section 2.3.

This is how this function has behaved since sep 2010, commit
5df13c31735fa0.

SSH: set non-blocking earlier

Introduce an INIT state for the SSH state machine and set libssh2
non-blocking in that so that it is set properly before
libssh2_session_startup() is called.

Bug: http://curl.haxx.se/mail/archive-2011-05/0001.html

curl_formfree.3: mention argument may be NULL

As the code already checks for it we can just as well make it official!

ConnectionExists: avoid NULL dereference

When checking for connections that are bound to a particular device we
must make sure we don't compare with a NULL pointer.

resolver_error: remove bad semicolon

RELEASE-NOTES: synced with c4bc1d473f324

anyauthput.c: stdint.h must not be included unconditionally

As it is already included by curlbuild.h if it exists on the platform it
was included here superfluously anyway.

Reported by: Dagobert Michelsen
Bug: http://curl.haxx.se/bug/view.cgi?id=3294509

gai_strerror: provide private implementation

There are systems (like NetWare) without its own gai_strerror()
function.

treaded-resolver: better error messages

Now use gai_strerror() to get proper error messages when getaddrinfo()
has failed. Detect the function in configure.

Code based on work and suggestions by Jeff Pohlmeyer and Guenter Knauf

proxy: allow socks:// prefix in proxy string

Inspired by a patch from OB.Conseil. Added test case 708 to verify.

CMake: improve library search, implement install.

Improved library search by check_function_exists_concat() macro:
it does not revert the list of libraries any more.

Improved OpenSSL library search: first find zlib, then search for
openssl libraries that may depend on zlib.

For Unix: openssl libraries can now be detected in nonstandard
locations. Supply CMAKE_LIBRARY_PATH to CMake on command line.

Added installation capability (very basic one yet).

multi-socks: fix connect to proxy

When connecting to a socks or similar proxy we do the proxy handshake at
once when we know the TCP connect is completed and we only consider the
"connection" complete after the proxy handshake. This fixes test 564
which is now no longer considered disabled.

Reported by: Dmitri Shubin
Bug: http://curl.haxx.se/mail/lib-2011-04/0127.html

cleanup: remove old unused debug code

Make checksrc.pl work on out-of-tree builds

Source files given with absolute paths do not have the
-D directory specifier prepended.

make: add checksrc.pl to dist

make: add 'checksrc' as target to check code style

The make target checksrc now works in the root makefile and in both the
src and lib directories.

It is also run automatically on "all" if configure --enable-debug was
used.

checksrc: whitespace and code style cleanup

Make everything adhere to the standards upheld by checksrc.pl and now
run checksrc from the makefile on debug builds.

checksrc: add -W to allow a file to be whitelisted

Useful when a known file just doesn't comply and there's no intention to
make it do so.

whitespace cleanup: no space first in conditionals

"if(a)" is our style, not "if( a )"

lib/make: run checksrc.pl once for all files

Since it now supports multiple files this will be faster and will show
problems for more file than one at a time - more convenient.

checksrc: scan many files, more checks

It now scans multiple files and outputs an error+warning count summary
at the end in case at least one was detected.

-D can be used to specify in which dir the files are located

The script now scans for conditions that starts with a space for
if/while/for lines.

Makefile: run checksrc if debug enabled

And make the build fail if a warning or error was detected

checksrc: exit error code if warnings or errors

SSH: move knownhost logic to separate function

source cleanup: unify look, style and indent levels

By the use of a the new lib/checksrc.pl script that checks that our
basic source style rules are followed.

Windows native IDN fixes.

For now provide prototypes instead of including the
non-standard normalisation.h which is only available in the
"Internationalized Domain Names Mitigation APIs" download.

Fixed compiler warning in Windows SSPI case

Fixed compilation when the synchronous resolver is used

Added new resolver sources to Symbian and VC6 build files

hostip: comment fixed to state current situation

async resolvers: further cleanups

asyn-ares.c and asyn-thread.c are two separate backends that implement
the same (internal) async resolver API for libcurl to use. Backend is
specified at build time.

The internal resolver API is defined in asyn.h for asynch resolvers.

asynch resolvers: cleanup

Fixed indents, coding conventions and white space edits.

Modified the c-ares completion callback function to again NOT read the
conn data when the ares handle is being taken down as then it may have
been freed already.

asynch resolvers: unified

Introducing an internal API for handling of different async resolver
backends.

Enabled OpenWatcom native Windows IDN build.

For now we directly import the Idn* symbols with the linker;
an upcoming release of OWC will have these added to the import
lib normaliz.lib, and prototypes are added to winnnls.h.

NTLM: work with unicode

Rewritten code from a patch brought by Matteo Rocco.

bump version: work towards 7.21.7

THANKS: contributors from 7.21.6

RELEASE-NOTES: two more contributors

Fixed test 1023 when using daily snapshots

Include unistd.h to declare close()

In lib/, change 'wanna' to 'want to'.

Found with codespell.

Fix spelling errors in buildconf

Found with codespell.

Fix spelling errors in src/

Found with codespell.

Fix spelling errors in include/

Fix a couple of spelling errors in lib/

Found with codespell.

transfer.c: Fixed indentation in readwrite_data.

Fixed closing test tag

RELEASE-NOTES: synced with 3242abd87a1262

SFTP: close file before postquote

Make sure that files are closed before the post quote commands run as if
they operate on the just transferred file they could otherwise easily
fail.

Patch by: Rajesh Naganathan (edited)

Fixed test 1022 when using daily snapshots

Curl_http_connect: detect HTTPS properly after CONNECT

libcurl failed to check the correct struct for HTTPS after CONNECT was
issued to the proxy, so it didn't do the TLS handshake and subsequently
failed the connection. A regression released in 7.21.5 (introduced
around commit 8831000bc07de).

Bug: http://curl.haxx.se/mail/lib-2011-04/0134.html
Reported by: Josue Andrade Gomes

curl_easy_setopt.3: CURLOPT_PROXYTYPE clarification

When set to a HTTP 1.0 proxy, that only affects the CONNECT request and
not the regular HTTP request.

CURL_DOES_CONVERSIONS: fixes

Made it compile and work again after the code move.

CURL_DOES_CONVERSIONS: cleanup

Massively reduce #ifdefs all over (23 #ifdef lines less so far)
Moved conversion-specific code to non-ascii.c

Improve MinGW static makefile builds.

It is now possible to use any combination of features without
having to 1st add makefile targets to the main makefile. The
main makefile now passes the 'mingw32-feat1-feat2' as var CFG,
and the ./[lib|src]/Makefile.m32 parses the CFG var to determine
the features to be enabled.

Enabled MinGW native Windows IDN build.

Windows native IDN fixes.

changed windows.h include to system header;
changed obsolete 2nd check for str_w to str_utf8 in order to catch
malloc() failure and avoid a free(NULL);
changed calls to GetLastError() to void to kill unsused var compiler
warnings;
moved one call to GetLastError() into else case so that its only
called when WideCharToMultiByte() really fails.

Windows native IDN fixes.

Provide prototype for curl_win32_idn_to_ascii();
remove wrong 3rd parameter from curl_win32_idn_to_ascii() call.

curl-config: fix version output

do the s/VERSION/CURLVERSION replacement for the human redable output
for --checkfor

Reported by: Ryan Schmidt

RELEASE-NOTES: synced with 5aae3c13e2

Updated default (recommended) dependency versions.

Updated default (recommended) dependency versions.

transfer-encoding: document the options

The new libcurl and command line options are now described.

transfer-encoding: added new option and cmdline

Added CURLOPT_TRANSFER_ENCODING as the option to set to request Transfer
Encoding in HTTP requests (if built zlib enabled). I also renamed
CURLOPT_ENCODING to CURLOPT_ACCEPT_ENCODING (while keeping the old name
around) to reduce the confusion when we have to encoding options for
HTTP.

--tr-encoding is now the new command line option for curl to request
this, and thus I updated the test cases accordingly.

CURLE_BAD_CONTENT_ENCODING: now used for transfer encoding too

TE: do the Connection: header

When TE: is inserted in the request, we must add a "Connection: TE" as
well to be HTTP 1.1 compliant. If a custom Connection: header is passed
in, we must use that and only append TE to it. Test case 1125 verifies
TE: + custom Connection:.

test1124: verify gzip AND chunked transfer-encoding