Merge insp20

m_ssl_openssl Store a pointer to the OpenSSLIOHook object in SSL objects

Release v2.0.18

Remove some dead code found by Coverity

Merge pull request #944 from SaberUK/insp20+gnutls-rnd

[2.0] Use gnutls_rnd instead of gcry_randomize on newer GnuTLS versions.

Use gnutls_rnd instead of gcry_randomize on newer GnuTLS versions.

This is a modified version of 690c372. Fixes #905.

Merge pull request #940 from SaberUK/insp20+fix-makefile

[2.0] Fix some small bugs in the makefile template.

Use clang++ on FreeBSD 10 instead of g++.

Fix some small bugs in the makefile template.

- GCC in C++ mode should be called g++ not gcc.
- ICC in C++ mode should be called icpc not icc.

m_delaymsg Add option to disallow NOTICEs too

Fix m_banredirect causing bans added for hosts being rewritten as nicks

Fail AAAA DNS queries if inet_ntop() fails

Check the return value of getnameinfo() on Windows in insp_inet_ntop() and return NULL if it fails

Set the parameter of -b to the banmask actually removed

This is needed because of the case insensitive match

m_ssl_openssl Add compile time option that allows disabling renegotiations

m_ssl_openssl Add compile time option to enable ECDH

m_ssl_gnutls Add compile time option for allowing sha256 certificate fingerprints

Update <nationalchars> tag documentation

Fix watch away numerics, #937

m_delayjoin Only send JOIN on mode change if the mode being changed is a prefix mode

Initialize all fields of issl_session on module load in SSL modules

Add interface to SSL modules that allows other modules to obtain the raw SSL session of a socket

m_ssl_gnutls Refcount GnuTLS objects, free them when they are no longer in use instead of at /rehash ssl time

m_ssl_openssl Fix debug message

m_ssl_openssl Reset data_to_write for new sessions

m_ssl_openssl Free the ssl_cert object as soon as the session is closed instead of waiting for the next VerifyCertificate() or new connection

m_ssl_openssl Remove bogus errno assignment from CloseSession()

Merge pull request #935 from SaberUK/insp20+fix-crlf

Add a workaround for compiling modules with Windows line endings.

m_ssl_openssl Return an error from the IOHook read and write functions if the handshake returns 0

The meaning of a 0 return value quoted from the manual:
The TLS/SSL handshake was not successful but was shut down controlled and by the specifications of the TLS/SSL protocol.

m_ssl_openssl Add user-friendly config options for setting a few OpenSSL context options

Add a workaround for compiling modules with Windows line endings.

m_nationalchars Ask m_spanningtree to rebuild its hashmaps when the national_case_insensitive_map changes

Issue #923

m_spanningtree Rebuild serverlist and sidlist when receiving a specific Request

Issue #923

m_nationalchars Rebuild the hash map in m_watch when the national_case_insensitive_map changes

Issue #923

m_nationalchars Rebuild core hashmaps when the national_case_insensitive_map changes

Issue #923

Merge pull request #934 from SaberUK/insp20+fix-various-issues

Fix various issues detected by Coverity.

Kill some logically dead code detected by Coverity.

- m_watch: wl has already been confirmed to not be NULL on L161.

Replace dodgy use of const char* and memcpy with std::string.

This was backported from commit e01df63 in master.

Merge pull request #933 from SaberUK/insp20+fix-llvm34

Fix various warnings when building with LLVM 3.5.

Avoid calling methods on NULL pointers wherever possible.

The trick we use to allow this is undefined behaviour and is not
liked by LLVM. We should stop using it but it has the potential to
break to many things for a minor release.

Fix various warnings when building with LLVM 3.5.

- warning: absolute value function 'abs' given an argument of type 'long' but has parameter of type 'int' which may cause truncation of value [-Wabsolute-value]
- warning: 'this' pointer cannot be null in well-defined C++ code; pointer may be assumed to always convert to true [-Wundefined-bool-conversion]

m_ssl_openssl Allow configuring raw OpenSSL context options

m_ssl_openssl Disable session caching and session tickets

m_ssl_openssl Enable single (EC)DH use and disable SSL v2

Options enabled:
- SSL_OP_NO_SSLv2
- SSL_OP_SINGLE_DH_USE
- SSL_OP_SINGLE_ECDH_USE (if it exists)

Partial backport of #856 by @jvehent

m_ssl_openssl Clear the error queue before every SSL_* call

Merge pull request #931 from SaberUK/insp20+fix-non-gnu-stl

Fix hash_map.h on non-GNU C++ standard library implementations.

Merge pull request #930 from rburchell/revertfix

Revert "Fix an off-by-one on registration timeout handling."

Fix hash_map.h on non-GNU C++ standard library implementations.

Revert "Fix an off-by-one on registration timeout handling."

This seemingly breaks (occasionally timing users out a second early), possibly
due to rounding issues on time (User::age and Time() both return int, but a
whole second is comprised of many ms).

The root cause needs investigation, but we cannot leave a potentially breaking
bug in the tree.

This reverts commit 4414d644a163f3906f90b35186e07ce0383161f4.

Merge pull request #929 from rburchell/insp20

Fix an off-by-one on registration timeout handling.

Fix an off-by-one on registration timeout handling.

The use of > instead of >= here added a second onto the configured timeout.

We won't be breaking any existing configuration, because users.h explicitly
increases the limit if it's 0.

m_ssl_gnutls Re-set DH params when the gnutls_certificate_credentials_t struct is reallocated

Store Membership objects physically in the nodes of Channel::MemberMap

Add the insp::aligned_storage template

Reject parameters that begin with a colon in Channel::SetDefaultModes()

Don't try to set a mode in Channel::SetDefaultModes() if it needs a parameter but does not have one

Thanks to @docwhat for helping to track this down, fixes issue #924

Forward-port of https://github.com/inspircd/inspircd-extras/commit/a311dbcaea1abad68b87ebbb283fc6d75a207a53 into master

Fix incorrect cleanup order in InspIRCd::Exit()

classbase objects such as callers call LogManager::Log() on destruction unless ServerInstance is NULL

Remove listmode hiding support from the core

This is now handled by m_hidelist

m_hidelist Make minimum rank required to view lists configurable

Add m_hidelist that allows hiding the lists of listmodes

Remove IPv6 address compaction.

This code is incorrect and can produce addresses with multiple double
colons. No other IRCds do this and most systems today will give us the
compact version anyway.

Issue #914

Do not use the result of the dns cache when the query type of the result is different from the type of the query. #66

cmode_l Reject negative limits

m_services_account Fix uninitialized variable usage introduced by insp20 merge e244cb2c63b1ac1d85bdbb4691f7b1bd940ae804

m_samode Simplify command handler logic

Unset oper-only modes in User::UnOper() in a more sensible way

Get the user mode list from the ModeParser instead of trying all possible mode letters

cmd_mode Exempt remote users and servers from max modes limitation when changing user modes

Migrate code from ModeParser into cmd_mode (core_user)

- Process() that takes a std::vector<std::string>
- DisplayCurrentModes()
- DisplayListModes()

core_user Expand the MODE handler into its own file

Split ModeParser::DisplayListMode() into two parts

ShowListModeList() sends the list of one listmode to a user, DisplayListMode() calls it for each mode letter

Remove unused parameter passed to ModeParser::DisplayCurrentModes()

m_namedmodes Build and process a Modes::ChangeList when handling PROP

Pass Modes::ChangeList references to the OnPreMode hook, make it modifiable

This gets rid of the duplicated mode parsing logic in m_namedmodes

Call the MODE command handler in several modules instead of the old ModeParser::Process()

m_spanningtree Simplify processing non-prefix modes in FJOINs

Use ModeParser::ModeParamsToChangeList() instead of duplicating logic

m_spanningtree Apply FMODE mode changes using the new ModeParser functions

Let callers customize the begin/end positions for ModeParser::ModeParamsToChangeList()

This helps spanningtree when it deals with a vector of parameters where the modes begin at different positions

m_services_account Call ModeHandler::RemoveMode() instead of duplicating code

Remove irc::modestacker

m_spanningtree Remove CommandFJoin::ApplyModeStack()

Use Modes::ChangeList in ModeHandler::RemoveMode()

Replace irc::modestacker usage with the new ModeParser::Process()

Add a ModeParser::Process() overload that can process an entire Modes::ChangeList

This is a wrapper that calls ProcessSingle() repeatedly until the entire changelist is processed

Make it possible to resume processing a partially processed Modes::ChangeList

Return number of processed mode changes from ModeParser::ProcessSingle() and add a begin index parameter

Split out ModeParser::ModeParamsToChangeList()

Handle mode merges in ModeParser::ProcessSingle()

Validate mode parameters from ModeParser::ProcessSingle()

Split out ModeParser::ProcessSingle() from Process()

This applies up to one MODE line's worth of mode changes from a Modes::ChangeList

Add mode process flag MODE_CHECKACCESS

Remove ModeHandler::m_paramtype and GetTranslateType()

Remove ModeParser::LastParseTranslate and GetLastParseTranslate()

Remove ModeParser::LastParseParams and GetLastParseParams()

Compare Limits.MaxModes to the size of LastChangeList in ModeParser::Process()

Remove ProtocolInterface::SendMode()

m_spanningtree Send MODE/FMODE from the OnMode hook

If the MODE_LOCALONLY flag is set the mode change is not propagated

m_spanningtree Add function to serialize parameters in a Modes::ChangeList::List into MODE/FMODE format

Pass the modes to the OnMode hook as a Modes::ChangeList, pass ModeProcessFlags too

Populate a Modes::ChangeList object in ModeParser::Process()

Add Modes::Change and Modes::ChangeList