adding a missing bug number

[software/mutt-debian.git] / recvcmd.c

diff --git a/recvcmd.c b/recvcmd.c
index 26312d69c7f143203cd23db0e2005341f39f2560..5f4511f6b7bba8bc67e10feddc03570e3fe4791b 100644 (file)
--- a/recvcmd.c
+++ b/recvcmd.c
@@ -139,6 +139,34 @@ void mutt_attach_bounce (FILE * fp, HEADER * hdr,
   /* one or more messages? */
   p = (cur || count_tagged (idx, idxlen) == 1);
 
   /* one or more messages? */
   p = (cur || count_tagged (idx, idxlen) == 1);
 

+  /* RfC 5322 mandates a From: header, so warn before bouncing
+   * messages without one */
+  if (cur)
+  {
+    if (!cur->hdr->env->from)
+    {
+      mutt_error _("Warning: message contains no From: header");
+      mutt_sleep (2);
+      mutt_clear_error ();
+    }
+  }
+  else
+  {
+    for (i = 0; i < idxlen; i++)
+    {
+      if (idx[i]->content->tagged)
+      {
+       if (!idx[i]->content->hdr->env->from)
+       {
+         mutt_error _("Warning: message contains no From: header");
+         mutt_sleep (2);
+         mutt_clear_error ();
+         break;
+       }
+      }
+    }
+  }
+

   if (p)
     strfcpy (prompt, _("Bounce message to: "), sizeof (prompt));
   else
   if (p)
     strfcpy (prompt, _("Bounce message to: "), sizeof (prompt));
   else


@@ -509,13 +537,13 @@ _("Can't decode all tagged attachments.  MIME-forward the others?"))) == -1)
     }
 
     if (mime_fwd_any && 
     }
 
     if (mime_fwd_any && 

-       (last = copy_problematic_attachments (fp, last, idx, idxlen, mime_fwd_all)) == NULL)
+       copy_problematic_attachments (fp, last, idx, idxlen, mime_fwd_all) == NULL)

       goto bail;
   }
   
   mutt_forward_trailer (tmpfp);
   
       goto bail;
   }
   
   mutt_forward_trailer (tmpfp);
   

-  fclose (tmpfp);
+  safe_fclose (&tmpfp);

   tmpfp = NULL;
 
   /* now that we have the template, send it. */
   tmpfp = NULL;
 
   /* now that we have the template, send it. */


@@ -526,7 +554,7 @@ _("Can't decode all tagged attachments.  MIME-forward the others?"))) == -1)
   
   if (tmpfp)
   {
   
   if (tmpfp)
   {

-    fclose (tmpfp);
+    safe_fclose (&tmpfp);

     mutt_unlink (tmpbody);
   }
 
     mutt_unlink (tmpbody);
   }
 


@@ -631,7 +659,7 @@ static void attach_forward_msgs (FILE * fp, HEADER * hdr,
        }
       }
     }
        }
       }
     }

-    fclose (tmpfp);
+    safe_fclose (&tmpfp);

   }
   else if (rc == M_YES)        /* do MIME encapsulation - we don't need to do much here */
   {
   }
   else if (rc == M_YES)        /* do MIME encapsulation - we don't need to do much here */
   {


@@ -907,12 +935,12 @@ void mutt_attach_reply (FILE * fp, HEADER * hdr,
        copy_problematic_attachments (fp, &tmphdr->content, idx, idxlen, 0) == NULL)
     {
       mutt_free_header (&tmphdr);
        copy_problematic_attachments (fp, &tmphdr->content, idx, idxlen, 0) == NULL)
     {
       mutt_free_header (&tmphdr);

-      fclose (tmpfp);
+      safe_fclose (&tmpfp);

       return;
     }
   }
 
       return;
     }
   }
 

-  fclose (tmpfp);
+  safe_fclose (&tmpfp);

   
   if (ci_send_message (flags, tmphdr, tmpbody, NULL, parent) == 0)
     mutt_set_flag (Context, hdr, M_REPLIED, 1);
   
   if (ci_send_message (flags, tmphdr, tmpbody, NULL, parent) == 0)
     mutt_set_flag (Context, hdr, M_REPLIED, 1);