From ec38ac38c7b10181b70cc1ed582136d8b61ac83d Mon Sep 17 00:00:00 2001 From: Steve Holme Date: Sat, 2 Mar 2013 20:35:03 +0000 Subject: [PATCH] imap: Changed successful response logic in imap_state_append_resp() For consistency changed the logic of the imap_state_append_resp() function to test for an unsucessful continuation response rather than a succesful one. --- lib/imap.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/lib/imap.c b/lib/imap.c index c34a0578d..263296142 100644 --- a/lib/imap.c +++ b/lib/imap.c @@ -1395,7 +1395,11 @@ static CURLcode imap_state_append_resp(struct connectdata *conn, (void)instate; /* No use for this yet */ - if(imapcode == '+') { + if(imapcode != '+') { + state(conn, IMAP_STOP); + return CURLE_UPLOAD_FAILED; + } + else { Curl_pgrsSetUploadSize(data, data->set.infilesize); Curl_setup_transfer(conn, -1, -1, FALSE, NULL, /* No download */ FIRSTSOCKET, NULL); @@ -1404,10 +1408,6 @@ static CURLcode imap_state_append_resp(struct connectdata *conn, state(conn, IMAP_STOP); return CURLE_OK; } - else { - state(conn, IMAP_STOP); - return CURLE_UPLOAD_FAILED; - } } /* For final APPEND responses performed after the upload */