ooops, make it compile again.
[apps/madmutt.git] / crypt.c
diff --git a/crypt.c b/crypt.c
index 5cc2cd8..c0c64ad 100644 (file)
--- a/crypt.c
+++ b/crypt.c
@@ -1,33 +1,37 @@
 /*
+ * Copyright notice from original mutt:
  * Copyright (C) 1996,1997 Michael R. Elkins <me@mutt.org>
  * Copyright (C) 1999-2000 Thomas Roessler <roessler@does-not-exist.org>
  * Copyright (C) 2001  Thomas Roessler <roessler@does-not-exist.org>
  *                     Oliver Ehli <elmy@acm.org>
  * Copyright (C) 2003  Werner Koch <wk@gnupg.org>
+ * Copyright (C) 2004 g10code GmbH
  *
- *     This program is free software; you can redistribute it and/or modify
- *     it under the terms of the GNU General Public License as published by
- *     the Free Software Foundation; either version 2 of the License, or
- *     (at your option) any later version.
- * 
- *     This program is distributed in the hope that it will be useful,
- *     but WITHOUT ANY WARRANTY; without even the implied warranty of
- *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *     GNU General Public License for more details.
- * 
- *     You should have received a copy of the GNU General Public License
- *     along with this program; if not, write to the Free Software
- *     Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
+ * This file is part of mutt-ng, see http://www.muttng.org/.
+ * It's licensed under the GNU General Public License,
+ * please see the file GPL in the top level source directory.
  */
 
+#if HAVE_CONFIG_H
+# include "config.h"
+#endif
+
+#include <lib-lib/str.h>
+#include <lib-lib/file.h>
+#include <lib-lib/ascii.h>
+#include <lib-lib/mem.h>
+#include <lib-lib/macros.h>
+
+#include <lib-mime/mime.h>
 
 #include "mutt.h"
+#include "handler.h"
 #include "mutt_curses.h"
-#include "mime.h"
 #include "copy.h"
 #include "mutt_crypt.h"
 #include "pgp.h"
 
+
 #include <sys/wait.h>
 #include <string.h>
 #include <stdlib.h>
@@ -50,7 +54,7 @@
 
 
 /* print the current time to avoid spoofing of the signature output */
-void crypt_current_time(STATE *s, char *app_name)
+void crypt_current_time (STATE * s, const char *app_name)
 {
   time_t t;
   char p[STRING], tmp[STRING];
@@ -58,9 +62,8 @@ void crypt_current_time(STATE *s, char *app_name)
   if (!WithCrypto)
     return;
 
-  if (option (OPTCRYPTTIMESTAMP))
-  {
-    t = time(NULL);
+  if (option (OPTCRYPTTIMESTAMP)) {
+    t = time (NULL);
     setlocale (LC_TIME, "");
     strftime (p, sizeof (p), _(" (current time: %c)"), localtime (&t));
     setlocale (LC_TIME, "C");
@@ -68,7 +71,8 @@ void crypt_current_time(STATE *s, char *app_name)
   else
     *p = '\0';
 
-  snprintf (tmp, sizeof (tmp), _("[-- %s output follows%s --]\n"), NONULL(app_name), p);
+  snprintf (tmp, sizeof (tmp), _("[-- %s output follows%s --]\n"),
+            NONULL (app_name), p);
   state_attach_puts (tmp, s);
 }
 
@@ -91,11 +95,10 @@ void crypt_forget_passphrase (void)
 
 static void disable_coredumps (void)
 {
-  struct rlimit rl = {0, 0};
+  struct rlimit rl = { 0, 0 };
   static short done = 0;
 
-  if (!done)
-  {
+  if (!done) {
     setrlimit (RLIMIT_CORE, &rl);
     done = 1;
   }
@@ -104,68 +107,31 @@ static void disable_coredumps (void)
 #endif /* HAVE_SETRLIMIT */
 
 
-int crypt_valid_passphrase(int flags)
+int crypt_valid_passphrase (int flags)
 {
-  time_t now = time (NULL);
+  int ret = 0;
 
 # if defined(HAVE_SETRLIMIT) &&(!defined(DEBUG))
   disable_coredumps ();
 # endif
 
   if ((WithCrypto & APPLICATION_PGP) && (flags & APPLICATION_PGP))
-  {
-    extern char PgpPass[STRING];
-    extern time_t PgpExptime;
-
-    if (pgp_use_gpg_agent())
-    {
-      *PgpPass = 0;
-      return 1; /* handled by gpg-agent */
-    }
-
-    if (now < PgpExptime) return 1; /* just use the cached copy. */
-    crypt_pgp_void_passphrase ();
-      
-    if (mutt_get_password (_("Enter PGP passphrase:"),
-                           PgpPass, sizeof (PgpPass)) == 0)
-    {
-      PgpExptime = time (NULL) + PgpTimeout;
-      return (1);
-    }
-    else
-      PgpExptime = 0;
-    }
+    ret = crypt_pgp_valid_passphrase ();
 
   if ((WithCrypto & APPLICATION_SMIME) && (flags & APPLICATION_SMIME))
-  {
-    extern char SmimePass[STRING];
-    extern time_t SmimeExptime;
-
-    if (now < SmimeExptime) return (1);
-    crypt_smime_void_passphrase ();
-      
-    if (mutt_get_password (_("Enter SMIME passphrase:"), SmimePass,
-                          sizeof (SmimePass)) == 0)
-    {
-      SmimeExptime = time (NULL) + SmimeTimeout;
-      return (1);
-    }
-    else
-      SmimeExptime = 0;
-  }
+    ret = crypt_smime_valid_passphrase ();
 
-  return (0);
+  return ret;
 }
 
 
 
-int mutt_protect (HEADER *msg, HEADER *cur, char *keylist)
+int mutt_protect (HEADER * msg, char *keylist)
 {
   BODY *pbody = NULL, *tmp_pbody = NULL;
   BODY *tmp_smime_pbody = NULL;
   BODY *tmp_pgp_pbody = NULL;
-  int traditional = 0;
-  int flags = (WithCrypto & APPLICATION_PGP)? msg->security: 0;
+  int flags = (WithCrypto & APPLICATION_PGP) ? msg->security : 0;
   int i;
 
   if (!WithCrypto)
@@ -174,59 +140,51 @@ int mutt_protect (HEADER *msg, HEADER *cur, char *keylist)
   if ((msg->security & SIGN) && !crypt_valid_passphrase (msg->security))
     return (-1);
 
-  if ((WithCrypto & APPLICATION_PGP) && (msg->security & APPLICATION_PGP))
-  {
-    if ((msg->content->type == TYPETEXT) &&
-       !ascii_strcasecmp (msg->content->subtype, "plain"))
-    {
-      if (cur && cur->security && option (OPTPGPAUTOTRAD)
-         && (option (OPTCRYPTREPLYENCRYPT)
-             || option (OPTCRYPTREPLYSIGN)
-             || option (OPTCRYPTREPLYSIGNENCRYPTED)))
-       {
-         if(mutt_is_application_pgp(cur->content))
-           traditional = 1;
-       }
-      else
-       {
-         if ((i = query_quadoption (OPT_PGPTRADITIONAL, _("Create a traditional (inline) PGP message?"))) == -1)
-           return -1;
-         else if (i == M_YES)
-           traditional = 1;
-       }
-    }
-    if (traditional)
-    {
-      if (!isendwin ()) mutt_endwin _("Invoking PGP...");
-      if (!(pbody = crypt_pgp_traditional_encryptsign (msg->content, flags, keylist)))
-       return -1;
+  if ((WithCrypto & APPLICATION_PGP)
+      && ((msg->security & PGPINLINE) == PGPINLINE)) {
+    /* they really want to send it inline... go for it */
+    if (!isendwin ())
+      mutt_endwin _("Invoking PGP...");
 
+    pbody = crypt_pgp_traditional_encryptsign (msg->content, flags, keylist);
+    if (pbody) {
       msg->content = pbody;
       return 0;
     }
+
+    /* otherwise inline won't work...ask for revert */
+    if ((i =
+         query_quadoption (OPT_PGPMIMEAUTO,
+                           _
+                           ("Message can't be sent inline.  Revert to using PGP/MIME?")))
+        != M_YES) {
+      mutt_error _("Mail not sent.");
+
+      return -1;
+    }
+
+    /* go ahead with PGP/MIME */
   }
 
-  if (!isendwin ()) mutt_endwin (NULL);
+  if (!isendwin ())
+    mutt_endwin (NULL);
 
   if ((WithCrypto & APPLICATION_SMIME))
     tmp_smime_pbody = msg->content;
   if ((WithCrypto & APPLICATION_PGP))
-    tmp_pgp_pbody   = msg->content;
+    tmp_pgp_pbody = msg->content;
 
-  if (msg->security & SIGN)
-  {
+  if (msg->security & SIGN) {
     if ((WithCrypto & APPLICATION_SMIME)
-        && (msg->security & APPLICATION_SMIME))
-    {
+        && (msg->security & APPLICATION_SMIME)) {
       if (!(tmp_pbody = crypt_smime_sign_message (msg->content)))
-       return -1;
+        return -1;
       pbody = tmp_smime_pbody = tmp_pbody;
     }
 
     if ((WithCrypto & APPLICATION_PGP)
         && (msg->security & APPLICATION_PGP)
-        && (!(flags & ENCRYPT) || option (OPTPGPRETAINABLESIG)))
-    {
+        && (!(flags & ENCRYPT) || option (OPTPGPRETAINABLESIG))) {
       if (!(tmp_pbody = crypt_pgp_sign_message (msg->content)))
         return -1;
 
@@ -234,86 +192,77 @@ int mutt_protect (HEADER *msg, HEADER *cur, char *keylist)
       pbody = tmp_pgp_pbody = tmp_pbody;
     }
 
-    if (WithCrypto
-        && (msg->security & APPLICATION_SMIME)
-       && (msg->security & APPLICATION_PGP))
-    {
-       /* here comes the draft ;-) */
+    if (WithCrypto && (msg->security & APPLICATION_SMIME)
+        && (msg->security & APPLICATION_PGP)) {
+      /* here comes the draft ;-) */
     }
   }
 
 
-  if (msg->security & ENCRYPT)
-  {
+  if (msg->security & ENCRYPT) {
     if ((WithCrypto & APPLICATION_SMIME)
-        && (msg->security & APPLICATION_SMIME))
-    {
+        && (msg->security & APPLICATION_SMIME)) {
       if (!(tmp_pbody = crypt_smime_build_smime_entity (tmp_smime_pbody,
-                                                        keylist)))
-      {
-       /* signed ? free it! */
-       return (-1);
+                                                        keylist))) {
+        /* signed ? free it! */
+        return (-1);
       }
       /* free tmp_body if messages was signed AND encrypted ... */
-      if (tmp_smime_pbody != msg->content && tmp_smime_pbody != tmp_pbody)
-      {
-       /* detatch and dont't delete msg->content,
-          which tmp_smime_pbody->parts after signing. */
-       tmp_smime_pbody->parts = tmp_smime_pbody->parts->next;
-       msg->content->next = NULL;
-       mutt_free_body (&tmp_smime_pbody);
+      if (tmp_smime_pbody != msg->content && tmp_smime_pbody != tmp_pbody) {
+        /* detatch and dont't delete msg->content,
+           which tmp_smime_pbody->parts after signing. */
+        tmp_smime_pbody->parts = tmp_smime_pbody->parts->next;
+        msg->content->next = NULL;
+        mutt_free_body (&tmp_smime_pbody);
       }
       pbody = tmp_pbody;
     }
 
     if ((WithCrypto & APPLICATION_PGP)
-        && (msg->security & APPLICATION_PGP))
-    {
+        && (msg->security & APPLICATION_PGP)) {
       if (!(pbody = crypt_pgp_encrypt_message (tmp_pgp_pbody, keylist,
-                                               flags & SIGN)))
-      {
-
-       /* did we perform a retainable signature? */
-       if (flags != msg->security)
-       {
-         /* remove the outer multipart layer */
-         tmp_pgp_pbody = mutt_remove_multipart (tmp_pgp_pbody);
-         /* get rid of the signature */
-         mutt_free_body (&tmp_pgp_pbody->next);
-       }
-
-       return (-1);
+                                               flags & SIGN))) {
+
+        /* did we perform a retainable signature? */
+        if (flags != msg->security) {
+          /* remove the outer multipart layer */
+          tmp_pgp_pbody = mutt_remove_multipart (tmp_pgp_pbody);
+          /* get rid of the signature */
+          mutt_free_body (&tmp_pgp_pbody->next);
+        }
+
+        return (-1);
       }
 
       /* destroy temporary signature envelope when doing retainable 
        * signatures.
+
        */
-      if (flags != msg->security)
-      {
-       tmp_pgp_pbody = mutt_remove_multipart (tmp_pgp_pbody);
-       mutt_free_body (&tmp_pgp_pbody->next);
+      if (flags != msg->security) {
+        tmp_pgp_pbody = mutt_remove_multipart (tmp_pgp_pbody);
+        mutt_free_body (&tmp_pgp_pbody->next);
       }
     }
   }
 
-  if(pbody)
-      msg->content = pbody;
+  if (pbody)
+    msg->content = pbody;
 
   return 0;
 }
 
 
-   
-     
-int mutt_is_multipart_signed (BODY *b)
+
+
+int mutt_is_multipart_signed (BODY * b)
 {
   char *p;
 
   if (!b || !(b->type == TYPEMULTIPART) ||
-      !b->subtype || ascii_strcasecmp(b->subtype, "signed"))
+      !b->subtype || ascii_strcasecmp (b->subtype, "signed"))
     return 0;
 
-  if (!(p = mutt_get_parameter("protocol", b->parameter)))
+  if (!(p = mutt_get_parameter ("protocol", b->parameter)))
     return 0;
 
   if (!(ascii_strcasecmp (p, "multipart/mixed")))
@@ -322,7 +271,7 @@ int mutt_is_multipart_signed (BODY *b)
   if ((WithCrypto & APPLICATION_PGP)
       && !(ascii_strcasecmp (p, "application/pgp-signature")))
     return PGPSIGN;
-    
+
   if ((WithCrypto & APPLICATION_SMIME)
       && !(ascii_strcasecmp (p, "application/x-pkcs7-signature")))
     return SMIMESIGN;
@@ -332,45 +281,45 @@ int mutt_is_multipart_signed (BODY *b)
 
   return 0;
 }
-   
-     
-int mutt_is_multipart_encrypted (BODY *b)
+
+
+int mutt_is_multipart_encrypted (BODY * b)
 {
-  if ((WithCrypto & APPLICATION_PGP))
-  {
+  if ((WithCrypto & APPLICATION_PGP)) {
     char *p;
-  
+
     if (!b || b->type != TYPEMULTIPART ||
         !b->subtype || ascii_strcasecmp (b->subtype, "encrypted") ||
         !(p = mutt_get_parameter ("protocol", b->parameter)) ||
         ascii_strcasecmp (p, "application/pgp-encrypted"))
       return 0;
-  
-     return PGPENCRYPT;
+
+    return PGPENCRYPT;
   }
 
   return 0;
 }
 
 
-int mutt_is_application_pgp (BODY *m)
+int mutt_is_application_pgp (BODY * m)
 {
   int t = 0;
   char *p;
-  
-  if (m->type == TYPEAPPLICATION)
-  {
-    if (!ascii_strcasecmp (m->subtype, "pgp") || !ascii_strcasecmp (m->subtype, "x-pgp-message"))
-    {
+
+  if (m->type == TYPEAPPLICATION) {
+    if (!ascii_strcasecmp (m->subtype, "pgp")
+        || !ascii_strcasecmp (m->subtype, "x-pgp-message")) {
       if ((p = mutt_get_parameter ("x-action", m->parameter))
-         && (!ascii_strcasecmp (p, "sign") || !ascii_strcasecmp (p, "signclear")))
-       t |= PGPSIGN;
+          && (!ascii_strcasecmp (p, "sign")
+              || !ascii_strcasecmp (p, "signclear")))
+        t |= PGPSIGN;
 
-      if ((p = mutt_get_parameter ("format", m->parameter)) && 
-         !ascii_strcasecmp (p, "keys-only"))
-       t |= PGPKEY;
+      if ((p = mutt_get_parameter ("format", m->parameter)) &&
+          !ascii_strcasecmp (p, "keys-only"))
+        t |= PGPKEY;
 
-      if(!t) t |= PGPENCRYPT;  /* not necessarily correct, but... */
+      if (!t)
+        t |= PGPENCRYPT;        /* not necessarily correct, but... */
     }
 
     if (!ascii_strcasecmp (m->subtype, "pgp-signed"))
@@ -379,46 +328,49 @@ int mutt_is_application_pgp (BODY *m)
     if (!ascii_strcasecmp (m->subtype, "pgp-keys"))
       t |= PGPKEY;
   }
-  else if (m->type == TYPETEXT && ascii_strcasecmp ("plain", m->subtype) == 0)
-  {
+  else if (m->type == TYPETEXT && ascii_strcasecmp ("plain", m->subtype) == 0) {
     if (((p = mutt_get_parameter ("x-mutt-action", m->parameter))
-        || (p = mutt_get_parameter ("x-action", m->parameter)) 
-        || (p = mutt_get_parameter ("action", m->parameter)))
-        && !ascii_strncasecmp ("pgp-sign", p, 8))
+         || (p = mutt_get_parameter ("x-action", m->parameter))
+         || (p = mutt_get_parameter ("action", m->parameter)))
+        && !ascii_strncasecmp ("pgp-sign", p, 8))
       t |= PGPSIGN;
     else if (p && !ascii_strncasecmp ("pgp-encrypt", p, 11))
       t |= PGPENCRYPT;
     else if (p && !ascii_strncasecmp ("pgp-keys", p, 7))
       t |= PGPKEY;
   }
+  if (t)
+    t |= PGPINLINE;
+
   return t;
 }
 
-int mutt_is_application_smime (BODY *m)
+int mutt_is_application_smime (BODY * m)
 {
-  char *t=NULL;
-  int len, complain=0;
+  char *t = NULL;
+  int len, complain = 0;
+
+  if (!m)
+    return 0;
 
-  if ((m->type & TYPEAPPLICATION) && m->subtype)
-  {
+  if ((m->type & TYPEAPPLICATION) && m->subtype) {
     /* S/MIME MIME types don't need x- anymore, see RFC2311 */
     if (!ascii_strcasecmp (m->subtype, "x-pkcs7-mime") ||
-       !ascii_strcasecmp (m->subtype, "pkcs7-mime"))
-    {
-      if ((t = mutt_get_parameter ("smime-type", m->parameter)))
-      {
-       if (!ascii_strcasecmp (t, "enveloped-data"))
-         return SMIMEENCRYPT;
-       else if (!ascii_strcasecmp (t, "signed-data"))
-         return (SMIMESIGN|SMIMEOPAQUE);
-       else return 0;
+        !ascii_strcasecmp (m->subtype, "pkcs7-mime")) {
+      if ((t = mutt_get_parameter ("smime-type", m->parameter))) {
+        if (!ascii_strcasecmp (t, "enveloped-data"))
+          return SMIMEENCRYPT;
+        else if (!ascii_strcasecmp (t, "signed-data"))
+          return (SMIMESIGN | SMIMEOPAQUE);
+        else
+          return 0;
       }
       /* Netscape 4.7 uses 
        * Content-Description: S/MIME Encrypted Message
        * instead of Content-Type parameter
        */
       if (!ascii_strcasecmp (m->description, "S/MIME Encrypted Message"))
-       return SMIMEENCRYPT;
+        return SMIMEENCRYPT;
       complain = 1;
     }
     else if (ascii_strcasecmp (m->subtype, "octet-stream"))
@@ -426,31 +378,32 @@ int mutt_is_application_smime (BODY *m)
 
     t = mutt_get_parameter ("name", m->parameter);
 
-    if (!t) t = m->d_filename;
-    if (!t) t = m->filename;
-    if (!t) 
-    {
+    if (!t)
+      t = m->d_filename;
+    if (!t)
+      t = m->filename;
+    if (!t) {
       if (complain)
-       mutt_message (_("S/MIME messages with no hints on content are unsupported."));
+        mutt_message (_
+                      ("S/MIME messages with no hints on content are unsupported."));
       return 0;
     }
 
     /* no .p7c, .p10 support yet. */
 
-    len = mutt_strlen (t) - 4;
-    if (len > 0 && *(t+len) == '.')
-    {
+    len = m_strlen(t) - 4;
+    if (len > 0 && *(t + len) == '.') {
       len++;
-      if (!ascii_strcasecmp ((t+len), "p7m"))
+      if (!ascii_strcasecmp ((t + len), "p7m"))
 #if 0
-       return SMIMEENCRYPT;
+        return SMIMEENCRYPT;
 #else
-      /* Not sure if this is the correct thing to do, but 
-         it's required for compatibility with Outlook */
-       return (SMIMESIGN|SMIMEOPAQUE);
+        /* Not sure if this is the correct thing to do, but 
+           it's required for compatibility with Outlook */
+        return (SMIMESIGN | SMIMEOPAQUE);
 #endif
-      else if (!ascii_strcasecmp ((t+len), "p7s"))
-       return (SMIMESIGN|SMIMEOPAQUE);
+      else if (!ascii_strcasecmp ((t + len), "p7s"))
+        return (SMIMESIGN | SMIMEOPAQUE);
     }
   }
 
@@ -462,59 +415,56 @@ int mutt_is_application_smime (BODY *m)
 
 
 
-int crypt_query (BODY *m)
+int crypt_query (BODY * m)
 {
   int t = 0;
 
   if (!WithCrypto)
     return 0;
-  
+
   if (!m)
     return 0;
 
-  if (m->type == TYPEAPPLICATION)
-  {
+  if (m->type == TYPEAPPLICATION) {
     if ((WithCrypto & APPLICATION_PGP))
-      t |= mutt_is_application_pgp(m);
-
-    if ((WithCrypto & APPLICATION_SMIME))
-    {
-      t |= mutt_is_application_smime(m);
-      if (t && m->goodsig) t |= GOODSIGN;
-      if (t && m->badsig) t |= BADSIGN;
+      t |= mutt_is_application_pgp (m);
+
+    if ((WithCrypto & APPLICATION_SMIME)) {
+      t |= mutt_is_application_smime (m);
+      if (t && m->goodsig)
+        t |= GOODSIGN;
+      if (t && m->badsig)
+        t |= BADSIGN;
     }
   }
-  else if ((WithCrypto & APPLICATION_PGP) && m->type == TYPETEXT)
-  {
+  else if ((WithCrypto & APPLICATION_PGP) && m->type == TYPETEXT) {
     t |= mutt_is_application_pgp (m);
     if (t && m->goodsig)
       t |= GOODSIGN;
   }
-  
-  if (m->type == TYPEMULTIPART)
-  {
-    t |= mutt_is_multipart_encrypted(m);
+
+  if (m->type == TYPEMULTIPART) {
+    t |= mutt_is_multipart_encrypted (m);
     t |= mutt_is_multipart_signed (m);
 
-    if (t && m->goodsig) 
+    if (t && m->goodsig)
       t |= GOODSIGN;
   }
 
-  if (m->type == TYPEMULTIPART || m->type == TYPEMESSAGE)
-  {
+  if (m->type == TYPEMULTIPART || m->type == TYPEMESSAGE) {
     BODY *p;
     int u, v, w;
-    
-    u = m->parts ? 0xffffffff : 0;     /* Bits set in all parts */
-    w = 0;                             /* Bits set in any part  */
-    for (p = m->parts; p; p = p->next)
-    {
-      v  = crypt_query (p);
-      u &= v; w |= v;
+
+    u = m->parts ? 0xffffffff : 0;      /* Bits set in all parts */
+    w = 0;                      /* Bits set in any part  */
+
+    for (p = m->parts; p; p = p->next) {
+      v = crypt_query (p);
+      u &= v;
+      w |= v;
     }
     t |= u | (w & ~GOODSIGN);
-    
+
     if ((w & GOODSIGN) && !(u & GOODSIGN))
       t |= PARTSIGN;
   }
@@ -525,7 +475,7 @@ int crypt_query (BODY *m)
 
 
 
-int crypt_write_signed(BODY *a, STATE *s, const char *tempfile)
+int crypt_write_signed (BODY * a, STATE * s, const char *tempfile)
 {
   FILE *fp;
   int c;
@@ -535,34 +485,31 @@ int crypt_write_signed(BODY *a, STATE *s, const char *tempfile)
   if (!WithCrypto)
     return -1;
 
-  if (!(fp = safe_fopen (tempfile, "w")))
-  {
+  if (!(fp = safe_fopen (tempfile, "w"))) {
     mutt_perror (tempfile);
     return -1;
   }
-      
-  fseek (s->fpin, a->hdr_offset, 0);
+
+  fseeko (s->fpin, a->hdr_offset, 0);
   bytes = a->length + a->offset - a->hdr_offset;
   hadcr = 0;
-  while (bytes > 0)
-  {
+  while (bytes > 0) {
     if ((c = fgetc (s->fpin)) == EOF)
       break;
-    
+
     bytes--;
-    
-    if  (c == '\r')
+
+    if (c == '\r')
       hadcr = 1;
-    else 
-    {
+    else {
       if (c == '\n' && !hadcr)
-       fputc ('\r', fp);
-      
+        fputc ('\r', fp);
+
       hadcr = 0;
     }
-    
+
     fputc (c, fp);
-    
+
   }
   fclose (fp);
 
@@ -571,36 +518,32 @@ int crypt_write_signed(BODY *a, STATE *s, const char *tempfile)
 
 
 
-void convert_to_7bit (BODY *a)
+void convert_to_7bit (BODY * a)
 {
   if (!WithCrypto)
     return;
 
-  while (a)
-  {
-    if (a->type == TYPEMULTIPART)
-    {
-      if (a->encoding != ENC7BIT)
-      {
+  while (a) {
+    if (a->type == TYPEMULTIPART) {
+      if (a->encoding != ENC7BIT) {
         a->encoding = ENC7BIT;
-       convert_to_7bit(a->parts);
+        convert_to_7bit (a->parts);
       }
       else if ((WithCrypto & APPLICATION_PGP) && option (OPTPGPSTRICTENC))
-       convert_to_7bit (a->parts);
-    } 
+        convert_to_7bit (a->parts);
+    }
     else if (a->type == TYPEMESSAGE &&
-            mutt_strcasecmp(a->subtype, "delivery-status"))
-    {
-      if(a->encoding != ENC7BIT)
-       mutt_message_to_7bit (a, NULL);
+             m_strcasecmp(a->subtype, "delivery-status")) {
+      if (a->encoding != ENC7BIT)
+        mutt_message_to_7bit (a, NULL);
     }
     else if (a->encoding == ENC8BIT)
       a->encoding = ENCQUOTEDPRINTABLE;
     else if (a->encoding == ENCBINARY)
       a->encoding = ENCBASE64;
     else if (a->content && a->encoding != ENCBASE64 &&
-            (a->content->from || (a->content->space && 
-                                  option (OPTPGPSTRICTENC))))
+             (a->content->from || (a->content->space &&
+                                   option (OPTPGPSTRICTENC))))
       a->encoding = ENCQUOTEDPRINTABLE;
     a = a->next;
   }
@@ -613,15 +556,14 @@ void crypt_extract_keys_from_messages (HEADER * h)
 {
   int i;
   char tempfname[_POSIX_PATH_MAX], *mbox;
-  ADDRESS *tmp = NULL;
+  address_t *tmp = NULL;
   FILE *fpout;
 
   if (!WithCrypto)
     return;
 
   mutt_mktemp (tempfname);
-  if (!(fpout = safe_fopen (tempfname, "w")))
-  {
+  if (!(fpout = safe_fopen (tempfname, "w"))) {
     mutt_perror (tempfname);
     return;
   }
@@ -629,100 +571,90 @@ void crypt_extract_keys_from_messages (HEADER * h)
   if ((WithCrypto & APPLICATION_PGP))
     set_option (OPTDONTHANDLEPGPKEYS);
 
-  if (!h)
-  {
-    for (i = 0; i < Context->vcount; i++)
-    {
-      if (Context->hdrs[Context->v2r[i]]->tagged)
-      {
-       mutt_parse_mime_message (Context, Context->hdrs[Context->v2r[i]]);
-       if (Context->hdrs[Context->v2r[i]]->security & ENCRYPT &&
-           !crypt_valid_passphrase (Context->hdrs[Context->v2r[i]]->security))
-       {
-         fclose (fpout);
-         break;
-       }
-
-       if ((WithCrypto & APPLICATION_PGP)
-            && (Context->hdrs[Context->v2r[i]]->security & APPLICATION_PGP))
-       {
-         mutt_copy_message (fpout, Context, Context->hdrs[Context->v2r[i]], 
-                            M_CM_DECODE|M_CM_CHARCONV, 0);
-         fflush(fpout);
-         
-         mutt_endwin (_("Trying to extract PGP keys...\n"));
-         crypt_pgp_invoke_import (tempfname);
-       }
-
-       if ((WithCrypto & APPLICATION_SMIME)
-            && (Context->hdrs[Context->v2r[i]]->security & APPLICATION_SMIME))
-       {
-         if (Context->hdrs[Context->v2r[i]]->security & ENCRYPT)
-           mutt_copy_message (fpout, Context, Context->hdrs[Context->v2r[i]],
-                              M_CM_NOHEADER|M_CM_DECODE_CRYPT
-                               |M_CM_DECODE_SMIME, 0);
-         else
-           mutt_copy_message (fpout, Context,
-                              Context->hdrs[Context->v2r[i]], 0, 0);
-         fflush(fpout);
+  if (!h) {
+    for (i = 0; i < Context->vcount; i++) {
+      if (Context->hdrs[Context->v2r[i]]->tagged) {
+        mutt_parse_mime_message (Context, Context->hdrs[Context->v2r[i]]);
+        if (Context->hdrs[Context->v2r[i]]->security & ENCRYPT &&
+            !crypt_valid_passphrase (Context->hdrs[Context->v2r[i]]->
+                                     security)) {
+          fclose (fpout);
+          break;
+        }
+
+        if ((WithCrypto & APPLICATION_PGP)
+            && (Context->hdrs[Context->v2r[i]]->security & APPLICATION_PGP)) {
+          mutt_copy_message (fpout, Context, Context->hdrs[Context->v2r[i]],
+                             M_CM_DECODE | M_CM_CHARCONV, 0);
+          fflush (fpout);
+
+          mutt_endwin (_("Trying to extract PGP keys...\n"));
+          crypt_pgp_invoke_import (tempfname);
+        }
+
+        if ((WithCrypto & APPLICATION_SMIME)
+            && (Context->hdrs[Context->v2r[i]]->security & APPLICATION_SMIME)) {
+          if (Context->hdrs[Context->v2r[i]]->security & ENCRYPT)
+            mutt_copy_message (fpout, Context, Context->hdrs[Context->v2r[i]],
+                               M_CM_NOHEADER | M_CM_DECODE_CRYPT
+                               | M_CM_DECODE_SMIME, 0);
+          else
+            mutt_copy_message (fpout, Context,
+                               Context->hdrs[Context->v2r[i]], 0, 0);
+          fflush (fpout);
 
           if (Context->hdrs[Context->v2r[i]]->env->from)
-           tmp = mutt_expand_aliases (h->env->from);
-         else if (Context->hdrs[Context->v2r[i]]->env->sender)
-           tmp = mutt_expand_aliases (Context->hdrs[Context->v2r[i]]
-                                                    ->env->sender);
+            tmp = mutt_expand_aliases (h->env->from);
+          else if (Context->hdrs[Context->v2r[i]]->env->sender)
+            tmp = mutt_expand_aliases (Context->hdrs[Context->v2r[i]]
+                                       ->env->sender);
           mbox = tmp ? tmp->mailbox : NULL;
-         if (mbox)
-         {
-           mutt_endwin (_("Trying to extract S/MIME certificates...\n"));
-           crypt_smime_invoke_import (tempfname, mbox);
-           tmp = NULL;
-         }
-       }
-
-       rewind (fpout);
+          if (mbox) {
+            mutt_endwin (_("Trying to extract S/MIME certificates...\n"));
+            crypt_smime_invoke_import (tempfname, mbox);
+            tmp = NULL;
+          }
+        }
+
+        rewind (fpout);
       }
     }
   }
-  else
-  {
+  else {
     mutt_parse_mime_message (Context, h);
-    if (!(h->security & ENCRYPT && !crypt_valid_passphrase (h->security)))
-    {
+    if (!(h->security & ENCRYPT && !crypt_valid_passphrase (h->security))) {
       if ((WithCrypto & APPLICATION_PGP)
-          && (h->security & APPLICATION_PGP))
-      {
-       mutt_copy_message (fpout, Context, h, M_CM_DECODE|M_CM_CHARCONV, 0);
-       fflush(fpout);
-       mutt_endwin (_("Trying to extract PGP keys...\n"));
-       crypt_pgp_invoke_import (tempfname);
+          && (h->security & APPLICATION_PGP)) {
+        mutt_copy_message (fpout, Context, h, M_CM_DECODE | M_CM_CHARCONV, 0);
+        fflush (fpout);
+        mutt_endwin (_("Trying to extract PGP keys...\n"));
+        crypt_pgp_invoke_import (tempfname);
       }
 
       if ((WithCrypto & APPLICATION_SMIME)
-          && (h->security & APPLICATION_SMIME))
-      {
-       if (h->security & ENCRYPT)
-         mutt_copy_message (fpout, Context, h, M_CM_NOHEADER
-                                                |M_CM_DECODE_CRYPT
-                                                |M_CM_DECODE_SMIME, 0);
-       else
-         mutt_copy_message (fpout, Context, h, 0, 0);
-
-       fflush(fpout);
-       if (h->env->from) tmp = mutt_expand_aliases (h->env->from);
-       else if (h->env->sender)  tmp = mutt_expand_aliases (h->env->sender); 
-       mbox = tmp ? tmp->mailbox : NULL;
-       if (mbox) /* else ? */
-       {
-         mutt_message (_("Trying to extract S/MIME certificates...\n"));
-         crypt_smime_invoke_import (tempfname, mbox);
-       }
+          && (h->security & APPLICATION_SMIME)) {
+        if (h->security & ENCRYPT)
+          mutt_copy_message (fpout, Context, h, M_CM_NOHEADER
+                             | M_CM_DECODE_CRYPT | M_CM_DECODE_SMIME, 0);
+        else
+          mutt_copy_message (fpout, Context, h, 0, 0);
+
+        fflush (fpout);
+        if (h->env->from)
+          tmp = mutt_expand_aliases (h->env->from);
+        else if (h->env->sender)
+          tmp = mutt_expand_aliases (h->env->sender);
+        mbox = tmp ? tmp->mailbox : NULL;
+        if (mbox) {             /* else ? */
+          mutt_message (_("Trying to extract S/MIME certificates...\n"));
+          crypt_smime_invoke_import (tempfname, mbox);
+        }
       }
     }
   }
-      
+
   fclose (fpout);
-  if (isendwin())
+  if (isendwin ())
     mutt_any_key_to_continue (NULL);
 
   mutt_unlink (tempfname);
@@ -733,7 +665,7 @@ void crypt_extract_keys_from_messages (HEADER * h)
 
 
 
-int crypt_get_keys (HEADER *msg, char **keylist)
+int crypt_get_keys (HEADER * msg, char **keylist)
 {
   /* Do a quick check to make sure that we can find all of the encryption
    * keys if the user has requested this service.
@@ -747,43 +679,38 @@ int crypt_get_keys (HEADER *msg, char **keylist)
 
   *keylist = NULL;
 
-  if (msg->security & ENCRYPT)
-  {
-     if ((WithCrypto & APPLICATION_PGP)
-         && (msg->security & APPLICATION_PGP))
-     {
-       if ((*keylist = crypt_pgp_findkeys (msg->env->to, msg->env->cc,
-                              msg->env->bcc)) == NULL)
-           return (-1);
-       unset_option (OPTPGPCHECKTRUST);
-     }
-     if ((WithCrypto & APPLICATION_SMIME)
-         && (msg->security & APPLICATION_SMIME))
-     {
-       if ((*keylist = crypt_smime_findkeys (msg->env->to, msg->env->cc,
-                                            msg->env->bcc)) == NULL)
-           return (-1);
-     }
+  if (msg->security & ENCRYPT) {
+    if ((WithCrypto & APPLICATION_PGP)
+        && (msg->security & APPLICATION_PGP)) {
+      if ((*keylist = crypt_pgp_findkeys (msg->env->to, msg->env->cc,
+                                          msg->env->bcc)) == NULL)
+        return (-1);
+      unset_option (OPTPGPCHECKTRUST);
+    }
+    if ((WithCrypto & APPLICATION_SMIME)
+        && (msg->security & APPLICATION_SMIME)) {
+      if ((*keylist = crypt_smime_findkeys (msg->env->to, msg->env->cc,
+                                            msg->env->bcc)) == NULL)
+        return (-1);
+    }
   }
-    
+
   return (0);
 }
 
 
 
-static void crypt_fetch_signatures (BODY ***signatures, BODY *a, int *n)
+static void crypt_fetch_signatures (BODY ***signatures, BODY * a, int *n)
 {
   if (!WithCrypto)
     return;
 
-  for (; a; a = a->next)
-  {
+  for (; a; a = a->next) {
     if (a->type == TYPEMULTIPART)
       crypt_fetch_signatures (signatures, a->parts, n);
-    else
-    {
-      if((*n % 5) == 0)
-       safe_realloc (signatures, (*n + 6) * sizeof (BODY **));
+    else {
+      if ((*n % 5) == 0)
+        p_realloc(signatures, *n + 6);
 
       (*signatures)[(*n)++] = a;
     }
@@ -795,133 +722,124 @@ static void crypt_fetch_signatures (BODY ***signatures, BODY *a, int *n)
  * This routine verifies a  "multipart/signed"  body.
  */
 
-void mutt_signed_handler (BODY *a, STATE *s)
+int mutt_signed_handler (BODY * a, STATE * s)
 {
   char tempfile[_POSIX_PATH_MAX];
   char *protocol;
   int protocol_major = TYPEOTHER;
   char *protocol_minor = NULL;
-  
+
   BODY *b = a;
   BODY **signatures = NULL;
   int sigcnt = 0;
   int i;
   short goodsig = 1;
+  int rc = 0;
 
   if (!WithCrypto)
-    return;
+    return (-1);
 
   protocol = mutt_get_parameter ("protocol", a->parameter);
   a = a->parts;
 
   /* extract the protocol information */
-  
-  if (protocol)
-  {
+
+  if (protocol) {
     char major[STRING];
     char *t;
 
-    if ((protocol_minor = strchr (protocol, '/'))) protocol_minor++;
-    
-    strfcpy (major, protocol, sizeof(major));
-    if((t = strchr(major, '/')))
+    if ((protocol_minor = strchr (protocol, '/')))
+      protocol_minor++;
+
+    m_strcpy(major, sizeof(major), protocol);
+    if ((t = strchr (major, '/')))
       *t = '\0';
-    
+
     protocol_major = mutt_check_mime_type (major);
   }
 
   /* consistency check */
 
-  if (!(a && a->next && a->next->type == protocol_major && 
-      !mutt_strcasecmp (a->next->subtype, protocol_minor)))
-  {
+  if (!(a && a->next && a->next->type == protocol_major &&
+        !m_strcasecmp(a->next->subtype, protocol_minor))) {
     state_attach_puts (_("[-- Error: "
                          "Inconsistent multipart/signed structure! --]\n\n"),
                        s);
-    mutt_body_handler (a, s);
-    return;
+    return mutt_body_handler (a, s);
   }
 
-  
+
   if ((WithCrypto & APPLICATION_PGP)
       && protocol_major == TYPEAPPLICATION
-      && !mutt_strcasecmp (protocol_minor, "pgp-signature"))
-    ;
+      && !m_strcasecmp(protocol_minor, "pgp-signature"));
   else if ((WithCrypto & APPLICATION_SMIME)
            && protocol_major == TYPEAPPLICATION
-          && !(mutt_strcasecmp (protocol_minor, "x-pkcs7-signature")
-              && mutt_strcasecmp (protocol_minor, "pkcs7-signature")))
-    ;
+           && !(m_strcasecmp(protocol_minor, "x-pkcs7-signature")
+                && m_strcasecmp(protocol_minor, "pkcs7-signature")));
   else if (protocol_major == TYPEMULTIPART
-          && !mutt_strcasecmp (protocol_minor, "mixed"))
-    ;
-  else
-  {
+           && !m_strcasecmp(protocol_minor, "mixed"));
+  else {
     state_printf (s, _("[-- Error: "
                        "Unknown multipart/signed protocol %s! --]\n\n"),
                   protocol);
-    mutt_body_handler (a, s);
-    return;
+    return mutt_body_handler (a, s);
   }
-  
-  if (s->flags & M_DISPLAY)
-  {
-    
+
+  if (s->flags & M_DISPLAY) {
+
     crypt_fetch_signatures (&signatures, a->next, &sigcnt);
-    
-    if (sigcnt)
-    {
+
+    if (sigcnt) {
       mutt_mktemp (tempfile);
-      if (crypt_write_signed (a, s, tempfile) == 0)
-      {
-       for (i = 0; i < sigcnt; i++)
-       {
-         if ((WithCrypto & APPLICATION_PGP)
-              && signatures[i]->type == TYPEAPPLICATION 
-             && !mutt_strcasecmp (signatures[i]->subtype, "pgp-signature"))
-         {
-           if (crypt_pgp_verify_one (signatures[i], s, tempfile) != 0)
-             goodsig = 0;
-           
-           continue;
-         }
-
-         if ((WithCrypto & APPLICATION_SMIME)
-              && signatures[i]->type == TYPEAPPLICATION 
-             && (!mutt_strcasecmp(signatures[i]->subtype, "x-pkcs7-signature")
-                 || !mutt_strcasecmp(signatures[i]->subtype, "pkcs7-signature")))
-         {
-           if (crypt_smime_verify_one (signatures[i], s, tempfile) != 0)
-             goodsig = 0;
-           
-           continue;
-         }
-
-         state_printf (s, _("[-- Warning: "
+      if (crypt_write_signed (a, s, tempfile) == 0) {
+        for (i = 0; i < sigcnt; i++) {
+          if ((WithCrypto & APPLICATION_PGP)
+              && signatures[i]->type == TYPEAPPLICATION
+              && !m_strcasecmp(signatures[i]->subtype, "pgp-signature")) {
+            if (crypt_pgp_verify_one (signatures[i], s, tempfile) != 0)
+              goodsig = 0;
+
+            continue;
+          }
+
+          if ((WithCrypto & APPLICATION_SMIME)
+              && signatures[i]->type == TYPEAPPLICATION
+              &&
+              (!m_strcasecmp(signatures[i]->subtype, "x-pkcs7-signature")
+               || !m_strcasecmp(signatures[i]->subtype,
+                                    "pkcs7-signature"))) {
+            if (crypt_smime_verify_one (signatures[i], s, tempfile) != 0)
+              goodsig = 0;
+
+            continue;
+          }
+
+          state_printf (s, _("[-- Warning: "
                              "We can't verify %s/%s signatures. --]\n\n"),
-                         TYPE(signatures[i]), signatures[i]->subtype);
-       }
+                        TYPE (signatures[i]), signatures[i]->subtype);
+        }
       }
-      
+
       mutt_unlink (tempfile);
 
       b->goodsig = goodsig;
-      b->badsig  = !goodsig;
-      
+      b->badsig = !goodsig;
+
       /* Now display the signed body */
       state_attach_puts (_("[-- The following data is signed --]\n\n"), s);
 
 
-      FREE (&signatures);
+      p_delete(&signatures);
     }
     else
-      state_attach_puts (_("[-- Warning: Can't find any signatures. --]\n\n"), s);
+      state_attach_puts (_("[-- Warning: Can't find any signatures. --]\n\n"),
+                         s);
   }
-  
-  mutt_body_handler (a, s);
-  
+
+  rc = mutt_body_handler (a, s);
+
   if (s->flags & M_DISPLAY && sigcnt)
     state_attach_puts (_("\n[-- End of signed data --]\n"), s);
-}
-
 
+  return (rc);
+}