imap: Changed successful response logic in imap_state_append_resp()
authorSteve Holme <steve_holme@hotmail.com>
Sat, 2 Mar 2013 20:35:03 +0000 (20:35 +0000)
committerSteve Holme <steve_holme@hotmail.com>
Sat, 2 Mar 2013 20:35:03 +0000 (20:35 +0000)
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

index c34a057..2632961 100644 (file)
@@ -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 */