Subtracted 1 from size argument in some calls to strncpy(3).

This fixed "specified bound [n] equals destination size" warnings
found by gcc -Wstringop-truncation.

Also, fixed harmless copy-and-paste error (sizeof(tmpfil) instead
of sizeof(drft) in uip/rcvdist.c) introduced in commit 2f689a1cb.

patch: Maildir support for "scan -file"

I have extended the "scan -file" command line so that the "file" can
also be a Maildir directory.

This feature complements the Maildir support added to the inc command
in 1.4.  Just as you may want to inc your mail from Maildir, you might
want to see what awaits you with a scan first.

Signed-off-by: David Levine <levinedl@acm.org>

Removed unused fail label.

Also, moved new tests to end of file.  Update to commit b7416e7959.

Patch for mhical to support "BYDAY=-1SU"-type RRULEs.

Notes:

- Added dmlastday() function to dtime.c, to obtain last day of month.
  Minor modifications to other parts here to reuse code.

- Modified rrule_clock() in datetime.c to handle negative BYDAY
  correctly.

- Added a couple of tests.

- Mainly this is to scratch an itch since UK timezone history makes
  heavy use of BYDAY=-1SU-type RRULEs, and the resultant complaints
  are annoying.  I had a brief look at "properly" extending
  icalparse.y, but I kindof think any serious extension of mhical
  should really instead adopt libical?

Fixed file descriptor leak in mhfixmsg when run on multiple input files.

Fixed source charset in mhfixmsg textcharset verbose output.

Thanks to Conrad Hughes for reporting this bug.

Removed obsolete spec for default password with -sasl for POP.

It was obsoleted by commit 90edb255effd0d2.

Include proper header without READLINE_SUPPORT.

Restored file execute permissions.

Fix to commit 62b9266aa31b2baced3252987b256b4e43ec9573.

Removed --depth 1 from git clone invocation.

That caused git describe in config/version.sh to fail.

Updated documentation and comments about sendmail/pipe.

Improve POP SASL error handling

Handle the case where we get an -ERR message during the SASL exchange.

Pass command line arguments first from rtm() to rt() to support its args.

Remove X.400 address parsing

Remove the X.400 address parsing used by the %(friendly) format escape.
Because ... you know, it's 2019 and X.400.

add documentation on friendly function

Change the default for noverify

The default for 'noverify' should be 'false'.  The certificate chain
SHOULD be verified by default.

Added warning if the profile contains a post entry.

Added warning from all programs that read the profile if the profile
contains a post entry, which is ignored, but does not contain a
postproc entry.  In other words, if you get this warning and want
to suppress it, your options include:
1) Remove your post profile entry.
2) Make your post profile entry a comment by prepending it with the #:
   comment indicator.
3) Add a postproc entry that points to the post that you use.  That can
   be viewed with "mhparam postproc".

Moved run_without_input() from test-version-check to common.sh.in.

Removed unnecessary warning about missing semicolon.

The warning was for Content-Type and Content-Disposition, and was
incorrect.  Thanks to Tom for providing output with it.  Also
removed multiple trailing newlines from warning output, thanks to
Ralph for tracking those down.

Added test of encoded Resent-To: and Resent-cc:.

No longer need trailing space after CFLAGS variable assignment.

Reorder configure options to be before variables

Reorder the configure options generated by build_nmh so they
will be before the variables, otherwise they end up IN things like
CFLAGS.  Fixes bug #56680.

Fix handling of fullname

Make it that if a Local-Mailbox profile entry was NOT specified
that the fullname would be generated BEFORE the local mailbox was
generated, so the user's name ended up in the email address properly.

show and mhl now decode more addresses in header fields.

Added decoding of To:, Cc:, Resent-To:, Resent-Cc:, and
Resent-From: addresses to mhl.format and mhl.headers.  Thanks to
Conrad Hughes for suggesting it for the first two components and
to Ralph for suggesting it for the Resent- components.

Handle teen ordinal suffixes correctly.

Create new mh-format function %(ordinal)

Implement a new mh-format function %(ordinal).  This will output
the appropriate ordinal prefix (st, nd, rd, st) based on the value
of the "num" register.

Support the TLS SNI extension

Add support for sending the TLS Server Name Indicator extension
during TLS connection negotiation.

Print port number in the connection error message

If a connection to a host fails, print the port number in the brief error
message.  This will hopefully eliminate problems when nmh is using a default
port number that users are not expecting.

- Fixed bcc to work with sendmail/pipe [Bug 55700].

And better documented that dcc doesn't work with sendmail/pipe.

Commit ddf3a8574f65 is a fix to commit af586ebe59b7.

Fixed inc(1) and %(me) to not obey Local-Mailbox profile component.

Thanks to Martin McCormick for reporting and Ken for diagnosing.

Added missing ')'.

Clean up more whitespace errors

Get rid of more whitespace errors; hopefully I got them all.

Remove the #ifdef UK for the default scan format

Remove the #ifdef UK that lets you select an alternate default
scan format; this is left over from old mhconfig program.

Fix whitespace errors

Remove final traces of TMA support

TMA (the TTI Trusted Mail Agent) was an early implementation of
encrypted mail and is long gone.  There were a few vestigial bits
of this support left in nmh, and this is finally removing them.
This includes the check for the "Encrypted" header field in various
scan formats and some code to return the code SCNENC from scan()
function where it would have special handling by programs like inc(1).

Added links to David Malone's MH 6.x archives.

Cleanup MIME header handling in mhbuild

Per discussion on nmh-workers, cleanup the handling of MIME headers
in mhbuild.  The following logic should now apply.

If mhbuild is run with -auto AND a MIME-Version header is detected, then
exit silently with no error (and leave the draft untouched).  Otherwise,
if a MIME-Version header or ANY Content-* header is found, exit with an
error.  Note that this means if mhbuild is run with -auto then an error
will be returned if any Content-* headers are seen.

Previously, mhbuild would silently eat any Content-Type headers seen
in an existing draft; this behavior was brought forward from MH 6 and
no one could explain the reason for this behavior.  This behavior has
been removed.

Added several historical source archives from the early days of MH.

SRI-NOSC, 2.9BSD, mh-jun-1982, and mh-nov-1983 predate those under
RCS/CVS source code control.

Thanks to M. Levinson, David Malone, Chad, Noel Chiappa, and Ralph
for advising of their discoveries of the various archive locations.

Removed docs/historical/mh-6.8.5/miscellany/mem/a.out because SPARC
executables shouldn't be archived.

Improve fmttest output

Improve the fmttest output; make it so more arguments to format
instructions are printed and clean up some of the argument handling.

Clarify %(profile) escape

Note that %(profile) can also pull entries from the context as well
as the profile

Remove support for message/partial

Improve SASL snoop output

Generate more information in the netsec layer when -snoop is enabled.
Specifically, enumerate the supported SASL mechanisms by the client and
server and explain which layer is using encryption.

Permit the matching of canonical MIME multipart boundaries

Allow the matching of MIME multipart boundaries if the message is
in canonical form (\r\n line endings).

Updated name of generated file in comment of generation script.

Removed test that hasn't been added to repo.

Fix to commit 3f845b1b0.

Commented out full output device test in test-scan.

Reverts commit d9b33232b.  It would pass on platforms that don't
have /dev/full, but was expected to fail.

Removed redundant initialization of local variable tls.

Fix to commit ec173fd2c.  gcc noticed that it, in effect, it
was declared as an int but initialized the second time with
a bool value.

Remove MIME content caching code

Remove all of the MIME content caching code.  Besides being of dubious
value and having a number of 'interesting' security implications,
it apparently only would cache content if you ran it using mhn
(the necessary entry point was not copied over when mhn was split
apart) and it would only work on the top-level MIME content of a
message.

Removal of Content-MD5 support

The generation and verification of Content-MD5 headers is no longer
done.  FINALLY.

Fix file descriptor leak in ruserpass()

Mention inc(1)/msgchk(1) changes

Set the close-on-exec flag for sockets

Since we no longer use closefds(), start cleaning up stray descriptors.
Start by making sure that any network sockets we create have the
close-on-exec flag set on them.

Merge branch 'pop-stls'

Loosen up valgrind suppression

Change the valgrind suppression rules so they exclude more things
on MacOS X.

Garbage collect closefds()

Remove the closefds() function; it was used sporadically and was
causing problems on High Sierra.  In the future look at opening
more descriptors with FD_CLOEXEC.

Final cleanup of POP STLS support

Perform final cleanup for support for the STLS command in POP.  Add
necessary switches to msgchk and add information to man pages.

Output the header name for the PUTADDR instruction

Add basic support for the STLS command in POP

Add support for the STLS command in POP (sometimes called STARTTLS).
Functionality not yet complete.

Added -checkbase switch to mhfixmsg(1).

Renamed build_multipart_alt() to build_multipart().

And some other minor code cleanup.

Add support for %(trimr) format function

Add support for a %(trimr) format function, which behaves exactly
like %(trim) but also returns a string.  Turns out this is literally
a one-line change that does not require a new format instruction.

Replaced use of hard-coded sizes with sizeof the char arrays.

Fix to commit 23024ffad75d997bf2a85b3a65fbfdfd35f8150c.

Added CONTEXT section to fmttest man page.

Added -attendee switch to mhical(1).

This allows the user to select which of their attendee addresses they are
responding about, when they have more than one in the invitation.
Added corresponding -a switch to calaccept, etc., functions in replaliases.
Updated mhical to retain DESCRIPTION lines.  Gmail puts a link to the event
DESCRIPTION line along with a message to no edit that section, though it
seems to accept responses if the DESCRIPTION line was removed.
Incremented mhical version number to 0.5.

Reduced quantities of data that could be written to a few buffers.

gcc 8 noticed that snprintfs could have overrun the buffers.

Increased sizes of a couple of buffers.

gcc 8 noticed that snprintf could have overrun them.  It might be
better to reduce the sizes of buffers used for display names, etc.
Or ever better, use dynamically sized buffers.

Resized buffer, and moved it to the only block where it's used.

gcc 8 noticed that snprintf could have overrun it.

Resized buffer and provided timezone its own buffer.

gcc 8 noticed that snprintfs could have overrun them.

Added full output test to test-scan.

From commit f54674f1b5d07148d9655fc3b337ad5f725272a7.  The test
fails now, so added to XFAIL_TESTS.

Revert "scan(1) now checks for full output device [Bug #50925]."

This reverts commit f54674f1b5d07148d9655fc3b337ad5f725272a7.

scan(1) now checks for full output device [Bug #50925].

Added fflush(3) after every output line:  may slow down scan on
output with large number of lines.

Added PROFILE COMPONENTS section and corrected CONTEXT.

Corrected comment: whom(1) does not use context_foil().

Increased sizes of a couple of buffers.

gcc 8 noticed that snprintfs could have overrun them.

Check for /etc/centos-release, on CentOS.

ruserpass.c: Return the password typed by the user.

Darren Provine reported that entering the password didn't work.  This
was due to ruserpass() failing to set the password.  Fixes 1d4977af.

mhlsbr.c: Use variable for strncpy(3)'s size, not sizeof.

Silences gcc 8.1.1's warning that the size of the source was being used,
not the destination.  The destination has just been allocated to be the
size of the source so no overflow can occur.  Move existing variable
that later holds the buffer size to before the allocation and strncpy so
it can be used for those too.

post.c: Remove `pure' attribute from void-returning chkadr().

Spoted by gcc 8.1.1.  Fixes 93a18dd7.

rcvtty.c: Use struct utmpx's ut_line[], not a copy.

Silences gcc 8.1.1's warning that strncpy(3)'s length is the sizeof of
the source, not the destination, by removing the strncpy and passing the
source to alert() where it's only read, not written.

man: Use `.B' instead of `.BR' when only one argument.

The sendmail config component/switch can be used with sendmail/smtp mts.

As well as with sendmail/pipe.

Change the default editor from 'vi' to 'prompter'.

Absent any profile environment variable setting, the default editor
for commands like 'comp', 'forw', and others is now the nmh program
'prompter'.

Fixed mhshow/test-charset to work on all tested platforms.

Restores the functionality of commit f81046da6.  Also, skips the
last two tests if iconv isn't enabled.  And removes the test
files if the last test, without iconv_elides_question_marks, behaves
as expected.  Finally, some comments have been updated.

Avoid free() in format engine for now.

This free() call messes up the buffer handling in scansbr.c:scan();
for now we're going to live with the leak and fix this properly when
we normalize the format engine's memory handling.

cpstripped(): Replace malloc()s with statics based on MB_LEN_MAX.

MB_CUR_MAX from stdlib.h isn't a compile-time constant so can't be used
for the size of a couple of static char arrays so malloc() was used at
run time, with the tiny claims never being freed.  valgrind(1) noticed
and David suggested MB_LEN_MAX from limits.h instead;  that header's
contents are suitable for #if expressions and thus compile-time
constants.

Remove extra semicolons at end of C statements.

trunccpy(): Fix typo in comment describing function.

mhshowsbr.c: Delete single-use global int `nolist'.

Was only used in one place for its value that was always zero.

Add a few more MacOS X valgrind suppressions

Clean up memory leaks by callers of scan().

Make sure we reuse the "scanl" argument to the scan() function, as that's
the signal to have scan() reuse it's buffers.  Otherwise repeated calls
to scan will leak a TON of memory.  Reported by Ralph Corderoy.

Compile the header format for each message.

Per Ken's suggestion, compile_header() and compile_marker() are now
called for each message.  That's done with reset_comptable disabled.
So, per Ralph's suggestion, fmt_free(NULL, 1) is called after all
messages are shown to free up memory held by the component table.

Generalized gdbm 1.13 Memcheck:Param suppression a bit.

gdbm 1.13-3 (Fedora 27) doesn't show the __write_nocancel call.

Fixed mhshow part markers when displaying multiple messages.

Added test/mhshow/test-markers, it reveals the problem.

Suppress unused parameter warning in builds without Cyrus SASL support.

Support building without Cyrus SASL or TLS.

If the user doesn't specify -y, and specifies n for either Cyrus
SASL or TLS when asked.

Removed unused hf_encoding from struct hfield.

I could not find any evidence of it ever being used.

Removed comment about testing noiconv portion of get_param_value().

I'm not sure if it ever did, but it doesn't seem to now.

test-charset: Fix test when HAVE_ICONV is false.

When the Content-Type's charset parameter's encoding is `invalid', it
makes no difference whether HAVE_ICONV is defined or not as the
parameter's value can't be decoded.

decode_rfc2047(): Stop unused-variable warning if HAVE_ICONV false.