X-Git-Url: http://git.madism.org/?p=apps%2Fmadmutt.git;a=blobdiff_plain;f=postpone.c;h=5fda6cec492ad0cd7ad99e5913c3017709a670da;hp=db84a3243fc871cae3605b2fd209568603042a02;hb=6ebff74ad242d4c56fb7762965b19cee14fd2daa;hpb=0f44dc85fc1280372ffab911d701e703d803fb4b diff --git a/postpone.c b/postpone.c index db84a32..5fda6ce 100644 --- a/postpone.c +++ b/postpone.c @@ -8,38 +8,31 @@ * 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 "mutt.h" -#include "mutt_menu.h" -#include "rfc1524.h" -#include "mime.h" -#include "mailbox.h" -#include "mapping.h" +#include "handler.h" #include "sort.h" -#ifdef USE_IMAP -#include "mx.h" -#include "imap.h" -#endif -#include "mutt_crypt.h" - -#include "lib/mem.h" -#include "lib/intl.h" -#include "lib/str.h" +#include "thread.h" +#include -#include -#include -#include -#include +#include static struct mapping_t PostponeHelp[] = { - {N_("Exit"), OP_EXIT}, - {N_("Del"), OP_DELETE}, + {N_("Exit"), OP_EXIT}, + {N_("Del"), OP_DELETE}, {N_("Undel"), OP_UNDELETE}, - {N_("Help"), OP_HELP}, - {NULL} + {N_("Help"), OP_HELP}, + {NULL, OP_NULL} }; @@ -74,28 +67,18 @@ int mutt_num_postponed (int force) if (!Postponed) return 0; -#ifdef USE_IMAP /* LastModify is useless for IMAP */ - if (mx_is_imap (Postponed)) { + if (imap_is_magic (Postponed, NULL) == M_IMAP) { if (force) { short newpc; newpc = imap_mailbox_check (Postponed, 0); if (newpc >= 0) { PostCount = newpc; - dprint (2, - (debugfile, - "mutt_num_postponed: %d postponed IMAP messages found.\n", - PostCount)); } - else - dprint (2, - (debugfile, - "mutt_num_postponed: using old IMAP postponed count.\n")); } return PostCount; } -#endif if (stat (Postponed, &st) == -1) { PostCount = 0; @@ -147,7 +130,7 @@ void mutt_update_num_postponed (void) UpdateNumPostponed = 1; } -static void post_entry (char *s, size_t slen, MUTTMENU * menu, int entry) +static void post_entry (char *s, ssize_t slen, MUTTMENU * menu, int entry) { CONTEXT *ctx = (CONTEXT *) menu->data; @@ -229,13 +212,13 @@ static HEADER *select_msg (void) * SENDREPLY recalled message is a reply */ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, - size_t fcclen) + ssize_t fcclen) { HEADER *h; int code = SENDPOSTPONED; - LIST *tmp; - LIST *last = NULL; - LIST *next; + string_list_t *tmp; + string_list_t *last = NULL; + string_list_t *next; char *p; int opt_delete; @@ -252,7 +235,7 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, if (!PostContext->msgcount) { PostCount = 0; mx_close_mailbox (PostContext, NULL); - FREE (&PostContext); + p_delete(&PostContext); mutt_error _("No postponed messages."); return (-1); @@ -264,13 +247,13 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, } else if ((h = select_msg ()) == NULL) { mx_close_mailbox (PostContext, NULL); - FREE (&PostContext); + p_delete(&PostContext); return (-1); } if (mutt_prepare_template (NULL, PostContext, hdr, h, 0) < 0) { mx_fastclose_mailbox (PostContext); - FREE (&PostContext); + p_delete(&PostContext); return (-1); } @@ -289,15 +272,14 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, mx_close_mailbox (PostContext, NULL); set_quadoption (OPT_DELETE, opt_delete); - FREE (&PostContext); + p_delete(&PostContext); for (tmp = hdr->env->userhdrs; tmp;) { if (ascii_strncasecmp ("X-Mutt-References:", tmp->data, 18) == 0) { if (ctx) { /* if a mailbox is currently open, look to see if the orignal message the user attempted to reply to is in this mailbox */ - p = tmp->data + 18; - SKIPWS (p); + p = vskipspaces(tmp->data + 18); if (!ctx->id_hash) ctx->id_hash = mutt_make_id_hash (ctx); *cur = hash_find (ctx->id_hash, p); @@ -310,15 +292,14 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, else hdr->env->userhdrs = tmp->next; tmp->next = NULL; - mutt_free_list (&tmp); + string_list_wipe(&tmp); tmp = next; if (*cur) code |= SENDREPLY; } else if (ascii_strncasecmp ("X-Mutt-Fcc:", tmp->data, 11) == 0) { - p = tmp->data + 11; - SKIPWS (p); - strfcpy (fcc, p, fcclen); + p = vskipspaces(tmp->data + 11); + m_strcpy(fcc, fcclen, p); mutt_pretty_mailbox (fcc); /* remove the X-Mutt-Fcc: header field */ @@ -328,14 +309,13 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, else hdr->env->userhdrs = tmp->next; tmp->next = NULL; - mutt_free_list (&tmp); + string_list_wipe(&tmp); tmp = next; } - else if ((WithCrypto & APPLICATION_PGP) - && (safe_strncmp ("Pgp:", tmp->data, 4) == 0 /* this is generated - * by old mutt versions - */ - || safe_strncmp ("X-Mutt-PGP:", tmp->data, 11) == 0)) { + else if ((m_strncmp("Pgp:", tmp->data, 4) == 0 /* this is generated + * by old mutt versions + */ + || m_strncmp("X-Mutt-PGP:", tmp->data, 11) == 0)) { hdr->security = mutt_parse_crypt_hdr (strchr (tmp->data, ':') + 1, 1); hdr->security |= APPLICATION_PGP; @@ -346,11 +326,10 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, else hdr->env->userhdrs = tmp->next; tmp->next = NULL; - mutt_free_list (&tmp); + string_list_wipe(&tmp); tmp = next; } - else if ((WithCrypto & APPLICATION_SMIME) - && safe_strncmp ("X-Mutt-SMIME:", tmp->data, 13) == 0) { + else if (m_strncmp("X-Mutt-SMIME:", tmp->data, 13) == 0) { hdr->security = mutt_parse_crypt_hdr (strchr (tmp->data, ':') + 1, 1); hdr->security |= APPLICATION_SMIME; @@ -361,15 +340,15 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, else hdr->env->userhdrs = tmp->next; tmp->next = NULL; - mutt_free_list (&tmp); + string_list_wipe(&tmp); tmp = next; } #ifdef MIXMASTER - else if (safe_strncmp ("X-Mutt-Mix:", tmp->data, 11) == 0) { + else if (m_strncmp("X-Mutt-Mix:", tmp->data, 11) == 0) { char *t; - mutt_free_list (&hdr->chain); + string_list_wipe(&hdr->chain); t = strtok (tmp->data + 11, " \t\n"); while (t) { @@ -383,7 +362,7 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, else hdr->env->userhdrs = tmp->next; tmp->next = NULL; - mutt_free_list (&tmp); + string_list_wipe(&tmp); tmp = next; } #endif @@ -404,12 +383,7 @@ int mutt_parse_crypt_hdr (char *p, int set_signas) char pgp_sign_as[LONG_STRING] = "\0", *q; char smime_cryptalg[LONG_STRING] = "\0"; - if (!WithCrypto) - return 0; - - SKIPWS (p); - for (; *p; p++) { - + for (p = vskipspaces(p); *p; p++) { switch (*p) { case 'e': case 'E': @@ -488,11 +462,11 @@ int mutt_parse_crypt_hdr (char *p, int set_signas) } /* the cryptalg field must not be empty */ - if ((WithCrypto & APPLICATION_SMIME) && *smime_cryptalg) - str_replace (&SmimeCryptAlg, smime_cryptalg); + if (*smime_cryptalg) + m_strreplace(&SmimeCryptAlg, smime_cryptalg); - if ((WithCrypto & APPLICATION_PGP) && (set_signas || *pgp_sign_as)) - str_replace (&PgpSignAs, pgp_sign_as); + if (set_signas || *pgp_sign_as) + m_strreplace(&PgpSignAs, pgp_sign_as); return pgp; } @@ -510,7 +484,7 @@ int mutt_prepare_template (FILE * fp, CONTEXT * ctx, HEADER * newhdr, int rv = -1; STATE s; - memset (&s, 0, sizeof (s)); + p_clear(&s, 1); if (!fp && (msg = mx_open_message (ctx, hdr->msgno)) == NULL) return (-1); @@ -522,21 +496,21 @@ int mutt_prepare_template (FILE * fp, CONTEXT * ctx, HEADER * newhdr, /* parse the message header and MIME structure */ - fseek (fp, hdr->offset, 0); + fseeko (fp, hdr->offset, 0); newhdr->offset = hdr->offset; newhdr->env = mutt_read_rfc822_header (fp, newhdr, 1, weed); newhdr->content->length = hdr->content->length; mutt_parse_part (fp, newhdr->content); - FREE (&newhdr->env->message_id); - FREE (&newhdr->env->mail_followup_to); /* really? */ + p_delete(&newhdr->env->message_id); + p_delete(&newhdr->env->mail_followup_to); /* really? */ /* decrypt pgp/mime encoded messages */ - if ((WithCrypto & (APPLICATION_PGP | APPLICATION_SMIME) & hdr->security) - && mutt_is_multipart_encrypted (newhdr->content)) { - int ccap = - WithCrypto & (APPLICATION_PGP | APPLICATION_SMIME) & hdr->security; + if ((APPLICATION_PGP | APPLICATION_SMIME) & hdr->security + && mutt_is_multipart_encrypted (newhdr->content)) + { + int ccap = (APPLICATION_PGP | APPLICATION_SMIME) & hdr->security; newhdr->security |= ENCRYPT | ccap; if (!crypt_valid_passphrase (ccap)) goto err; @@ -550,14 +524,14 @@ int mutt_prepare_template (FILE * fp, CONTEXT * ctx, HEADER * newhdr, || b == NULL) { err: mx_close_message (&msg); - mutt_free_envelope (&newhdr->env); - mutt_free_body (&newhdr->content); + envelope_delete(&newhdr->env); + body_list_wipe(&newhdr->content); mutt_error _("Decryption failed."); return -1; } - mutt_free_body (&newhdr->content); + body_list_wipe(&newhdr->content); newhdr->content = b; mutt_clear_error (); @@ -568,19 +542,16 @@ int mutt_prepare_template (FILE * fp, CONTEXT * ctx, HEADER * newhdr, * resending messages */ - if (WithCrypto && mutt_is_multipart_signed (newhdr->content)) { + if (mutt_is_multipart_signed (newhdr->content)) { newhdr->security |= SIGN; - if ((WithCrypto & APPLICATION_PGP) - && - ascii_strcasecmp (mutt_get_parameter - ("protocol", newhdr->content->parameter), - "application/pgp-signature") == 0) + if (ascii_strcasecmp(parameter_getval(newhdr->content->parameter, "protocol"), + "application/pgp-signature") == 0) newhdr->security |= APPLICATION_PGP; - else if ((WithCrypto & APPLICATION_SMIME)) + else newhdr->security |= APPLICATION_SMIME; /* destroy the signature */ - mutt_free_body (&newhdr->content->parts->next); + body_list_wipe(&newhdr->content->parts->next); newhdr->content = mutt_remove_multipart (newhdr->content); } @@ -609,8 +580,8 @@ int mutt_prepare_template (FILE * fp, CONTEXT * ctx, HEADER * newhdr, file[0] = '\0'; if (b->filename) { - strfcpy (file, b->filename, sizeof (file)); - b->d_filename = safe_strdup (b->filename); + m_strcpy(file, sizeof(file), b->filename); + b->d_filename = m_strdup(b->filename); } else { /* avoid Content-Disposition: header with temporary filename */ @@ -623,44 +594,42 @@ int mutt_prepare_template (FILE * fp, CONTEXT * ctx, HEADER * newhdr, if (b->type == TYPETEXT) { if (!ascii_strcasecmp - ("yes", mutt_get_parameter ("x-mutt-noconv", b->parameter))) + ("yes", parameter_getval(b->parameter, "x-mutt-noconv"))) b->noconv = 1; else { s.flags |= M_CHARCONV; b->noconv = 0; } - mutt_delete_parameter ("x-mutt-noconv", &b->parameter); + parameter_delval(&b->parameter, "x-mutt-noconv"); } - mutt_adv_mktemp (file, sizeof (file)); - if ((s.fpout = safe_fopen (file, "w")) == NULL) + s.fpout = m_tempfile(file, sizeof(file), NONULL(Tempdir), file); + if (!s.fpout) goto bail; - - if ((WithCrypto & APPLICATION_PGP) - && (mutt_is_application_pgp (b) & (ENCRYPT | SIGN))) { + if (mutt_is_application_pgp (b) & (ENCRYPT | SIGN)) { mutt_body_handler (b, &s); newhdr->security |= mutt_is_application_pgp (newhdr->content); b->type = TYPETEXT; - str_replace (&b->subtype, "plain"); - mutt_delete_parameter ("x-action", &b->parameter); + m_strreplace(&b->subtype, "plain"); + parameter_delval(&b->parameter, "x-action"); } else mutt_decode_attachment (b, &s); - if (safe_fclose (&s.fpout) != 0) + if (m_fclose(&s.fpout) != 0) goto bail; - str_replace (&b->filename, file); + m_strreplace(&b->filename, file); b->unlink = 1; mutt_stamp_attachment (b); - mutt_free_body (&b->parts); + body_list_wipe(&b->parts); if (b->hdr) b->hdr->content = NULL; /* avoid dangling pointer */ } @@ -668,12 +637,9 @@ int mutt_prepare_template (FILE * fp, CONTEXT * ctx, HEADER * newhdr, /* Fix encryption flags. */ /* No inline if multipart. */ - if (WithCrypto && (newhdr->security & INLINE) && newhdr->content->next) + if ((newhdr->security & INLINE) && newhdr->content->next) newhdr->security &= ~INLINE; - /* Do we even support multiple mechanisms? */ - newhdr->security &= WithCrypto | ~(APPLICATION_PGP | APPLICATION_SMIME); - /* Theoretically, both could be set. Take the one the user wants to set by default. */ if ((newhdr->security & APPLICATION_PGP) && (newhdr->security & APPLICATION_SMIME)) { @@ -689,13 +655,13 @@ bail: /* that's it. */ if (bfp != fp) - fclose (bfp); + m_fclose(&bfp); if (msg) mx_close_message (&msg); if (rv == -1) { - mutt_free_envelope (&newhdr->env); - mutt_free_body (&newhdr->content); + envelope_delete(&newhdr->env); + body_list_wipe(&newhdr->content); } return rv;