# include "config.h"
#endif
+#include <lib-lib/mem.h>
+#include <lib-lib/str.h>
+#include <lib-lib/ascii.h>
+#include <lib-lib/macros.h>
+#include <lib-lib/file.h>
+
#include "mutt.h"
#include "enter.h"
-#include "ascii.h"
#include "mutt_curses.h"
#include "rfc2047.h"
+#include "rfc3676.h"
#include "keymap.h"
#include "mime.h"
#include "copy.h"
#include "url.h"
#include "attach.h"
-#include "lib/mem.h"
-#include "lib/intl.h"
-#include "lib/str.h"
#include "lib/debug.h"
#include <ctype.h>
return top;
}
-static int edit_address (ADDRESS ** a, /* const */ char *field)
+static int edit_address (ADDRESS ** a, const char *field)
{
char buf[HUGE_STRING];
char *err = NULL;
mutt_error (_("Error: '%s' is a bad IDN."), err);
mutt_refresh ();
mutt_sleep (2);
- mem_free (&err);
+ p_delete(&err);
}
}
while (idna_ok != 0);
#ifdef USE_NNTP
if (option (OPTNEWSSEND)) {
if (en->newsgroups)
- strfcpy (buf, en->newsgroups, sizeof (buf));
+ m_strcpy(buf, sizeof(buf), en->newsgroups);
else
buf[0] = 0;
if (mutt_get_field ("Newsgroups: ", buf, sizeof (buf), 0) != 0)
return (-1);
- mem_free (&en->newsgroups);
- en->newsgroups = str_dup (buf);
+ p_delete(&en->newsgroups);
+ en->newsgroups = m_strdup(buf);
if (en->followup_to)
- strfcpy (buf, en->followup_to, sizeof (buf));
+ m_strcpy(buf, sizeof(buf), en->followup_to);
else
buf[0] = 0;
if (option (OPTASKFOLLOWUP)
&& mutt_get_field ("Followup-To: ", buf, sizeof (buf), 0) != 0)
return (-1);
- mem_free (&en->followup_to);
- en->followup_to = str_dup (buf);
+ p_delete(&en->followup_to);
+ en->followup_to = m_strdup(buf);
if (en->x_comment_to)
- strfcpy (buf, en->x_comment_to, sizeof (buf));
+ m_strcpy(buf, sizeof(buf), en->x_comment_to);
else
buf[0] = 0;
if (option (OPTXCOMMENTTO) && option (OPTASKXCOMMENTTO)
&& mutt_get_field ("X-Comment-To: ", buf, sizeof (buf), 0) != 0)
return (-1);
- mem_free (&en->x_comment_to);
- en->x_comment_to = str_dup (buf);
+ p_delete(&en->x_comment_to);
+ en->x_comment_to = m_strdup(buf);
}
else
#endif
if (option (OPTFASTREPLY))
return (0);
else
- strfcpy (buf, en->subject, sizeof (buf));
+ m_strcpy(buf, sizeof(buf), en->subject);
}
else {
char *p;
buf[0] = 0;
for (; uh; uh = uh->next) {
if (ascii_strncasecmp ("subject:", uh->data, 8) == 0) {
- p = uh->data + 8;
- SKIPWS (p);
- strncpy (buf, p, sizeof (buf));
+ p = vskipspaces(uh->data + 8);
+ m_strcpy(buf, sizeof(buf), p);
}
}
}
}
#ifdef USE_NNTP
-char *nntp_get_header (const char *s)
+char *nntp_get_header(const char *s)
{
- SKIPWS (s);
- return str_dup (s);
+ return m_strdup(skipspaces(s));
}
#endif
}
else
last = env->userhdrs = mutt_new_list ();
- last->data = str_dup (uh->data);
+ last->data = m_strdup(uh->data);
}
}
}
*/
if (!(flags & SENDGROUPREPLY) && mutt_is_list_cc (0, env->to, env->cc)) {
switch (query_quadoption (OPT_LISTREPLY,
- _
- ("Message came from a mailing list. Reply to author only?")))
+ _("Message came from a mailing list. List-reply to mailing list?")))
{
- case M_NO:
+ case M_YES:
tmp = find_mailing_lists (env->to, env->cc);
rfc822_append (to, tmp);
rfc822_free_address (&tmp);
if (e->message_id) {
t = mutt_new_list ();
- t->data = str_dup (e->message_id);
+ t->data = m_strdup(e->message_id);
t->next = l;
l = t;
}
env->to = mutt_remove_duplicates (env->to);
env->cc = mutt_remove_duplicates (env->cc);
env->cc = mutt_remove_xrefs (env->to, env->cc);
+
+ if (env->cc && !env->to) {
+ env->to = env->cc;
+ env->cc = NULL;
+ }
}
void mutt_make_forward_subject (ENVELOPE * env, CONTEXT * ctx, HEADER * cur)
* been taken from a List-Post header. Is that correct?
*/
if (curenv->real_subj) {
- mem_free (&env->subject);
- env->subject = mem_malloc (str_len (curenv->real_subj) + 5);
+ p_delete(&env->subject);
+ env->subject = p_new(char, m_strlen(curenv->real_subj) + 5);
sprintf (env->subject, "Re: %s", curenv->real_subj); /* __SPRINTF_CHECKED__ */
}
else if (!env->subject)
- env->subject = str_dup ("Re: your mail");
+ env->subject = m_strdup("Re: your mail");
#ifdef USE_NNTP
if (option (OPTNEWSSEND) && option (OPTXCOMMENTTO) && curenv->from)
- env->x_comment_to = str_dup (mutt_get_name (curenv->from));
+ env->x_comment_to = m_strdup(mutt_get_name (curenv->from));
#endif
}
if (curenv->message_id) {
*q = mutt_new_list ();
- (*q)->data = str_dup (curenv->message_id);
+ (*q)->data = m_strdup(curenv->message_id);
}
if (pp)
if ((flags & SENDNEWS)) {
/* in case followup set Newsgroups: with Followup-To: if it present */
if (!env->newsgroups && curenv &&
- str_casecmp (curenv->followup_to, "poster"))
- env->newsgroups = str_dup (curenv->followup_to);
+ m_strcasecmp(curenv->followup_to, "poster"))
+ env->newsgroups = m_strdup(curenv->followup_to);
}
else
#endif
#ifdef USE_NNTP
if (option (OPTNEWSSEND)) {
if (!e->followup_to && e->newsgroups && (strrchr (e->newsgroups, ',')))
- e->followup_to = str_dup (e->newsgroups);
+ e->followup_to = m_strdup(e->newsgroups);
return;
}
#endif
if (tmp) {
tmp = rfc822_cpy_adr_real (tmp);
if (!option (OPTREVREAL))
- mem_free (&tmp->personal);
+ p_delete(&tmp->personal);
if (!tmp->personal)
- tmp->personal = str_dup (Realname);
+ tmp->personal = m_strdup(Realname);
}
return (tmp);
}
adr = rfc822_cpy_adr_real (From);
else if (option (OPTUSEDOMAIN)) {
adr = rfc822_new_address ();
- adr->mailbox =
- mem_malloc (str_len (Username) + str_len (fqdn) + 2);
+ adr->mailbox = p_new(char, m_strlen(Username) + m_strlen(fqdn) + 2);
sprintf (adr->mailbox, "%s@%s", NONULL (Username), NONULL (fqdn)); /* __SPRINTF_CHECKED__ */
}
else {
adr = rfc822_new_address ();
- adr->mailbox = str_dup (NONULL (Username));
+ adr->mailbox = m_strdup(NONULL (Username));
}
return (adr);
}
}
+static void fix_end_of_file (const char *data)
+{
+ FILE *fp;
+ int c;
+
+ if ((fp = safe_fopen (data, "a+")) == NULL)
+ return;
+ fseeko (fp, -1, SEEK_END);
+ if ((c = fgetc (fp)) != '\n')
+ fputc ('\n', fp);
+ safe_fclose (&fp);
+}
+
int mutt_resend_message (FILE * fp, CONTEXT * ctx, HEADER * cur)
{
HEADER *msg = mutt_new_header ();
char *pgpkeylist = NULL;
/* save current value of "pgp_sign_as" */
- char *signas = NULL;
- char *tag = NULL, *err = NULL;
+ char *signas = NULL, *err = NULL;
+ const char *tag = NULL;
char *ctype;
int rv = -1;
if ((WithCrypto & APPLICATION_PGP) && (flags & SENDPOSTPONED))
- signas = str_dup (PgpSignAs);
+ signas = m_strdup(PgpSignAs);
/* Delay expansion of aliases until absolutely necessary--shouldn't
* be necessary unless we are prompting the user or about to execute a
pbody->next = msg->content; /* don't kill command-line attachments */
msg->content = pbody;
- ctype = str_dup (ContentType);
+ if (!(ctype = m_strdup(ContentType)))
+ ctype = m_strdup("text/plain");
mutt_parse_content_type (ctype, msg->content);
- mem_free (&ctype);
+ p_delete(&ctype);
msg->content->unlink = 1;
msg->content->use_disp = 0;
if (!tempfile) {
mutt_mktemp (buffer);
tempfp = safe_fopen (buffer, "w+");
- msg->content->filename = str_dup (buffer);
+ msg->content->filename = m_strdup(buffer);
}
else {
tempfp = safe_fopen (tempfile, "a+");
- msg->content->filename = str_dup (tempfile);
+ msg->content->filename = m_strdup(tempfile);
}
if (!tempfp) {
#ifdef USE_NNTP
if ((flags & SENDNEWS) && ctx && ctx->magic == M_NNTP
&& !msg->env->newsgroups)
- msg->env->newsgroups = str_dup (((NNTP_DATA *) ctx->data)->group);
+ msg->env->newsgroups = m_strdup(((NNTP_DATA *) ctx->data)->group);
#endif
if (!(flags & SENDMAILX) &&
if (option (OPTSIGONTOP)
&& (!(flags & (SENDMAILX | SENDKEY)) && Editor
- && str_cmp (Editor, "builtin") != 0))
+ && m_strcmp(Editor, "builtin") != 0))
append_signature (tempfp);
/* include replies/forwarded messages, unless we are given a template */
if (!option (OPTSIGONTOP)
&& (!(flags & (SENDMAILX | SENDKEY)) && Editor
- && str_cmp (Editor, "builtin") != 0))
+ && m_strcmp(Editor, "builtin") != 0))
append_signature (tempfp);
/*
that $realname can be set in a send-hook */
if (msg->env->from && !msg->env->from->personal
&& !(flags & (SENDRESEND | SENDPOSTPONED)))
- msg->env->from->personal = str_dup (Realname);
+ msg->env->from->personal = m_strdup(Realname);
if (!((WithCrypto & APPLICATION_PGP) && (flags & SENDKEY)))
safe_fclose (&tempfp);
query_quadoption (OPT_FORWEDIT,
_("Edit forwarded message?")) == M_YES)) {
/* If the this isn't a text message, look for a mailcap edit command */
- if (mutt_needs_mailcap (msg->content))
- mutt_edit_attachment (msg->content);
- else if (!Editor || str_cmp ("builtin", Editor) == 0)
+ if (mutt_needs_mailcap (msg->content)) {
+ if (!mutt_edit_attachment (msg->content))
+ goto cleanup;
+ } else if (!Editor || m_strcmp("builtin", Editor) == 0)
mutt_builtin_editor (msg->content->filename, msg, cur);
else if (option (OPTEDITHDRS)) {
mutt_env_to_local (msg->env);
sizeof (fcc));
mutt_env_to_idna (msg->env, NULL, NULL);
}
- else
+ else {
mutt_edit_file (Editor, msg->content->filename);
+ if (stat (msg->content->filename, &st) == 0) {
+ if (mtime != st.st_mtime)
+ fix_end_of_file (msg->content->filename);
+ } else
+ mutt_perror (msg->content->filename);
+ }
+
+ if (option (OPTTEXTFLOWED))
+ rfc3676_space_stuff (msg);
+
mutt_message_hook (NULL, msg, M_SEND2HOOK);
}
if (mutt_env_to_idna (msg->env, &tag, &err)) {
mutt_error (_("Bad IDN in \"%s\": '%s'"), tag, err);
- mem_free (&err);
+ p_delete(&err);
if (!(flags & SENDBATCH))
goto main_loop;
else
mutt_protect (msg, pgpkeylist) == -1) {
msg->content = mutt_remove_multipart (msg->content);
- mem_free (&pgpkeylist);
+ p_delete(&pgpkeylist);
decode_descriptions (msg->content);
goto main_loop;
fcc[0] = '\0';
#endif
- if (*fcc && str_cmp ("/dev/null", fcc) != 0) {
+ if (*fcc && m_strcmp("/dev/null", fcc) != 0) {
BODY *tmpbody = msg->content;
BODY *save_sig = NULL;
BODY *save_parts = NULL;
/* check to see if the user wants copies of all attachments */
if (!option (OPTFCCATTACH) && msg->content->type == TYPEMULTIPART) {
if (WithCrypto
- && (str_cmp (msg->content->subtype, "encrypted") == 0 ||
- str_cmp (msg->content->subtype, "signed") == 0)) {
+ && (m_strcmp(msg->content->subtype, "encrypted") == 0 ||
+ m_strcmp(msg->content->subtype, "signed") == 0)) {
if (clear_content->type == TYPEMULTIPART) {
if (!(msg->security & ENCRYPT) && (msg->security & SIGN)) {
/* save initial signature and attachments */
#endif
if (WithCrypto && (msg->security & ENCRYPT))
- mem_free (&pgpkeylist);
+ p_delete(&pgpkeylist);
if (WithCrypto && free_clear_content)
mutt_free_body (&clear_content);
if ((WithCrypto & APPLICATION_PGP) && (flags & SENDPOSTPONED)) {
if (signas) {
- mem_free (&PgpSignAs);
+ p_delete(&PgpSignAs);
PgpSignAs = signas;
}
}