+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
+ modes specified as ASCII constants in the nmh code started with a
+ leading zero (signifying octal), which I did as it would work if
+ internal constants were ever changed to hex. Unfortunately I was
+ unaware of the "Folder-Protect:" .mh_profile entry, which
+ mh-profile.man documents as an octal-only constant, with no
+ leading zero required. I've changed the strtoul() call to an
+ atooi() call and removed the misleading leading zeroes on the
+ ASCII octal constants in the code and man pages. Also changed the
+ "Folder-Protect:" example in the man page to something more
+ interesting than a duplication of the default.
+
+ * When I added my --enable-masquerade option, you'll note that I
+ didn't make it --enable-nmh-masquerade. I find the --enable-nmh-*
+ options too wordy and I'm not sure why Richard went that route.
+ I've renamed them to just --enable-*, but the old versions will
+ still work as well (they just aren't advertised).
+
+ * 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,
* 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>
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.
Thu Jan 27 12:22:25 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
- * makedir() had multiple bugs dating back to MH. It passed an
- octal constant to atoi(), which interpreted it as decimal,
- resulting in directories with no user read or execute permissions,
- making nested directory creation fail. Also, when creating a
- nested directory, correct permissions were only set on the inner one.
+ * makedir() had multiple bugs dating back to MH. An octal
+ constant was apparently being interpreted as decimal, resulting in
+ directories with no user read or execute permissions, making
+ nested directory creation fail. And there wasn't even an
+ _attempt_ to set desired permissions (e.g. from "Folder-Protect:"
+ in .mh_profile) on the outer directories of a nested directory.
* A second `make install' would always fail because the check for
whether mh_profile.5 existed was written incorrectly.
projects / mmh / blobdiff