don't translate strings in debug_print(), log_print(), log_message(), and log_warning()
[claws.git] / src / pop.c
index 09de1e109a64b00850026ea377316199076115b4..637421c4bea343dca6b9d4b8afc24b6cf9176d1b 100644 (file)
--- a/src/pop.c
+++ b/src/pop.c
@@ -1,6 +1,6 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2003 Hiroyuki Yamamoto
+ * Copyright (C) 1999-2004 Hiroyuki Yamamoto
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
 #include "md5.h"
 #include "prefs_account.h"
 #include "utils.h"
-#include "inc.h"
 #include "recv.h"
-/* disable sd
-#include "selective_download.h"
-*/
+
 #include "log.h"
+#include "hooks.h"
 
 static gint pop3_greeting_recv         (Pop3Session *session,
                                         const gchar *msg);
@@ -59,16 +57,16 @@ static gint pop3_getrange_last_recv (Pop3Session *session,
                                         const gchar *msg);
 static gint pop3_getrange_uidl_send    (Pop3Session *session);
 static gint pop3_getrange_uidl_recv    (Pop3Session *session,
-                                        const gchar *msg);
+                                        const gchar *data,
+                                        guint        len);
 static gint pop3_getsize_list_send     (Pop3Session *session);
 static gint pop3_getsize_list_recv     (Pop3Session *session,
-                                        const gchar *msg);
+                                        const gchar *data,
+                                        guint        len);
 static gint pop3_retr_send             (Pop3Session *session);
 static gint pop3_retr_recv             (Pop3Session *session,
                                         const gchar *data,
                                         guint        len);
-static gint pop3_retr_eom_recv         (Pop3Session *session,
-                                        const gchar *msg);
 static gint pop3_delete_send           (Pop3Session *session);
 static gint pop3_delete_recv           (Pop3Session *session);
 static gint pop3_logout_send           (Pop3Session *session);
@@ -83,7 +81,7 @@ static gint pop3_write_msg_to_file    (const gchar    *file,
                                         guint           len);
 
 static Pop3State pop3_lookup_next      (Pop3Session    *session);
-static gint pop3_ok                    (Pop3Session    *session,
+static Pop3ErrorValue pop3_ok          (Pop3Session    *session,
                                         const gchar    *msg);
 
 static gint pop3_session_recv_msg              (Session        *session,
@@ -91,11 +89,8 @@ static gint pop3_session_recv_msg            (Session        *session,
 static gint pop3_session_recv_data_finished    (Session        *session,
                                                 guchar         *data,
                                                 guint           len);
-/* disable sd
-static gboolean pop3_sd_get_next (Pop3State *state);
-static void pop3_sd_new_header(Pop3State *state);
-gboolean pop3_sd_state(Pop3State *state, gint cur_state, guint *next_state);
-*/
+
+
 static gint pop3_greeting_recv(Pop3Session *session, const gchar *msg)
 {
        session->state = POP3_GREETING;
@@ -152,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 "
-                             "in greeting\n"));
+               log_warning("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_warning("Timestamp syntax error in greeting\n");
                session->error_val = PS_PROTOCOL;
                return -1;
        }
@@ -185,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_warning("POP3 protocol error\n");
                session->error_val = PS_PROTOCOL;
                return -1;
        } else {
@@ -212,13 +207,14 @@ static gint pop3_getrange_last_recv(Pop3Session *session, const gchar *msg)
        gint last;
 
        if (sscanf(msg, "%d", &last) == 0) {
-               log_warning(_("POP3 protocol error\n"));
+               log_warning("POP3 protocol error\n");
                session->error_val = PS_PROTOCOL;
                return -1;
        } else {
-               if (session->count > last)
+               if (session->count > last) {
+                       session->new_msg_exist = TRUE;
                        session->cur_msg = last + 1;
-               else
+               else
                        session->cur_msg = 0;
        }
 
@@ -232,46 +228,51 @@ static gint pop3_getrange_uidl_send(Pop3Session *session)
        return PS_SUCCESS;
 }
 
-static gint pop3_getrange_uidl_recv(Pop3Session *session, const gchar *msg)
+static gint pop3_getrange_uidl_recv(Pop3Session *session, const gchar *data,
+                                   guint len)
 {
        gchar id[IDLEN + 1];
+       gchar buf[POPBUFSIZE];
+       gint buf_len;
        gint num;
        time_t recv_time;
-       gint next_state;
+       const gchar *p = data;
+       const gchar *lastp = data + len;
+       const gchar *newline;
 
-       if (msg[0] == '.') {
-               session->uidl_is_valid = TRUE;
-               return PS_SUCCESS;
-       }
+       while (p < lastp) {
+               if ((newline = memchr(p, '\r', lastp - p)) == NULL)
+                       return -1;
+               buf_len = MIN(newline - p, sizeof(buf) - 1);
+               memcpy(buf, p, buf_len);
+               buf[buf_len] = '\0';
 
-       if (sscanf(msg, "%d %" Xstr(IDLEN) "s", &num, id) != 2)
-               return -1;
-       if (num <= 0 || num > session->count)
-               return -1;
+               p = newline + 1;
+               if (p < lastp && *p == '\n') p++;
 
-       session->msg[num].uidl = g_strdup(id);
+               if (sscanf(buf, "%d %" Xstr(IDLEN) "s", &num, id) != 2)
+                       return -1;
+               if (num <= 0 || num > session->count)
+                       return -1;
 
-       if (!session->uidl_table)
-               return PS_CONTINUE;
+               session->msg[num].uidl = g_strdup(id);
 
-       recv_time = (time_t)g_hash_table_lookup(session->uidl_table, id);
-       session->msg[num].recv_time = recv_time;
+               recv_time = (time_t)g_hash_table_lookup(session->uidl_table, id);
+               session->msg[num].recv_time = recv_time;
 
-       if (!session->ac_prefs->getall && recv_time != RECV_TIME_NONE)
-               session->msg[num].received = TRUE;
+               if (!session->ac_prefs->getall && recv_time != RECV_TIME_NONE)
+                       session->msg[num].received = TRUE;
 
-       if (!session->new_msg_exist &&
-           (session->ac_prefs->getall || recv_time == RECV_TIME_NONE ||
-            session->ac_prefs->rmmail)) {
-               session->cur_msg = num;
-               session->new_msg_exist = TRUE;
+               if (!session->new_msg_exist &&
+                   (session->ac_prefs->getall || recv_time == RECV_TIME_NONE ||
+                    session->ac_prefs->rmmail)) {
+                       session->cur_msg = num;
+                       session->new_msg_exist = TRUE;
+               }
        }
 
-/* disable sd
-       if (pop3_sd_state(state, POP3_GETRANGE_UIDL_RECV, &next_state))
-               return next_state;
-*/
-       return PS_CONTINUE;
+       session->uidl_is_valid = TRUE;
+       return PS_SUCCESS;
 }
 
 static gint pop3_getsize_list_send(Pop3Session *session)
@@ -281,77 +282,40 @@ static gint pop3_getsize_list_send(Pop3Session *session)
        return PS_SUCCESS;
 }
 
-static gint pop3_getsize_list_recv(Pop3Session *session, const gchar *msg)
+static gint pop3_getsize_list_recv(Pop3Session *session, const gchar *data,
+                                  guint len)
 {
+       gchar buf[POPBUFSIZE];
+       gint buf_len;
        guint num, size;
-       gint next_state;
-
-       if (msg[0] == '.')
-               return PS_SUCCESS;
-
-       if (sscanf(msg, "%u %u", &num, &size) != 2) {
-               session->error_val = PS_PROTOCOL;
-               return -1;
-       }
+       const gchar *p = data;
+       const gchar *lastp = data + len;
+       const gchar *newline;
 
-       if (num > 0 && num <= session->count)
-               session->msg[num].size = size;
-       if (num > 0 && num < session->cur_msg)
-               session->cur_total_bytes += size;
+       while (p < lastp) {
+               if ((newline = memchr(p, '\r', lastp - p)) == NULL)
+                       return -1;
+               buf_len = MIN(newline - p, sizeof(buf) - 1);
+               memcpy(buf, p, buf_len);
+               buf[buf_len] = '\0';
 
-/* disable sd
-       if (pop3_sd_state(state, POP3_GETSIZE_LIST_RECV, &next_state))
-               return next_state;
-*/
-       return PS_CONTINUE;
-}
-/* disable sd 
-gint pop3_top_send(SockInfo *sock, gpointer data)
-{
-       Pop3State *state = (Pop3State *)data;
+               p = newline + 1;
+               if (p < lastp && *p == '\n') p++;
 
-       inc_progress_update(state, POP3_TOP_SEND); 
+               if (sscanf(buf, "%u %u", &num, &size) != 2) {
+                       session->error_val = PS_PROTOCOL;
+                       return -1;
+               }
 
-       pop3_gen_send(sock, "TOP %i 0", state->cur_msg );
+               if (num > 0 && num <= session->count)
+                       session->msg[num].size = size;
+               if (num > 0 && num < session->cur_msg)
+                       session->cur_total_bytes += size;
+       }
 
-       return POP3_TOP_RECV;
+       return PS_SUCCESS;
 }
 
-gint pop3_top_recv(SockInfo *sock, gpointer data)
-{
-       Pop3State *state = (Pop3State *)data;
-       gchar *filename, *path;
-       gint next_state;
-       gint write_val;
-       if (pop3_ok(sock, NULL) != PS_SUCCESS) 
-               return POP3_LOGOUT_SEND;
-
-       path = g_strconcat(get_header_cache_dir(), G_DIR_SEPARATOR_S, NULL);
-
-       if ( !is_dir_exist(path) )
-               make_dir_hier(path);
-       
-       filename = g_strdup_printf("%s%i", path, state->cur_msg);
-                                  
-       if ( (write_val = recv_write_to_file(sock, filename)) < 0) {
-               state->error_val = (write_val == -1 ? PS_IOERR : PS_SOCKET);
-               g_free(path);
-               g_free(filename);
-               return -1;
-       }
-       
-       g_free(path);
-       g_free(filename);
-       
-       pop3_sd_state(state, POP3_TOP_RECV, &next_state);
-       
-       if (state->cur_msg < state->count) {
-               state->cur_msg++;
-               return POP3_TOP_SEND;
-       } else
-               return POP3_LOGOUT_SEND;
-}
-*/
 static gint pop3_retr_send(Pop3Session *session)
 {
        session->state = POP3_RETR;
@@ -363,26 +327,30 @@ static gint pop3_retr_recv(Pop3Session *session, const gchar *data, guint len)
 {
        gchar *file;
        gint drop_ok;
-       gint next_state;
+       MailReceiveData mail_receive_data;
+
+       mail_receive_data.session = session;
+       mail_receive_data.data = g_strndup(data, len);
+       hooks_invoke(MAIL_RECEIVE_HOOKLIST, &mail_receive_data);
 
        file = get_tmp_file();
-       if (pop3_write_msg_to_file(file, data, len) < 0) {
+       if (pop3_write_msg_to_file(file, mail_receive_data.data,
+               strlen(mail_receive_data.data)) < 0) {
                g_free(file);
+               g_free(mail_receive_data.data);
                session->error_val = PS_IOERR;
                return -1;
        }
+       g_free(mail_receive_data.data);
 
        /* drop_ok: 0: success 1: don't receive -1: error */
-       drop_ok = inc_drop_message(file, session);
+       drop_ok = session->drop_message(session, file);
        g_free(file);
        if (drop_ok < 0) {
-               session->error_val = PS_ERROR;
+               session->error_val = PS_IOERR;
                return -1;
        }
-/* disable sd
-       if (pop3_sd_state(state, POP3_RETR_RECV, &next_state))
-               return next_state;
-*/     
+
        session->cur_total_bytes += session->msg[session->cur_msg].size;
        session->cur_total_recv_bytes += session->msg[session->cur_msg].size;
        session->cur_total_num++;
@@ -394,19 +362,6 @@ static gint pop3_retr_recv(Pop3Session *session, const gchar *data, guint len)
        return PS_SUCCESS;
 }
 
-static gint pop3_retr_eom_recv(Pop3Session *session, const gchar *msg)
-{
-       if (msg[0] == '.' && msg[1] == '\0')
-               return PS_SUCCESS;
-       else {
-               g_warning("pop3_retr_eom_recv(): "
-                         "invalid end of message: '%s'.\n"
-                         "Maybe given size and actual one is different?\n",
-                         msg);
-               return PS_PROTOCOL;
-       }
-}
-
 static gint pop3_delete_send(Pop3Session *session)
 {
        session->state = POP3_DELETE;
@@ -443,156 +398,7 @@ static void pop3_gen_send(Pop3Session *session, const gchar *format, ...)
 
        session_send_msg(SESSION(session), SESSION_MSG_NORMAL, buf);
 }
-/* disable sd
-static void pop3_sd_new_header(Pop3State *state)
-{
-       HeaderItems *new_msg;
-       if (state->cur_msg <= state->count) {
-               new_msg = g_new0(HeaderItems, 1); 
-               
-               new_msg->index              = state->cur_msg;
-               new_msg->state              = SD_UNCHECKED;
-               new_msg->size               = state->msg[state->cur_msg].size; 
-               new_msg->received           = state->msg[state->cur_msg].received;
-               new_msg->del_by_old_session = FALSE;
-               
-               state->ac_prefs->msg_list = g_slist_append(state->ac_prefs->msg_list, 
-                                                          new_msg);
-               debug_print("received ?: msg %i, received: %i\n",new_msg->index, new_msg->received); 
-       }
-}
-
-gboolean pop3_sd_state(Pop3State *state, gint cur_state, guint *next_state) 
-{
-       gint session = state->ac_prefs->session;
-       guint goto_state = -1;
 
-       switch (cur_state) { 
-       case POP3_GETRANGE_UIDL_RECV:
-               switch (session) {
-               case STYPE_POP_BEFORE_SMTP:
-                       goto_state = POP3_LOGOUT_SEND;
-                       break;
-               case STYPE_DOWNLOAD:
-               case STYPE_DELETE:
-               case STYPE_PREVIEW_ALL:
-                       goto_state = POP3_GETSIZE_LIST_SEND;
-               default:
-                       break;
-               }
-               break;
-       case POP3_GETSIZE_LIST_RECV:
-               switch (session) {
-               case STYPE_PREVIEW_ALL:
-                       state->cur_msg = 1;
-               case STYPE_PREVIEW_NEW:
-                       goto_state = POP3_TOP_SEND;
-                       break;
-               case STYPE_DELETE:
-                       if (pop3_sd_get_next(state))
-                               goto_state = POP3_DELETE_SEND;          
-                       else
-                               goto_state = POP3_LOGOUT_SEND;
-                       break;
-               case STYPE_DOWNLOAD:
-                       if (pop3_sd_get_next(state))
-                               goto_state = POP3_RETR_SEND;
-                       else
-                               goto_state = POP3_LOGOUT_SEND;
-               default:
-                       break;
-               }
-               break;
-       case POP3_TOP_RECV: 
-               switch (session) { 
-               case STYPE_PREVIEW_ALL:
-               case STYPE_PREVIEW_NEW:
-                       pop3_sd_new_header(state);
-               default:
-                       break;
-               }
-               break;
-       case POP3_RETR_RECV:
-               switch (session) {
-               case STYPE_DOWNLOAD:
-                       if (state->ac_prefs->sd_rmmail_on_download) 
-                               goto_state = POP3_DELETE_SEND;
-                       else {
-                               if (pop3_sd_get_next(state)) 
-                                       goto_state = POP3_RETR_SEND;
-                               else
-                                       goto_state = POP3_LOGOUT_SEND;
-                       }
-               default:        
-                       break;
-               }
-               break;
-       case POP3_DELETE_RECV:
-               switch (session) {
-               case STYPE_DELETE:
-                       if (pop3_sd_get_next(state)) 
-                               goto_state = POP3_DELETE_SEND;
-                       else
-                               goto_state =  POP3_LOGOUT_SEND;
-                       break;
-               case STYPE_DOWNLOAD:
-                       if (pop3_sd_get_next(state)) 
-                               goto_state = POP3_RETR_SEND;
-                       else
-                               goto_state = POP3_LOGOUT_SEND;
-               default:
-                       break;
-               }
-       default:
-               break;
-               
-       }                 
-
-       *next_state = goto_state;
-       if (goto_state != -1)
-               return TRUE;
-       else 
-               return FALSE;
-}
-
-gboolean pop3_sd_get_next(Pop3State *state)
-{
-       GSList *cur;
-       gint deleted_msgs = 0;
-       
-       switch (state->ac_prefs->session) {
-       case STYPE_DOWNLOAD:
-       case STYPE_DELETE:      
-               for (cur = state->ac_prefs->msg_list; cur != NULL; cur = cur->next) {
-                       HeaderItems *items = (HeaderItems*)cur->data;
-
-                       if (items->del_by_old_session)
-                               deleted_msgs++;
-
-                       switch (items->state) {
-                       case SD_REMOVE:
-                               items->state = SD_REMOVED;
-                               break;
-                       case SD_DOWNLOAD:
-                               items->state = SD_DOWNLOADED;
-                               break;
-                       case SD_CHECKED:
-                               state->cur_msg = items->index - deleted_msgs;
-                               if (state->ac_prefs->session == STYPE_DELETE)
-                                       items->state = SD_REMOVE;
-                               else
-                                       items->state = SD_DOWNLOAD;
-                               return TRUE;
-                       default:
-                               break;
-                       }
-               }
-               return FALSE;
-       default:
-               return FALSE;
-       }
-}
-*/
 Session *pop3_session_new(PrefsAccount *account)
 {
        Pop3Session *session;
@@ -601,12 +407,9 @@ Session *pop3_session_new(PrefsAccount *account)
 
        session = g_new0(Pop3Session, 1);
 
+       session_init(SESSION(session));
+
        SESSION(session)->type = SESSION_POP3;
-       SESSION(session)->server = NULL;
-       SESSION(session)->port = 0;
-       SESSION(session)->sock = NULL;
-       SESSION(session)->state = SESSION_READY;
-       SESSION(session)->data = NULL;
 
        SESSION(session)->recv_msg = pop3_session_recv_msg;
        SESSION(session)->recv_data_finished = pop3_session_recv_data_finished;
@@ -619,6 +422,7 @@ Session *pop3_session_new(PrefsAccount *account)
        session->uidl_table = pop3_get_uidl_table(account);
        session->current_time = time(NULL);
        session->error_val = PS_SUCCESS;
+       session->error_msg = NULL;
 
        return SESSION(session);
 }
@@ -642,6 +446,7 @@ static void pop3_session_destroy(Session *session)
        g_free(pop3_session->greeting);
        g_free(pop3_session->user);
        g_free(pop3_session->pass);
+       g_free(pop3_session->error_msg);
 }
 
 GHashTable *pop3_get_uidl_table(PrefsAccount *ac_prefs)
@@ -654,6 +459,8 @@ GHashTable *pop3_get_uidl_table(PrefsAccount *ac_prefs)
        time_t recv_time;
        time_t now;
 
+       table = g_hash_table_new(g_str_hash, g_str_equal);
+
        path = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S,
                           "uidl", G_DIR_SEPARATOR_S, ac_prefs->recv_server,
                           "-", ac_prefs->userid, NULL);
@@ -666,13 +473,11 @@ GHashTable *pop3_get_uidl_table(PrefsAccount *ac_prefs)
                if ((fp = fopen(path, "rb")) == NULL) {
                        if (ENOENT != errno) FILE_OP_ERROR(path, "fopen");
                        g_free(path);
-                       return NULL;
+                       return table;
                }
        }
        g_free(path);
 
-       table = g_hash_table_new(g_str_hash, g_str_equal);
-
        now = time(NULL);
 
        while (fgets(buf, sizeof(buf), fp) != NULL) {
@@ -765,6 +570,9 @@ static gint pop3_write_msg_to_file(const gchar *file, const gchar *data,
                else
                        prev = cur + 1;
 
+               if (prev - data < len - 1 && *prev == '.' && *(prev + 1) == '.')
+                       prev++;
+
                if (prev - data >= len)
                        break;
        }
@@ -816,7 +624,7 @@ static Pop3State pop3_lookup_next(Pop3Session *session)
                    msg->recv_time != RECV_TIME_KEEP &&
                    session->current_time - msg->recv_time >=
                    ac->msg_leave_time * 24 * 60 * 60) {
-                       log_print(_("POP3: Deleting expired message %d\n"),
+                       log_print("POP3: Deleting expired message %d\n",
                                  session->cur_msg);
                        pop3_delete_send(session);
                        return POP3_DELETE;
@@ -824,7 +632,7 @@ static Pop3State pop3_lookup_next(Pop3Session *session)
 
                if (size_limit_over)
                        log_print
-                               (_("POP3: Skipping message %d (%d bytes)\n"),
+                               ("POP3: Skipping message %d (%d bytes)\n",
                                  session->cur_msg, size);
 
                if (size == 0 || msg->received || size_limit_over) {
@@ -842,9 +650,11 @@ static Pop3State pop3_lookup_next(Pop3Session *session)
        return POP3_RETR;
 }
 
-static gint pop3_ok(Pop3Session *session, const gchar *msg)
+static Pop3ErrorValue pop3_ok(Pop3Session *session, const gchar *msg)
 {
-       gint ok;
+       Pop3ErrorValue ok;
+
+       log_print("POP3< %s\n", msg);
 
        if (!strncmp(msg, "+OK", 3))
                ok = PS_SUCCESS;
@@ -852,34 +662,62 @@ static gint pop3_ok(Pop3Session *session, const gchar *msg)
                if (strstr(msg + 4, "lock") ||
                    strstr(msg + 4, "Lock") ||
                    strstr(msg + 4, "LOCK") ||
-                   strstr(msg + 4, "wait"))
+                   strstr(msg + 4, "wait")) {
+                       log_warning("mailbox is locked\n");
                        ok = PS_LOCKBUSY;
-               else
-                       ok = PS_PROTOCOL;
+               } else if (strcasestr(msg + 4, "timeout")) {
+                       log_warning("session timeout\n");
+                       ok = PS_ERROR;
+               } else {
+                       switch (session->state) {
+#if USE_OPENSSL
+                       case POP3_STLS:
+                               log_warning("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");
+                               ok = PS_AUTHFAIL;
+                               break;
+                       case POP3_GETRANGE_LAST:
+                       case POP3_GETRANGE_UIDL:
+                               log_warning("command not supported\n");
+                               ok = PS_NOTSUPPORTED;
+                               break;
+                       default:
+                               log_warning("error occurred on POP3 session\n");
+                               ok = PS_ERROR;
+                       }
+               }
 
+               g_free(session->error_msg);
+               session->error_msg = g_strdup(msg);
                fprintf(stderr, "POP3: %s\n", msg);
        } else
                ok = PS_PROTOCOL;
 
+       session->error_val = ok;
        return ok;
 }
 
 static gint pop3_session_recv_msg(Session *session, const gchar *msg)
 {
        Pop3Session *pop3_session = POP3_SESSION(session);
-       gint val;
-       const gchar *body;
+       Pop3ErrorValue val = PS_SUCCESS;
+       const guchar *body;
 
        body = msg;
        if (pop3_session->state != POP3_GETRANGE_UIDL_RECV &&
-           pop3_session->state != POP3_GETSIZE_LIST_RECV &&
-           pop3_session->state != POP3_RETR_EOM) {
-               log_print("POP3< %s\n", msg);
+           pop3_session->state != POP3_GETSIZE_LIST_RECV) {
                val = pop3_ok(pop3_session, msg);
                if (val != PS_SUCCESS) {
-                       pop3_session->state = POP3_ERROR;
-                       pop3_session->error_val = val;
-                       return -1;
+                       if (val != PS_NOTSUPPORTED) {
+                               pop3_session->state = POP3_ERROR;
+                               return -1;
+                       }
                }
 
                if (*body == '+' || *body == '-')
@@ -930,7 +768,9 @@ static gint pop3_session_recv_msg(Session *session, const gchar *msg)
                        pop3_logout_send(pop3_session);
                break;
        case POP3_GETRANGE_LAST:
-               if (pop3_getrange_last_recv(pop3_session, body) < 0)
+               if (val == PS_NOTSUPPORTED)
+                       pop3_session->error_val = PS_SUCCESS;
+               else if (pop3_getrange_last_recv(pop3_session, body) < 0)
                        return -1;
                if (pop3_session->cur_msg > 0)
                        pop3_getsize_list_send(pop3_session);
@@ -938,52 +778,21 @@ static gint pop3_session_recv_msg(Session *session, const gchar *msg)
                        pop3_logout_send(pop3_session);
                break;
        case POP3_GETRANGE_UIDL:
-               pop3_session->state = POP3_GETRANGE_UIDL_RECV;
-               return 1;
-       case POP3_GETRANGE_UIDL_RECV:
-               val = pop3_getrange_uidl_recv(pop3_session, body);
-               if (val == PS_CONTINUE)
-                       return 1;
-               else if (val == PS_SUCCESS) {
-                       if (pop3_session->new_msg_exist)
-                               pop3_getsize_list_send(pop3_session);
-                       else
-                               pop3_logout_send(pop3_session);
-               } else
-                       return -1;
+               if (val == PS_NOTSUPPORTED) {
+                       pop3_session->error_val = PS_SUCCESS;
+                       pop3_getrange_last_send(pop3_session);
+               } else {
+                       pop3_session->state = POP3_GETRANGE_UIDL_RECV;
+                       session_recv_data(session, 0, ".\r\n");
+               }
                break;
        case POP3_GETSIZE_LIST:
                pop3_session->state = POP3_GETSIZE_LIST_RECV;
-               return 1;
-       case POP3_GETSIZE_LIST_RECV:
-               val = pop3_getsize_list_recv(pop3_session, body);
-               if (val == PS_CONTINUE)
-                       return 1;
-               else if (val == PS_SUCCESS) {
-                       if (pop3_lookup_next(pop3_session) == POP3_ERROR)
-                               return -1;
-               } else
-                       return -1;
+               session_recv_data(session, 0, ".\r\n");
                break;
        case POP3_RETR:
                pop3_session->state = POP3_RETR_RECV;
-               session_recv_data
-                       (session,
-                        pop3_session->msg[pop3_session->cur_msg].size, TRUE);
-               break;
-       case POP3_RETR_EOM:
-               if (pop3_retr_eom_recv(pop3_session, body) != PS_SUCCESS)
-                       return -1;
-               if (pop3_session->ac_prefs->rmmail &&
-                   pop3_session->ac_prefs->msg_leave_time == 0)
-                       pop3_delete_send(pop3_session);
-               else if (pop3_session->cur_msg == pop3_session->count)
-                       pop3_logout_send(pop3_session);
-               else {
-                       pop3_session->cur_msg++;
-                       if (pop3_lookup_next(pop3_session) == POP3_ERROR)
-                               return -1;
-               }
+               session_recv_data(session, 0, ".\r\n");
                break;
        case POP3_DELETE:
                pop3_delete_recv(pop3_session);
@@ -1010,14 +819,48 @@ static gint pop3_session_recv_data_finished(Session *session, guchar *data,
                                            guint len)
 {
        Pop3Session *pop3_session = POP3_SESSION(session);
+       Pop3ErrorValue val = PS_SUCCESS;
 
-       if (len == 0)
-               return -1;
+       switch (pop3_session->state) {
+       case POP3_GETRANGE_UIDL_RECV:
+               val = pop3_getrange_uidl_recv(pop3_session, data, len);
+               if (val == PS_SUCCESS) {
+                       if (pop3_session->new_msg_exist)
+                               pop3_getsize_list_send(pop3_session);
+                       else
+                               pop3_logout_send(pop3_session);
+               } else
+                       return -1;
+               break;
+       case POP3_GETSIZE_LIST_RECV:
+               val = pop3_getsize_list_recv(pop3_session, data, len);
+               if (val == PS_SUCCESS) {
+                       if (pop3_lookup_next(pop3_session) == POP3_ERROR)
+                               return -1;
+               } else
+                       return -1;
+               break;
+       case POP3_RETR_RECV:
+               if (pop3_retr_recv(pop3_session, data, len) < 0)
+                       return -1;
 
-       if (pop3_retr_recv(pop3_session, data, len) < 0)
+               if (pop3_session->ac_prefs->rmmail &&
+                   pop3_session->ac_prefs->msg_leave_time == 0 &&
+                   pop3_session->msg[pop3_session->cur_msg].recv_time
+                   != RECV_TIME_KEEP)
+                       pop3_delete_send(pop3_session);
+               else if (pop3_session->cur_msg == pop3_session->count)
+                       pop3_logout_send(pop3_session);
+               else {
+                       pop3_session->cur_msg++;
+                       if (pop3_lookup_next(pop3_session) == POP3_ERROR)
+                               return -1;
+               }
+               break;
+       case POP3_ERROR:
+       default:
                return -1;
+       }
 
-       pop3_session->state = POP3_RETR_EOM;
-
-       return 1;
+       return 0;
 }