tests: Added SMTP NOOP and RSET tests

test928: Corrected typo in expected data from commit df58084695ce9b

tests: Added SMTP HELP test

smtp: Changed the default command to HELP when no options are specified

Otherwise a NOOP operation would be performed which a) only returns a
single line response and not a multiline response where -I needs to be
used, and b) provides an inconsistent user experience compared to that
of the POP3 and IMAP protocols.

smtp:Fixed memory leak from commit dac01ff6d788b2

The buffer allocated by smtp_parse_custom_request() was not freed.

connect: Close open but unconnected socket in singleipconnect()

singleipconnect() could return the file descriptor of an open socket
even though the function returned a CURLE_COULDNT_CONNECT error code
from commit ed1662c374361a and 02fbc26d59c591.

This could cause tests 19, 704 and 1233 to fail on FreeBSD, AIX and
Solaris.

tests: Added SMTP EXPN command test

test926: Corrected unknown user reply from commit 0d735c29f92748

The error code should not be sent as data as it isn't passed onto the
client as body data, so cannot be compared in the test suite against
expected data.

tests: Corrected missing data reply sections from 3d50e91aeef8cc

tests: Added SMTP VRFY command tests

ftpserver.pl: Added support for new SMTP commands

smtp: Fixed broken RCPT TO from commit 0ea4a80bb23666

smtp_state_mail_resp: removed unused variable 'smtp'

DOCS: Updated curl_easy_setopt.3 following recent SMTP changes

* Added information about the verify and expand commands to
  CURLOPT_MAIL_RCPT.
* Reworked CURLOPT_CUSTOMREQUEST section, adding information about IMAP
  and SMTP custom commands.

DOCS: Updated manpage following recent SMTP modifications

* Added SMTP section to --request
* Expanded --mail-rcpt to describe the usage when using the verify and
  expand commands.

smtp: Simplified the next RCPT TO logic

lib1507.c: Added missing set of CURLOPT_UPLOAD option

Although this option should have already been set, the SMTP module can
now download information from and send instructional commands to, an
SMTP server, requiring the option to be set in order to perform a mail
transfer.

smtp.c: Fixed trailing whitespace

smtp: Added support for VRFY and EXPN commands

smtp: Added support for NOOP and RSET commands

smtp: Fixed handling of multiline server greeting responses

Incorrectly processed multiline server greeting responses as "wanted"
continue responses in smtp_endofresp(), from commit f16c0de4e9bbe3,
which in turn broke the SMTP server detection in the test suite,
because the EHLO command would not be sent.

smtp.c: Fixed compilation error from commit f16c0de4e9bbe3

warning: unused variable 'smtpc'

smtp: Fixed processing of more than one response when sent in same packet

Added a loop to smtp_statemach_act() in which Curl_pp_readresp() is
called until the cache is drained. Without this multiple responses
received in a single packet could result in a hang or delay.

smtp: Moved EHLO response handling to smtp_state_ehlo_resp()

Similar to the processing of untagged CAPABILITY responses in IMAP moved
the processing of multiline EHLO responses to smtp_state_ehlo_resp() and
introduced an internal response code of one to differentiate a multiline
continuation from the end of command. This also allows for the separate
processing of multiline responses from commands such as VRFY and EXPN.

connect: Forgot to correct line endings before push in commit ed1662c374361a

connect: Return the socket descriptor even on fail

singleipconnect() did not return the open socket descriptor on some
errors, thereby sometimes causing a socket leak. This patch ensures
the socket is always returned.

configure: Fix test with -Werror=implicit-function-declaration

The ipv6 auto-detect test in configure returns a false negative when
CFLAGS contains -Werror=implicit-function-declaration. (I have been
using this flag to detect code issues that would result in SEGVs on
x86_64-cygwin.)

Patch-by: Yaakov Selkowitz
Bug: http://curl.haxx.se/bug/view.cgi?id=1304

test825: Corrected typo from commit b29217d0d682d4

RELEASE-NOTES: Synced with bde901ad89a6f1

test922: Corrected title to match other OAuth 2.0 tests

tests: Added IMAP OAuth 2.0 authentication with initial response test

tests: Added IMAP NTLM authentication with initial response test

tests: Added IMAP login authentication with initial response test

tests: Added IMAP plain authentication with initial response test

test873: Use proper padding in NTLM responses

darwinssl: check for SSLSetSessionOption() presence when toggling BEAST

Even though this is only a formality (since not many people build on
Mavericks while targeting Leopard), since we still support Leopard
at the earliest, we might as well be pedantic.

darwinssl: PKCS#12 import feature now requires Lion or later

It turns out that some of the constants necessary to make this feature
work are missing from Snow Leopard's Security framework even though
they are defined in the headers.

Bug: http://curl.haxx.se/mail/lib-2013-11/0076.html
Reported by: myriachan

tests: Added POP3 OAuth 2.0 authentication with initial response test

tests: Added POP3 NTLM authentication with initial response test

tests: Added POP3 login authentication with initial response test

tests: Added POP3 plain authentication with initial response test

TODO: Added auth= in URLs to the wish list for HTTP

curl_easy_setopt: Added the ability to set the login options separately

Rather than set the authentication options as part of the login details
specified in the URL, or via the older CURLOPT_USERPWD option, added a
new libcurl option to allow the login options to be set separately.

curl.1: mention that -O does no URL decoding

curl_easy_setopt.3: clarify CURLOPT_SSL_VERIFYHOST documentation

- better describe what happens when 1 is specified as parameter
- clarify what "is ignored" means for NSS builds

runtests.pl: Added SSPI detection

multi: Small code tidy up to avoid hard return

tests: Updated CRAM-MD5 tests to use test user details

multi: Set read socket when returning READSOCK(0)

This patch fixes and issue introduced in commit 7d7df831981fee, if the
tunnel state was TUNNEL_CONNECT, waitconnect_getsock() would return a
bitmask indicating a readable socket but never stored the socket in the
return array.

bump: next release will be 7.34.0

Due to all the news and changes.

connect: Close temporary sockets in conn_free()

The temporary sockets used for Happy Eyeballs were not closed properly,
if curl exited prematurely, which this patch fixes.

tests: Corrected titles of POP3 and SMTP OAuth 2.0 tests

test823: Fixed expected authentication text from commit e10a26a9d6d6de

Fixed authentication text due to incorrect digest-uri property.

test821: Fixed expected authentication text from commit 2d5455feac9984

RELEASE-NOTES: Synced with 0f81fbe5da6643

ares: Fixed compilation under Visual Studio 2012

tests: Added IMAP OAuth 2.0 authentication test

tests: Added IMAP DIGEST-MD5 authentication test

tests: Added IMAP NTLM authentication test

tests: Added IMAP CRAM-MD5 authentication test

test819: Fixed expected authentication text from commit 76f924131c9fd3

ftpserver.pl: Reworked custom reply handling code

1) To fix issues with IMAP custom replies
2) So initial space is not required in IMAP display text
3) To be more readable and understandable

ftpserver.pl: Reworked unrecognised command responses

As the IMAP regex could fail and $1 would not contain the command id
updated the unrecognised command response to be more generic and
realistic (like those used in the command handlers).

Additionally updated the POP3, SMTP and FTP responses.

ftpserver.pl: Fixed processing of IMAP authentication strings

cmake: fix Windows build with IPv6 support

Patch-by: "Z98"

Revert "ftpserver.pl: Corrected logic from commit 27b7b1062f9d97"

This reverts commit 558034ab7002d1 as it appears to break the auto
builds. More thought is required for this!

ftpserver.pl: Corrected logic from commit 27b7b1062f9d97

ftpserver.pl: Fixed IMAP cmdid being sent on custom responses

tests: Added IMAP login authentication test

tests: Added IMAP plain authentication test

url.c: Very small amount of policing

url.c: Remove superfluous for loop

The reason for this loop's existence was removed in commit
02fbc26d59c591.

KNOWN_BUGS: STARTTRANSFER for POST requests

added 85. Wrong STARTTRANSFER timer accounting for POST requests

Curl_ssl_push_certinfo_len: don't %.*s non-zero-terminated string

Our own printf() replacement clearly can't properly handle %.*s with a
string that isn't zero terminated. Instead of fixing the printf code or
even figuring out what the proper posix behavior is, I reverted this
piece of the code back to the previous version where it does malloc +
memcpy instead.

Regression added in e839446c2a5, released in curl 7.32.0.

Reported-by: Felix Yan
Bug: http://curl.haxx.se/bug/view.cgi?id=1295

RELEASE-NOTES: Synced with 7fc3b2ce382ed6

connect.h: Updated copyright year for last edit

connect: Add connection delay to Happy Eyeballs.

This patch adds a 200ms delay between the first and second address
family socket connection attempts.

It also iterates over IP addresses in the order returned by the
system, meaning most dual-stack systems will try IPv6 first.

Additionally, it refactors the connect code, removing most code that
handled synchronous connects. Since all sockets are now non-blocking,
the logic can be made simpler.

winbind: Fixed ntlm_auth expecting eol following commit e17c1b25bc33eb

nss.c: Fixed compilation warnings (Take Two)

nss.c:702: warning: pointer targets in passing argument 3 of
'Curl_extract_certinfo' differ in signedness

nss.c:702: warning: pointer targets in passing argument 4 of
'Curl_extract_certinfo' differ in signedness

Made sure the cast was correctly "unsigned char *" to "char *" and not
"unsigned char *" to "unsigned char *".

nss.c: Fixed compilation warnings

nss.c:700: warning: pointer targets in passing argument 3 of
'Curl_extract_certinfo' differ in signedness

nss.c:700: warning: pointer targets in passing argument 4 of
'Curl_extract_certinfo' differ in signedness

tests: Added test for IMAP NOOP command

tests: Added test for IMAP COPY command

tests: Disable IMAP CLOSE and EXPUNGE command tests

This is temporary until curl supports either multiple custom commands
or post-quote commands in IMAP.

tests: Added tests for IMAP CLOSE and EXPUNGE commands

connect.c: Code policing on commit 5094bb53f4a027

No need for a rhs condition on a bitwise compare.

nss.c: Fixed compilation warnings

warning: implicit declaration of function 'Curl_extract_certinfo'

x509asn1.c: Fixed compilation warnings

warning: declaration of 'chsize' shadows a global declaration

DOCS: Expanded --request description to include POP3 and IMAP details

tool_help: Updated --list-only description to include POP3

DOCS: Updated --list-only description to include POP3

Additionally corrected typos in --oauth2-bearer protocol list.

base64: Fixed compilation warnings when using Curl_base64_decode()

curl_sasl.c:294: warning: dereferencing type-punned pointer will break
strict-aliasing rules

getpart.c:201: warning: dereferencing type-punned pointer will break
strict-aliasing rules

connect: Fixed "Whut?" no server connection failures

Introduced in commit 7d7df831981fee curl would loop displaying "Whut?"
if it was trying to connect to an address and port that didn't have
anything listening on it.

http: Post base64 decoding tidy up

Renamed copy_header_value() to Curl_copy_header_value() as this
function is now non static.

Simplified proxy flag in Curl_http_input_auth() when calling
sub-functions.

Removed unnecessary white space removal when using negotiate as it had
been missed in commit cdccb422671aeb.

glob_range: pass the closing bracket for a-z ranges

Regression since commit 5ca96cb844102 (release in 7.33.0)

Reported-by: Marcin Gryszkalis

getpart: Fixed base64 encoded parts following commit e17c1b25bc33eb

http: Added proxy tunnel authentication message header value extraction

...following recent changes to Curl_base64_decode() rather than trying
to parse a header line for the authentication mechanisms which is CRLF
terminated and inline zero terminate it.

http: Added authentication message header value extraction

...following recent changes to Curl_base64_decode() rather than trying
to parse a header line for the authentication mechanisms which is CRLF
terminated and inline zero terminate it.

curl_multi_wait: accept 0 from multi_timeout() as valid timeout

The code rejected 0 as a valid timeout while in fact the function could
indeed legitimately return that and it should be respected.

Reported-by: Bjorn Stenberg

email: Corrected a couple of typos from commit aa0eaef4838ccd

TODO: Removed the 'Graceful base64 decoding failure' sections

Updated following the recent changes to support graceful failures
during the authentication phrase.