charybdis 4.0.1

ircd: use correct buffer sizes

This fixes an issue with truncated SJOINs.

.travis.yml: don't use specific clang version

libratbox/src/mbedtls.c: check public/private keys match

modules/cap_server_time.c: fix logic error

cap_server_time.c:60:76: warning: comparison of unsigned expression < 0
                                  is always false
                                  [-Wtautological-unsigned-zero-compare]

    if (strftime(buf, sizeof(buf), "%Y-%m-%dT%H:%M:%S.", gmtime(&tv.tv_sec)) < 0)

.gitignore: don't include shtool and install-sh anymore

Subsequent to commit b5e8833c78f50028 these files are automatically
generated now.

autogen.sh: report correct exit status from failed commands

test(1) may overwrite the $? special variable, so cache its value
first.

Rename extensions/README to index.txt

Update license to more recent GPLv2

Remove appveyor and have osx install shtool before build

Trim autogen.sh a bit and add shtool to it

Fix regression introduced in previous commit

update export-syms.txt

Various Makefile improvements

doc/reference.conf: clarify that server link fingerprints aren't optional

[ci skip]

help/Makefile.am: silence construction of indexes, make build depend on index

Merge pull request #253 from Mkaysi/sno_whois

extensions/sno_whois.c: fix comment

Merge pull request #252 from Mkaysi/doc-snomask

help/opers/snomask: add extension snomasks

extensions/sno_whois.c: fix comment

When discussing #252 it was noticed that the comment is wrong.

help/opers/snomask: add extension snomasks

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.