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!

Avoid double newline for the 'last commits' log in testcurl.pl

The backtick command which extracts 'git log' lines come with a
newline, so chomp the newline before calling logit(), as the logit
function adds a newline by itself.

Change to version-independent git option for 'git log --oneline'

'git log --oneline' is a relatively recent Git function. It is
documented to be the same as 'git log --pretty=oneline --abbrev-commit',
so use that instead. It works all the way back to Git 1.5.0.

show 5 commits even if no git pull was made

don't touch ares/aclocal.m4 and show recent git commits

since c-ares no longer embedded, we must not touch such files
anymore

we show the 5 last git commits if git was proven in use, to help
us see exactly what's being tested

use CURL_SIZEOF_LONG instead of SIZEOF_LONG

That's the symbol we have or generate in include/curl/curlbuild.h

s/CVS/DEV in the version string from the git repo

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

Add .gitattributes files to turn off CRLF translation for some files

provide a version number as today's date

It should at least help visualize which autobuilds that are
using this script.

testcurl now uses git instead of CVS

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

Enable LDAP by default since it is now disabled when ldap.h is not found,

CMake fixes for Linux.

Make sure <sys/socket.h> is included if around when testing/using
socklen_t.  Also, disable LDAP if LDAP_H is not found on the system.

fix: timeout after last data chunk was handled

Bob Richmond: There's an annoying situation where libcurl will
read new HTTP response data from a socket, then check if it's a
timeout if one is set. If the last packet received constitutes
the end of the response body, libcurl still treats it as a
timeout condition and reports a message like:

"Operation timed out after 3000 milliseconds with 876 out of 876
bytes received"

It should only a timeout if the timer lapsed and we DIDN'T
receive the end of the response body yet.

avoid compiler warning without USE_ALARM_TIMEOUT

Fix curl CMake build.

This commit fixes the cmake build of curl, and cleans up the
cmake code a little.  It removes some commented out code and
some trailing whitespace.  To get curl to build the binary
tree include/curl directory needed to be added to the include
path. Also, SIZEOF_SHORT needed to be added.  A check for the
lack of defines of SIZEOF_* for warnless.c was added.

remove debug printfs

RTSP GET_PARAMETER fix

Christopher Conroy fixed a problem with RTSP and GET_PARAMETER
reported to us by Massimo Callegari. There's a new test case 572
that verifies this now.

remove trace of CVS

Fix RTSP GET_PARAMETER empty and non-empty operation.

Test coverage included. Thanks to Massimo Callegari for the bug report

s/CVS/DEV/ in the version string for repo versions

scrapped all left-over TODOs

In order to get back on track, I've removed all the plans for
stuff I had in the queue. I will instead focus on fixing bugs and
relying on that people who truly want things added will come back
on the mailing list and nag and provide patches.

7.20.1 should be possible to release in April 2010

restore executable bits on some files

remove the CVSish $Id$ lines

The 'ares' subtree has been removed from the source repository

s/CVS/git

update to current state

remove the ares subtree

c-ares is now hosted entirely separate from the curl project
see http://c-ares.haxx.se/ for all details concerning c-ares,
its source repository and more.

mark connection as connected

Kenny To filed the bug report #2963679 with patch to fix a
problem he experienced with doing multi interface HTTP POST over
a proxy using PROXYTUNNEL. He found a case where it would connect
fine but bits.tcpconnect was not set correct so libcurl didn't
work properly.

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