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

file_range: Value stored to 'totalsize' is never read

changelog: GnuTLS: SSL handshake phase is non-blocking

GnuTLS: make the connection phase non-blocking

When multi interface is used, the SSL handshake is no longer
blocking when GnuTLS is used.

krb5_auth: fix my previous change to compile

OS400 version V5R2M0 not supported anymore by IBM: default target release changed to V5R3M0.

changelog: GnuTLS fix, no reverse loopkups and fixed GSS detection

SSL_RECV: EOF is not an error here

The recent overhaul of the SSL recv function made this treat a
zero returned from gnutls_record_recv() as an error, and this
caused our HTTPS test cases to fail. We leave it to upper layer
code to detect if an EOF is a problem or not.

configure: GSSAPI detection on ancient Linux distros

On some ancient distributions such as RHEL-3, <gssapi/gssapi_krb5.h> needs
to be processed after <gssapi/gssapi.h>, but does not include it itself.
This patch checks for <gssapi/gssapi.h> first and then includes it
in the test for <gssapi/gssapi_krb5.h>, resolving the problem.

Without the patch, <gssapi/gssapi_krb5.h> is "present but cannot be
compiled".

resolvers: no more using AI_CANONNAME

No resolver anymore needs to use AI_CANONNAME and do reverse
lookups.  We should work hard to avoid having code that relies on
it.

KRB5: use given host name instead of reverse lookup'ed name

This code would previously use dns_entry->addr->ai_canonname
instead of the given host name, which caused us grief and
problems since not all our resolver options do the reverse lookup
and I would also guess that it caused problems with KRB5/GSS with
virtual name-based hosts. Now the host name from the URL is used.

Remove redundant conditional

Eliminated an unlikely race condition in some tests.

Based on a patch from the FreeBSD ports by Peter Pentchev.

changelog: prevent needless reverse name lookups

Curl_ipv4_resolve_r: only set AI_CANONNAME when needed

As reported in bug report #2987196, the code for ipv6 already did
the setting of this bit correctly so we copied that logic into
the Curl_ipv4_resolve_r() function as well. KRB code is the only
code we know that might need the cannonical name so only resolve
it for such requests!

IGNORE: files generated by maketgz

bumped to start the journey towards 7.20.2

added contributors from the 7.20.1 RELEASE-NOTES

ignore files generated by 'maketgz'

7.20.1: 14 April 2010

Use correct directory for c-ares git pull
Signed-off-by: Tor Arntsen <tor@spacetec.no>

fix compiler warning: variable might be clobbered by longjmp or vfork

added last git commit output for c-ares too.

changed the git update block to take care of c-ares repo if detected.

updated timestamp of the script.

removed obsolete var in gitpull() function
no need to create a var - lets just return the status var itself.

added a cast to silent compiler warning with 64bit systems.

fixed a path typo in src/Makefile.netware.

Added text for How To Make a Patch with git

update the section on timeouts

The section that describes how to work with timeouts was
misleading and could easily trick users to use the wrong API.

update URL and cut out wrong info on ipv6

c-ares has had its own URL for a while and we should point
people to that. It also works with IPv6 since a long time.

refer to CURLMOPT_TIMERFUNCTION for multi_socket users

curl_multi_timeout(3) is simply the wrong function to use
if you're using the multi_socket API and this document now
states this pretty clearly to help guiding users.

s/CVS/git

modified to use the git file, not cvs

I've done this blindly, and the last piece that works with ares
should possibly be done differently now that c-ares isn't a
subtree within the curl tree anymore...

mention missing test servers for <server>

FTP quote commands prefixed with '*' now can fail without aborting

Prefixing the FTP quote commands with an asterisk really only
worked for the postquote actions. This is now fixed and test case
227 has been extended to verify.

qssl: reflect recent code changes in SSL interface

Reported by Guenter Knauf.

nss: handle client certificate related errors

ssl: Fix build when SSL isn't enabled

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

refactorize interface of Curl_ssl_recv/Curl_ssl_send

simplify code of Curl_resolv_timeout()