LDAP: properly implemented as a curl_handler

makes the LDAP code much cleaner, nicer and in general being a
better libcurl citizen. If a new enough OpenLDAP version is
detect, the new and shiny lib/openldap.c code is then used
instead of the old cruft

Code by Howard, minor cleanups by Daniel.

curl_fnmatch: Use int not bool when function returns int

bool in curl internals is unsigned char and should not be used
to receive return value from functions returning int - this fails
when using IBM VisualAge and Tru64 compilers.

TFTP: send legal timeout value

Eric Mertens posted bug #3003705: when we made TFTP use the
correct timeout option when sent to the server (fixed May 18th
2010) it became obvious that libcurl used invalid timeout values
(300 by default while the RFC allows nothing above 255). While of
course it is obvious that as TFTP has worked thus far without
being able to set timeout at all, just removing the setting
wouldn't make any difference in behavior. I decided to still keep
it (but fix the problem) as it now actually allows for easier
(future) customization of the timeout.

(http://curl.haxx.se/bug/view.cgi?id=3003705)

TFTP: don't ack if wrong block num is received

If an unexpected block number was received, break out of the
switch loop.

TFTP: block id wrap bug fix

In a normal expression, doing [unsigned short] + 1 will not wrap
at 16 bits so the comparisons and outputs were done wrong. I
added a macro do make sure it gets done right.

Douglas Kilpatrick filed bug report #3004787 about it:
http://curl.haxx.se/bug/view.cgi?id=3004787

Fix build warnings.

Signed-off-by: Ben Greear <greearb@candelatech.com>

setopt: Fix setting of set.is_fwrite_set

Signed-off-by: Ben Greear <greearb@candelatech.com>

build: allow curl to build with Microsoft VC10

By undefing a bunch of E* defines that VC10 has started to define
but that we redefine internally to their WSA* alternatives when
building for Windows.

Test 573: Use correct type for CURLINFO_CONNECT_TIME

curl_easy_getinfo() called with a pointer to long instead of double
would sigbus on RISC processors (e.g. MIPS) due to wrong alignment
of pointer address.

lib: Fix AIX build failure

Fixed some memory leaks in the POP3 torture tests

Fixed a memory leak in the SMTP torture tests

TFTP: send timeout option correctly

Eric Mertens posted bug report #3003005 pointing out that the
libcurl TFTP code was not sending the timeout option properly to
the server, and suggested a fix.

(http://curl.haxx.se/bug/view.cgi?id=3003005)

lib: Change some CRLF line endings to LF

An update had added a couple of lines with DOS line endings,
and some compilers will choke on that (e.g. the Tru64 compiler).

ftp wildcard: a new option CURLOPT_FNMATCH_DATA

RMTP: the version code is now rtmp aware

RTMP: fix wrong #ifdef

ftp wildcard: fix int32_t and size/group mixups

Fixed test 577 to work when --enable-hidden-symbols is configured

OpenSSL: multi interface handshake could hang

John-Mark Bell filed bug #3000052 that identified a problem (with
an associated patch) with the OpenSSL handshake state machine
when the multi interface is used:

Performing an https request using a curl multi handle and using
select or epoll to wait for events results in a hang. It appears
that the cause is the fix for bug #2958179, which makes
ossl_connect_common unconditionally return from the step 2 loop
when fetching from a multi handle.

When ossl_connect_step2 has completed, it updates
connssl->connecting_state to ssl_connect_3. ossl_connect_common
will then return to the caller, as a multi handle is in
use. Eventually, the client code will call curl_multi_fdset to
obtain an updated fdset to select or epoll on. For https
requests, curl_multi_fdset will cause https_getsock to be called.
https_getsock will only return a socket handle if the
connecting_state is ssl_connect_2_reading or
ssl_connect_2_writing.  Therefore, the client will never obtain a
valid fdset, and thus not drive the multi handle, resulting in a
hang.

(http://curl.haxx.se/bug/view.cgi?id=3000052)

changelog: add link to bug report

Added directories.pm to the source tar ball

follow redirect: ignore response-body on redirect even if compressed

Sebastian V reported bug #3000056 identifying a problem with
redirect following. It showed that when curl followed redirects
it didn't properly ignore the response body of the 30X response
if that response was using compressed Content-Encoding!

(http://curl.haxx.se/bug/view.cgi?id=3000056)

version: we're now going for 7.21.0

Remove support for BSD version of PolarSSL

"The BSD version of PolarSSL was made for migratory purposes only and is not
maintained. The GPL version of PolarSSL is actually the only actively
developed version, so I would be very reluctant to use the BSD version." /
Paul Bakker, PolarSSL hacker.

Signed-off-by: Hoi-Ho Chan <hoiho.chan@gmail.com>

Added Polar SSL and RTMP files to the non-autoconf build files

I didn't bother with a few that have little hope of running the required
dependent libraries.

Added the new ftp source files to the non-autoconf build files

Copy the license file so it's seen by the Android build system

updated with symbols added in recent commits for 7.21.0

changelogs: mention RTMP and the FTP wildcard support

ftp wildcards: mention they're added in 7.21.0

style: minor whitespace change

syntax: cleanups

FTP: WILDCARDMATCH/CHUNKING/FNMATCH added

RTMP: initial support added, powered by librtmp

librtmp is found at http://rtmpdump.mplayerhq.hu/

sendrecv: make them two pairs of send/recv to properly deal with FTPS

FTP(S) use two connections that can be set to different recv and
send functions independently, so by introducing recv+send pairs
in the same manner we already have sockets/connections we can
work with FTPS fine.

This commit fixes the FTPS regression introduced in change d64bd82.

changelog: fixed CRL support in libcurl-NSS

nss: make it possible to read ASCII and DER CRL

nss: add CRL to cache instead of read-only NSS db

git: how to write a fine commit message

findtool: file name as a full path requires a slash

Kalle Vahlman's patch applied a while ago broke how the findtool
function searches for tools, as it would always check if "$file"
was present first, which thus made the bad assumption that a file
in the current directory would be a match.

I noticed when it found 'libtool' in the current directory but
libtoolize is not there, which confused the script.

moved vars into conditional since seems that winsock implementation doesnt use them.

multi interface: missed storing connection time

Dirk Manske reported a regression. When connecting with the multi
interface, there were situations where libcurl wouldn't store
connect time correctly as it used to (and is documented to) do.

Using his fine sample program we could repeat it, and I wrote up
test case 573 using that code. The problem does not easily show
itself using the local test suite though.

The fix, also as suggested by Dirk, is a bit on the ugly side as
it adds yet another call to Curl_verboseconnect() and setting the
TIMER_CONNECT time.  That situation is subject for some closer
inspection in the future.

verboseconnect: so the verbose checking within the function

As the function is used more than once and libcurl can be built
without it, do the conditional check within the verboseconnect()
function itself.

changelogs: split the I/O handling

sendrecv: split the I/O handling into private handler

Howard Chu brought the bulk work of this patch that properly
moves out the sending and recving of data to the parts of the
code that are properly responsible for the various ways of doing
so.

Daniel Stenberg assisted with polishing a few bits and fixed some
minor flaws in the original patch.

Another upside of this patch is that we now abuse CURLcodes less
with the "magic" -1 return codes and instead use CURLE_AGAIN more
consistently.

changelog: PolarSSL

PolarSSL: initial support added

This is Hoi-Ho Chan's patch with some minor fixes by me. There
are some potential issues in this, but none worse than we can
sort out on the list and over time.

TODO: we've done PRET already, consider HOST for the future

... and GnuTLS connects are non-blocking, TFTP is better
integrated as a "real" protocol and RTSP is supported.

TODO: GnuTLS connects are now non-blocking

Since commit c288860 by Jerome Vouillon

INTERNALS: tftp is decent now, ldap is not

It's not quite fair to list TFTP is a "crappy" member of the
libcurl family so I removed its mentioning.

changelog: mention Ben Greear's telnet work

telnet: Allow programatic use of telnet.

The main change is to allow input from user-specified methods,
when they are specified with CURLOPT_READFUNCTION.
All calls to fflush(stdout) in telnet.c were removed, which makes
using 'curl telnet://foo.com' painful since prompts and other data
are not always returned to the user promptly.  Use
'curl --no-buffer telnet://foo.com' instead.  In general,
the user should have their CURLOPT_WRITEFUNCTION do a fflush
for interactive use.

Also fix assumption that reading from stdin never returns < 0.
Old code could crash in that case.

Call progress functions in telnet main loop.

Signed-off-by: Ben Greear <greearb@candelatech.com>

test: enable valgrind for 604, seems to work

add 1s post-command delay to tests 513 and 514

addressing http://curl.haxx.se/mail/lib-2009-12/0031.html

Allow tools to be defined with full path in buildconf

This is required in Scratchbox where
LIBTOOL=/targets/links/arch_tools/bin/libtool
is set in the environment.

progress callback: can be called more than once per sec

SSH: init and cleanup libssh2 in global_init/cleanup

The necessary libssh2 functions require libssh2 1.2.5 or later.

new configure option --enable-threaded-resolver

configure: check for libssh2_init and libssh2_exit

nss: fix SSL handshake timeout underflow

encourage users to take latest lib dependencies.

socks5: please static code analyzer

Make sure we don't call memcpy() if the argument is NULL even
though we also passed a zero length then, as the clang analyzer
whined and we want to limit warnings (even false positives) when
they're this easy to fix.

The change of (char) to (unsigned char) will fix long user names
and passwords on systems that have the char type signed by
default.

gzip: Value stored to 'data' is never read

RELEASE-NOTES: update top numbers

changelog: added the --proto and -proto-redir options

curl: added --proto and --proto-redir

--proto tells curl to use the listed protocols for its initial
retrieval

--proto-redir tells curl to use the listed protocols after a
redirect

test536: do not fail with threaded DNS resolver

Also tweaked comments in certain examples using curl_multi_fdset().

curl: -O crash on windows

The -O option caused curl to crash on windows and DOS due to the
tool writing out of boundary memory.

hmac.c related compilation adjustment

hmac.c related compilation adjustment

Add compilation directives for hmac in Watcom,riscos and vc6 platform-specific makefiles.

replaced wsock32.lib usage with ws2_32.lib in MSVC makefiles

Merge branch 'master' of github.com:bagder/curl

Remove null-effect leftover code.

changelog: -J/--remote-header-name strips CRLF

parse_filename: strip trailing CRs and LFs

The feature that uses the file name given in a
Content-disposition: header didn't properly skip trailing
carriage returns and linefeed characters from the end of the file
name when it was given without quotes.

Curl_HMAC_MD5: fix the array init to not warn with picky compilers

Fix GnuTLS compilation problem in md5.c

Fix compilation problem: declare Curl_HMAC_MD5 as extern in include file.

Merge branch 'master' of github.com:bagder/curl

Implement SMTP authentication

parseconfig: Value stored to 'line' is never read

Make the function call with (void) as we don't care about the
return code.

parsedate: Value stored to 'found' is never read

check_gzip_header: Value stored to 'data' is never read

dprintf_formatf: Value stored to 'left' is never read

curl_version: remove superfluous assignments

FTP PORT: Value stored to 'rc' is never read

Curl_setup_transfer: no longer returns anything

This function could only return CURLE_OK and by changing it to
a void instead, we can simplify code all over.

PASV response: Value stored to 'rc' is never read

Curl_perform: Value stored to 'res2' is never read

sftp range: remove unnecessary check for NULL pointer

ftp_range: remove unnecessary check for NULL pointer

file_range: remove unnecessary check for NULL pointer

SOCKS4: Value stored to 'rc' is never read

FTP PASV: Value stored to 'rc' is never read

ftp_range: Value stored to 'totalsize' is never read

Simplified the code by removing a local variable completely.

SOCKS5: when name resolves fail return immediately

This makes the code flow more obvious and reacts on the return
code properly, even if the code acted the same way before.

POP3: when USER command fails, don't even try PASS

tftp_rx: Value stored to 'sbytes' is never read