2007-03-30 [colin] 2.8.1cvs69
[claws.git] / src / pop.c
index fe24b0609c688ed15d7ce781f3518a76f2793a02..370e6b6a5b2bd8827b8545ba72c93ffd867b93b4 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-2006 Hiroyuki Yamamoto and the Claws Mail team
+ * Copyright (C) 1999-2007 Hiroyuki Yamamoto and the Claws Mail team
  *
  * 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
@@ -147,14 +147,14 @@ static gint pop3_getauth_apop_send(Pop3Session *session)
        session->state = POP3_GETAUTH_APOP;
 
        if ((start = strchr(session->greeting, '<')) == NULL) {
-               log_error(_("Required APOP timestamp not found "
+               log_error(LOG_PROTOCOL, _("Required APOP timestamp not found "
                            "in greeting\n"));
                session->error_val = PS_PROTOCOL;
                return -1;
        }
 
        if ((end = strchr(start, '>')) == NULL || end == start + 1) {
-               log_error(_("Timestamp syntax error in greeting\n"));
+               log_error(LOG_PROTOCOL, _("Timestamp syntax error in greeting\n"));
                session->error_val = PS_PROTOCOL;
                return -1;
        }
@@ -180,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_error(_("POP3 protocol error\n"));
+               log_error(LOG_PROTOCOL, _("POP3 protocol error\n"));
                session->error_val = PS_PROTOCOL;
                return -1;
        } else {
@@ -207,7 +207,7 @@ 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(LOG_PROTOCOL, _("POP3 protocol error\n"));
                session->error_val = PS_PROTOCOL;
                return -1;
        } else {
@@ -253,7 +253,7 @@ static gint pop3_getrange_uidl_recv(Pop3Session *session, const gchar *data,
 
                if (sscanf(buf, "%d %" Xstr(IDLEN) "s", &num, id) != 2 ||
                    num <= 0 || num > session->count) {
-                       log_warning(_("invalid UIDL response: %s\n"), buf);
+                       log_warning(LOG_PROTOCOL, _("invalid UIDL response: %s\n"), buf);
                        continue;
                }
 
@@ -490,9 +490,9 @@ static void pop3_gen_send(Pop3Session *session, const gchar *format, ...)
        va_end(args);
 
        if (!g_ascii_strncasecmp(buf, "PASS ", 5))
-               log_print("POP3> PASS ********\n");
+               log_print(LOG_PROTOCOL, "POP3> PASS ********\n");
        else
-               log_print("POP3> %s\n", buf);
+               log_print(LOG_PROTOCOL, "POP3> %s\n", buf);
 
        session_send_msg(SESSION(session), SESSION_MSG_NORMAL, buf);
 }
@@ -633,9 +633,16 @@ void pop3_get_uidl_table(PrefsAccount *ac_prefs, Pop3Session *session)
        return;
 }
 
+#define TRY(func) \
+if (!(func)) \
+{ \
+       g_warning("failed to write\n"); \
+       goto err_write;                 \
+} \
+
 gint pop3_write_uidl_list(Pop3Session *session)
 {
-       gchar *path;
+       gchar *path, *tmp_path;
        FILE *fp;
        Pop3MsgInfo *msg;
        gint n;
@@ -650,29 +657,52 @@ gint pop3_write_uidl_list(Pop3Session *session)
                           "uidl", G_DIR_SEPARATOR_S,
                           session->ac_prefs->recv_server,
                           "-", sanitized_uid, NULL);
-       
+       tmp_path = g_strconcat(path, ".tmp", NULL);
+
        g_free(sanitized_uid);
 
-       if ((fp = g_fopen(path, "wb")) == NULL) {
-               FILE_OP_ERROR(path, "fopen");
-               g_free(path);
-               return -1;
+       if ((fp = g_fopen(tmp_path, "wb")) == NULL) {
+               FILE_OP_ERROR(tmp_path, "fopen");
+               goto err_write;
        }
 
        for (n = 1; n <= session->count; n++) {
                msg = &session->msg[n];
                if (msg->uidl && msg->received &&
                    (!msg->deleted || session->state != POP3_DONE))
-                       fprintf(fp, "%s\t%ld\t%d\n", 
-                               msg->uidl, (long int) msg->recv_time, msg->partial_recv);
+                       TRY(fprintf(fp, "%s\t%ld\t%d\n", 
+                               msg->uidl, (long int) 
+                               msg->recv_time, 
+                               msg->partial_recv) 
+                           > 0);
        }
 
-       if (fclose(fp) == EOF) FILE_OP_ERROR(path, "fclose");
+       if (fclose(fp) == EOF) {
+               FILE_OP_ERROR(tmp_path, "fclose");
+               fp = NULL;
+               goto err_write;
+       }
+       fp = NULL;
+#ifdef G_OS_WIN32
+       g_unlink(path);
+#endif
+       if (g_rename(tmp_path, path) < 0) {
+               FILE_OP_ERROR(path, "rename");
+               goto err_write;
+       }
        g_free(path);
-
+       g_free(tmp_path);
        return 0;
+err_write:
+       if (fp)
+               fclose(fp);
+       g_free(path);
+       g_free(tmp_path);
+       return -1;
 }
 
+#undef TRY
+
 static gint pop3_write_msg_to_file(const gchar *file, const gchar *data,
                                   guint len, const gchar *prefix)
 {
@@ -774,9 +804,9 @@ static Pop3State pop3_lookup_next(Pop3Session *session)
                    msg->partial_recv == POP3_TOTALLY_RECEIVED &&
                    session->current_time - msg->recv_time >=
                    ac->msg_leave_time * 24 * 60 * 60) {
-                       log_message
-                               (_("POP3: Deleting expired message "
-                                  "%d\n"), session->cur_msg);
+                       log_message(LOG_PROTOCOL, 
+                                       _("POP3: Deleting expired message %d\n"),
+                                       session->cur_msg);
                        session->cur_total_bytes += size;
                        pop3_delete_send(session);
                        return POP3_DELETE;
@@ -790,9 +820,9 @@ static Pop3State pop3_lookup_next(Pop3Session *session)
                        } else if (msg->partial_recv == POP3_MUST_COMPLETE_RECV)
                                break;
 
-                       log_message
-                               (_("POP3: Skipping message %d (%d bytes)\n"),
-                                  session->cur_msg, size);
+                       log_message(LOG_PROTOCOL, 
+                                       _("POP3: Skipping message %d (%d bytes)\n"),
+                                       session->cur_msg, size);
                }
                
                if (size == 0 || msg->received || size_limit_over) {
@@ -814,7 +844,7 @@ static Pop3ErrorValue pop3_ok(Pop3Session *session, const gchar *msg)
 {
        Pop3ErrorValue ok;
 
-       log_print("POP3< %s\n", msg);
+       log_print(LOG_PROTOCOL, "POP3< %s\n", msg);
 
        if (!strncmp(msg, "+OK", 3))
                ok = PS_SUCCESS;
@@ -823,34 +853,34 @@ static Pop3ErrorValue pop3_ok(Pop3Session *session, const gchar *msg)
                    strstr(msg + 4, "Lock") ||
                    strstr(msg + 4, "LOCK") ||
                    strstr(msg + 4, "wait")) {
-                       log_error(_("mailbox is locked\n"));
+                       log_error(LOG_PROTOCOL, _("mailbox is locked\n"));
                        ok = PS_LOCKBUSY;
                } else if (strcasestr(msg + 4, "timeout")) {
-                       log_error(_("Session timeout\n"));
+                       log_error(LOG_PROTOCOL, _("Session timeout\n"));
                        ok = PS_ERROR;
                } else {
                        switch (session->state) {
 #if USE_OPENSSL
                        case POP3_STLS:
-                               log_error(_("couldn't start TLS session\n"));
+                               log_error(LOG_PROTOCOL, _("couldn't start TLS session\n"));
                                ok = PS_ERROR;
                                break;
 #endif
                        case POP3_GETAUTH_USER:
                        case POP3_GETAUTH_PASS:
                        case POP3_GETAUTH_APOP:
-                               log_error(_("error occurred on authentication\n"));
+                               log_error(LOG_PROTOCOL, _("error occurred on authentication\n"));
                                ok = PS_AUTHFAIL;
                                break;
                        case POP3_GETRANGE_LAST:
                        case POP3_GETRANGE_UIDL:
                        case POP3_TOP:
-                               log_warning(_("command not supported\n"));
+                               log_warning(LOG_PROTOCOL, _("command not supported\n"));
                                ok = PS_NOTSUPPORTED;
                                break;
                                
                        default:
-                               log_error(_("error occurred on POP3 session\n"));
+                               log_error(LOG_PROTOCOL, _("error occurred on POP3 session\n"));
                                ok = PS_ERROR;
                        }
                }
@@ -1044,7 +1074,7 @@ static gint pop3_session_recv_data_finished(Session *session, guchar *data,
                }
                break;
        case POP3_TOP:
-               log_warning(_("TOP command unsupported\n"));
+               log_warning(LOG_PROTOCOL, _("TOP command unsupported\n"));
                if (pop3_session->cur_msg == pop3_session->count)
                        pop3_logout_send(pop3_session);
                else {