X-Git-Url: http://git.madism.org/?p=apps%2Fmadmutt.git;a=blobdiff_plain;f=attach.c;h=dfdab79bca384385e360b279562fcdbf3fd0e7c9;hp=0b97d5ab3566d4f97e5524d80c916f9cd77a3f50;hb=617e7d83d14e14e6a520a48e75437211b16c8834;hpb=e83dbdbc6200a71981d06773869cd0492f81f16c

diff --git a/attach.c b/attach.c
index 0b97d5a..dfdab79 100644
--- a/attach.c
+++ b/attach.c
@@ -12,6 +12,8 @@
 # include "config.h"
 #endif
 
+#include <lib-lib/mem.h>
+
 #include "mutt.h"
 #include "ascii.h"
 #include "handler.h"
@@ -26,7 +28,6 @@
 #include "mx.h"
 #include "mutt_crypt.h"
 
-#include "lib/mem.h"
 #include "lib/intl.h"
 #include "lib/str.h"
 #include "lib/debug.h"
@@ -144,19 +145,19 @@ int mutt_compose_attachment (BODY * a)
               b->parameter = NULL;
             }
             if (b->description) {
-              mem_free (&a->description);
+              p_delete(&a->description);
               a->description = b->description;
               b->description = NULL;
             }
             if (b->form_name) {
-              mem_free (&a->form_name);
+              p_delete(&a->form_name);
               a->form_name = b->form_name;
               b->form_name = NULL;
             }
 
             /* Remove headers by copying out data to another file, then 
              * copying the file back */
-            fseek (fp, b->offset, 0);
+            fseeko (fp, b->offset, 0);
             mutt_mktemp (tempfile);
             if ((tfp = safe_fopen (tempfile, "w")) == NULL) {
               mutt_perror (_("Failure to open file to strip headers."));
@@ -238,11 +239,14 @@ int mutt_edit_attachment (BODY * a)
                                   command, sizeof (command))) {
         /* For now, editing requires a file, no piping */
         mutt_error _("Mailcap Edit entry requires %%s");
+        goto bailout;
       }
       else {
         mutt_endwin (NULL);
-        if (mutt_system (command) == -1)
+        if (mutt_system (command) == -1) {
           mutt_error (_("Error running \"%s\"!"), command);
+          goto bailout;
+        }
       }
     }
   }
@@ -332,9 +336,9 @@ void mutt_check_lookup_list (BODY * b, char *type, int len)
         debug_print (1, ("\"%s\" -> %s\n", b->filename, type));
       }
       if (tmp.subtype)
-        mem_free (&tmp.subtype);
+        p_delete(&tmp.subtype);
       if (tmp.xtype)
-        mem_free (&tmp.xtype);
+        p_delete(&tmp.xtype);
     }
   }
 }
@@ -449,7 +453,7 @@ int mutt_view_attachment (FILE * fp, BODY * a, int flag, HEADER * hdr,
 
     if (fp) {
       /* recv case: we need to save the attachment to a file */
-      mem_free (&fname);
+      p_delete(&fname);
       if (mutt_save_attachment (fp, a, tempfile, 0, NULL) == -1)
         goto return_error;
     }
@@ -721,7 +725,7 @@ int mutt_save_attachment (FILE * fp, BODY * m, char *path, int flags,
       hn->msgno = hdr->msgno;   /* required for MH/maildir */
       hn->read = 1;
 
-      fseek (fp, m->offset, 0);
+      fseeko (fp, m->offset, 0);
       if (fgets (buf, sizeof (buf), fp) == NULL)
         return -1;
       if (mx_open_mailbox (path, M_APPEND | M_QUIET, &ctx) == NULL)
@@ -756,7 +760,7 @@ int mutt_save_attachment (FILE * fp, BODY * m, char *path, int flags,
         mutt_perror ("fopen");
         return (-1);
       }
-      fseek ((s.fpin = fp), m->offset, 0);
+      fseeko ((s.fpin = fp), m->offset, 0);
       mutt_decode_attachment (m, &s);
 
       if (fclose (s.fpout) != 0) {