extensions/extb_ssl.c: make certfp parameter case-insensitive

I had the idea that maybe these should be case-sensitive because some
encodings (like Base-64) are. But it turns out it's better to
prioritise not breaking existing configurations / channel mode lists,
and just revisit this in future maybe.

[ci skip]

extensions/extb_ssl.c: clean up this logic

This should also be using strcmp(3), not irccmp().
No practical consequence (yet), it just makes more sense.

Support ECDH X25519 for TLSv1.3 (OpenSSL 1.1.1)

Charybdis 3.5.7

Revert "ircd: Fix umode orphan scheme."

This reverts commit c1fc044c3594798cc5389486c361a7d9a43bc4f8.

m_sasl: Don't process authentication messages if SASL has been aborted

Revert "m_sasl: Don't process authentication messages if SASL has been aborted"

SASL does not work with this commit in the tree.

This reverts commit f44a0d7ea219371cf3e85c7efe80b06984795091.

m_sasl: Don't process authentication messages if SASL has been aborted

Merge branch fakelist into release/3.5

src/newconf.c: fakechannel: guard against repeated config options

Also correct another minor memory leak

Fixed syntax error

Initialize fakechan topic to NULL, allocate an empty string if not overridden

Proper range for fakechannel user count

Use irccmp instead of strcasecmp for fakechannels

Disable fakechannels by default

Fixed fakechannel removal

Check if users_max is less than users_min

Allow fakechannels with 0 users

Added fakechannels to /LIST

charybdis 3.5.6

libratbox/src/mbedtls_ratbox.h: remove unnecessary inclusion

m_nick/m_sasl/m_user: restore check for mixing of client and server protocol

m_pass: store unverified SID in preClient for use in m_server

MbedTLS: Support ChaCha20-Poly1305 in TLSv1.2+

authd: always use an empty buffer to read ident reply

Otherwise we could read uninitialised data beyond the actual reply

m_sasl: check if the agent is present after every client_exit

When a server disconnects the client_exit hook will only be called once
but there could be multiple servers and clients behind that server.

After any client exits, check if the agent is still present.

doc: build with travis

doc: there is no _static directory

Merge pull request #263 from anarcat/extban-doc

point users towards HELP EXTBAN for inline help

point users towards HELP EXTBAN for inline help

modules/m_sasl.c: prevent abort_sasl() sending 906 twice

modules/m_sasl.c: abort session if we receive '*' as data

Otherwise we'd send the * on to services as actual data, which is likely
to fail to decode it (it's not valid Base-64) and reply with an SASL ...
D F which will result in us sending a 904 numeric instead of a 906.

cf. https://github.com/ircv3/ircv3-specifications/pull/298#issuecomment-271336287

Reported-By: James Wheare

libratbox/src/mbedtls.c: check public/private keys match

src/s_user.c: don't send fake MODE for clients who have CHGHOST

Reported-by: Samuel Hoffman <redacted>

doc/reference.conf: clarify that server link fingerprints aren't optional

[ci skip]

Update configure from configure.ac

configure.ac: Adjust dlopen/dlsym checks

In modern gcc/clang, libasan (the address sanitizer runtime) exports a
weak definition of `dlopen` so that it can intercept certain flags. If
one tried to `./configure` with address sanitizer enabled, this would
cause AC_SEARCH_LIBS to conclude that dlopen doesn't require any link
flags to use. However libasan does not export `dlsym` and this caused
AC_CHECK_FUNC to fail because it didn't try linking with `-ldl`.

modules/static_modules.c.SH: use correct header file

Commit 4016731b1cf38951 missed a file.

Reported-by: mniip (Freenode)
[ci skip]

Merge pull request #247 from anarcat/cmode-help-pointer

point to the CMODE help page for more modes

point to the CMODE help page for more modes

exit_unknown_client: don't delete servers from the client hash

Outgoing servers are not added to the client hash until they reach
IsServer() status, so if they're unknown when they exit then don't
attempt to delete them.

m_sasl: indicate client connection type for SASL

libratbox/src/commio.c: misc cleanups for compiler warnings

commio.c:2244:11: warning: variable length array used [-Wvla]
commio.c:2253:29: warning: comparison of integers of different signs:
                  'unsigned int' and 'int' [-Wsign-compare]

This use of alloca(3) is okay-ish considering the corresponding function
`rb_recv_fd_buf()` already uses it too, for much the same purpose. Also
remove a redundant return statement.

modules/core/m_join.c: remove global variable parabuf

Edit by @aaronmdjones: m_join() doesn't need it

modules/core/m_join.c: remove global variable modebuf

modules/core/m_join.c: remove global variable para

modules/core/m_join.c: remove global variable pargs

modules/core/m_join.c: remove global variable mbuf

libratbox/src/ratbox_lib.c: avoid clang static analysis warning

ssld: avoid clang static analysis warning

Don't set `x = 0` twice.

Edit by @aaronmdjones: fix for loop initialisation and inner condition

ssld: avoid clang static analysis warning

librb: rb_linebuf_copy_raw: remove unused assignment

ircd: hostmask: avoid clang static analysis warning

arec->Mask.ipa.bits is unused if arec->masktype == HM_HOST

libratbox/src/commio.c: Must set addrlen before every call to accept()

If an IPv4 connection is dropped by the pre-callback, and there is a
pending IPv6 connection on the same listening socket then the retried
accept() will be unable to populate `st` because `addrlen` will be too
small. Also initialise `st` each time to avoid a clang static analysis
warning.

src/hash.c: misc cleanup for compiler warnings

hash.c:714:36: warning: comparison of integers of different signs:
               'uint32_t' (aka 'unsigned int') and 'int'
               [-Wsign-compare]

    (... and 1 more of the same)

libratbox/src/crypt.c: misc cleanup for compiler warning

crypt.c:1979:7: warning: '_STRING_ARCH_unaligned' is not defined,
                evaluates to 0 [-Wundef]

include/s_user.h: convert raw attribute into already-existing macro call

I overlooked this existing macro when adding this attribute in an earlier
commit.

modules/core/m_mode.c: misc cleanup for compiler warning

core/m_mode.c:378:11: warning: possible misuse of comma operator here
                      [-Wcomma]

extensions/hurt.c: misc cleanups for compiler warnings

hurt.c:196:16: warning: possible misuse of comma operator here [-Wcomma]

    (... and 3 more of the same)

extensions/m_sendbans.c: misc cleanup for compiler warning

m_sendbans.c:76:15: warning: possible misuse of comma operator here
                    [-Wcomma]

modules/m_rehash.c: misc cleanups for compiler warnings

m_rehash.c:380:17: warning: possible misuse of comma operator here
                   [-Wcomma]

    (... and 3 more of the same)

src/supported.c: misc cleanups for compiler warnings

supported.c:172:22: warning: possible misuse of comma operator here
                    [-Wcomma]

    (... and 4 more of the same)

src/chmode.c: misc cleanups for compiler warnings

chmode.c:417:12: warning: possible misuse of comma operator here
                 [-Wcomma]

    (... and 12 more of the same)

libratbox/src/commio.c: misc cleanup for compiler warning

commio.c:1269:17: warning: possible misuse of comma operator here
                  [-Wcomma]

What an ugly way to use commas!

lots of misc cleanups for compiler warnings

ratbox_lib.c:159:1: warning: function 'rb_lib_restart' could be declared
                    with attribute 'noreturn' [-Wmissing-noreturn]

ratbox_lib.c:220:1: warning: function 'rb_lib_loop' could be declared
                    with attribute 'noreturn' [-Wmissing-noreturn]

restart.c:55:1: warning: function 'server_reboot' could be declared with
                attribute 'noreturn' [-Wmissing-noreturn]

src/ircd_signal.c: misc cleanup for compiler warning

ircd_signal.c:59:1: warning: function 'sigterm_handler' could be declared
                    with attribute 'noreturn' [-Wmissing-noreturn]

rb_helper: misc cleanups for compiler warning

helper.c:291:1: warning: function 'rb_helper_loop' could be declared
                with attribute 'noreturn' [-Wmissing-noreturn]

getopt: misc cleanups for compiler warning

getopt.c:124:1: warning: function 'usage' could be declared with
                attribute 'noreturn' [-Wmissing-noreturn]

src/bandbi.c: misc cleanup for compiler warning

bandbi.c:389:1: warning: function 'bandb_handle_failure' could be
                declared with attribute 'noreturn'
                [-Wmissing-noreturn]

libratbox/src/balloc.c: misc cleanup for compiler warning

balloc.c:111:1: warning: function '_rb_bh_fail' could be declared with
                attribute 'noreturn' [-Wmissing-noreturn]

modules/core/m_server.c: misc cleanup for compiler warnings

core/m_server.c:138:3: warning: 'break' will never be executed
                       [-Wunreachable-code-break]

    (... and 3 more of the same)

Why put an unreachable comment in the code *and then write a
statement following it* ? O_o

modules/core/m_die.c: misc cleanup for compiler warning

core/m_die.c:76:9: warning: 'return' will never be executed
                   [-Wunreachable-code-return]

modules/m_whois.c: misc cleanup for compiler warning

m_whois.c:331:8: warning: declaration shadows a local variable [-Wshadow]

include/s_user.h: misc cleanup for compiler warning

s_user.c:1428:26: warning: format string is not a string literal
                  [-Wformat-nonliteral]

Adding the printf attribute to the function will make the compiler
assume that the 'format' argument to the function is a string
literal (by warning about the *callers* of the function *not* using
a string literal), thus avoiding the warning in the function.

src/res.c: misc cleanup for compiler warnings

res.c:704:6: warning: 'break' will never be executed
             [-Wunreachable-code-break]

    (... and 2 more of the same)

libratbox/src/commio.c: misc cleanup for compiler warnings

commio.c:1368:3: warning: 'break' will never be executed
                 [-Wunreachable-code-break]

    (... and 2 more of the same)

src/chmode.c: misc cleanup for compiler warning

chmode.c:782:3: warning: 'break' will never be executed
                [-Wunreachable-code-break]

src/modules.c: misc cleanup for compiler warnings

modules.c:799:37: warning: cast from function call of type 'void *' to
                  non-matching type 'uintptr_t' (aka 'unsigned long')
                  [-Wbad-function-cast]

    (... and 1 more of the same)

Redundant double-cast removed.

libratbox/src/openssl_ratbox.h: misc cleanup for compiler warning

openssl.c:459:47: warning: cast from 'const char *' to 'char *' drops
                  const qualifier [-Wcast-qual]

(I find it remarkable that SSL_CTX_set1_curves_list() does not accept a
 'const char *' argument...)

modules/m_stats.c: misc cleanup for compiler warnings

m_stats.c:181:22: warning: this function declaration is not a prototype
                  [-Wstrict-prototypes]

m_stats.c:1502:24: warning: format string is not a string literal
                   [-Wformat-nonliteral]
    (... and 2 more of the same)

tools/mkpasswd.c: misc cleanup for compiler warnings

mkpasswd.c:516:1: warning: function 'full_usage' could be declared with
                  attribute 'noreturn' [-Wmissing-noreturn]

mkpasswd.c:537:1: warning: function 'brief_usage' could be declared with
                  attribute 'noreturn' [-Wmissing-noreturn]

libratbox/src/crypt.c: misc cleanup for compiler warnings

crypt.c:49:4: warning: 'break' will never be executed
              [-Wunreachable-code-break]
    (... and 3 more of the same)

crypt.c:627:7: warning: variable 'f' may be uninitialized when used
               here [-Wconditional-uninitialized]

crypt.c:539:12: note: initialize the variable 'f' to silence this
                warning

ssld/ssld.c: misc cleanup for compiler warning

ssld.c:1251:14: warning: signed shift result (0x80000000) sets the sign
                bit of the shift expression's type ('int') and becomes
                negative [-Wshift-sign-overflow]

bandb/bantool.c: misc cleanup for compiler warning

bantool.c:149:4: warning: 'break' will never be executed
                 [-Wunreachable-code-break]

ircd: misc cleanup for compiler warnings

ircd.c:125:1: warning: function 'ircd_shutdown' could be declared with
              attribute 'noreturn' [-Wmissing-noreturn]

ircd.c:437:1: warning: function 'ircd_die_cb' could be declared with
              attribute 'noreturn' [-Wmissing-noreturn]

bandb/bandb.c: misc cleanup for compiler warnings

bandb.c:243:1: warning: function 'error_cb' could be declared with
               attribute 'noreturn' [-Wmissing-noreturn]

bandb.c:289:1: warning: function 'db_error_cb' could be declared with
               attribute 'noreturn' [-Wmissing-noreturn]

bandb.c:293:13: warning: signed shift result (0x80000000) sets the sign
                bit of the shift expression's type ('int') and becomes
                negative [-Wshift-sign-overflow]

bandb/bantool.c: misc cleanup for compiler warning

bantool.c:872:1: warning: function 'print_help' could be declared with
                 attribute 'noreturn' [-Wmissing-noreturn]

m_webirc: set sockhost before potentially using it to set host

Remove extra IP check, it's not necessary.

sslproc: check number of arguments to zipstats command

src/channel.c: don't use the bancache in is_banned()/is_quieted()

The bancache will be re-architected onto clients in future for easier
invalidation, but this is a good-enough temporary fix for issue #243

Fixes #243

mbedtls.c: minor fixups

- Add (void) casts for unused function parameters
- Rearrange member in `struct rb_mbedtls_cfg_context' for data alignment
- Document a `clang-4.0 -Weverything' (-Wcast-qual) diagnostic
- Avoid pointless conversions between positive/negative error codes
- Use capital hexadecimals in error codes and properly cast to
  (unsigned int) for %x/%X

m_webirc: use rb_inet_ntop_sock to populate sockhost

ircd: s_conf: fix use of strlcpy in strip_tabs

strlcpy should be called with the size of the destination buffer, not
the length of the source string.

When the source is an empty string, the destination buffer isn't
written at all, resulting in it trying to output uninitialised data.

This could also cause a buffer overflow on very long invalid config
lines.

configure: allow exact PID file prefix to be specified

MbedTLS: Don't use a dummy CA certificate on new library (no longer required)

extensions/extb_ssl.c: add support for matching by certificate fingerprint

README.md: Update channel

The domain expired and then someone else seems to have reacquired it,
so point users back at Freenode for now as recovery is unlikely.

Merge pull request #241 from anarcat/rst-guide

fix more headings

fix two headings to be toplevel