Merge pull request #73 from Argure/master

Explicitly drop SSLv3 connections (SSL_OP_NO_SSLv3) - might break TLS-ca...

Merge pull request #75 from attilamolnar/master+nullcharfix

Fix sending null char after ERROR when the server is full

Fix sending null char after ERROR when the server is full

Explicitly drop SSLv3 connections (SSL_OP_NO_SSLv3) - might break TLS-capable clients that still depend on SSLv23 handshake

ban: Fix build breakage.

A normal 'make' did not rebuild m_ban.c even though dependencies had
changed.

tools: Remove Hybrid 6 conversion tools.

Hybrid 6 is old enough that the conversion tools can go away now.
They are for I and K lines; the ircd.conf converter was already removed.

This removes compiler/analyzer warnings about these tools.

bandb: Don't apply empty ban list when bandb starts sending bans.

When bandb sends the ban list, it first sends 'C', then all bans and
finally 'F'. Only when 'F' is sent is ircd supposed to apply the bans.
Because of a missing break, 'C' also did 'F', clearing the ircd active
permanent bans until bandb sent 'F'.

The effect is pretty limited because having bandb send the ban list via
/rehash bans is uncommon and most bans will be enforced when reset.

libratbox: Add comment that case fallthrough is deliberate.

Gitignore vim swap files.

server: Don't read beyond the bounds of ServerInfo.ip and ServerInfo.ip6.

server: Remove two dead stores.

Explicitly pass the current time to deactivate_conf().

Some places depend on the ban not being destroyed.

Fix crash when there are two blacklist hosts without intervening reason.

This is invalid configuration; the first host is supposed to be ignored.

Merge pull request #70 from grobe0ba/master

Remove incorrect comment regarding location of functions in src/modules.c

Removed comment from src/modules.c that states certain function were moved into a file that no longer exists. (Circa ~2002)

doc/sasl.txt: forgot to wrap lines and sort references

doc/sasl.txt: drop dead links, add links to IRCv3 spec and relevant SASL RFCs

chmode: Check mlock when a local oper changes cmode +L/+P.

This check was erroneously removed when fixing /mode #channel f when +f is
mlocked. Mlock checks were restricted to the places requiring chanops
(other than viewing +eI lists); cmode +L/+P do not require chanops, but
still constitute a mode change that must be checked against mlock.

BAN: Don't schedule check_klines for 0 seconds in the future.

When receiving bans from a bursting server, if kline_delay is set to 0
(the default), rb_event_addonce will be called to schedule an event for
0 seconds in the future. While this works fine for the fallback
rb_event_run function, the epoll implementation ends up scheduling a
timerfd for the event in the past, which is then never executed.

While fixing this, I also made rb_event_add and rb_event_addonce reject
attempts to add events scheduled for 0 seconds in the future; they're
instead rewritten to run 1 second in the future.

Update 3.5 NEWS.

SJOIN: Fix crash when both mode parameter and UID list are missing.

Closes #63

conf: Make channel::channel_target_change option actually work.

Channel target change was forcibly enabled.

extensions/extb_channel: Allow the channel itself as target even if +s/+p.

extensions/chm_sslonly: Use some 4xx numeric for the join failure.

Merge pull request #64 from maxanton/fix_path_to_ssl_cert

example configs: change ssl_cert to etc/ssl.pem

example configs: change ssl_cert to etc/ssl.pem

The genssl script writes to etc/ssl.pem by default.

smoketest: Remove race conditions causing occasional test failures.

Chase change from example.conf to ircd.conf.example in various places.

configure: Rerun autoconf.

configure: Restore libratbox build.

This fixes building from scratch.

Merge branch 'master+sharedsqlite' of https://github.com/lstarnes1024/charybdis

configure: use pkg-config for sqlite

aclocal.m4: rerun aclocal

m4/pkg.m4: add pkg-config-0.24 macros

configure: move local autoconf macros to m4/charybdis.m4

configure: fix linking with shared sqlite on *BSD

configure: use --with-shared-sqlite instead of --enable-shared-sqlite

bandb: make Makefile work with BSD make again

ircd_parser: Fix inconsistent declaration of yyerror().

configure: add --enable-shared-sqlite option

When --enable-shared-sqlite is used, charybdis will be compiled with a shared
sqlite library if it is available.

channel: Remove write-only field.

channel: Stop pretending to support per-mode server capabilities.

After a change for dynamic server capabilities, the code to send out mode
changes was changed to use the capabilities belonging to the last mode
being sent out. This does not make sense; therefore, just use no
capabilities and remove supporting infrastructure.

server: Require EX and IE capabilities (+e and +I cmodes).

The code to send each channel mode only to servers supporting it was
broken a while ago and was not very useful anyway. Therefore, require
all connecting servers to support all standard channel modes.

Merge branch 'patch-1' of https://github.com/auscompgeek/charybdis

Merge branch 'master+umodes' of https://github.com/Adam-/charybdis

Fix buffer overflow in introduce_client and burst_TS6

If the client being introduced has more than 10 user modes send_umode()
will overflow ubuf

help: Update indexes

`make index` (add PRIVS for opers, CHANTRACE for users)

help: Add MONITOR to the user symlinks

Apparently the overrides are done at `make install` time, okay.

m_version: remove spaces from version string

"to make it machine parseable again", as in ircd-seven commit 03b2176b88a1.

m_version: display charybdis version in /version

Otherwise it just crashes when /version is used.

Closes: #60

info: Change N-line to connect block.

strip_colour: Strip '\x0F' (^O, formatting off)

Reported by ssbr on freenode:

chmode +c doesn't strip ^O, which turns off all previous formatting.
This can cause clients that internally use mIRC formatting to render messages weirdly,
e.g. highlighted messages in HexChat: <https://i.imgur.com/eDX8Aif.png>.

example confs: Remove AHBL blacklists, which are no longer available.

mkpasswd: Prompt for password twice

Fix const qualifier on raw_mask

Fixes compilation warning about losing const qualifier in assignment to
non-const variable

(cherry picked from commit 6d9c3f50944e1da3bf3a1be6454f85d6d6f7ab37)

Fix format string type mismatch in src/capability.c.

The type 'long' was correct on most machines but not according to the C
standard and Windows 64-bit.

Restore trailing whitespace in sqlite3 amalgamation.

Stripping the whitespace from the upstream-maintained sqlite3 code just
causes merge conflicts with no benefit.

Fix fd leak if /dev/urandom can be opened but not read.

Add first cut at 3.5.0 NEWS. [ci skip]

blacklist: Make the snote more like the +c one.

Merge pull request #53 from ShadowNinja/clarify_U+R

Clarify UMODE +R help text

Add DNSBL snotes for snomask +r.

Remove snotes on +r about GET/PUT/POST commands.

The server notice "HTTP Proxy disconnected: [<user>@<host>]" is
confusing and not particularly useful.

Remove trailing whitespace from all .c and .h files.

3134 bytes were removed.

s_conf: Don't leak log file paths when loading default conf

This change prevents the log file paths from being leaked when
rehashing. Additionally, fname_killlog was added to two places where it
was previously forgotten.

s_conf: Don't leak strings when loading default conf

This change prevents conf strings from being leaked when resetting the
conf to default prior to a rehash. Additionally, some default strings
are now rb_strdup'd into the ConfigFileEntry structure after loading the
conf so that they aren't allocated and then immediately freed by the
conf loading process.

libratbox openssl: Don't leak EC_KEY structures.

newconf.c: Remove end-of-line whitespace.

conf parsing: Fix freeing section names upon section end.

Avoid possible null dereference.

Fix a couple more string leaks.

commio: Properly zero a struct msghdr.

This bug is probably not noticeable because all specified fields are
initialized later.

commio: Explicitly ignore return value from setsockopt(TCP_DEFER_ACCEPT).

sslproc: Add missing break, still allowing zlib when ssl cannot be set up.

log_client_name: Remove MASK_IP case.

The MASK_IP case in log_client_name() was broken (because of a missing
break, it behaved as HIDE_IP). However, log_client_name() with MASK_IP
does not make sense anyway and is not used.

findforwards: Don't compare array to NULL as this is always true.

listener: Move error messages to something visible by default.

If a listening port cannot be opened, send error messages to opers with
snomask +s and ircd.log, instead of snomask +d and the ioerror log, which
both are usually disabled.

Also, restore information about what listener is having problems. This
was lost when report_error() was replaced.

doc/extban: Correct description of EXTBAN token in 005.

The change from EXTBAN=$:<types> to EXTBAN=$,<types> was made later on
and the documentation was not adjusted.

conf parsing: Fix memory leaks and clean up code a bit.

Charybdis currently leaks about 45-50k per configuration parse,
including every rehash. This change plugs these leaks by properly
iterating through all conf_parm_t structures to seek all strings that
should be freed and also by freeing the conf_parm_t structures
themselves.

These leaks have been present since the original rewrite of the
configuration parsing system in ircd-ratbox r11953.

Additionally, this change also cleans up and documents the parsing code
a bit.

ircd_parser: Fix whitespace errors.

SIGNON: Do not send uninitialized data if login name starts with '*'.

libratbox: Abort if the restart callback returns.

stats c: Simplify construction of flags string.

Perhaps this avoids false positives in static analysis.

commio: Add error string for SSL trouble.

server: Use rb_strlcpy() instead of strcpy().

An overflow should be impossible here, but be paranoid.

Simplify module path list, removing strcpy use.

Fix use after free.

conf: Remove dead store.

Fix various printf arg types.

part: Fix constness issue with part hook.

modlist: Fix address display on 64-bit systems.

modules: Fix bad sizeofs.

Replace double semicolons with single; place null statement on its own line.

No functional change is intended.

conf: Do not leak memory when modules::module is already loaded.

extensions/mkpasswd: Do not leak /dev/random fd.

balloc: Fix memory leak when get_block() fails.

bantool: Close .conf files when done reading.

Not very many files are read, so leaving them open is not a major issue.

extensions/roleplay: Fix memory leak on every use.

startup: Check return value of open /dev/null. Don't fclose stdin/stdout/stderr.

Open /dev/null for standard fds earlier, so a failure can be reported.
Do not fclose stdin/stdout/stderr but just overwrite the fds with
/dev/null.

webirc: Check validity of given IP.