lock_file.c: close(2) file descriptor on failure, avoiding leak.

[nmh] / uip / mhfree.c

diff --git a/uip/mhfree.c b/uip/mhfree.c
index ec0f8d3b2c5ae377b5cd3b7198268423c4f79778..6a827e080beb568f913c88ca92c98ccaf7965f6f 100644 (file)
--- a/uip/mhfree.c
+++ b/uip/mhfree.c
@@ -1,6 +1,4 @@
-
-/*
- * mhfree.c -- routines to free the data structures used to
+/* mhfree.c -- routines to free the data structures used to

  *          -- represent MIME messages
  *
  * This code is Copyright (c) 2002, by the authors of nmh.  See the
  *          -- represent MIME messages
  *
  * This code is Copyright (c) 2002, by the authors of nmh.  See the


@@ -12,21 +10,16 @@
 #include <h/utils.h>
 #include <h/mime.h>
 #include <h/mhparse.h>
 #include <h/utils.h>
 #include <h/mime.h>
 #include <h/mhparse.h>

+#include "sbr/m_mktemp.h"
+#include "mhfree.h"

 
 /* The list of top-level contents to display */
 CT *cts = NULL;
 
 
 /* The list of top-level contents to display */
 CT *cts = NULL;
 

-/*
- * prototypes
- */
-void free_header (CT);
-void free_ctinfo (CT);
-void free_encoding (CT, int);
-void freects_done (int);
-

 /*
  * static prototypes
  */
 /*
  * static prototypes
  */

+static void free_header (CT);

 static void free_text (CT);
 static void free_multi (CT);
 static void free_partial (CT);
 static void free_text (CT);
 static void free_multi (CT);
 static void free_partial (CT);


@@ -49,9 +42,9 @@ free_content (CT ct)
      */
     free_header (ct);
 
      */
     free_header (ct);
 

-    mh_xfree(ct->c_partno);
-    mh_xfree(ct->c_vrsn);
-    mh_xfree(ct->c_ctline);
+    free(ct->c_partno);
+    free(ct->c_vrsn);
+    free(ct->c_ctline);

     ct->c_partno = ct->c_vrsn = ct->c_ctline = NULL;
 
     free_ctinfo (ct);
     ct->c_partno = ct->c_vrsn = ct->c_ctline = NULL;
 
     free_ctinfo (ct);


@@ -77,26 +70,28 @@ free_content (CT ct)
            }
            break;
 
            }
            break;
 

-       case CT_TEXT:
+       default:
+           /* Assume that the ct is for text.  mhfixmsg(1) uses it for
+              decoding application content. */

            free_text (ct);
            break;
     }
 
            free_text (ct);
            break;
     }
 

-    mh_xfree(ct->c_showproc);
-    mh_xfree(ct->c_termproc);
-    mh_xfree(ct->c_storeproc);
+    free(ct->c_showproc);
+    free(ct->c_termproc);
+    free(ct->c_storeproc);

     ct->c_showproc = ct->c_termproc = ct->c_storeproc = NULL;
 
     ct->c_showproc = ct->c_termproc = ct->c_storeproc = NULL;
 

-    mh_xfree(ct->c_celine);
+    free(ct->c_celine);

     ct->c_celine = NULL;
 
     /* free structures for content encodings */
     free_encoding (ct, 1);
 
     ct->c_celine = NULL;
 
     /* free structures for content encodings */
     free_encoding (ct, 1);
 

-    mh_xfree(ct->c_id);
-    mh_xfree(ct->c_descr);
-    mh_xfree(ct->c_dispo);
-    mh_xfree(ct->c_dispo_type);
+    free(ct->c_id);
+    free(ct->c_descr);
+    free(ct->c_dispo);
+    free(ct->c_dispo_type);

     ct->c_id = ct->c_descr = ct->c_dispo = ct->c_dispo_type = NULL;
     free_pmlist (&ct->c_dispo_first);
 
     ct->c_id = ct->c_descr = ct->c_dispo = ct->c_dispo_type = NULL;
     free_pmlist (&ct->c_dispo_first);
 


@@ -111,8 +106,8 @@ free_content (CT ct)
        ct->c_fp = NULL;
     }
 
        ct->c_fp = NULL;
     }
 

-    mh_xfree(ct->c_storage);
-    mh_xfree(ct->c_folder);
+    free(ct->c_storage);
+    free(ct->c_folder);

     ct->c_storage = ct->c_folder = NULL;
 
     free (ct);
     ct->c_storage = ct->c_folder = NULL;
 
     free (ct);


@@ -124,7 +119,7 @@ free_content (CT ct)
  * for this content.
  */
 
  * for this content.
  */
 

-void
+static void

 free_header (CT ct)
 {
     HF hp1, hp2;
 free_header (CT ct)
 {
     HF hp1, hp2;


@@ -151,12 +146,12 @@ free_ctinfo (CT ct)
     CI ci;
 
     ci = &ct->c_ctinfo;
     CI ci;
 
     ci = &ct->c_ctinfo;

-    mh_xfree(ci->ci_type);
-    mh_xfree(ci->ci_subtype);
+    free(ci->ci_type);
+    free(ci->ci_subtype);

     ci->ci_type = ci->ci_subtype = NULL;
     free_pmlist(&ci->ci_first_pm);
     ci->ci_type = ci->ci_subtype = NULL;
     free_pmlist(&ci->ci_first_pm);

-    mh_xfree(ci->ci_comment);
-    mh_xfree(ci->ci_magic);
+    free(ci->ci_comment);
+    free(ci->ci_magic);

     ci->ci_comment = ci->ci_magic = NULL;
 }
 
     ci->ci_comment = ci->ci_magic = NULL;
 }
 


@@ -169,7 +164,7 @@ free_text (CT ct)
     if (!(t = (struct text *) ct->c_ctparams))
        return;
 
     if (!(t = (struct text *) ct->c_ctparams))
        return;
 

-    free ((char *) t);
+    free(t);

     ct->c_ctparams = NULL;
 }
 
     ct->c_ctparams = NULL;
 }
 


@@ -183,19 +178,19 @@ free_multi (CT ct)
     if (!(m = (struct multipart *) ct->c_ctparams))
        return;
 
     if (!(m = (struct multipart *) ct->c_ctparams))
        return;
 

-    mh_xfree(m->mp_start);
-    mh_xfree(m->mp_stop);
+    free(m->mp_start);
+    free(m->mp_stop);

     free (m->mp_content_before);
     free (m->mp_content_after);
        
     for (part = m->mp_parts; part; part = next) {
        next = part->mp_next;
        free_content (part->mp_part);
     free (m->mp_content_before);
     free (m->mp_content_after);
        
     for (part = m->mp_parts; part; part = next) {
        next = part->mp_next;
        free_content (part->mp_part);

-       free ((char *) part);
+       free(part);

     }
     m->mp_parts = NULL;
 
     }
     m->mp_parts = NULL;
 

-    free ((char *) m);
+    free(m);

     ct->c_ctparams = NULL;
 }
 
     ct->c_ctparams = NULL;
 }
 


@@ -208,9 +203,9 @@ free_partial (CT ct)
     if (!(p = (struct partial *) ct->c_ctparams))
        return;
 
     if (!(p = (struct partial *) ct->c_ctparams))
        return;
 

-    mh_xfree(p->pm_partid);
+    free(p->pm_partid);

 
 

-    free ((char *) p);
+    free(p);

     ct->c_ctparams = NULL;
 }
 
     ct->c_ctparams = NULL;
 }
 


@@ -224,9 +219,9 @@ free_external (CT ct)
        return;
 
     free_content (e->eb_content);
        return;
 
     free_content (e->eb_content);

-    mh_xfree(e->eb_body);
-    mh_xfree(e->eb_url);
-    free ((char *) e);
+    free(e->eb_body);
+    free(e->eb_url);
+    free(e);

     ct->c_ctparams = NULL;
 }
 
     ct->c_ctparams = NULL;
 }
 


@@ -237,10 +232,10 @@ free_pmlist (PM *p)
     PM pm = *p, pm2;
 
     while (pm != NULL) {
     PM pm = *p, pm2;
 
     while (pm != NULL) {

-        mh_xfree(pm->pm_name);
-        mh_xfree(pm->pm_value);
-        mh_xfree(pm->pm_charset);
-        mh_xfree(pm->pm_lang);
+        free(pm->pm_name);
+        free(pm->pm_value);
+        free(pm->pm_charset);
+        free(pm->pm_lang);

        pm2 = pm->pm_next;
        free(pm);
        pm = pm2;
        pm2 = pm->pm_next;
        free(pm);
        pm = pm2;


@@ -279,7 +274,7 @@ free_encoding (CT ct, int toplevel)
 }
 
 
 }
 
 

-void
+void NORETURN

 freects_done (int status)
 {
     CT *ctp;
 freects_done (int status)
 {
     CT *ctp;