(frm): Make fromheader and subjheader automatic variables, no longer

use malloc.  Also, initialize the whole buffers (fixing memory issues
picked up by valgrind).

diff --git a/mdfrm.c b/mdfrm.c
index 36c4a34..3e51e93 100644 (file)
--- a/mdfrm.c
+++ b/mdfrm.c
@@ -87,7 +87,8 @@ frm(char *dirname)
     DIR *dir;
     struct dirent *dirent;
     char *filename;
     DIR *dir;
     struct dirent *dirent;
     char *filename;

-    char *fromheader, *subjheader;
+    char fromheader[FROMCOUNT + 1] = { '\0' };
+    char subjheader[SUBJCOUNT + 1] = { '\0' };

 
     if (chdir(dirname) != 0) {
         err(EX_OSERR, "Failed chdir(%s)", dirname);
 
     if (chdir(dirname) != 0) {
         err(EX_OSERR, "Failed chdir(%s)", dirname);


@@ -102,8 +103,6 @@ frm(char *dirname)
         err(EX_OSERR, "Failed opendir(%s/new)", dirname);
     }
 
         err(EX_OSERR, "Failed opendir(%s/new)", dirname);
     }
 

-    fromheader = malloc(FROMCOUNT + 1);
-    subjheader = malloc(SUBJCOUNT + 1);

     for (dirent = readdir(dir); dirent; dirent = readdir(dir)) {
         filename = dirent->d_name;
         if (filename[0] == '.') {
     for (dirent = readdir(dir); dirent; dirent = readdir(dir)) {
         filename = dirent->d_name;
         if (filename[0] == '.') {