mhfixmsg: Use folder name to open second message of a sequence.

`mhfixmsg foo' where foo was sequence of more than one message would
sucessfully process the lowest numbered message of the sequence, but
fail to open(2) the second one as the folder was missing from the path.
For example, /home/ralph/mail/inbox/1 was used to open the first, but
/home/ralph/mail/2 was attempted for the second.  I guess this was
because a static array was being trampled during the first's processing,
though didn't try too hard to find where.  Making a copy of it fixed the
problem.

diff --git a/uip/mhfixmsg.c b/uip/mhfixmsg.c
index 2a701f0057478e467480a23af24ca073b966fd65..a0c6efda9030b61f157f95d0a8e0a4de1b9d92e8 100644 (file)
--- a/uip/mhfixmsg.c
+++ b/uip/mhfixmsg.c
@@ -406,7 +406,7 @@ main (int argc, char **argv) {
         if (! folder) {
             folder = getfolder (1);
         }
         if (! folder) {
             folder = getfolder (1);
         }

-        maildir = m_maildir (folder);
+        maildir = mh_xstrdup(m_maildir (folder));

 
         /* chdir so that error messages, esp. from MIME parser, just
            refer to the message and not its path. */
 
         /* chdir so that error messages, esp. from MIME parser, just
            refer to the message and not its path. */


@@ -505,6 +505,7 @@ main (int argc, char **argv) {
         status = 1;
     }
 
         status = 1;
     }
 

+    mh_xfree(maildir);

     free (cts);
 
     if (fx.fixtypes != NULL) { svector_free (fx.fixtypes); }
     free (cts);
 
     if (fx.fixtypes != NULL) { svector_free (fx.fixtypes); }