[nmh-commits] [SCM] The nmh Mail Handling System branch, master, updated. 1.7-branchpoint-884-gf116eb31

[Ralph Corderoy]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The nmh Mail Handling System".

The branch, master has been updated
       via  f116eb314cba0e77261966a99277868939f1f6bd (commit)
       via  caf881271bae730e20e262e0e1671ba01a0959b9 (commit)
       via  ab820458dca06091276250bed73446780c8faa53 (commit)
       via  5226c536d06124a36d881b8b46bfe115fd4c3ca0 (commit)
       via  78596e01aa5a694bbc34ad5bcde7f2f61d9be5e6 (commit)
       via  d2035263fbbe1075e402c2b4ba012509c2b6aa9e (commit)
       via  357d13bfc512b5f423a4dd1193cab1d319fff333 (commit)
       via  7eb67af8e7b066c292b09b3ba5dcf8ec670e6b07 (commit)
       via  0eb7dd6ae4be185e8266b3ee95661993a20a0e50 (commit)
       via  8b8100f788091b188ce19bf79123d9267b64ad5c (commit)
       via  462b6bf016c9f55ce50608f687f8bc0d9529b4b0 (commit)
       via  d059de38f28fd60875ea1283d199552ffae7ed8f (commit)
       via  43fd3407af4494b8e562b6a26f003d260811e6a7 (commit)
       via  5aca931cacf210019a6ca65d93fb5598c04101d6 (commit)
       via  90b0b47a53812c709d32cbb627e630c0ab6b0c9a (commit)
       via  e1fa26dcd75f8f7fa092c481a8428e50adf50713 (commit)
       via  34a18f2939d9301d06b713704d466b3dffb55078 (commit)
       via  512f636aae2b1a44e573c93e09c73d5d505b99b8 (commit)
      from  f87d58a3365b46d0d5d8ae9f8c2e2637eeffaa97 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit f116eb314cba0e77261966a99277868939f1f6bd
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Fri Dec 30 17:18:09 2022 +0000

    uip/mhlistsbr.c: switch some printf(3) for puts(3), and similar.

commit caf881271bae730e20e262e0e1671ba01a0959b9
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Fri Dec 30 16:39:56 2022 +0000

    test/valgrind.supp: suppress memmove(3)-overlaps warning.
    
    memmove()'s raison d’être is to handle an overlapping source and
    destination.

commit ab820458dca06091276250bed73446780c8faa53
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Fri Dec 30 15:58:20 2022 +0000

    uip/mhlistsbr.c: ditch ‘LSTFMT*’ macros.
    
    Each macro defined a printf(3) format-string and was used only once.
    Although this gathered the formats together, the reader saw ‘LSTFMT2b’
    passed to printf() instead of a format string so it wasn't obvious what
    formatting was occurring.  This may be why mhlist(1) fails to document
    all the effects of ‘-verbose’.

commit 5226c536d06124a36d881b8b46bfe115fd4c3ca0
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Fri Dec 30 15:24:09 2022 +0000

    test/valgrind.supp: suppress valgrind(1) on Manjaro's iconv_open(3).
    
    The parameters passed to iconv_open() seem to be C strings so assume
    it's a glibc problem.

commit 78596e01aa5a694bbc34ad5bcde7f2f61d9be5e6
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Fri Dec 30 12:56:21 2022 +0000

    uip/slocal.c: stop formatting NULL with ‘%s’ in debug.
    
    glob() had a conditional debug print of a vars[] p_value but that can
    still be NULL, e.g. for a name of ‘reply-to’, so explicitly test for
    NULL and produce nice output.

commit d2035263fbbe1075e402c2b4ba012509c2b6aa9e
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Fri Dec 30 12:37:12 2022 +0000

    uip/mhlsbr.c: replace strncpy() with strcpy(), silencing warnings.
    
    The strncpy()s could leave the destination unterminated.
    The source and destination char[] are the same size so strcpy() can't
    cause problems.

commit 357d13bfc512b5f423a4dd1193cab1d319fff333
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Fri Dec 30 11:32:29 2022 +0000

    uip/mhlsbr.c: fix overflow of char[] in parse().
    
    The code still silently truncates the token but at least it doesn't
    overflow the char[] by one when it does so.

commit 7eb67af8e7b066c292b09b3ba5dcf8ec670e6b07
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Fri Dec 30 09:51:47 2022 +0000

    test/*: replace egrep(1) with grep(1); regexp unchanged.
    
    The Basic Regular Expression ‘^DTSTAMP:’ can be processed by grep.

commit 0eb7dd6ae4be185e8266b3ee95661993a20a0e50
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Fri Dec 30 09:49:30 2022 +0000

    SPECS/nmh.spec: replace egrep(1) with ‘grep -E’.
    
    Some egreps have started to spew brain-damage on stderr.

commit 8b8100f788091b188ce19bf79123d9267b64ad5c
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Thu Dec 29 17:31:55 2022 +0000

    uip/*.c: delete -check and -nocheck Content-MD5 field options.
    
    Fixes 43d9833b which documented them as deprecated even though the
    functionality had been removed, not marked for removal.  The options
    were retained but did nothing, misleading the user.  Remove the options
    so the user learns of the change.
    
    Also remove ‘-nocheck’ from the DEFAULTS section of man pages; it had
    already been removed from the SYNOPSIS.  And a reference to RFC 1864
    from mhbuild(1).

commit 462b6bf016c9f55ce50608f687f8bc0d9529b4b0
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Thu Dec 29 15:16:57 2022 +0000

    uip/dropsbr.c: detect string truncation in mbx_copy().
    
    Also avoids a memory allocation for the temporary string.

commit d059de38f28fd60875ea1283d199552ffae7ed8f
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Thu Dec 29 15:15:02 2022 +0000

    sbr/dtime.c: use new ‘DCTIME_SIZEOF’ macro for string's size.
    
    It also allows callers to avoid hard-coding ‘26’ too.

commit 43fd3407af4494b8e562b6a26f003d260811e6a7
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Wed Dec 28 13:21:34 2022 +0000

    uip/popsbr.c: rework multiline() to have one string-copying line.
    
    The incoming line needs copying to the destination, but without a
    possible prefix of ‘.’.  Instead of having two methods of copying, track
    the first byte to copy and have a single overflow-checking abortcpy().
    
    Also, restore the larger buffer size so the destination is the limit on
    the line length.

commit 5aca931cacf210019a6ca65d93fb5598c04101d6
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Wed Dec 28 12:59:03 2022 +0000

    mts/smtp/smtp.h: fix misleading S_STARTTLS, ... TLS-flags.
    
    The S_STARTTLS, etc., macros are bit masks but the definitions ran
    1,2,3,4 rather than 1,2,4 because 3 was a mask of convenience for 1|2
    except it wasn't defined using S_STARTTLS and S_INITTLS.  The run of
    1,2,3,4 suggested they were exclusive values where the next should be 5
    rather than 8.  Instead, re-order to 1,2,4,3 with the last being defined
    in terms of the first two.

commit 90b0b47a53812c709d32cbb627e630c0ab6b0c9a
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Wed Dec 28 12:54:46 2022 +0000

    mts/smtp/smtp.h: remove partial client() prototype in comment.
    
    It's unclear if it's a commented-out early prototype or commentary which
    continues in the next comment but if the latter it seems incorrect.

commit e1fa26dcd75f8f7fa092c481a8428e50adf50713
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Wed Dec 28 12:49:19 2022 +0000

    sbr/datetime.c: have parse_datetime() return bool for success.
    
    It was returning OK or NOTOK but this meant ‘if (parse_datetime())’
    indicated an error whereas now that's ‘if (!parse_datetime())’.

commit 34a18f2939d9301d06b713704d466b3dffb55078
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Wed Dec 28 12:47:57 2022 +0000

    uip/aliasbr.c: comment some of the data and functions.
    
    The behaviour doesn't match the documentation and is surprising.
    The comments were a first step in uncovering the oddness.  More are
    required.

commit 512f636aae2b1a44e573c93e09c73d5d505b99b8
Author: Ralph Corderoy <ralph@inputplus.co.uk>
Date:   Wed Dec 28 12:37:47 2022 +0000

    sbr/utils.c: rework scan_input() to return 0/-1 and bool.
    
    Move commentary away from an ‘8-bit byte’ to ‘top-bit-set byte’ as
    non-ASCII is what's meant.  Change the pointer-argument from an int to a
    bool.
    
    The old code returned -1 or OK; the latter presumably isn't -1.
    Instead, return -1 or 0 as is familiar for syscall-like functions.

-----------------------------------------------------------------------

Summary of changes:
 SPECS/nmh.spec             |  2 +-
 docs/pending-release-notes | 11 ++---------
 man/mhbuild.man            |  3 ---
 man/mhlist.man             |  1 -
 man/mhshow.man             |  1 -
 man/mhstore.man            |  1 -
 mts/smtp/smtp.h            | 14 ++++++--------
 sbr/datetime.c             | 24 ++++++++++++------------
 sbr/dtime.c                | 11 +++--------
 sbr/dtime.h                |  3 +++
 sbr/utils.c                | 28 ++++++++++++++--------------
 sbr/utils.h                |  2 +-
 test/mhical/test-mhical    | 16 ++++++++--------
 test/repl/test-convert     |  2 +-
 test/valgrind.supp         | 18 ++++++++++++++++++
 uip/aliasbr.c              | 19 +++++++++++++++++++
 uip/dropsbr.c              | 16 ++++++----------
 uip/mhbuild.c              |  7 -------
 uip/mhlist.c               |  7 -------
 uip/mhlistsbr.c            | 42 +++++++++++++-----------------------------
 uip/mhlsbr.c               |  7 ++++---
 uip/mhn.c                  |  7 -------
 uip/mhshow.c               |  7 -------
 uip/mhstore.c              |  7 -------
 uip/popsbr.c               | 14 ++++++++------
 uip/post.c                 |  3 +--
 uip/slocal.c               | 15 ++++++++++++---
 27 files changed, 132 insertions(+), 156 deletions(-)


hooks/post-receive
-- 
The nmh Mail Handling System