X-Git-Url: http://git.madism.org/?a=blobdiff_plain;f=imap%2Fimap.c;h=06e14edbc6fa15be2f79fda4d5757b68eb4041fc;hb=618ceafdc9564dbb8f3bf45c3869297a1d5a3320;hp=6b372300bdd11885687760ab758ab09f80d27949;hpb=c36aba5f86a5001a110d21ba9766426f4806b17e;p=apps%2Fmadmutt.git diff --git a/imap/imap.c b/imap/imap.c index 6b37230..06e14ed 100644 --- a/imap/imap.c +++ b/imap/imap.c @@ -16,6 +16,8 @@ #endif #include "mutt.h" +#include "ascii.h" +#include "buffer.h" #include "mutt_curses.h" #include "mx.h" #include "globals.h" @@ -26,6 +28,7 @@ #if defined(USE_SSL) || defined(USE_GNUTLS) # include "mutt_ssl.h" #endif +#include "buffy.h" #include "lib/mem.h" #include "lib/intl.h" @@ -64,7 +67,7 @@ int imap_access (const char *path, int flags) if (!(idata = imap_conn_find (&mx.account, option (OPTIMAPPASSIVE) ? M_IMAP_CONN_NONEW : 0))) { - FREE (&mx.mbox); + mem_free (&mx.mbox); return -1; } @@ -72,11 +75,11 @@ int imap_access (const char *path, int flags) /* we may already be in the folder we're checking */ if (!ascii_strcmp(idata->mailbox, mx.mbox)) { - FREE (&mx.mbox); + mem_free (&mx.mbox); return 0; } - FREE (&mx.mbox); + mem_free (&mx.mbox); imap_munge_mbox_name (mbox, sizeof (mbox), mailbox); if (mutt_bit_isset (idata->capabilities, IMAP4REV1)) @@ -136,7 +139,7 @@ int imap_delete_mailbox (CONTEXT * ctx, IMAP_MBOX mx) if (!(idata = imap_conn_find (&mx.account, option (OPTIMAPPASSIVE) ? M_IMAP_CONN_NONEW : 0))) { - FREE (&mx.mbox); + mem_free (&mx.mbox); return -1; } } @@ -239,7 +242,7 @@ void imap_expunge_mailbox (IMAP_DATA * idata) if (idata->cache[cacheno].uid == HEADER_DATA (h)->uid && idata->cache[cacheno].path) { unlink (idata->cache[cacheno].path); - FREE (&idata->cache[cacheno].path); + mem_free (&idata->cache[cacheno].path); } imap_free_header_data (&h->data); @@ -332,6 +335,7 @@ IMAP_DATA *imap_conn_find (const ACCOUNT * account, int flags) CONNECTION *conn; IMAP_DATA *idata; ACCOUNT *creds; + int new = 0; if (!(conn = mutt_conn_find (NULL, account))) return NULL; @@ -368,6 +372,7 @@ IMAP_DATA *imap_conn_find (const ACCOUNT * account, int flags) conn->data = idata; idata->conn = conn; + new = 1; } if (idata->state == IMAP_DISCONNECTED) @@ -381,10 +386,15 @@ IMAP_DATA *imap_conn_find (const ACCOUNT * account, int flags) else mutt_account_unsetpass (&idata->conn->account); - FREE (&idata->capstr); + mem_free (&idata->capstr); } - if (idata->state == IMAP_AUTHENTICATED) + if (new && idata->state == IMAP_AUTHENTICATED) { imap_get_delim (idata); + if (option (OPTIMAPCHECKSUBSCRIBED)) { + mutt_message _("Checking mailbox subscriptions"); + imap_exec (idata, "LSUB \"\" \"*\"", 0); + } + } return idata; } @@ -399,10 +409,6 @@ int imap_open_connection (IMAP_DATA * idata) idata->state = IMAP_CONNECTED; if (imap_cmd_step (idata) != IMAP_CMD_CONTINUE) { - mutt_error (_("Unexpected response received from server: %s"), - idata->cmd.buf); - mutt_sleep (1); - mutt_socket_close (idata->conn); idata->state = IMAP_DISCONNECTED; return -1; @@ -414,11 +420,14 @@ int imap_open_connection (IMAP_DATA * idata) goto bail; #if defined(USE_SSL) || defined(USE_GNUTLS) /* Attempt STARTTLS if available and desired. */ - if (mutt_bit_isset (idata->capabilities, STARTTLS) && !idata->conn->ssf) { + if (!idata->conn->ssf && (option(OPTSSLFORCETLS) || + mutt_bit_isset (idata->capabilities, STARTTLS))) { int rc; - if ((rc = query_quadoption (OPT_SSLSTARTTLS, - _("Secure connection with TLS?"))) == -1) + if (option (OPTSSLFORCETLS)) + rc = M_YES; + else if ((rc = query_quadoption (OPT_SSLSTARTTLS, + _("Secure connection with TLS?"))) == -1) goto err_close_conn; if (rc == M_YES) { if ((rc = imap_exec (idata, "STARTTLS", IMAP_CMD_FAIL_OK)) == -1) @@ -442,13 +451,19 @@ int imap_open_connection (IMAP_DATA * idata) } } } + + if (option(OPTSSLFORCETLS) && ! idata->conn->ssf) { + mutt_error _("Encrypted connection unavailable"); + mutt_sleep (1); + goto err_close_conn; + } #endif } else if (ascii_strncasecmp ("* PREAUTH", idata->cmd.buf, 9) == 0) { idata->state = IMAP_AUTHENTICATED; if (imap_check_capabilities (idata) != 0) goto bail; - FREE (&idata->capstr); + mem_free (&idata->capstr); } else { imap_error ("imap_open_connection()", buf); @@ -461,7 +476,7 @@ err_close_conn: mutt_socket_close (idata->conn); idata->state = IMAP_DISCONNECTED; bail: - FREE (&idata->capstr); + mem_free (&idata->capstr); return -1; } @@ -543,12 +558,12 @@ int imap_open_mailbox (CONTEXT * ctx) /* Clean up path and replace the one in the ctx */ imap_fix_path (idata, mx.mbox, buf, sizeof (buf)); - FREE (&(idata->mailbox)); - idata->mailbox = safe_strdup (buf); + mem_free (&(idata->mailbox)); + idata->mailbox = str_dup (buf); imap_qualify_path (buf, sizeof (buf), &mx, idata->mailbox); - FREE (&(ctx->path)); - ctx->path = safe_strdup (buf); + mem_free (&(ctx->path)); + ctx->path = str_dup (buf); idata->ctx = ctx; @@ -676,8 +691,8 @@ int imap_open_mailbox (CONTEXT * ctx) } ctx->hdrmax = count; - ctx->hdrs = safe_calloc (count, sizeof (HEADER *)); - ctx->v2r = safe_calloc (count, sizeof (int)); + ctx->hdrs = mem_calloc (count, sizeof (HEADER *)); + ctx->v2r = mem_calloc (count, sizeof (int)); ctx->msgcount = 0; if (count && (imap_read_headers (idata, 0, count - 1) < 0)) { mutt_error _("Error opening mailbox"); @@ -687,14 +702,14 @@ int imap_open_mailbox (CONTEXT * ctx) } debug_print (2, ("msgcount is %d\n", ctx->msgcount)); - FREE (&mx.mbox); + mem_free (&mx.mbox); return 0; fail: if (idata->state == IMAP_SELECTED) idata->state = IMAP_AUTHENTICATED; fail_noidata: - FREE (&mx.mbox); + mem_free (&mx.mbox); return -1; } @@ -713,7 +728,7 @@ int imap_open_mailbox_append (CONTEXT * ctx) * ctx is brand new and mostly empty */ if (!(idata = imap_conn_find (&(mx.account), 0))) { - FREE(&mx.mbox); + mem_free(&mx.mbox); return (-1); } conn = idata->conn; @@ -723,7 +738,7 @@ int imap_open_mailbox_append (CONTEXT * ctx) imap_fix_path (idata, mx.mbox, mailbox, sizeof (mailbox)); - FREE(&mx.mbox); + mem_free(&mx.mbox); /* really we should also check for W_OK */ if (!imap_access (ctx->path, F_OK)) @@ -747,8 +762,8 @@ void imap_logout (IMAP_DATA * idata) idata->status = IMAP_BYE; imap_cmd_start (idata, "LOGOUT"); while (imap_cmd_step (idata) == IMAP_CMD_CONTINUE); - FREE (&idata->cmd.buf); - FREE (&idata); + mem_free (&idata->cmd.buf); + mem_free (&idata); } /* @@ -775,7 +790,7 @@ static void imap_set_flag (IMAP_DATA * idata, int aclbit, int flag, { if (mutt_bit_isset (idata->rights, aclbit)) if (flag) - safe_strcat (flags, flsize, str); + str_cat (flags, flsize, str); } /* imap_make_msg_set: make an IMAP4rev1 UID message set out of a set of @@ -800,7 +815,7 @@ int imap_make_msg_set (IMAP_DATA * idata, BUFFER * buf, int flag, int changed) int started = 0; /* make copy of header pointers to sort in natural order */ - hdrs = safe_calloc (idata->ctx->msgcount, sizeof (HEADER *)); + hdrs = mem_calloc (idata->ctx->msgcount, sizeof (HEADER *)); memcpy (hdrs, idata->ctx->hdrs, idata->ctx->msgcount * sizeof (HEADER *)); if (Sort != SORT_ORDER) { @@ -856,7 +871,7 @@ int imap_make_msg_set (IMAP_DATA * idata, BUFFER * buf, int flag, int changed) } } - FREE (&hdrs); + mem_free (&hdrs); return count; } @@ -1039,10 +1054,10 @@ int imap_sync_mailbox (CONTEXT * ctx, int expunge, int *index_hint) rc = 0; out: if (cmd.data) - FREE (&cmd.data); + mem_free (&cmd.data); if (appendctx) { mx_fastclose_mailbox (appendctx); - FREE (&appendctx); + mem_free (&appendctx); } return rc; } @@ -1068,7 +1083,7 @@ void imap_close_mailbox (CONTEXT * ctx) } idata->reopen &= IMAP_REOPEN_ALLOW; - FREE (&(idata->mailbox)); + mem_free (&(idata->mailbox)); mutt_free_list (&idata->flags); idata->ctx = NULL; } @@ -1080,7 +1095,7 @@ void imap_close_mailbox (CONTEXT * ctx) for (i = 0; i < IMAP_CACHE_LEN; i++) { if (idata->cache[i].path) { unlink (idata->cache[i].path); - FREE (&idata->cache[i].path); + mem_free (&idata->cache[i].path); } } } @@ -1102,7 +1117,7 @@ int imap_check_mailbox (CONTEXT * ctx, int *index_hint, int force) idata = (IMAP_DATA *) ctx->data; - if ((force || time (NULL) > idata->lastread + Timeout) + if ((force || time (NULL) >= idata->lastread + Timeout) && imap_exec (idata, "NOOP", 0) != 0) return -1; @@ -1149,13 +1164,13 @@ int imap_mailbox_check (char *path, int new) connflags = M_IMAP_CONN_NONEW; if (!(idata = imap_conn_find (&(mx.account), connflags))) { - FREE (&mx.mbox); + mem_free (&mx.mbox); return -1; } conn = idata->conn; imap_fix_path (idata, mx.mbox, buf, sizeof (buf)); - FREE (&mx.mbox); + mem_free (&mx.mbox); imap_munge_mbox_name (mbox, sizeof (mbox), buf); strfcpy (mbox_unquoted, buf, sizeof (mbox_unquoted)); @@ -1164,9 +1179,9 @@ int imap_mailbox_check (char *path, int new) * command on a mailbox that you have selected */ - if (mutt_strcmp (mbox_unquoted, idata->mailbox) == 0 + if (str_cmp (mbox_unquoted, idata->mailbox) == 0 || (ascii_strcasecmp (mbox_unquoted, "INBOX") == 0 - && safe_strcasecmp (mbox_unquoted, idata->mailbox) == 0)) { + && str_casecmp (mbox_unquoted, idata->mailbox) == 0)) { strfcpy (buf, "NOOP", sizeof (buf)); } else if (mutt_bit_isset (idata->capabilities, IMAP4REV1) || @@ -1191,8 +1206,8 @@ int imap_mailbox_check (char *path, int new) /* The mailbox name may or may not be quoted here. We could try to * munge the server response and compare with quoted (or vise versa) * but it is probably more efficient to just strncmp against both. */ - if (safe_strncmp (mbox_unquoted, s, mutt_strlen (mbox_unquoted)) == 0 - || safe_strncmp (mbox, s, mutt_strlen (mbox)) == 0) { + if (str_ncmp (mbox_unquoted, s, str_len (mbox_unquoted)) == 0 + || str_ncmp (mbox, s, str_len (mbox)) == 0) { s = imap_next_word (s); s = imap_next_word (s); if (isdigit ((unsigned char) *s)) { @@ -1291,6 +1306,8 @@ int imap_subscribe (char *path, int subscribe) IMAP_DATA *idata; char buf[LONG_STRING]; char mbox[LONG_STRING]; + char errstr[STRING]; + BUFFER err, token; IMAP_MBOX mx; if (mx_get_magic (path) == M_IMAP || imap_parse_path (path, &mx)) { @@ -1298,37 +1315,106 @@ int imap_subscribe (char *path, int subscribe) return -1; } - if (!(idata = imap_conn_find (&(mx.account), 0))) goto fail; conn = idata->conn; imap_fix_path (idata, mx.mbox, buf, sizeof (buf)); + + if (option (OPTIMAPCHECKSUBSCRIBED)) { + memset (&token, 0, sizeof (token)); + err.data = errstr; + err.dsize = sizeof (errstr); + snprintf (mbox, sizeof (mbox), "%smailboxes \"%s\"", + subscribe ? "" : "un", path); + if (mutt_parse_rc_line (mbox, &token, &err)) + debug_print (1, ("Error adding subscribed mailbox: %s\n", errstr)); + mem_free (&token.data); + } + if (subscribe) mutt_message (_("Subscribing to %s..."), buf); else mutt_message (_("Unsubscribing to %s..."), buf); imap_munge_mbox_name (mbox, sizeof (mbox), buf); - snprintf (buf, sizeof (buf), "%s %s", subscribe ? "SUBSCRIBE" : - "UNSUBSCRIBE", mbox); + snprintf (buf, sizeof (buf), "%sSUBSCRIBE %s", subscribe ? "" : "UN", mbox); if (imap_exec (idata, buf, 0) < 0) goto fail; - FREE (&mx.mbox); + mem_free (&mx.mbox); return 0; fail: - FREE (&mx.mbox); + mem_free (&mx.mbox); return -1; } +/* trim dest to the length of the longest prefix it shares with src, + * returning the length of the trimmed string */ +static int longest_common_prefix (char *dest, const char* src, + int start, size_t dlen) { + int pos = start; + + while (pos < dlen && dest[pos] && dest[pos] == src[pos]) + pos++; + dest[pos] = '\0'; + + return pos; +} + +/* look for IMAP URLs to complete from defined mailboxes. Could be extended + * to complete over open connections and account/folder hooks too. */ +static int imap_complete_hosts (char *dest, size_t len) { + BUFFY* mailbox; + CONNECTION* conn; + int rc = -1; + int matchlen; + int i = 0; + + matchlen = str_len (dest); + if (list_empty (Incoming)) + return (-1); + for (i = 0; i < Incoming->length; i++) { + mailbox = (BUFFY*) Incoming->data[i]; + if (!str_ncmp (dest, mailbox->path, matchlen)) { + if (rc) { + strfcpy (dest, mailbox->path, len); + rc = 0; + } else + longest_common_prefix (dest, mailbox->path, matchlen, len); + } + } + + for (conn = mutt_socket_head (); conn->next; conn = conn->next) { + ciss_url_t url; + char urlstr[LONG_STRING]; + + if (conn->account.type != M_ACCT_TYPE_IMAP) + continue; + + mutt_account_tourl (&conn->account, &url); + /* FIXME: how to handle multiple users on the same host? */ + url.user = NULL; + url.path = NULL; + url_ciss_tostring (&url, urlstr, sizeof (urlstr), 0); + if (!str_ncmp (dest, urlstr, matchlen)) { + if (rc) { + strfcpy (dest, urlstr, len); + rc = 0; + } else + longest_common_prefix (dest, urlstr, matchlen, len); + } + } + + return rc; +} + /* imap_complete: given a partial IMAP folder path, return a string which * adds as much to the path as is unique */ -int imap_complete (char *dest, size_t dlen, char *path) -{ +int imap_complete (char *dest, size_t dlen, char *path) { CONNECTION *conn; IMAP_DATA *idata; char list[LONG_STRING]; @@ -1339,18 +1425,20 @@ int imap_complete (char *dest, size_t dlen, char *path) char completion[LONG_STRING]; int clen, matchlen = 0; int completions = 0; - int pos = 0; IMAP_MBOX mx; - /* verify passed in path is an IMAP path */ - if (imap_parse_path (path, &mx)) { - debug_print (2, ("bad path %s\n", path)); - return -1; + if (imap_parse_path (path, &mx) || !mx.mbox) { + strfcpy (dest, path, dlen); + return imap_complete_hosts (dest, dlen); } - /* don't open a new socket just for completion */ - if (!(idata = imap_conn_find (&(mx.account), M_IMAP_CONN_NONEW))) - goto fail; + /* don't open a new socket just for completion. Instead complete over + * known mailboxes/hooks/etc */ + if (!(idata = imap_conn_find (&(mx.account), M_IMAP_CONN_NONEW))) { + mem_free (&mx.mbox); + strfcpy (dest, path, dlen); + return imap_complete_hosts (dest, dlen); + } conn = idata->conn; /* reformat path for IMAP list, and append wildcard */ @@ -1380,25 +1468,19 @@ int imap_complete (char *dest, size_t dlen, char *path) /* if the folder isn't selectable, append delimiter to force browse * to enter it on second tab. */ if (noselect) { - clen = mutt_strlen (list_word); + clen = str_len (list_word); list_word[clen++] = delim; list_word[clen] = '\0'; } /* copy in first word */ if (!completions) { strfcpy (completion, list_word, sizeof (completion)); - matchlen = mutt_strlen (completion); + matchlen = str_len (completion); completions++; continue; } - pos = 0; - while (pos < matchlen && list_word[pos] && - completion[pos] == list_word[pos]) - pos++; - completion[pos] = '\0'; - matchlen = pos; - + matchlen = longest_common_prefix (completion, list_word, 0, matchlen); completions++; } } @@ -1409,12 +1491,10 @@ int imap_complete (char *dest, size_t dlen, char *path) imap_qualify_path (dest, dlen, &mx, completion); mutt_pretty_mailbox (dest); - FREE (&mx.mbox); + mem_free (&mx.mbox); return 0; } -fail: - FREE (&mx.mbox); return -1; }