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.

verify_password(): more error checking, better logic, fix minor bug

* ci->crypt() and ci->salt() can return NULL in rare circumstances
* don't duplicate logic for generating a new password hash
* when the user's password is encrypted but a crypto module is not
  loaded, don't complain if the /encrypted password/ is '*', not
  if the /user-supplied password/ is '*'.
* remove extra line at end of file

Atheme 7.2.9

Merge pull request #541 from mniip/free

saslserv/main: Fix use after free during impersonation

saslserv/main: Fix use after free during impersonation

Atheme 7.2.8

This is a security release. Release 7.2.7 is affected.

saslserv/main: free sasl_sourceinfo_t after use

memoserv: unregister hooks when unloading

This will avoid a crash discovered by Xenthys in issue #531

memoserv: let user know (on identify and /away) when their inbox is full

c.f. issue #531

Merge pull request #530 from aaronmdjones/pbkdf2v2-improv

Improve the PBKDF2v2 module

pbkdf2v2: make digest and rounds configurable at runtime

atheme.conf.example: better highlight the pbkdf2v2 crypto module

pbkdf2v2: remove obsolete compatibility function

All modern supported versions of OpenSSL provide this function

c.f. issue #528

pbkdf2: remove obsolete compatibility function

All modern supported versions of OpenSSL provide this function

Fixes #528

email templates: Fix leading whitespace

The leading whitespace is barely visible in many email clients
and was often cause for sending the message directly to the channel
for clients that do not interpret " /command" as a command. This
was most notable for qwebirc.

(cherry picked from commit 72821fa6eea3e7a20c0e3286f709fe6cc0e214f5)

atheme 7.2.7

NEWS: update for 7.2.7

[ci skip]

(cherry picked from commit bec88cb93d546b8783167c4a6613da59dc5a9311)

i18n: update po/POTFILES.in

(cherry picked from commit 13ae2f56ef829a07649e7e1e7af0e1b6ebdfcdd2)

nickserv/{setpass,sendpass_user}: fix a typo breaking the build

(cherry picked from commit e5ce76c8e6c20f12445b35e23e0ae54eb97eb1e0)

chanserv/help: include INFO in short help instead of RECOVER

The RECOVER command should rarely be needed unless handing out channel
access like popcorn is the norm in some channel, in which case we can't
really help them anyway; as such there's little point in listing it by
default. INFO on the other hand is widely used as *the* way to ask
services for basic information on a channel, so it makes sense to be in
the short help (which nickserv/help does as well).

(In fact, on freenode, the most common use of RECOVER seems to be by
people who use it as the primary way to get opped, given that network's
recommendation to avoid auto-op and their default flag set reflecting
this. I suppose some people misinterpret this as having lost control of
their channel. Presumably it's debatable who's to blame here, but maybe
removing RECOVER from the default short help listing will help avoid
that sort of problem. Either way, it shouldn't hurt.)

(cherry picked from commit 4396f465e9f7fd993432d84e275a38703d215aea)

nickserv/{setpass,sendpass_user}: more helpful message for logged-in users

Sometimes, people try SETPASS or SENDPASS when all they actually need is
SET PASSWORD, so point them there if applicable.

(cherry picked from commit 1565bfa466b8bb4e643f111981f0feb93d24a902)

helpserv/ticket: better integration of close reason

(cherry picked from commit 9ebde9161004143f5852eca5ca4528beec65bf05)

nickserv/info: also show "Channels" line if mu == si->smu

(cherry picked from commit 09e3d0f4e957b822e4ba32e0b63c91b2ffe4e930)

atheme.conf.example: fix case in previous commit

proxyscan/dnsbl: use kline_add and correct db

entries.  also add modules and example to conf.example.

proxyscan/dnsbl: mark strings for i18n; style

As pointed out by @maxteufel, uppercase the name of the configuration
entry (dnsbl_action) for consistency.

(cherry picked from commit 796fb830251a0dcbd1524ca1041a32be3183c2b6)

proxyscan/dnsbl: fix crash on scanning users without IPv4 address

"not too concerned right now", they said.
"A sscanf worked fine for chary for many years, it'll be fine here",
they said.

(cherry picked from commit 407fb1cfcac1898ea97c6b77f9b82fe6a5f322b3)

proxyscan/dnsbl: log /os set dnsblaction

(cherry picked from commit db3b37b409e189e96cd58c5d95c9d77631104454)

proxyscan/dnsbl: use enums where appropriate

(cherry picked from commit b2566f40fc5c347bdd6b97af030a1e07e8fa9bfb)

proxyscan/dnsbl: properly abort DNS queries

Fixes shalture/shalture#49.

(cherry picked from commit f653a85ff90dae6313c5a70b2aa85adb97ba0dc0)

proxyscan/dnsbl: fix module header

(cherry picked from commit 4a0347ae3ff3327bb2d759bf504e2a29b6732f02)

help/default/nickserv/cert: more clarification about fingerprints

Clarify the it is even entirely agnostic of the fingerprint format.
Furthermore, add a note about the usage of CERT ADD without any
arguments.

(cherry picked from commit 322ebac90d614fa7fd88a2b93d5ea15b874b4ede)

help/default/nickserv/cert: clarification about fingerprints

Add a note about the fact that the CertFP implementation is agnostic
with regards to the fingerprint algorithm used by the IRCd. Otherwise,
users could be confused due to the examples.

(cherry picked from commit 9304ca9006c893298a2033f9cb5a6781f180bc6b)

InspIRCd: Fix atoi logic error preventing maximum rejoindelay value

c.f. issue #509