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()

eliminate a race condition in Curl_resolv_timeout()

fixed compiler warnings

updated contributor count

add contributors from the 7.20.0 release notes

fix SFTP download hang

Matt Wixson found and fixed a bug in the SCP/SFTP area where the
code treated a 0 return code from libssh2 to be the same as
EAGAIN while in reality it isn't. The problem caused a hang in
SFTP transfers from a MessageWay server.

Fixed misleading test message

update the generic copyright year range

removed README.cmake due to the improved situation

fix compiler warning with a cast.

make folks use latest available dependent libraries.

Call curl_global_cleanup() in test 560 to avoid a memory leak

Allow test 538 to run even when proxy support is disabled

use (s)size_t for string lengths to fix compiler warns

use size_t to hold string length

using int is not fine on 64bit systems

Fix compile warnings in ssh.c

strlen() returns size_t, but ssh libraries are wanting 'unsigned int'.  Add
explicit casts and use _ex versions of the ssh library calls.

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

fix smtp compile warning

Use ssize_t instead of int for the Curl_smtp_escape_eob nread
argument.

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

Ben's POP3 change

pop3: Get message listing if no mailbox in URL

If you pass a URL to pop3 that does not contain a message ID as
part of the URL, it will currently ask for 'INBOX' which just
causes the pop3 server to return an error.

The change makes libcurl treat en empty message ID as a request
for LIST (list of pop3 message IDs).  User's code could then
parse this and download individual messages as desired.

Allow running ./tests/testcurl.pl from within git repo.

My first instinct was to run the test script within the checked out
repository.  This small change to the script allows that to work as
expected.

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

minor language fix

allow user+password in the URL for all protocols

Ben Greear brought a patch that from now on allows all protocols
to specify name and user within the URL, in the same manner HTTP
and FTP have been allowed to in the past - although far from all
of the libcurl supported protocols actually have that feature in
their URL definition spec.

ignore pid files and stunnel.conf

all used while running tests

make sure git pull is actually done!

changelogged: smoother rate limiting

Make rate-limitation logic smoother

This gives a smoother rate limitation performance by using
sub-second pauses and also taking the buffer sizes into
account.

remove all .cvsignore files

PROT_CLOSEACTION should not include TFTP

TFTP is not a protocol that uses close actions so it should
not be set in that bitmask!