mkpasswd: use urandom for salts, cleanup

Using /dev/random for salt generation is pointless -- it can block, and
any extra randomness it would provide (which is debatable) is not needed,
as salts only need to be unique, not unpredictable.

openssl: Avoid use-after-free when rehashing fails to load new files

Commit cf12678 introduced a fix for issue #186 by freeing the old SSL_CTX
structure before constructing a new one, which could disconnect existing
clients otherwise.

Unfortunately, the freeing is done first, which means that if setting up
a new structure fails for any reason, there will be no usable structures
left, but they are still referenced.

This fix moves the freeing to the end of the function, using intermediate
new variables in the meantime. This problem was discovered while testing
against OpenSSL 1.1.0 RC6.

charybdis 4-rc3.

m_grant: convert jevolk's rewritten version to AV2

Merge pull request #207 from charybdis-ircd/jevolk-patch-1

authd: Fix use after reference count decrement.

authd: Fix use after reference count decrement.

Merge pull request #205 from jevolk/release/4

m_grant: rewrite.

m_grant: rewrite.

ircd: Fix missing operhash reference decrement from b02a913b.

Fix erroneous return value.

reference.conf: Document fingerprint generation

[ci skip]

Fix stupid compiler errors by my stupidity and tiredness

blacklist: add blacklist_cancel_none

blacklist: add return statement. d'oh.

blacklist: fix precedence lossage

blacklist: if no blacklists were checked, properly terminate query.

build_rdns: minor cleanup

blacklist: tweak message for timeout

blacklist: give different messages for cancellation and timeout

charybdis 4-rc2.

Merge branch 'master' into release/4

Fix bug. Note: The second hunk is just an assumption. It's not called from anywhere.

Merge pull request #196 from jevolk/master

I typed MODRESTART by accident

Core modules cannot be unloaded, otherwise bad things happen.
Additionally some information is logged and passed to the operator
conducting a MODRESTART.

ircd: Fix capability entry name string ownership.
The entry->cap must be copied and exclusive to the entry for the
cap to be orphaned, even if literals are expected. Because modules.

charybdis 4-rc1.

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.