From: Ralph Corderoy <ralph@inputplus.co.uk>
Date: Tue, 17 Jan 2017 12:54:02 +0000 (+0000)
Subject: mhfixmsg: Use folder name to open second message of a sequence.
X-Git-Url: https://diplodocus.org/git/nmh/commitdiff_plain/66fa74b004d50df011eecdeac09b5e4f59181e12?ds=inline;hp=103abacb78b3ed17dce5d47dfa65e97bf4f5377d

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 2a701f00..a0c6efda 100644
--- a/uip/mhfixmsg.c
+++ b/uip/mhfixmsg.c
@@ -406,7 +406,7 @@ main (int argc, char **argv) {
         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. */
@@ -505,6 +505,7 @@ main (int argc, char **argv) {
         status = 1;
     }
 
+    mh_xfree(maildir);
     free (cts);
 
     if (fx.fixtypes != NULL) { svector_free (fx.fixtypes); }