charybdis 4.0

charybdis 4-rc7

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.

tests: add serv_connect1 (exit_unknown_client)

Test serv_connect followed by exit_unknown_client.

An outgoing connection should not delete an incoming connection that
occurs during the connection attempt.

s_serv: connect using both IPv4 and IPv6 if unspecified

authd: assume all providers are running

Otherwise ident returns without setting itself running causing problems.

Move opm/blacklist before ident/rdns so that they can receive completion
notifications.

m_sasl: indicate client connection type for SASL

librb: remove zstring

rb_zstring_serialized leaks memory while doing nothing with its
arguments... nothing is using zstring so remove it

tests: add rb_dictionary1

This doesn't do much because replacing existing elements isn't allowed.

librb: dictionary_add mishandles replacing elements

This can only happen with soft asserts otherwise it will crash instead.

Nothing currently uses the return value of rb_dictionary_add.

charybdis 4-rc6

tests: send1: add tag tests to all functions

ircd: send tags on every message

Simplify linebuf by introducing fsnprint to manage a list of printfs.
Add a msgbuf unparse cache for send functions that loop.

tests: send1: fix sendto_channel_opmod (remote) to cover all scenarios

tests: send1: fix sendto_channel_opmod (remote) to call the right function

tests: send1: add missing CAP_SERVER_TIME test to sendto_channel_flags from remote users

tests: send1: test MODE_MODERATED with sendto_channel_opmod

tests: send1: load modules in a predictable order

So that tags are added in the same order every time.

# wanted: @time=2017-07-14T02:40:00.000Z;account=test :LChanPeon!username@example.test TEST #placeholder :Hello World!
#   seen: @account=test;time=2017-07-14T02:40:00.000Z :LChanPeon!username@example.test TEST #placeholder :Hello World!

tests: add missing modules symlinks

tests: add remaining sendto_* tests

ircd: check caps for user in sendto_common_channels_local

tests: add remote server message tests

Test sendto_one, sendto_one_prefix, sendto_one_notice and complete
sendto_one_numeric (for unregistered clients).

tests: add client util make_local_person_oper()

tests: add sendto_* test framework

ircd: log command in mod_add_cmd/mod_del_cmd error scenarios

ircd: substitution: fix buffer overrun if variable name is too long

Also fix the "ptr = ptr + (pptr - ptr)" aka "ptr = pptr" mess by
removing pptr.

remove unused variables

ssld: avoid clang static analysis warning

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

tests: fix warnings

remove unused variables

m_webirc: set sockhost before potentially using it to set host

Remove extra IP check, it's not necessary.

cap_server_time: check return values of string functions

cap_server_time: provide time with millisecond resolution

charybdis 4-rc5

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_cap: use rn_snprintf_try_append

tests: add rb_snprintf_append, rb_snprintf_try_append tests

librb: add rb_snprintf_try_append

For when it might fit, or it might not.

librb: rb_snprinf_append: handle error return value from vsnprintf()

tests: add rb_linebuf_put tests

librb: linebuf: reduce the number of "put" implementations from 4 to 1

Revert "Core modules cannot be unloaded, otherwise bad things happen."

This reverts commit b5cfad03195d566cd259154d212875fb238f5d80.

m_join: remove global variable parabuf

m_join: remove global variable modebuf

m_join: remove global variable para

m_join: remove global variable pargs

m_join: remove global variable mbuf

librb: linebuf needs to use the buffer limit from msgbuf_unparse_prefix

ircd: send: these buffers can be static

m_cap: Fix CAP LS generation

The `caplen` variable was unused, so the cap to be output wasn't considered
when determining whether or not it would fit.

librb: rb_rawbuf_length: do something if soft assert fails

correct the value of rb->len

tests: add msgbuf_unparse tests

tests: add substitution_parse tests

tests: add msgbuf_parse tests

msgbuf: correctly split buffers into IRCv3 tags and RFC1459 message data

linebuf: increase buffer size to accommodate IRCv3 tags

tests: add C TAP harness

librb: remove unnecessary NULL from the end of rb_string_to_array output

ircd: attach_conf: avoid clang static analysis warning

In the impossible scenario where ClassPtr(aconf) is NULL, reject the
client instead of dereferencing the NULL pointer.

make soft asserts better by allowing them to be used in expressions

wsockd: avoid clang static analysis warning

Don't set `x = 0` twice.

authd: opm: avoid clang static analysis warning

Remove unused `c` variable.

ircd: send_to_channel_flags: avoid clang static analysis warning

Set current_capmask and then use it, instead of referencing target_p->localClient->caps again.

This makes the purpose of current_capmask a bit clearer.

ircd: sendto_one_notice: avoid clang static analysis warning

target_p->from can't be NULL, and if it is then get_id(..., target_p)
dereferences it later in the function anyway

librb: rb_lib_loop: avoid clang static analysis warning

ssld: avoid clang static analysis warning

Don't set `x = 0` twice.

ircd: sendto_one_numeric: avoid clang static analysis warning

target_p->from can't be NULL, and if it is then get_id(..., target_p)
dereferences it later in the function anyway

authd: opm: avoid clang static analysis warning

Remove unused `lookup` variables.

ircd: get_or_create_channel: avoid clang static analysis warning

Use `len` after setting it.

ssld: avoid clang static analysis warning

ircd: sendto_one_prefix: avoid clang static analysis warning

target_p->from can't be NULL, and if it is then get_id(..., target_p)
dereferences it later in the function anyway

exit_remote_server: avoid clang static analysis warning

if IsServer(source_p) then the following must be all non-NULL:
 ->serv
 ->servptr
 ->servptr->serv

Remove unnecessary and inconsistent pointer checks.

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

librb: commio: 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.

ircd: listener: Document check_reject() behaviour

wsockd: Initialise ws_frame_hdr_t before using it

Outgoing messages have uninitialised data from the stack in the 3 reserved
bits of the opcode_rsv_fin value.

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

Revert "no really, it's dead people"

This reverts commit 40ac82e87820d9941bdd38571d0390122008e5d3.

charybdis 4-rc4

msgbuf: remove unused field parselen

ircd: substitution: fix NULL termination buffer overrun when the output is too large for the buffer

ircd: substitution: fix overrun with unterminated '}' in format string

handle a variable following an unterminated variable better

msgbuf: s_assert is not a substitute for proper code

(dns, m_alias, m_stats updated as msgbuf no longer includes s_assert.h)

ircd: check_server: don't allow a connection if that would exceed the class limit

ircd: serv_connect: don't try to connect if that would exceed the class limit

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.

no really, it's dead people

CREDITS: fix my nickname and update my email address

MbedTLS: Disable TLSv1.0

OpenSSL: Disable TLSv1.0

Also some misc other cleanups/additions to bring it in line with the
release/3.5 backend

GNUTLS: Provide a default priority string, disable TLSv1.0 in it

The user can still override this choice with the ssl_cipher_list option
in ircd.conf -- this is the only backend that will allow you to do so.

extensions/extb_channel: Allow matching secret channels.

This change modifies extb_channel to allow matching users in secret
channels, which prevents trivial ban evasion by setting the target
channel +s. Information leak due to this change is unlikely since the
attacker would have to know that the target channel exists, the name of
the channel (or guess it), have a specific user they wanted to know
whether was in the channel (and not know already), and the target user
would need to have something like autojoin-on-invite enabled (or any of
the other various ways hostname cloaking is attacked).

Documentation: Comment-out the OPM block and its options by default

The feature is not yet stable and is causing several issues.

When a remote MODRESTART command is received, it will pass through the
ENCAP module. The ms_encap function is responsible for dispatching the
command handler and then the modules will eventually be reloaded.

However, if the ENCAP module is reloaded to a different address, the
stack now contains the address of a function that no longer exists.

Also, in this version of the IRCd, the module restarting functionality
was located in a function that is itself located in a module, so things
will also go badly if that module is reloaded to a different address,
too.

Return immediately from the command handler and have the event loop
call the function responsible for reloading the modules instead.

c.f. release/3.5 commit db05a3621058

Reported-by: mniip (Freenode)

mkpasswd: avoid strdup(NULL) and the like if rb_crypt() fails

authd: don't exit() on OPM duplicate scanner errors, just ignore it

I run into this code even with a single kind of scanner with a single
port, so I'm not even sure how it thinks it already exists.

Perhaps it's being parsed twice, or something similar.

c.f. issue #229

mr_server: Handle certificate validation errors.

When certificate validation fails, the certificate fingerprint won't be
calculated, resulting in an attempt to format NULL into a log line
showing the fingerprint. Instead, add a different error message for
missing fingerprint (i.e. validation failed).