X-Git-Url: http://git.madism.org/?p=apps%2Fmadmutt.git;a=blobdiff_plain;f=postpone.c;h=afcc9b1cdfb9e16ff49e08e7de611bed7cdcb2f4;hp=eb62d22f12e2b4c2f1884b16f749a42c6598a07f;hb=3cb53df6efcb6b137bbab53c16da4eddbc5a1b2b;hpb=7f7a0be369840b290248e5b0302beb447fa1b3cd diff --git a/postpone.c b/postpone.c index eb62d22..afcc9b1 100644 --- a/postpone.c +++ b/postpone.c @@ -14,16 +14,18 @@ #include #include +#include #include +#include +#include + +#include #include "mutt.h" -#include "ascii.h" #include "enter.h" #include "handler.h" #include "mutt_menu.h" #include "rfc1524.h" -#include "mime.h" -#include "mapping.h" #include "sort.h" #include "thread.h" #include "mx.h" @@ -297,8 +299,7 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, 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); @@ -317,9 +318,8 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, 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 */ @@ -332,11 +332,10 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, mutt_free_list (&tmp); tmp = next; } - else if ((WithCrypto & APPLICATION_PGP) - && (str_ncmp ("Pgp:", tmp->data, 4) == 0 /* this is generated - * by old mutt versions - */ - || str_ncmp ("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; @@ -350,8 +349,7 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, mutt_free_list (&tmp); tmp = next; } - else if ((WithCrypto & APPLICATION_SMIME) - && str_ncmp ("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; @@ -367,7 +365,7 @@ int mutt_get_postponed (CONTEXT * ctx, HEADER * hdr, HEADER ** cur, char *fcc, } #ifdef MIXMASTER - else if (str_ncmp ("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); @@ -405,12 +403,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': @@ -489,11 +482,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; } @@ -511,7 +504,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); @@ -534,10 +527,10 @@ int mutt_prepare_template (FILE * fp, CONTEXT * ctx, HEADER * newhdr, /* 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; @@ -569,15 +562,13 @@ 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 + if (ascii_strcasecmp (mutt_get_parameter ("protocol", newhdr->content->parameter), "application/pgp-signature") == 0) newhdr->security |= APPLICATION_PGP; - else if ((WithCrypto & APPLICATION_SMIME)) + else newhdr->security |= APPLICATION_SMIME; /* destroy the signature */ @@ -610,7 +601,7 @@ int mutt_prepare_template (FILE * fp, CONTEXT * ctx, HEADER * newhdr, file[0] = '\0'; if (b->filename) { - strfcpy (file, b->filename, sizeof (file)); + m_strcpy(file, sizeof(file), b->filename); b->d_filename = m_strdup(b->filename); } else { @@ -639,15 +630,14 @@ int mutt_prepare_template (FILE * fp, CONTEXT * ctx, HEADER * newhdr, 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"); + m_strreplace(&b->subtype, "plain"); mutt_delete_parameter ("x-action", &b->parameter); } else @@ -656,7 +646,7 @@ int mutt_prepare_template (FILE * fp, CONTEXT * ctx, HEADER * newhdr, if (safe_fclose (&s.fpout) != 0) goto bail; - str_replace (&b->filename, file); + m_strreplace(&b->filename, file); b->unlink = 1; mutt_stamp_attachment (b); @@ -669,12 +659,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)) {