2005-01-07 [paul] 0.9.13cvs29.1
[claws.git] / src / pop.c
index 6054ce9beb13bc1b5e39d3c33816c84e32561698..98846460a750ff58a1bfc7cbbd8d06f3227f165c 100644 (file)
--- a/src/pop.c
+++ b/src/pop.c
@@ -36,7 +36,7 @@
 #include "prefs_account.h"
 #include "utils.h"
 #include "recv.h"
-
+#include "partial_download.h"
 #include "log.h"
 #include "hooks.h"
 
@@ -91,9 +91,6 @@ static gint pop3_session_recv_data_finished   (Session        *session,
                                                 guchar         *data,
                                                 guint           len);
 
-static gchar *pop3_get_filename_for_partial_mail(Pop3Session   *session, 
-                                                gchar          *muidl);
-
 static gint pop3_greeting_recv(Pop3Session *session, const gchar *msg)
 {
        session->state = POP3_GREETING;
@@ -150,14 +147,14 @@ static gint pop3_getauth_apop_send(Pop3Session *session)
        session->state = POP3_GETAUTH_APOP;
 
        if ((start = strchr(session->greeting, '<')) == NULL) {
-               log_warning(_("Required APOP timestamp not found "
+               log_error(_("Required APOP timestamp not found "
                            "in greeting\n"));
                session->error_val = PS_PROTOCOL;
                return -1;
        }
 
        if ((end = strchr(start, '>')) == NULL || end == start + 1) {
-               log_warning(_("Timestamp syntax error in greeting\n"));
+               log_error(_("Timestamp syntax error in greeting\n"));
                session->error_val = PS_PROTOCOL;
                return -1;
        }
@@ -183,7 +180,7 @@ static gint pop3_getrange_stat_send(Pop3Session *session)
 static gint pop3_getrange_stat_recv(Pop3Session *session, const gchar *msg)
 {
        if (sscanf(msg, "%d %d", &session->count, &session->total_bytes) != 2) {
-               log_warning(_("POP3 protocol error\n"));
+               log_error(_("POP3 protocol error\n"));
                session->error_val = PS_PROTOCOL;
                return -1;
        } else {
@@ -254,10 +251,11 @@ static gint pop3_getrange_uidl_recv(Pop3Session *session, const gchar *data,
                p = newline + 1;
                if (p < lastp && *p == '\n') p++;
 
-               if (sscanf(buf, "%d %" Xstr(IDLEN) "s", &num, id) != 2)
-                       return -1;
-               if (num <= 0 || num > session->count)
-                       return -1;
+               if (sscanf(buf, "%d %" Xstr(IDLEN) "s", &num, id) != 2 ||
+                   num <= 0 || num > session->count) {
+                       log_warning(_("invalid UIDL response: %s\n"), buf);
+                       continue;
+               }
 
                session->msg[num].uidl = g_strdup(id);
 
@@ -268,8 +266,10 @@ static gint pop3_getrange_uidl_recv(Pop3Session *session, const gchar *data,
                partial_recv = (gint)g_hash_table_lookup(
                                        session->partial_recv_table, id);
 
-               if (!session->ac_prefs->getall && recv_time != RECV_TIME_NONE) {
-                       session->msg[num].received = (partial_recv != 2);
+               if ((!session->ac_prefs->getall && recv_time != RECV_TIME_NONE)
+               || partial_recv != POP3_TOTALLY_RECEIVED) {
+                       session->msg[num].received = 
+                               (partial_recv != POP3_MUST_COMPLETE_RECV);
                        session->msg[num].partial_recv = partial_recv;
 
                }
@@ -339,13 +339,18 @@ static gint pop3_retr_recv(Pop3Session *session, const gchar *data, guint len)
        gint drop_ok;
        MailReceiveData mail_receive_data;
 
-       mail_receive_data.session = session;
-       mail_receive_data.data = g_strndup(data, len);
+       /* NOTE: we allocate a slightly larger buffer with a zero terminator
+        * because some plugins may think that it has a C string. */ 
+       mail_receive_data.session  = session;
+       mail_receive_data.data     = g_new0(gchar, len + 1);
+       mail_receive_data.data_len = len;
+       memcpy(mail_receive_data.data, data, len); 
+       
        hooks_invoke(MAIL_RECEIVE_HOOKLIST, &mail_receive_data);
 
        file = get_tmp_file();
-       if (pop3_write_msg_to_file(file, mail_receive_data.data,
-               strlen(mail_receive_data.data), NULL) < 0) {
+       if (pop3_write_msg_to_file(file, mail_receive_data.data, 
+                                  mail_receive_data.data_len, NULL) < 0) {
                g_free(file);
                g_free(mail_receive_data.data);
                session->error_val = PS_IOERR;
@@ -354,22 +359,21 @@ static gint pop3_retr_recv(Pop3Session *session, const gchar *data, guint len)
        g_free(mail_receive_data.data);
 
        if (session->msg[session->cur_msg].partial_recv 
-           == POP3_PARTIAL_DLOAD_DELE) {
-               gchar *old_file = pop3_get_filename_for_partial_mail(
-                               session, session->msg[session->cur_msg].uidl);
-               if (old_file != NULL) {
-                       move_file(file, old_file, TRUE);
-                       g_free(file);
-                       file = old_file;
+           == POP3_MUST_COMPLETE_RECV) {
+               gchar *old_file = partial_get_filename(
+                               session->ac_prefs->recv_server,
+                               session->ac_prefs->userid,
+                               session->msg[session->cur_msg].uidl);
+               
+               if (old_file) {
+                       partial_delete_old(old_file);
+                       g_free(old_file);
                }
-               /* drop_ok: 0: success 1: don't receive -1: error */
-               drop_ok = session->drop_message(
-                               session, file, old_file != NULL);
-       } else {
-               /* drop_ok: 0: success 1: don't receive -1: error */
-               drop_ok = session->drop_message(
-                               session, file, FALSE);
-       }
+       } 
+
+       /* drop_ok: 0: success 1: don't receive -1: error */
+       drop_ok = session->drop_message(session, file);
+
        g_free(file);
        if (drop_ok < 0) {
                session->error_val = PS_IOERR;
@@ -381,7 +385,7 @@ static gint pop3_retr_recv(Pop3Session *session, const gchar *data, guint len)
        session->cur_total_num++;
 
        session->msg[session->cur_msg].received = TRUE;
-       session->msg[session->cur_msg].partial_recv = FALSE;
+       session->msg[session->cur_msg].partial_recv = POP3_TOTALLY_RECEIVED;
 
        session->msg[session->cur_msg].recv_time =
                drop_ok == 1 ? RECV_TIME_KEEP : session->current_time;
@@ -404,11 +408,17 @@ static gint pop3_top_recv(Pop3Session *session, const gchar *data, guint len)
        MailReceiveData mail_receive_data;
        gchar *partial_notice = NULL;
        
-       mail_receive_data.session = session;
-       mail_receive_data.data = g_strndup(data, len);
+       /* NOTE: we allocate a slightly larger buffer with a zero terminator
+        * because some plugins may think that it has a C string. */ 
+       mail_receive_data.session  = session;
+       mail_receive_data.data     = g_new0(gchar, len + 1);
+       mail_receive_data.data_len = len;
+       memcpy(mail_receive_data.data, data, len);
+       
        hooks_invoke(MAIL_RECEIVE_HOOKLIST, &mail_receive_data);
 
-       partial_notice = g_strdup_printf("SC-Partially-Retrieved: %s\n"
+       partial_notice = g_strdup_printf("SC-Marked-For-Download: 0\n"
+                                        "SC-Partially-Retrieved: %s\n"
                                         "SC-Account-Server: %s\n"
                                         "SC-Account-Login: %s\n"
                                         "SC-Message-Size: %d",
@@ -418,7 +428,8 @@ static gint pop3_top_recv(Pop3Session *session, const gchar *data, guint len)
                                         session->msg[session->cur_msg].size);
        file = get_tmp_file();
        if (pop3_write_msg_to_file(file, mail_receive_data.data,
-               strlen(mail_receive_data.data), partial_notice) < 0) {
+                                  mail_receive_data.data_len,  
+                                  partial_notice) < 0) {
                g_free(file);
                g_free(mail_receive_data.data);
                session->error_val = PS_IOERR;
@@ -429,7 +440,7 @@ static gint pop3_top_recv(Pop3Session *session, const gchar *data, guint len)
        g_free(partial_notice);
 
        /* drop_ok: 0: success 1: don't receive -1: error */
-       drop_ok = session->drop_message(session, file, FALSE);
+       drop_ok = session->drop_message(session, file);
        g_free(file);
        if (drop_ok < 0) {
                session->error_val = PS_IOERR;
@@ -441,7 +452,7 @@ static gint pop3_top_recv(Pop3Session *session, const gchar *data, guint len)
        session->cur_total_num++;
 
        session->msg[session->cur_msg].received = TRUE;
-       session->msg[session->cur_msg].partial_recv = TRUE;
+       session->msg[session->cur_msg].partial_recv = POP3_PARTIALLY_RECEIVED;
        session->msg[session->cur_msg].recv_time =
                drop_ok == 1 ? RECV_TIME_KEEP : session->current_time;
 
@@ -477,7 +488,7 @@ static void pop3_gen_send(Pop3Session *session, const gchar *format, ...)
        g_vsnprintf(buf, sizeof(buf) - 2, format, args);
        va_end(args);
 
-       if (!strncasecmp(buf, "PASS ", 5))
+       if (!g_ascii_strncasecmp(buf, "PASS ", 5))
                log_print("POP3> PASS ********\n");
        else
                log_print("POP3> %s\n", buf);
@@ -581,23 +592,29 @@ void pop3_get_uidl_table(PrefsAccount *ac_prefs, Pop3Session *session)
                gchar tmp[POPBUFSIZE];
                strretchomp(buf);
                recv_time = RECV_TIME_NONE;
-               partial_recv = 0;
+               partial_recv = POP3_TOTALLY_RECEIVED;
                
-               if (sscanf(buf, "%s\t%ld\t%s", uidl, &recv_time, &tmp) < 2) {
-                       if (sscanf(buf, "%s", uidl) != 1)
-                               continue;
-                       else {
-                               recv_time = now;
+               if (sscanf(buf, "%s\t%ld\t%s", uidl, &recv_time, tmp) < 3) {
+                       if (sscanf(buf, "%s\t%ld", uidl, &recv_time) != 2) {
+                               if (sscanf(buf, "%s", uidl) != 1)
+                                       continue;
+                               else {
+                                       recv_time = now;
+                                       strcpy(tmp, "0");
+                               }
+                       } else {
+                               strcpy(tmp, "0");
                        }
                }
+
                if (recv_time == RECV_TIME_NONE)
                        recv_time = RECV_TIME_RECEIVED;
                g_hash_table_insert(table, g_strdup(uidl),
                                    GINT_TO_POINTER(recv_time));
                if (strlen(tmp) == 1)
-                       partial_recv = atoi(tmp);
+                       partial_recv = atoi(tmp); /* totally received ?*/
                else
-                       partial_recv = POP3_PARTIAL_DLOAD_DELE;
+                       partial_recv = POP3_MUST_COMPLETE_RECV;
 
                g_hash_table_insert(partial_recv_table, g_strdup(uidl),
                                    GINT_TO_POINTER(partial_recv));
@@ -610,254 +627,6 @@ void pop3_get_uidl_table(PrefsAccount *ac_prefs, Pop3Session *session)
        return;
 }
 
-int pop3_msg_in_uidl_list(const gchar *server, const gchar *login, 
-                         const gchar *muidl)
-{
-       gchar *path;
-       FILE *fp;
-       gchar buf[POPBUFSIZE];
-       gchar uidl[POPBUFSIZE];
-       time_t recv_time;
-       time_t now;
-       gint partial_recv;
-       
-       path = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S,
-                          "uidl", G_DIR_SEPARATOR_S, server,
-                          "-", login, NULL);
-       if ((fp = fopen(path, "rb")) == NULL) {
-               if (ENOENT != errno) FILE_OP_ERROR(path, "fopen");
-               g_free(path);
-               path = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S,
-                                  "uidl-", server,
-                                  "-", login, NULL);
-               if ((fp = fopen(path, "rb")) == NULL) {
-                       if (ENOENT != errno) FILE_OP_ERROR(path, "fopen");
-                       g_free(path);
-                       return FALSE;
-               }
-       }
-       g_free(path);
-
-       now = time(NULL);
-
-       while (fgets(buf, sizeof(buf), fp) != NULL) {
-               gchar tmp[POPBUFSIZE];
-               strretchomp(buf);
-               recv_time = RECV_TIME_NONE;
-               partial_recv = 0;
-               
-               if (sscanf(buf, "%s\t%ld\t%s", uidl, &recv_time, &tmp) < 2) {
-                       if (sscanf(buf, "%s", uidl) != 1)
-                               continue;
-                       else {
-                               recv_time = now;
-                       }
-               }
-               if (!strcmp(uidl, muidl)) {
-                       fclose(fp);
-                       return TRUE;
-               }
-       }
-
-       fclose(fp);     
-       return FALSE;
-}
-
-static gchar *pop3_get_filename_for_partial_mail(Pop3Session *session, 
-                                                gchar *muidl)
-{
-       gchar *path;
-       gchar *result = NULL;
-       FILE *fp;
-       gchar buf[POPBUFSIZE];
-       gchar uidl[POPBUFSIZE];
-       time_t recv_time;
-       time_t now;
-       gint partial_recv;
-       
-       path = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S,
-                          "uidl", G_DIR_SEPARATOR_S, 
-                          session->ac_prefs->recv_server,
-                          "-", session->ac_prefs->userid, NULL);
-       if ((fp = fopen(path, "rb")) == NULL) {
-               if (ENOENT != errno) FILE_OP_ERROR(path, "fopen");
-               g_free(path);
-               path = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S,
-                                  "uidl-", session->ac_prefs->recv_server,
-                                  "-", session->ac_prefs->userid, NULL);
-               if ((fp = fopen(path, "rb")) == NULL) {
-                       if (ENOENT != errno) FILE_OP_ERROR(path, "fopen");
-                       g_free(path);
-                       return result;
-               }
-       }
-       g_free(path);
-
-       now = time(NULL);
-
-       while (fgets(buf, sizeof(buf), fp) != NULL) {
-               gchar tmp[POPBUFSIZE];
-               strretchomp(buf);
-               recv_time = RECV_TIME_NONE;
-               partial_recv = 0;
-               
-               if (sscanf(buf, "%s\t%ld\t%s", uidl, &recv_time, &tmp) < 2) {
-                       if (sscanf(buf, "%s", uidl) != 1)
-                               continue;
-                       else {
-                               recv_time = now;
-                       }
-               }
-               if (!strcmp(muidl, uidl)) {
-                       result = strdup(tmp);
-                       break;
-               }
-       }
-
-       fclose(fp);
-       
-       return result;
-}
-
-static int pop3_uidl_mark_mail(const gchar *server, const gchar *login, 
-                         const gchar *muidl, const gchar *filename, 
-                         int download)
-{
-       gchar *path;
-       gchar *pathnew;
-       FILE *fp;
-       FILE *fpnew;
-       gchar buf[POPBUFSIZE];
-       gchar uidl[POPBUFSIZE];
-       time_t recv_time;
-       time_t now;
-       int len;
-       int start = TRUE;
-       gchar partial_recv[POPBUFSIZE];
-       
-       path = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S,
-                          "uidl", G_DIR_SEPARATOR_S, server,
-                          "-", login, NULL);
-       if ((fp = fopen(path, "rb")) == NULL) {
-               perror("fopen1");
-               if (ENOENT != errno) FILE_OP_ERROR(path, "fopen");
-               g_free(path);
-               path = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S,
-                                  "uidl-", server,
-                                  "-", login, NULL);
-               if ((fp = fopen(path, "rb")) == NULL) {
-                       if (ENOENT != errno) FILE_OP_ERROR(path, "fopen");
-                       g_free(path);
-               }
-               return -1;
-       }
-
-       pathnew = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S,
-                          "uidl", G_DIR_SEPARATOR_S, server,
-                          "-", login, ".new", NULL);
-       if ((fpnew = fopen(pathnew, "wb")) == NULL) {
-               perror("fopen2");
-               fclose(fp);
-               g_free(pathnew);
-               return -1;      
-       }
-       
-       now = time(NULL);
-
-       while (fgets(buf, sizeof(buf), fp) != NULL) {
-               strretchomp(buf);
-               recv_time = RECV_TIME_NONE;
-               sprintf(partial_recv,"0");
-               
-               if (sscanf(buf, "%s\t%ld\t%s", 
-                          uidl, &recv_time, &partial_recv) < 2) {
-                       if (sscanf(buf, "%s", uidl) != 1)
-                               continue;
-                       else {
-                               recv_time = now;
-                       }
-               }
-               if (strcmp(muidl, uidl)) {
-                       fprintf(fpnew, "%s\t%ld\t%s\n", 
-                               uidl, recv_time, partial_recv);
-               } else {
-                       gchar *stat = "0";
-                       if (download == POP3_PARTIAL_DLOAD_DLOAD)
-                               stat = filename;
-                       if (download == POP3_PARTIAL_DLOAD_UNKN)
-                               stat = "1";
-                       fprintf(fpnew, "%s\t%ld\t%s\n", 
-                               uidl, recv_time, stat);
-               }
-       }
-       fclose(fpnew);
-       fclose(fp);
-
-       move_file(pathnew, path, TRUE);
-
-       g_free(path);
-       g_free(pathnew);
-       
-       if ((fp = fopen(filename,"rb")) == NULL) {
-               perror("fopen3");
-               return -1;
-       }
-       pathnew = g_strdup_printf("%s.new", filename);
-       if ((fpnew = fopen(pathnew, "wb")) == NULL) {
-               perror("fopen4");
-               fclose(fp);
-               g_free(pathnew);
-               return -1;
-       }
-       
-       while ((len = fread(buf, sizeof(gchar), sizeof(buf), fp)) > 0) {
-               buf[len]='\0';
-               if (start) {
-                       start = FALSE;
-                       if (download != POP3_PARTIAL_DLOAD_UNKN)
-                               fprintf(fpnew, "SC-Marked-For-Download: %d\n", 
-                                       download);
-                       
-                       if(strlen(buf) > strlen("SC-Marked-For-Download: x\n")
-                       && !strncmp(buf, "SC-Marked-For-Download:", 
-                                   strlen("SC-Marked-For-Download:"))) {
-                               fprintf(fpnew, "%s", 
-                                buf+strlen("SC-Marked-For-Download: x\n"));
-                               continue;
-                       }
-               }
-               fprintf(fpnew, "%s", buf);
-       }
-       fclose(fpnew);
-       fclose(fp);
-       unlink(filename);
-       rename(pathnew, filename);
-       
-       g_free(pathnew);
-       return 0;
-}
-
-int pop3_mark_for_delete(const gchar *server, const gchar *login, 
-                        const gchar *muidl, const gchar *filename)
-{
-       return pop3_uidl_mark_mail(server, login, muidl, filename, 
-               POP3_PARTIAL_DLOAD_DELE);
-}
-
-int pop3_mark_for_download(const gchar *server, const gchar *login, 
-                         const gchar *muidl, const gchar *filename)
-{
-       return pop3_uidl_mark_mail(server, login, muidl, filename, 
-               POP3_PARTIAL_DLOAD_DLOAD);
-}
-
-int pop3_unmark(const gchar *server, const gchar *login, 
-               const gchar *muidl, const gchar *filename)
-{
-       return pop3_uidl_mark_mail(server, login, muidl, filename, 
-               POP3_PARTIAL_DLOAD_UNKN);
-}
-
 gint pop3_write_uidl_list(Pop3Session *session)
 {
        gchar *path;
@@ -879,10 +648,10 @@ gint pop3_write_uidl_list(Pop3Session *session)
 
        for (n = 1; n <= session->count; n++) {
                msg = &session->msg[n];
-               if (msg->uidl && msg->received && !msg->deleted) {
+               if (msg->uidl && msg->received &&
+                   (!msg->deleted || session->state != POP3_DONE))
                        fprintf(fp, "%s\t%ld\t%d\n", 
                                msg->uidl, msg->recv_time, msg->partial_recv);
-               }
        }
 
        if (fclose(fp) == EOF) FILE_OP_ERROR(path, "fclose");
@@ -908,7 +677,7 @@ static gint pop3_write_msg_to_file(const gchar *file, const gchar *data,
                FILE_OP_ERROR(file, "chmod");
 
        if (prefix != NULL) {
-               fprintf(fp, prefix);
+               fprintf(fp, "%s", prefix);
                fprintf(fp, "\n");
        }
        
@@ -916,7 +685,8 @@ static gint pop3_write_msg_to_file(const gchar *file, const gchar *data,
         * ^data              ^prev            ^cur             data+len-1^ */
 
        prev = data;
-       while ((cur = memchr(prev, '\r', len - (prev - data))) != NULL) {
+       while ((cur = (gchar *)my_memmem(prev, len - (prev - data), "\r\n", 2))
+              != NULL) {
                if ((cur > prev && fwrite(prev, cur - prev, 1, fp) < 1) ||
                    fputc('\n', fp) == EOF) {
                        FILE_OP_ERROR(file, "fwrite");
@@ -988,7 +758,7 @@ static Pop3State pop3_lookup_next(Pop3Session *session)
                if (ac->rmmail &&
                    msg->recv_time != RECV_TIME_NONE &&
                    msg->recv_time != RECV_TIME_KEEP &&
-                   msg->partial_recv == POP3_PARTIAL_DLOAD_UNKN &&
+                   msg->partial_recv == POP3_TOTALLY_RECEIVED &&
                    session->current_time - msg->recv_time >=
                    ac->msg_leave_time * 24 * 60 * 60) {
                        log_message
@@ -999,10 +769,11 @@ static Pop3State pop3_lookup_next(Pop3Session *session)
                }
 
                if (size_limit_over) {
-                       if (!msg->received && msg->partial_recv != 2) {
+                       if (!msg->received && msg->partial_recv != 
+                           POP3_MUST_COMPLETE_RECV) {
                                pop3_top_send(session, ac->size_limit);
                                return POP3_TOP;
-                       } else if (msg->partial_recv == POP3_PARTIAL_DLOAD_DELE)
+                       } else if (msg->partial_recv == POP3_MUST_COMPLETE_RECV)
                                break;
 
                        log_message
@@ -1038,23 +809,23 @@ static Pop3ErrorValue pop3_ok(Pop3Session *session, const gchar *msg)
                    strstr(msg + 4, "Lock") ||
                    strstr(msg + 4, "LOCK") ||
                    strstr(msg + 4, "wait")) {
-                       log_warning(_("mailbox is locked\n"));
+                       log_error(_("mailbox is locked\n"));
                        ok = PS_LOCKBUSY;
                } else if (strcasestr(msg + 4, "timeout")) {
-                       log_warning(_("session timeout\n"));
+                       log_error(_("Session timeout\n"));
                        ok = PS_ERROR;
                } else {
                        switch (session->state) {
 #if USE_OPENSSL
                        case POP3_STLS:
-                               log_warning(_("can't start TLS session\n"));
+                               log_error(_("can't start TLS session\n"));
                                ok = PS_ERROR;
                                break;
 #endif
                        case POP3_GETAUTH_USER:
                        case POP3_GETAUTH_PASS:
                        case POP3_GETAUTH_APOP:
-                               log_warning(_("error occurred on authentication\n"));
+                               log_error(_("error occurred on authentication\n"));
                                ok = PS_AUTHFAIL;
                                break;
                        case POP3_GETRANGE_LAST:
@@ -1065,7 +836,7 @@ static Pop3ErrorValue pop3_ok(Pop3Session *session, const gchar *msg)
                                break;
                                
                        default:
-                               log_warning(_("error occurred on POP3 session\n"));
+                               log_error(_("error occurred on POP3 session\n"));
                                ok = PS_ERROR;
                        }
                }
@@ -1114,7 +885,7 @@ static gint pop3_session_recv_msg(Session *session, const gchar *msg)
                        pop3_stls_send(pop3_session);
                else
 #endif
-               if (pop3_session->ac_prefs->protocol == A_APOP)
+               if (pop3_session->ac_prefs->use_apop_auth)
                        pop3_getauth_apop_send(pop3_session);
                else
                        pop3_getauth_user_send(pop3_session);
@@ -1123,7 +894,7 @@ static gint pop3_session_recv_msg(Session *session, const gchar *msg)
        case POP3_STLS:
                if (pop3_stls_recv(pop3_session) != PS_SUCCESS)
                        return -1;
-               if (pop3_session->ac_prefs->protocol == A_APOP)
+               if (pop3_session->ac_prefs->use_apop_auth)
                        pop3_getauth_apop_send(pop3_session);
                else
                        pop3_getauth_user_send(pop3_session);
@@ -1193,6 +964,7 @@ static gint pop3_session_recv_msg(Session *session, const gchar *msg)
                }
                break;
        case POP3_LOGOUT:
+               pop3_session->state = POP3_DONE;
                session_disconnect(session);
                break;
        case POP3_ERROR: