Use ERR_YOUREBANNEDCREEP instead of NOTICE when a user is banned.

This is specified in RFC 1459 so we should probably use it.

Remove pointless fd == INT_MAX check from StreamSocket::DoWrite()

Remove some unnecessary NULL checks

Reduce std::string::substr() usage

substr() returns a new string while erase() and assign() modify the existing one

Remove the unused OnGlobalOper hook

Merge pull request #959 from Alef-Burzmali/master+fixcloaking

Fix a regression in m_cloaking

cmd_invite Send RPL_AWAY after RPL_INVITING if the target user is away

This is consistent with other server implementations and the RFC

Fix a regression in m_cloaking

5c7db14a5 made the cloak use hex-encoded hashes instead
of the binary output.

Merge branch 'master+listmode'

Merge branch 'master+flatmap'

Merge branch 'master+serverlimits'

Merge pull request #953 from SaberUK/master+configure-oversights

Fix some oversights from #895.

Fix some oversights from #895.

- Always write the configure cache file as it is needed for updating
  the makefile when --update is called.
- List the modules which have been enabled in the "configuration
  complete" message.
- Silence some harmless warnings about installing PEM files which
  don't exist.

Drop setter is empty check when displaying listmodes

The setter cannot be empty unless the nick of a client is empty which is not possible

Display listmodes in the order in which they are in the container instead of reverse iterating

Change listmode storage type to vector

Change type of some associative containers to their flat versions, including Extensible storage

Add flat_[multi]{map,set} containers

Add typedef ModeParser::ModeWatcherMap, remove ModeWatchIter

Add typedef PrivSet for OperInfo::AllowedOperCommands and AllowedPrivs

Merge branch 'master+callerid' into master+flatmap

m_callerid Change CallerIdDataSet (wholistsme) to be a vector

m_callerid Add typedefs for containers in callerid_data

m_callerid Simplify code interacting with containers

Pass the empty tag to Limits when constructing a ServerConfig, remove default ServerLimits constructor

Add ServerLimits constructor that reads limits from a ConfigTag and use it

Initialize ServerConfig::EmptyTag using the init list

Merge branch 'master+timeout'

Link m_ldap against libldap_r

Merge pull request #941 from SaberUK/master+test-build

Add support for extra modules and custom compilers to test-build.

Add support for extra modules and custom compilers to test-build.

Merge pull request #895 from SaberUK/master+configure-misc-cleanup

Various improvements to the build system.

Fix builds on OpenBSD spamming the console with warnings.

Fix warnings in m_ssl_openssl about the use of `long long` on C++03.

Move common code to make::common from make::utilities.

The other code in that file will be removed very soon so it has
been left alone to avoid merge conflicts.

This will help prevent insp20 merge conflicts in the future.

Implement support for distribution specific version labels.

This will help us determine whether a version has been packaged by
a downstream distribution who quite often make arbitrary changes
which are a pain to provide support for.

Debian, we're looking at you.

Improve build system compiler detection.

- Check the CXX environment variable inside find_compiler instead
  of doing it separately.
- Use a test file instead of checking for version output. This
  ensures we are using a working compiler.
- Pipe the output through run_test which makes debugging problems
  considerably easier.
- Detect Xcode versions of Clang properly by returning the LLVM
  version number instead of the Xcode one.

Improve configure cache file handling.

- Add a version number to the configure cache file.
- Disable configure cache file in non-interactive mode.
- Rename configure cache file to .configure.cache to avoid 2.0 files.
- Use run_test to produce the "reading .configure.cache" message.

Miscellaneous improvements to configure.

- Clean up various minor sections of the code.
- Remove OpenSSL and GnuTLS detection plumbing.
  - This will soon be the job of modulemanager.
  - This did not work in non-interactive mode unlike --enable-extras.
- Rework runas user handling.
  - Add the --gid configure option.
  - Accept either an id or a name in --gid and --uid.
- Rework the question flow in interactive mode.
  - User is no longer asked detailed questions unless they want them.
  - Socket engine questions have been removed.
- Automatically enable non-interactive mode if STDIN or STDOUT are not
  a tty.

Add Perl module for console related code.

- Move prompt_* methods to this module.
- Add methods for printing errors and warnings easily.
- Add colour code helpers and switch all code to use them.

m_flashpolicyd, m_httpd Store sockets in a intrusive list

m_flashpolicyd, m_httpd Handle timeouts using the Timer system

m_httpd Clean up HttpServerSocket constructor

m_httpd Remove unused container

m_httpd Always timeout connections, set default timeout to 10 seconds

Add stdalgo::isin() and use it to simplify code

Add stdalgo::erase() and use it to simplify code

Remove if (!this) check from ConfigTag::readString()

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