modules/saslserv/main: downgrade severity of no mechanism log message

It was discovered while testing some improvements to contrib/sasl_blacklist
that clients can trigger this message remotely and at will, just by not
adhering to the SASL specification.

Add examples to NickServ help entries.
For consistenty with other entries.
Adds examples to set_accountname, set_enforce, set_pubkey, and setpass.

chanserv/flags help: only mention groups if groupserv exists

The Great Migration

help/default/nickserv/drop: fix command help

Add match_masks_through_vhost option for Solanum

This reflects a similar Solanum PR: solanum-ircd/solanum#150

It might be slightly more proper to have this option apply specifically
to the solanum protocol module once we have one, but we don't, plus
there isn't currently any precedent for config options added from
protocol modules.

The option is deliberately rehashable as the Solanum option is as well,
allowing a synchronized change in configuration across a network without
requiring restarts.

Update version in SECURITY.md

libathemecore/object.c: add method for deleting object privatedata

include/atheme/hooktypes.in: add a hook for password (hash) change

This will allow modules to detect when a user's account password
(or its hash) has been changed (after the fact; use the hook added
in the previous commit if you need to access the plaintext password
for some reason).

nickserv/set_password: allow a hook to deny a password change

This prevents bypassing nickserv/pwquality, by initially registering
with a password that it does not object to, and then changing it to
a more insecure one.

libathemecore/connection.c: free vhost_addr after using it.

Caught by Clang's AddressSanitizer, eventually, after a few inexplicable
corrupt addresses were output. I'm not sure why it didn't catch this
immediately.

Fixes: 642759fc2907efe5e0ba ("libathemecore/connection.c: use
                              connection->name to store addrs & ports")

libathemecore/connection.c: fix cut/paste mistake

Fixes: 642759fc2907efe5e0ba ("libathemecore/connection.c: use
                              connection->name to store addrs & ports")

include/atheme/global.h: remove obsolete/unused global maxfd vars

modules/protocol/charybdis: check_forward(): add +oO flags to check

If the user has the *ability* to obtain channel operator status (via
services), consider that as good as the user *having* channel operator
status.

Also document why the existing code is the way it is, and allow checks
against myuser to succeed even if user is present.

Reported-by: grumble <redacted>

libathemecore/connection.c: connection_stats(): don't justify fds

This only ends up making the output look weird.

libathemecore/connection.c: connection_stats(): indicate if listening

libathemecore/connection.c: connection_stats(): indicate the uplink

libathemecore/connection.c: use connection->name to store addrs & ports

Also obtain IP addresses where the sockets are created, instead of in
connection_add(). This is a better approach than that taken by commit
1bb7e1e587306239ca87, and restores the ability to perform non-blocking
connections.

Finally, since connection_add() creates the mowgli.pollable object for
the fd, don't bother testing if the fd is valid. Add an assertion for a
valid fd being passed, because an invalid fd will only result in
nothing working anyway. Likewise assert that the connection name is
valid, and that at least one I/O handler was supplied.

include/atheme/connection.h: add an intrusive connection list node

include/atheme/connection.h: rearrange struct in member size order

include/atheme/connection.h: remove obsolete/unused struct member

libathemecore/connection.c: tidy up all functions

Apply const-correctness, move variables to where they are needed, etc.

include/atheme/connection.h: add a typedef to reduce boilerplate

libathemecore/connection.c: ask getaddrinfo(3) to fill in the port instead

libathemecore/connection.c: refactor how to ignore errno on connect(2)

libathemecore/connection.c: tidy up macro definitions

libathemecore/connection.c: move empty_handler() up and rename it

include/atheme/connection.h: reindent and tidy up CF_ macros

Also add missing CF_IS_*() macros, and use them directly.

contrib/: add script for converting pbkdf2 hashes to pbkdf2v2 hashes

modules/contrib/: transition to independent build system

Having to duplicate configure tests in all supported Atheme release and
development branches, to enable contrib modules to be built reliably on
all of our supported platforms, was quickly becoming untenable.

dist/atheme.conf.example: expand on how to use pseudoservice::access{}

dist/atheme.conf.example: upgrade URI scheme for commented helpurl

modules/operserv/{readonly,set_commitinterval}: change fault code

cf. https://github.com/atheme/atheme/pull/767#pullrequestreview-600277627

.gitattributes: update for commit 8c3926d73eb23f567a25

m4/atheme-libtest-mowgli.m4: add a usability test if using external lib

When we are not using the internal libmowgli submodule, do a link test
to see if the library is usable, just like we do with the other library
tests.

This one is fairly extensive, as this codebase makes heavy use of this
library.

Build System: Improve handling of enabling and configuring submodules

- When performing the libmowgli test, don't pollute CFLAGS / CPPFLAGS /
  LDFLAGS / LIBS variables. Set library-specific variables, just like
  the other library tests do

  This allows us to configure submodules later without having to save
  the environment variables first

- When --with-libmowgli=yes is given, require that pkg-config is
  available and that it can detect the library, erroring out otherwise

- Put the logic for handling enabling and configuring submodules into
  its own M4 file

modules/Makefile: build module subdirectories alphabetically

extra.mk.in: Tidy up and document some groups of variables

m4/atheme-libtest-ldap.m4: prefix library variables with LIB

This more closely aligns it with the other library tests.

configure.ac: move digest/RNG API tests to immediately after libraries

The library tests set the variables that these use, so it makes sense
that they appear immediately after.

configure.ac: move warning about pkg-config being absent

This belongs with where it is tested for. Also break it up a
bit to make it easier to read.

configure.ac: rename AC_CONFIG_HEADER to AC_CONFIG_HEADERS

The former was never documented anywhere and is now obsolete.

configure.ac: move AC_CONFIG_FILES to the top

This is an autoconf configuration instantiation, just like all of the
other macros around it.

configure.ac: rename autoconf/ to build-aux/

This is the name that a lot of other projects use, the name that is
recommended by the GNU autoconf documentation, and the name that is
used by the upstream buildsys project [1] (though we have diverged
from that significantly).

[1] https://fossil.nil.im/buildsys/timeline

configure: put directory macros in a header file, not in CPPFLAGS

This will be necessary for my near-future intention to change the contrib
modules repository to be self-building; i.e. to have its own configure
script and build system.

Also don't try to expand directories like MODDIR for pretty printing,
because it sometimes doesn't work depending on the directory arguments
given to ./configure. Yeah, the config output looks worse, but oh well.

m4/atheme-api-random-frontend.m4: detect more secure arc4random(3)s

It turns out that a lot more than recent OpenBSD systems have a
secure algorithm backing their arc4random(3) implementation.

Remove the check for just OpenBSD, and add checks for:

- Mac OS 10.12+
- FreeBSD 12.0+
- NetBSD 7.0+
- OpenBSD 5.4+

Created with input from rqsd from IRC, in response to commit
8ff53e4d2a843c5fdd7a.

Note that this is not a statement that Atheme will successfully
compile & run on the above platforms.

m4/atheme-libtest-mowgli.m4: warn when pkg-config is unavailable

m4/atheme-*.m4: use autoconf flow control macros exclusively

The AS_IF and AS_CASE macros enable one to write "if" and "case"
shell statements in a portable way. They still generate more or
less the same output, but code which works better on various
obscure platforms.

They also allow autoconf to examine the conditional block bodies
for macro calls, to ensure those macros are available. This will
become more necessary in future versions of autoconf.

Also use these to replace an unguarded grep(1) invocation in the
Perl library testing macro.

libathemecore/: rename openbsd random backend to arc4random

There may well be platforms in the future that have a secure algorithm
backing their arc4random(3) implementation. For the moment we continue
to support only OpenBSD, but make it easier to adjust in the future.

configure.ac: remove checks for programs that aren't used

modules/**/Makefile: remove PLUGIN_CFLAGS & PLUGIN_LDFLAGS

The buildsys.mk.in and buildsys.module.mk files already add these
variables to CFLAGS and LDFLAGS during execution of CompileModule,
CompilePlugin, and Link steps.

libathemecore/Makefile: remove LIB_CFLAGS from CFLAGS

The buildsys.mk.in file already adds this variable to CFLAGS during
execution of CompileLib steps.

configure.ac: approach noexecstack differently

This can also be passed as -Wl,-z,noexecstack which saves Clang warning
about it being unused during compilation. Since -Wl arguments are only
used by the linker, take it out of CFLAGS. This continues to work on
GCC.

configure.ac: prepend LIBPCRE_CFLAGS and LIBPCRE_LIBS

The entire codebase depends on libpcre if it is detected successfully,
so we need to build all of the .c files with the appropriate CFLAGS &
LIBS.

This was taken care of in libathemecore/Makefile, but nowhere else.
It's better to just have configure.ac take care of adding it if they
are non-empty.

modules/saslserv/ecdh-x25519-challenge: fix ifdef soup

m4/atheme-featuretest-contrib.m4: fix test for res_query(3)

Most platforms have these as enums, which are converted to int (for use
as arguments) by the compiler automatically. However, some platforms do
not have these as enums, and use macros to define their values instead.

The former platforms also have those macros to define them in terms of
the corresponding enum, so use the macros instead of the enums for
broader compatibility, defining them in terms of their enums when they
don't exist.

glibc2 and musl have enums and compatibility macros, uclibc and
uclibc-ng have enums only, and OpenBSD libc has macros only.

doc/VERSION_LETTERS: sync with reality

Version: Indicate if --enable-fhs-paths was given

include/atheme/mkserno.sh: exit early in presence of an environment variable

This makes life easier for people who want to package snapshots
of a git branch. They must provide their own serno.h in this
case, containing also, perhaps, the date the snapshot was
downloaded or such.

Build System: Compiler Sanitizers: Some small improvements

- Remove a few levels of indentation by swapping the following approach:

  < Try to enable LTO >
  if (that succeeded) {
    < Try to enable some sanitizers >
    if (one of those succeeded) {
      AC_DEFINE(...)
    } else {
      AC_MSG_FAILURE(...)
    }
  } else {
    AC_MSG_FAILURE(...)
  }

  ... for this equivalent approach instead:

  < Try to enable LTO >
  if (that failed) {
    AC_MSG_FAILURE(...)
  }
  < Try to enable some sanitizers >
  If (none of those succeeded) {
    AC_MSG_FAILURE(...)
  }
  AC_DEFINE(...)

  This is equivalent because AC_MSG_FAILURE terminates the script

- Add support for -fsanitize=bounds

  This enables some checks not enabled by -fsanitize=undefined on Clang

- Add support for falling back to individual undefined behaviour
  sanitizers if the compiler does not support -fsanitize=undefined

  This may seem pointless, but it may benefit older compilers, and
  compilers that are not GCC or Clang.

- Adjust the compiler sanitizers driver to take the name of a sanitizer
  rather than its whole -fsanitize= option. Use this to collect a list
  of enabled sanitizers and report it in the output of ./configure

GitHub Actions: CI: Tidy up a bit

- Clone the repository first

  This is likely to be much faster than downloading the dependencies,
  which will result in the job aborting much earlier and wasting less
  data if for some reason the clone cannot succeed

- Remove a pointless single-valued option from the build matrix

This more closely aligns it with the Coverity Scan action.

GitHub Actions: Coverity Scan: Pleeeeeeease work damn you!

GitHub is throwing an incredibly vague and non-specific "No such file or
directory" error when trying to start this job.

I have no idea why.

Remove all the fancy bells and whistles.

Allow it to be scheduled manually.

GitHub Actions: Coverity Scan: Check out source repo first

Merge branch 'list-mlock' (PR #762)

modules/operserv/readonly: error out on no write backend

Build System: Detect user error w/ downloading sourcecode

A frequent complaint we receive is that serno.h is missing. This turns
out to be users downloading release tarballs from GitHub, which does
not include the .git directory, thus making the target for serno.h
fail.

Detect whether .git/ exists; reference GIT-Access.txt then. Otherwise,
this must be a release tarball, so see if it has a pre-supplied serno.h
and error if not (this could only be caused by downloading a GitHub
source code link instead of an asset/release tarball).

Also tidy up mkserno.sh a bit. It still can't use git-describe(1) on
this branch (lack of recent tags), but will be made to do so on release
branches.

Thanks to ilbelkyr for the idea (pointing out that the sourcecode links
on GitHub don't include the .git/ directory, making it possible to
distinguish between that and git-clone(1)).

Support rehashing of general::commit_interval

modules/nickserv/listlogins: correct module name & help path

This was overlooked when backporting this module from contrib/.

Build System: Several small improvements

- Don't let autoconf add "-O2 -g" to the CFLAGS variable. Detect
  optimisations automatically (if sanitizers are not enabled) or
  explicitly disable them (if they are). Detect debugging flags
  automatically, preferring DWARF, then GDB, then regular -g as
  autoconf does. This allows more accurate debugging when supported
  by the toolchain. Allow debugging symbols to be disabled; enable
  them by default.

- When requested to enable compiler sanitizers, bail out with an
  error when they cannot be enabled. Update the comment on
  ATHEME_ENABLE_COMPILER_SANITIZERS to reflect that sanitizers are
  enabled; rather than just the configure argument given.

- Move the logic for testing CFLAGS / CPPFLAGS / LDFLAGS / some
  combination of them to a dedicated separate file. Tidy up those
  functions to use the same M4sh coding style as the other files.
  Explicitly provide a program with both a header and main body when
  doing compiler and linker tests. Use a unified function and variable
  name scheme.

- Rewrite the compiler sanitizers driver function to use the new
  combined compiler and linker test logic.

- Update the comment in the compiler sanitizers feature file to explain
  why we are trying to enable LTO (Clang sanitizers require it).

- Clean up temporary _SAVED variables at the end of function execution
  in various feature and library tests.

- Remove 2 unsubstituted and unused variables from extra.mk.in.

- Support the -Wa,--noexecstack flag to the compiler and linker; enable
  it by default.

GitHub Actions: Coverity Scan: Truncate version hash

The web UI truncates excessively long version fields; 20 characters
is more than long enough, and is already what we use in e.g. serno.h.

GitHub Actions: Coverity Scan: Ensure version string is valid

curl(1) reads this file for the value of the 'version' form field.
We should not be submitting a field with a newline character in it.

modules/contrib/: bump to latest HEAD

libathemecore/logger.c: logfile_new(): fix up close-on-exec logic

We were calling fcntl(2) F_SETFD without first obtaining the current
file descriptor flags with F_GETFD. Furthermore, we were not checking
the return value of the F_SETFD operation. Now we obtain the current
flags to bitwise-OR FD_CLOEXEC them with, and we warn if the operation
fails.

However, it would be better if we didn't have to try to call it in the
first place, so try opening the file descriptor with the O_CLOEXEC flag
first, which is safer too.

While we're at it, make sure that the log file doesn't end up being
world-readable if services' umask is not sufficient to prevent this.

Identified by Coverity Scan.

libathemecore/commandhelp.c: help_not_available(): move buffer

We cannot (safely) assign the address of this block-scope buffer to a
function-scope variable and then use it after the block ends.

Identified by Coverity Scan.

libathemecore/version.c: infotext[]: add missing comma delimiter

Identified by Coverity Scan.

libathemecore/connection.c: connection_add(): library calling fixes

We should check if getpeername(2) fails, and we must use inet_ntop(3)
properly.

Previously the code assumed that "(struct sockaddr_in *)->sin_addr" and
"(struct sockaddr_in6 *)->sin6_addr" both started at the same offset.

While we're at it, use the proper "struct sockaddr_storage" type for
passing to inet_pton(3), capable of holding any kind of sockaddr
structure, and remove it from `struct connection', as it was not
referenced anywhere. Also remove some unused macros and the now-unused
sockaddr_any union.

Finally, remove the non-blocking invocation from connection_open_tcp();
this prevents getpeername(2) from functioning alltogether (the socket
is not yet connected). connection_add() itself sets the socket non-
blocking.

Identified by Coverity Scan.

libathemecore/connection.c: connection_open_tcp(): correct argument

From setsockopt(2):

    Most socket-level options utilize an int argument for optval. For
    setsockopt(2), the argument should be non-zero to enable a boolean
    option, or zero if the option is to be disabled. For a description
    of the available socket options see socket(7) and the appropriate
    protocol man pages.

From socket(7):

    The socket options listed below can be set by using setsockopt(2)
    and read with getsockopt(2) with the socket level set to SOL_SOCKET
    for all sockets. Unless otherwise noted, optval is a pointer to an
    int.

    SO_REUSEADDR
        Indicates that the rules used in validating addresses supplied
        in a bind(2) call should allow reuse of local addresses. For
        AF_INET sockets this means that a socket may bind, except when
        there is an active listening socket bound to the address. When
        the listening socket is bound to INADDR_ANY with a specific port
        then it is not possible to bind to this port for any local
        address. Argument is an integer boolean flag.

Therefore, the argument must be a pointer to int. Further, check the
return value of setsockopt(2) to ensure that it succeeds.

Identified by Coverity Scan.

libathemecore/connection.c: connection_open_listener_tcp(): correct argument

From setsockopt(2):

    Most socket-level options utilize an int argument for optval. For
    setsockopt(2), the argument should be non-zero to enable a boolean
    option, or zero if the option is to be disabled. For a description
    of the available socket options see socket(7) and the appropriate
    protocol man pages.

From socket(7):

    The socket options listed below can be set by using setsockopt(2)
    and read with getsockopt(2) with the socket level set to SOL_SOCKET
    for all sockets. Unless otherwise noted, optval is a pointer to an
    int.

    SO_REUSEADDR
        Indicates that the rules used in validating addresses supplied
        in a bind(2) call should allow reuse of local addresses. For
        AF_INET sockets this means that a socket may bind, except when
        there is an active listening socket bound to the address. When
        the listening socket is bound to INADDR_ANY with a specific port
        then it is not possible to bind to this port for any local
        address. Argument is an integer boolean flag.

Therefore, the argument must be a pointer to int. Further, check the
return value of setsockopt(2) to ensure that it succeeds.

Identified by Coverity Scan.

libathemecore/services.c: get_storage_oper_name(): complete NULL checks

Code above this already tests if si->v is NULL; indicating that it can
be NULL (and it can be). Therefore, check if it's NULL again before
attempting to dereference it again.

At present this cannot be the case, but guard against it anyway, making
sure to fill the result buffer with something in any case.

Identified by Coverity Scan.

libathemecore/services.c: get_oper_name(): complete NULL checks

Code above this already tests if si->v is NULL; indicating that it can
be NULL (and it can be). Therefore, check if it's NULL again before
attempting to dereference it again.

At present this cannot be the case, but guard against it anyway, making
sure to fill the result buffer with something in any case.

Identified by Coverity Scan.

libathemecore/services.c: get_source_mask(): complete NULL checks

Code above this already tests if si->v is NULL; indicating that it can
be NULL (and it can be). Therefore, check if it's NULL again before
attempting to dereference it again.

At present this cannot be the case, but guard against it anyway, making
sure to fill the result buffer with something in any case.

Identified by Coverity Scan.

libathemecore/services.c: get_source_name(): complete NULL checks

Code above this already tests if si->v is NULL; indicating that it can
be NULL (and it can be). Therefore, check if it's NULL again before
attempting to dereference it again.

At present this cannot be the case, but guard against it anyway, making
sure to fill the result buffer with something in any case.

Identified by Coverity Scan.

chanserv/{voice,op,halfop,owner,protect,kick}: check channel exists

These modules would not check whether the target channel existed
ircd-side, instead failing on their chanuser_find call and causing an
assertion failure. Add a proper check instead.

GitHub Actions: Add periodic Coverity Scan runner

GitHub Actions: CI: Increase dependencies wrapping character column

GitHub Actions: CI: Use version 2 of actions/checkout & name it

GitHub Actions: CI: Use /tmp for dependency install step

GitHub Actions: CI: Ignore modifications to periodic actions

GitHub Actions: CI: Ensure the build-essential package is installed

atheme.conf.example: listownmail_canon: apply style updates

This is a follow-up to PR #745 to make the added bit in the example
configuration follow the style as since adjusted on the master branch.

Merge PR #745 (make LISTOWNMAIL canonicalization optional)

Merge branch 'infoserv-improvements'

libmowgli-2/: bump to latest HEAD

modules/scripting/perl: make non-unloadable

Reloading this module causes services to segfault because the
PERL_SYS_INIT3() macro should only be called once during the entire
lifetime of the process [1]. Allowing it to be unloaded thus carries the
risk that it will be reloaded (or, at a later date, loaded again).

I tried to (ab)use Mowgli's global storage API to only call it once
(regardless of how many times it's loaded), and to not call the
PERL_SYS_TERM() macro at all, but this only lead to a different crash on
reload: trying to allocate a little less than 100 TiB of memory (!).

If a Perl expert comes along and weighs in, this commit can be reverted
and the underlying problem fixed. Nonetheless, libperl did exhibit *dozens*
of uses of unintialized data on reload (confirmed by valgrind), followed by
a segmentation fault even if we skip calling PERL_SYS_INIT3() again
(because, naturally, such a large allocation will usually return NULL, and
it apparently doesn't deal with that).

[1] https://perldoc.perl.org/perlembed

modules/proxyscan/dnsbl: make reload-only to prevent data loss

modules/chanserv/moderate: make reload-only to prevent data loss

infoserv/main: fix signedness on format parameter

(spotted by @aaronmdjones)

infoserv/main: store list node in struct logoninfo

This avoids having to dynamically allocate the node (as suggested during
review of PR #765).

infoserv/main: address issues found in review

See discussion on PR #765 for details.

README.md, GIT-Access.txt: Some small improvements

- Recommend an explicit directory name for the clone, to avoid cloning
  into ~/atheme/ if the user runs the `git clone` operation in their
  home directory. Atheme defaults to installing to ~/atheme/, and you
  cannot install Atheme to its source directory. [1]

- Recommend the use of the `--recursive` option to `git clone` in
  `README.md`, as it was already recommended in `GIT-Access.txt`. [1]

- Provide alternative command sequences for people who have very old
  versions of git, which may not even support the `--init` option of
  `git submodule update`, let alone the `--recursive` option of
  `git clone`.

- Quote a filesystem path.

- Tidy up the more information section in the bottom of `README.md`.

[1] Suggested by GitHub user @PeGaSuS-Coder in PR #764

Closes #764