Whoah, this should have been changed a long time ago. For one thing, Richard
[mmh] / ChangeLog
index a4a698c..c15f910 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,4 +1,51 @@
-Wed Mar 15 15:43:51 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+Wed Apr 05 21:09:28 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * Applied Eric Schnoebelen <eric@cirr.com>'s mhshowsbr.c patch
+       fixing apparent bugs in Dan Winship's new security quoting code:
+       
+            Since upgrading, I've been getting the following errors
+            while attempting to process some MIME messages:
+
+                (1) Syntax error: Unterminated quoted string
+                    exit 2
+       
+            and:
+
+                (2) line 1/10 (END)Segmentation fault (core dumped)
+
+           (2) appears to be due to the testing of an unset pointer in
+           mhshowsbr.c:show_multi_aux().  (1) appears to be caused by
+           mis-quoting a filename being handed to the shell in
+           mhshowsbr.c:show_content_aux().
+
+           Resolving the pointer reference issue in
+           mhshowsbr.c:show_multi_aux() turned up a similar mis-quoting
+           problem in the routine.
+
+Tue Mar 28 16:17:39 2000 Doug Morris <doug@mhost.com>
+
+       * Applied Todd.Miller@courtesan.com's patch to dropsbr.c to
+       prevent core dumping on packf. Here's the note from his message:
+
+           Since sizeof(buffer) == sizeof(tmpbuffer) packf will dump
+           core on a file w/o a From line with a line >= BUFSIZ.
+           I noticed this because I had a junk file in my mail
+           spool somehow.
+
+Fri Mar 17 11:59:33 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * wesley.craig@umich.edu did not document his previous KPOP patch,
+       so I did so, and asked him to check what I wrote.  Unfortunately
+       he didn't notice my misunderstanding of his patch.  I wrote that
+       if you #define POPSERVICE "kpop", inc and msgchk will use KPOP
+       exclusively, but if you leave it as "pop3", you can use Wesley's
+       new -kpop switch on a given invocation.  Instead, however, -kpop
+       turned out to be necessary on every invocation, and a KPOP user
+       complained.  Applied Wesley's new patch, which makes things work
+       like I thought his original patch did.  After that, did one more
+       clarifying pass to the documentation in inc.man and msgchk.man.
+
+Wed Mar 15 18:45:45 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
 
        * When I fixed the long-standing makedir() bugs in January, I had
        the code call strtoul(..., 0), which I believed to be safe as all
@@ -22,6 +69,16 @@ Wed Mar 15 15:43:51 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
        * Added a line to the "nmh configuration" output saying whether
        POP is enabled.
 
+       * Added a new README.developers file.  From the file:
+
+           This file is intended to provide a few tips for anyone doing
+           development on nmh.  Developers who learn things "the hard
+           way" about the nmh codebase (as opposed to local info best
+           encoded in a comment) are encouraged to share their wisdom
+           here.
+
+       Currently the topics are "autoconf files" and "directory structure".
+
 Tue Mar 14 12:41:48 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
 
        * Applied, after some finessing,
@@ -204,13 +261,18 @@ Fri Feb 04 12:29:12 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
 
        * Whoever originally added the -help switch to all the commands
        got too cute and had the option itself print out as "-(help)" in
-       the -help output.  I guess the idea was to make reference to the
-       fact that clearly you know about the -help option since you're
-       currently looking at its output.  I think it's a bad idea to
-       overload the meaning of the parentheses, however -- they're
+       the -help output.  One theory is that they were making reference
+       to the fact that clearly you know about the -help option since
+       you're currently looking at its output.  I think it's a bad idea
+       to overload the meaning of the parentheses, however -- they're
        supposed to indicate what abbreviated prefix of the switch you're
-       allowed to specify.  It doesn't make sense to show that you're
-       allowed to "abbreviate" the switch to its entire length.
+       allowed to specify.
+
+       The other theory is that because you can say something like
+       "mhstore -" and get "mhstore: - ambiguous.  It matches" followed
+       by the same list of switches you get with -help, they were saying
+       you can "sorta" abbreviate "-help" as "-".  You don't get the
+       "Usage:" string, though, so it's not really the same thing.
 
 Thu Feb 03 17:52:01 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
 
@@ -221,8 +283,8 @@ Thu Feb 03 17:52:01 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
              also makes "-kpop" a command line option, for users who
              would like to use both "kpop" and "pop".
 
-         Did no testing of the new features, as I don't have access to a
-         KPOP server.
+       Did no testing of the new features, as I don't have access to a
+       KPOP server.
 
        * Modified inc.man and msgchk.man to document Wesley's new -kpop.