modules/chanserv/close: check correct flags variable for log target

A user reported that the ChanServ CLOSE command was not working for
their channel, saying that the channel could not be closed.

MC_HOLD and CHAN_LOG both have the same value (1), but the former is
for flags in `struct mychan`, and the latter is for `struct channel`.

This lead me to believe that the channel was defined as a log target,
when in reality it was checking the wrong flags field and deducing
that the channel was a log target because it was held.

Check the correct flags field.

(Cherry-picked from commit 51e84e63e5b696ef39d5)

Reported-by: @hello-smile6

Atheme IRC Services v7.2.12

modules/saslserv/main: backport 7.3 commits for pending EID login

This backports commits 4e664c75d0b280a052eb & ceb0235695e6736ce2ab
from the master branch.

The IRCv3.1 SASL specification contains the following wording:

    If the client completes registration (with CAP END, NICK, USER
    and any other necessary messages) while the SASL authentication
    is still in progress, the server SHOULD abort it and send a 906
    numeric, then register the client without authentication.

We were relying on this behaviour (which was our mistake; it's a
SHOULD, not a MUST), which turned out to be implemented in every
IRC server daemon (that supports SASL) that we are aware of. This
means that if someone completes registration without having completed
an SASL negotiation, the SASL session would be aborted before the
client is introduced to the network. At that point, the session would
not exist and the client would not be logged in.

The InspIRCd developers changed this behaviour in the
inspircd/inspircd@407b2e004cf66e442771 commit. It no longer aborts
negotiation when a client prematurely completes registration.

This means that if the client is attempting a multi-step (challenge-
response) authentication mechanism, and that mechanism caches user
credentials at some point before completion, the client can pre-
maturely end negotiation and get logged in as that user.

Worse still, SASL impersonation lets the attacker set the authzid to
their intended victim, allowing them to login as anyone, even if they
don't have a challenge-response authentication credential configured.
This does not exist in version 7.1; the victim's account there has to
have such a credential to be vulnerable to this attack.

Vulnerable configurations are as follows:

- All of:
  - InspIRCd 3+
  - Any of:
    - Atheme 7.1 (any version)
    - Atheme 7.2 (any version before 7.2.12; this commit)
    - Atheme 7.3 (any version before commit 4e664c75d0b280a052eb)
  - Any of:
    - The saslserv/scram module is loaded
    - The saslserv/ecdh-x25519-challenge module is loaded
    - The saslserv/ecdsa-nist256p-challenge module is loaded

This is a fix for a security vulnerability. The master (7.3) branch
was already fixed in 4e664c75d0b280a052eb, but the scope of the
problem was not fully known at that time. The 7.1 branch is no longer
supported, is not receiving security updates, and will not be patched;
users of the 7.1 series (using an IRCd that does not abort the SASL
session when the client prematurely completes registration) must
upgrade, or unload the `saslserv/ecdsa-nist256p-challenge` module.

This problem was discovered by and reported by @edk0.

corestorage: don't try to write if we couldn't open a database

(cherry picked from commit f26b1b762e650d1a39f8800deedf20ea9da92c9c)

Merge pull request #785 from fasterthanlime/patch-1

Comment typo: nore ➡️ node

Comment typo: nore ➡️ node

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.

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.

include/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.

Git tree (not tarballs): include/mkserno.sh: remove --broken switch

Some very old gits do not support this; it was added in 2.13.0 (2017).

We already sort of detect broken submodules in the configure script,
so this should be safe to remove. Unless the user deliberately
corrupts their repository, there's no difference.

Atheme IRC Services v7.2.11

libathemecore/conf.c: sfree() does not exist in this branch, use free()

Disable Travis CI

This no longer works.

configure, README.md, GIT-Access.txt: backport from master & adjust

This will enable future releases on this branch to be more user friendly.

modules/contrib/: bump to latest HEAD

libmowgli-2/: bump to latest HEAD

libathemecore/conf.c: fix minor memory leak with hide_xop

The entries in global_template_dict are heap-allocated structures;
hide_xop was deleting the entries without freeing them, leaking a few
bytes each time the config was loaded.

(cherry picked from commit f22b8f34a5174e87d3cc2a008ca3317fe33ac61e)

atheme.conf.example: add +e to default template example

(cherry picked from commit e66fccfc08b4043290bf64ca2ea6d78f8cfc632a)

CA_ constants: include CA_EXEMPT (+e) where appropriate

Some CA_ constants including CA_REMOVE weren't updated when CA_EXEMPT
was split off from it; add it to CA_SUCCESSOR_0 and CA_SOP_DEF.

CA_USEDUPDATE remains without +e on purpose as that flag merely
signifies ability to join the channel, comparable to +i.

(copied from commit ff6708a7672e82a393a60c84dab743b79892acfe)

Fixes #695.

nickserv/multimark: forbid unloading due to data loss

nickserv/multimark: use irccasecanon for restored nicks

modules/nickserv/register: check e-mail address validity earlier

modules/contrib/: bump to latest HEAD

modules/chanserv/akick: fix unload crash with akicks that have timeouts

The module did not take care to cancel any outstanding expiry timers on
deinit, leading the event loop to (eventually) call a function that no
longer exists.

Backport of commit f28269362bb59ef6673d from master.

Reported-By: Ed Kellet <redacted>

Travis CI: Bump configuration to same state as master

Add SECURITY.md

Quick summary:

- we'll support the current and previous release series once 7.3 is out
  (for now, only 7.2 is supported)
- poke us on IRC or email security@atheme.org to report stuff
- we do coordinated disclosure, full disclosure after two weeks

atheme.conf.example: document needoper not being inherited

modules/crypto/pbkdf2v2: add HMAC-MD5 support (verify-only)

This is cheap (the necessary scaffolding is already present) and easy,
and will enable the migration of password hashes from more sources.

It will always re-encrypt the password, after verification, with a
stronger digest algorithm, because you cannot configure the module to
use this one.

This is a back-port of commit 7b1204391f5bd25b5409728aac46b2855f7f9433
from the master (v7.3) branch, for forward-compatibility.

modules/contrib/: bump to latest HEAD

[ci skip]

Add Preliminary Turkish Translation

modules/contrib/: bump to latest HEAD

configure: reliably test for res_query(3) if building modules/contrib/

This makes building contrib modules on non-Linux platforms possible,
where res_query(3) is in -lc, not -lresolv.

modules/contrib/: bump to latest HEAD

Atheme v7.2.10-r2

modules/crypto/pbkdf2v2: fix regression from v7.2.9

Interoperability tests were performed between 7.2.9 and master,
and 7.2.10 and master, but not 7.2.9 and 7.2.10. 7.2.10 and
master use base64-encoded salts for new hashes, and master has
a verify function which doesn't require string-equal output.

However, 7.2 doesn't have that feature, so crypt() must output
a string-equal hash to all of the previous outputs over all
versions. What I failed to notice was that crypt() was assuming
the salt was base64-encoded, so if it wasn't, password
verification would fail; the hash is the same byte-wise, but it
is encoded as a string differently.

While we're at it, make sure the parameter upgrade system takes
account of various salt lengths, allow the user to configure a
different salt length (as if on master), and make sure we erase
the password from the stack after we're done with it.

Reported-by: xerox123

Atheme 7.2.10

modules/misc/httpd: mod_deinit: close listener & children immediately

Closes #628

libathemecore/connection: add func to close listener & children immediately

modules/chanserv/{access,flags}: use flags setter information if available

modules/crypto/argon2d.c: argon2d_hash_raw(): fix uninitialised data path

If the first blake2b_long() call in argon2d_hash_raw() fails, it would pass
uninitialised data in bhash_bytes[] to argon2d_load_block(). Similarly if
the second call fails, same thing. Lastly, the return value should depend on
whether the final call succeeds.

Fix this by testing if it fails, and add a function attribute to all boolean
functions that will result in a diagnostic if their return value is not
tested (or under Clang, explicitly discarded). Adjust other callers of those
functions to also test for failure.

This issue was found by the Clang static analyzer.

modules/crypto/pbkdf2v2: use SASL mechanism name for SCRAM digests

[ci skip]

include/common.h: bump e-mail max length to 254 characters

<http://www.rfc-editor.org/errata_search.php?rfc=3696&eid=1690>

[ci skip]

modules/contrib/: bump to latest HEAD

include/common.h: add internal error fault code from master (needed for new contrib)

libathemecore, modules: some spelling fixes

groupserv/main: use correct database backend functions

These 2 functions behave identically in opensex in this scenario but
that is an implementation detail that should not be relied upon.

groupserv/main: write out group definitions before their access lists

If a group '!a' includes an access list entry for group '!b' (which is
lexicographically after) then upon read-back (when restarting services)
the access list contains an entry for a (for the moment) non-existent
group which is ignored. This results in data loss.

The fix is simple: Write out all group definitions (& their flags and
metadata) and then all group access lists.

Reported-by: Samuel Hoffman <redacted> (sjh)
Reported-by: Aaron M. D. Jones <redacted> (amdj)
Reviewed-by: Janik Kleinhoff <redacted> (ilbelkyr)

Travis CI: Use old config option for old branch

[ci skip]

Travis CI: Backport configuration from master

[ci skip]

include/protocol/*.h: use better include guards

While authoring the last commit I neglected to look at the other files.

[ci skip]

include/protocol/inspircd.h: rename include guard

[ci skip]

modules/saslserv/ecdsa-nist256p-challenge: use o2i_ECPublicKey properly

Check its return value, and fail if it does.

modules/saslserv/ecdsa-nist256p-challenge: use arc4random_buf instead

RAND_pseudo_bytes() is dangerous and we already have our own RNG.

modules/crypto/pbkdf2v2: backport from master

See commits 68044e2a9cd2b5d3e97b...cf8e0677e90beb832665 for details.

Build System: backport from master

See commits bcae7b4e445e149179b4...e2872439e4a6f4eb353e for details.

libathemecore: backport base64 codec from master

GitHub: We don't use C++, stop labelling our headers as so

libathemecore/crypto.c: crypt_string(): generate salt if not given one

remove unnecessary space

modules/crypto/pbkdf2v2: return NULL instead of false

clang trunk missed this.

modules/crypto/pbkdf2v2: backport e3b0d7bdb831db694272 from master

modules/crypto/pbkdf2v2: add debugging log messages

modules/crypto/pbkdf2v2: backport 376472c9ac22e1832cd2 from master

modules/crypto/pbkdf2v2: add SCRAM-SHA support

modules/crypto/pbkdf2v2: backport b3a33071d4e9d5dbca16 from master

modules/crypto/pbkdf2v2: don't ask OpenSSL for digest length

We know it's only ever going to be SHA1 (20 bytes), SHA2-256 (32 bytes)
or SHA2-512 (64 bytes), and we're already in a switch() statement
converting a PRF ID into a digest algorithm, so we may as well set the
digest length in there too.

modules/crypto/pbkdf2v2: reduce size of sscanf(3) buffers

0x2000 (8 KiB) is definitely sufficient to mitigate any potential
inadvertant overflow, and an attacker with a crafted malicious
database would have been able to overflow the old 0x8000 (32 KiB)
buffers anyway.

modules/crypto/argon2d: reduce size of sscanf(3) buffers

0x2000 (8 KiB) is definitely sufficient to mitigate any potential
inadvertant overflow, and an attacker with a crafted malicious
database would have been able to overflow the old 0x8000 (32 KiB)
buffers anyway.

modules/crypto/pbkdf2v2: offload most computation to dedicated function

modules/crypto/pbkdf2v2: rename variables consistent with master

modules/crypto/pbkdf2v2: allow variable salt length for possible migrations

modules/crypto/pbkdf2v2: use PRF macro for default variable

modules/crypto/pbkdf2v2: adjust macro names for iter count

modules/crypto/pbkdf2v2: add macros for PRF algorithms

modules/crypto/pbkdf2v2: fix up some format strings

modules/crypto/pbkdf2v2: add support for HMAC-SHA1

The master branch (what will become 7.3) got this too, but it was
added there for future possibility of implementing SASL SCRAM support.

We will not be implementing support for SCRAM in version 7.2, but
people who used the pbkdf2v2 module with SHA1 in version 7.3 might
want to downgrade to version 7.2, so we need support for verifying
those hashes too.

Fix '--docdir' configuration option.

Documentation should now be installed according to the '--docdir'
argument passed to configure.

modules/crypto/argon2d: don't mix format strings for scanning & printing

The inttypes.h header provides format specifier macros for scanning, so
use those for parsing instead of the printing ones, just incase we're
building on a machine where they are different.

NEWS.md: Document new crypto module

modules/crypto/argon2d: backport module+documentation from master

modules/crypto/pbkdf2v2: reuse previous idiom for length of salt array

This is equivalent but more robust.

modules/crypto/pbkdf2v2: reorder variables by usage

modules/crypto/pbkdf2v2: use correct variable types to function calls

PKCS5_PBKDF2_HMAC() and base64_encode()

modules/crypto/pbkdf2v2: replace switch() with if-elseif-else

It's so short anyway...

modules/crypto/pbkdf2v2: move variables to where they are needed

modules/crypto/pbkdf2v2: realign macros with spaces not tabs

modules/crypto/pbkdf2v2: make mod(de)init static and prefix their name(s)

Calls to these functions should be made through the pointers in the MAPI
header anyway.

modules/crypto/pbkdf2v2: silence unused parameter warnings

modules/crypto/pbkdf2v2: rename conf_pbkdf2v2_table to pbkdf2v2_conf_table

This is consistent with the prefix on the other symbols.

modules/crypto/pbkdf2v2: rename another function

modules/crypto/pbkdf2v2: fix wrong variable name

modules/crypto/pbkdf2v2: rename function

modules/crypto/pbkdf2v2: align member assignment in crypt_impl_t

modules/crypto/pbkdf2v2: use 0/1 instead of false/true in bool function

modules/crypto/pbkdf2v2: rename a function argument for consistency

modules/crypto/pbkdf2v2: test for exact required return value of sscanf(3)

modules/crypto/pbkdf2v2: add appropriate qualifiers to function arguments

modules/crypto/pbkdf2v2: put function names on their own line

Makes grepping for '^name' easy.

modules/crypto/pbkdf2v2: return error if hash generation would overflow

modules/crypto/pbkdf2v2: return error if salt generation would overflow

modules/crypto/pbkdf2v2: use arc4random_buf() to generate salt

This is slightly more efficient and less ugly than calling arc4random()
multiple times.

modules/crypto/pbkdf2v2: remove unnecessary memset(3) calls

These are left over from module development where they made
debugging easier.

modules/crypto/posix: fix potential NULL deref on password verify

If we have an encrypted password from the database that does not
contain a '$' in it (as is the case for the original deprecated
pbkdf2 module) then when posix tries to verify the hash it will
segfault because we assume a '$' is present.