From 69ba0da8272d7bddeef518670d4c1298b0627ad7 Mon Sep 17 00:00:00 2001 From: Steve Holme Date: Sat, 2 Jun 2012 22:11:37 +0100 Subject: pop3: Fixed the issue of having to supply the user name for all requests Previously it wasn't possible to connect to POP3 and not specify the user name as a CURLE_ACCESS_DENIED error would be returned. This error occurred because USER would be sent to the server with a blank user name if no mailbox user was specified as the server would reply with -ERR. This wasn't a problem prior to the 7.26.0 release but with the introduction of custom commands the user and/or application developer might want to issue a CAPA command without having to log in as a specific mailbox user. Additionally this fix won't send the newly introduced AUTH command if no user name is specified. --- lib/pop3.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/pop3.c b/lib/pop3.c index 579e77bb5..64efb255f 100644 --- a/lib/pop3.c +++ b/lib/pop3.c @@ -305,12 +305,20 @@ static void state(struct connectdata *conn, pop3state newstate) static CURLcode pop3_state_auth(struct connectdata *conn) { - CURLcode result; + CURLcode result = CURLE_OK; struct pop3_conn *pop3c = &conn->proto.pop3c; pop3c->authmechs = 0; /* No known authentication mechanisms yet */ pop3c->authused = 0; /* Clear the authentication mechanism used */ + /* Check we have a username and password to authenticate with and end the + connect phase if we don't */ + if(!conn->bits.user_passwd) { + state(conn, SMTP_STOP); + + return result; + } + /* send AUTH */ result = Curl_pp_sendf(&pop3c->pp, "AUTH"); -- cgit v1.2.3