Merge branch 'master' into release/4

librb: remove one more VMS detritius

librb: we have no plans to support VMS

Merge branch 'master' into release/4

ircd: relocate_paths() back on windows only now

m_modules: use new module api

conf_parser: warning fixes

modules: serious cleanups

modules: cleanups

ircd: fix compile of relocate_paths()

ircd: make relocate_paths() available always

ircd: call relocate_paths() in all cases

messages: fix format string warning reported by latest clang

ircd: print runtime path configuration

librb: dictionaries which use integer keys may use 0 legitimately

invite: do not send duplicate invite messages (closes #194)

invite: do not send duplicate invite messages (closes #194)

mbedtls backend: indicate reason for TLS session termination

[ci skip]

mbedtls backend: indicate reason for TLS session termination

[ci skip]

CREDITS: charybdis official channel will now be on irc.charybdis.io.

CREDITS: charybdis official channel will now be on irc.charybdis.io.

CREDITS: adjust to reflect present situation

CREDITS: adjust to reflect present situation

Fix regression introduced by commit 2f361bfc

Fix regression introduced by commit 1863a0f8

Preliminary code tidying complete.

These issues (commits 47a66e15 through 906fd91f inclusive) were
identified with the Clang compiler. Please raise concerns about
them on the issue tracker or in the support channel.

tools/mkfingerprint: strlen(3) on a raw byte array is wrong -- use its length as indicated by the function that filled it

tools/mkpasswd: functions that call exit(3) should be marked noreturn

bandb/bandb: functions that call exit(3) should be marked noreturn, avoid sign overflow in integer function argument

bandb/bantool: a function that calls exit(3) should be marked noreturn

authd/res: make function used only within this unit static, remove unused macros

authd/provider: remove shadowed double variable decl

authd/authd: a function that calls exit(3) should be marked noreturn

wsockd: various fixes

* Use correct sign for comparing data lengths
* Don't return a void statement in a void function
* Remove unused functions and macros

ssld: remove unused macros, avoid sign overflow in integer function argument

wsproc: compile out dead code

Investigation is required to determine if this function should
actually be used

restart: functions that call exit(3) should be marked noreturn

ircd_signal: a function that tailcalls a noreturn function should be marked noreturn

ircd: functions that call exit(3) should be marked noreturn

getopt: a function that calls exit(3) should be marked noreturn

dns: make function used only within this unit static

class: remove unused macros

chmode: remove unreachable break statement

chmode: silence harmless uninitialised variable warning

channel: silence harmless uninitialised variable warning

bandbi: a function that calls exit(3) should be marked noreturn

authproc: don't shadow variable decls, avoid reserved name

librb: silence some fairly harmless compiler warnings

These include warnings about "break" statements that will never be
executed (because they are after "return" statements), unused macros
(lost to code refactoring or never even used in the first place),
functions that call abort() or loop indefinitely but aren't marked with
the "noreturn" attribute, and use of variables possibly uninitialised
(a false positive).

Preliminary code tidying complete.

These issues (commits 92706fd5 through 707bc7cd inclusive) were
identified with the Clang compiler. Please raise concerns about
them on the issue tracker or in the support channel.

tools/mkfingerprint: strlen(3) on a raw byte array is wrong -- use its length as indicated by the function that filled it

tools/mkpasswd: functions that call exit(3) should be marked noreturn

bandb/bandb: functions that call exit(3) should be marked noreturn, avoid sign overflow in integer function argument

bandb/bantool: a function that calls exit(3) should be marked noreturn

authd/res: make function used only within this unit static, remove unused macros

authd/provider: remove shadowed double variable decl

authd/authd: a function that calls exit(3) should be marked noreturn

wsockd: various fixes

* Use correct sign for comparing data lengths
* Don't return a void statement in a void function
* Remove unused functions and macros

ssld: remove unused macros, avoid sign overflow in integer function argument

wsproc: compile out dead code

Investigation is required to determine if this function should
actually be used

restart: functions that call exit(3) should be marked noreturn

ircd_signal: a function that tailcalls a noreturn function should be marked noreturn

ircd: functions that call exit(3) should be marked noreturn

getopt: a function that calls exit(3) should be marked noreturn

dns: make function used only within this unit static

class: remove unused macros

chmode: remove unreachable break statement

chmode: silence harmless uninitialised variable warning

channel: silence harmless uninitialised variable warning

bandbi: a function that calls exit(3) should be marked noreturn

authproc: don't shadow variable decls, avoid reserved name

librb: silence some fairly harmless compiler warnings

These include warnings about "break" statements that will never be
executed (because they are after "return" statements), unused macros
(lost to code refactoring or never even used in the first place),
functions that call abort() or loop indefinitely but aren't marked with
the "noreturn" attribute, and use of variables possibly uninitialised
(a false positive).

openssl: use snprintf(3) instead of rb_snprintf()

openssl: use snprintf(3) instead of rb_snprintf()

openssl: More LibreSSL compatibility

LibreSSL does not have the new version macros & functions that OpenSSL
1.1.0 implements. This causes a compile-time failure against LibreSSL.

Further, the runtime function for returning the library version returns
the wrong number (the hardcoded constant number SSLEAY_VERSION_NUMBER
aka OPENSSL_VERSION_NUMBER, instead of LIBRESSL_VERSION_NUMBER).

Add more ifdef soup to remedy the situation.

openssl: More LibreSSL compatibility

LibreSSL does not have the new version macros & functions that OpenSSL
1.1.0 implements. This causes a compile-time failure against LibreSSL.

Further, the runtime function for returning the library version returns
the wrong number (the hardcoded constant number SSLEAY_VERSION_NUMBER
aka OPENSSL_VERSION_NUMBER, instead of LIBRESSL_VERSION_NUMBER).

Add more ifdef soup to remedy the situation.

openssl: change how we load DH parameters

The code already assumes the presence of fopen(3) and errno, and, by
extension, fclose(3) and strerror(3), so just use those instead of the
BIO wrappers.

Additionally, don't fail to initialise if the DH file does exist but
parsing it fails, as per the pre-existing comment about them being
optional.

openssl: change how we load DH parameters

The code already assumes the presence of fopen(3) and errno, and, by
extension, fclose(3) and strerror(3), so just use those instead of the
BIO wrappers.

Additionally, don't fail to initialise if the DH file does exist but
parsing it fails, as per the pre-existing comment about them being
optional.

strcpy: mass-migrate to strlcpy where appropriate

strcpy: mass-migrate to strlcpy where appropriate

appveyor: correct version

[ci skip]

appveyor: correct version

[ci skip]

ircd_lexer: fix another crash with the same cause

ircd_lexer: fix another crash with the same cause

ircd_lexer: fix crash with very large config option strings

minor spring cleaning: remove/relocate duplicate/unused includes & macros

[ci skip]

makerelease: fix DATECODE output

NEWS: document websocket availability

config: document websocket options

newconf: ensure wsock and defer_accept are default-to-disable for now, for consistency sake on rehashes

starttls: Allow command usage with backends other than OpenSSL

m_alias: correctly construct string to be sent

bandb: sqlite3: use getpagesize() on FreeBSD

wsockd: include stdinc.h so that sys/types.h is included on FreeBSD 4.8

librb: define UINT32_MAX for FreeBSD 4.8

librb: support rb_path_to_self on FreeBSD 4.8

client: call authd_abort_client with the client that is exiting, not the originator

Merge pull request #191 from GLolol/patch-1

doc: add extensions/chm_nonotice to example confs

doc: add extensions/chm_nonotice to example confs

[Documentation] Increase bitlength recommendation for DH parameters

Also clarify the behaviour of TLS backends and the consequences for
not providing any parameters at all.

[ci skip]

[Documentation] Reflect that ssl_private_key is now optional

[ci skip]