X-Git-Url: http://git.madism.org/?p=apps%2Fmadmutt.git;a=blobdiff_plain;f=pop%2Fpop.c;h=40df8781bab09e82406547b396fdbf3e09ca32fc;hp=572c79e4463a9e1891b7e6d92b515412d9bce6e7;hb=916e4872caf252a5850e64f79427b9dd7808435d;hpb=05a3bbbe420e4afc76e0eea24ce32f859405dc4a diff --git a/pop/pop.c b/pop/pop.c index 572c79e..40df878 100644 --- a/pop/pop.c +++ b/pop/pop.c @@ -7,23 +7,12 @@ * please see the file GPL in the top level source directory. */ -#if HAVE_CONFIG_H -# include "config.h" -#endif - -#include -#include - -#include -#include -#include -#include -#include +#include #include +#include #include "mutt.h" -#include "mx.h" #include "pop.h" #include @@ -50,22 +39,22 @@ static int fetch_message (char *line, void *file) static pop_query_status pop_read_header (POP_DATA * pop_data, HEADER * h) { FILE *f; - int index; + int idx; pop_query_status ret; long length; char buf[LONG_STRING]; char tempfile[_POSIX_PATH_MAX]; - mutt_mktemp (tempfile); - if (!(f = safe_fopen (tempfile, "w+"))) { - mutt_perror (tempfile); + f = m_tempfile(tempfile, sizeof(tempfile), NONULL(Tempdir), NULL); + if (!f) { + mutt_error(_("Could not create temporary file")); return PFD_FUNCT_ERROR; } - snprintf (buf, sizeof (buf), "LIST %d\r\n", h->refno); + snprintf (buf, sizeof (buf), "string_list_t %d\r\n", h->refno); ret = pop_query (pop_data, buf, sizeof (buf)); if (ret == PQ_OK) { - sscanf (buf, "+OK %d %ld", &index, &length); + sscanf (buf, "+OK %d %ld", &idx, &length); snprintf (buf, sizeof (buf), "TOP %d 0\r\n", h->refno); ret = pop_fetch_data (pop_data, buf, NULL, fetch_message, f); @@ -73,14 +62,10 @@ static pop_query_status pop_read_header (POP_DATA * pop_data, HEADER * h) if (pop_data->cmd_top == CMD_UNKNOWN) { if (ret == PQ_OK) { pop_data->cmd_top = CMD_AVAILABLE; - - debug_print (1, ("set TOP capability\n")); } if (ret == PQ_ERR) { pop_data->cmd_top = CMD_NOT_AVAILABLE; - - debug_print (1, ("unset TOP capability\n")); snprintf (pop_data->err_msg, sizeof (pop_data->err_msg), _("Command TOP is not supported by server.")); } @@ -118,7 +103,7 @@ static pop_query_status pop_read_header (POP_DATA * pop_data, HEADER * h) } } - fclose (f); + m_fclose(&f); unlink (tempfile); return ret; } @@ -126,18 +111,16 @@ static pop_query_status pop_read_header (POP_DATA * pop_data, HEADER * h) /* parse UIDL */ static int fetch_uidl (char *line, void *data) { - int i, index; + int i, idx; CONTEXT *ctx = (CONTEXT *) data; POP_DATA *pop_data = (POP_DATA *) ctx->data; - sscanf (line, "%d %s", &index, line); + sscanf (line, "%d %s", &idx, line); for (i = 0; i < ctx->msgcount; i++) if (!m_strcmp(line, ctx->hdrs[i]->data)) break; if (i == ctx->msgcount) { - debug_print (1, ("new header %d %s\n", index, line)); - if (i >= ctx->hdrmax) mx_alloc_memory (ctx); @@ -145,11 +128,11 @@ static int fetch_uidl (char *line, void *data) ctx->hdrs[i] = header_new(); ctx->hdrs[i]->data = m_strdup(line); } - else if (ctx->hdrs[i]->index != index - 1) + else if (ctx->hdrs[i]->index != idx - 1) pop_data->clear_cache = 1; - ctx->hdrs[i]->refno = index; - ctx->hdrs[i]->index = index - 1; + ctx->hdrs[i]->refno = idx; + ctx->hdrs[i]->index = idx - 1; return 0; } @@ -182,14 +165,11 @@ static int pop_fetch_headers (CONTEXT * ctx) if (pop_data->cmd_uidl == CMD_UNKNOWN) { if (ret == PQ_OK) { pop_data->cmd_uidl = CMD_AVAILABLE; - - debug_print (1, ("set UIDL capability\n")); } if (ret == PQ_ERR && pop_data->cmd_uidl == CMD_UNKNOWN) { pop_data->cmd_uidl = CMD_NOT_AVAILABLE; - debug_print (1, ("unset UIDL capability\n")); snprintf (pop_data->err_msg, sizeof (pop_data->err_msg), _("Command UIDL is not supported by server.")); } @@ -226,25 +206,25 @@ static int pop_fetch_headers (CONTEXT * ctx) } /* open POP mailbox - fetch only headers */ -int pop_open_mailbox (CONTEXT * ctx) +static int pop_open_mailbox (CONTEXT * ctx) { int ret; char buf[LONG_STRING]; CONNECTION *conn; - ACCOUNT acct; + ACCOUNT act; POP_DATA *pop_data; ciss_url_t url; - if (pop_parse_path (ctx->path, &acct)) { + if (pop_parse_path (ctx->path, &act)) { mutt_error (_("%s is an invalid POP path"), ctx->path); mutt_sleep (2); return -1; } - mutt_account_tourl (&acct, &url); + mutt_account_tourl (&act, &url); url.path = NULL; url_ciss_tostring (&url, buf, sizeof (buf), 0); - conn = mutt_conn_find (NULL, &acct); + conn = mutt_conn_find (NULL, &act); if (!conn) return -1; @@ -288,8 +268,6 @@ static void pop_clear_cache (POP_DATA * pop_data) if (!pop_data->clear_cache) return; - debug_print (1, ("delete cached messages\n")); - for (i = 0; i < POP_CACHE_LEN; i++) { if (pop_data->cache[i].path) { unlink (pop_data->cache[i].path); @@ -299,7 +277,7 @@ static void pop_clear_cache (POP_DATA * pop_data) } /* close POP mailbox */ -void pop_close_mailbox (CONTEXT * ctx) +static void pop_close_mailbox (CONTEXT * ctx) { POP_DATA *pop_data = (POP_DATA *) ctx->data; @@ -371,11 +349,10 @@ int pop_fetch_message (MESSAGE * msg, CONTEXT * ctx, int msgno) bar.msg = _("Fetching message..."); mutt_progress_bar (&bar, 0); - mutt_mktemp (path); - msg->fp = safe_fopen (path, "w+"); + msg->fp = m_tempfile(path, sizeof(path), NONULL(Tempdir), NULL); if (!msg->fp) { - mutt_perror (path); - mutt_sleep (2); + mutt_error(_("Could not create temporary file")); + mutt_sleep(2); return -1; } @@ -385,7 +362,7 @@ int pop_fetch_message (MESSAGE * msg, CONTEXT * ctx, int msgno) if (ret == PQ_OK) break; - safe_fclose (&msg->fp); + m_fclose(&msg->fp); unlink (path); if (ret == PQ_ERR) { @@ -431,7 +408,9 @@ int pop_fetch_message (MESSAGE * msg, CONTEXT * ctx, int msgno) } /* update POP mailbox - delete messages from server */ -pop_query_status pop_sync_mailbox (CONTEXT * ctx, int unused, int *index_hint) +static pop_query_status +pop_sync_mailbox (CONTEXT * ctx, int unused __attribute__ ((unused)), + int *index_hint __attribute__ ((unused))) { int i; pop_query_status ret; @@ -474,7 +453,9 @@ pop_query_status pop_sync_mailbox (CONTEXT * ctx, int unused, int *index_hint) } /* Check for new messages and fetch headers */ -int pop_check_mailbox (CONTEXT * ctx, int *index_hint, int unused) +static int pop_check_mailbox (CONTEXT * ctx, + int *index_hint __attribute__ ((unused)), + int unused __attribute__ ((unused))) { int ret; POP_DATA *pop_data = (POP_DATA *) ctx->data; @@ -516,7 +497,7 @@ void pop_fetch_mail (void) CONNECTION *conn; CONTEXT ctx; MESSAGE *msg = NULL; - ACCOUNT acct; + ACCOUNT act; POP_DATA *pop_data; if (!PopHost) { @@ -532,14 +513,14 @@ void pop_fetch_mail (void) } strcpy (p, PopHost); /* __STRCPY_CHECKED__ */ - ret = pop_parse_path (url, &acct); + ret = pop_parse_path (url, &act); p_delete(&url); if (ret) { mutt_error (_("%s is an invalid POP path"), PopHost); return; } - conn = mutt_conn_find (NULL, &acct); + conn = mutt_conn_find (NULL, &act); if (!conn) return; @@ -658,3 +639,36 @@ fail: mutt_socket_close (conn); p_delete(&pop_data); } + +static int pop_is_magic (const char* path, struct stat* st __attribute__ ((unused))) { + url_scheme_t s = url_check_scheme (NONULL (path)); + return ((s == U_POP || s == U_POPS) ? M_POP : -1); +} + +static int acl_check_pop (CONTEXT* ctx __attribute__ ((unused)), int bit) { + switch (bit) { + case ACL_INSERT: /* editing messages */ + case ACL_WRITE: /* change importance */ + return (0); + case ACL_DELETE: /* (un)deletion */ + case ACL_SEEN: /* mark as read */ + return (1); + default: + return (0); + } +} + +mx_t const pop_mx = { + M_POP, + 0, + pop_is_magic, + NULL, + NULL, + pop_open_mailbox, + NULL, + acl_check_pop, + pop_check_mailbox, + pop_close_mailbox, + pop_sync_mailbox, + NULL, +};