Merge pull request #416 from SaberUK/insp20+clang42-warnings

Remove unused variables detected by Clang 4.2.

Remove unused variables detected by Clang 4.2.

Merge pull request #414 from SaberUK/insp20+hashmap

[2.0] Use the preprocessor to detect whether hash_map is available.

Merge pull request #413 from SaberUK/insp20+httpd

[2.0] Use a better server name in m_httpd.

Use the preprocessor to detect whether hash_map is available.

Use a better server name in m_httpd.

m_ssl_gnutls Fix build with GnuTLS 1.x

Fixes #409 reported by @SaberUK

m_chanhistory Fix durations containing 'S' being rejected

Use GNU install on Solaris.

Merge pull request #407 from SaberUK/insp20+fix-openssl-detection

Fix OpenSSL detection on distributions which modify the version number.

Fix OpenSSL detection on distributions which modify the version number.

Merge pull request #403 from SaberUK/insp20+wextra-fixes

Fix warning: "const type qualifier on return type has no effect".

Fix warning: "const type qualifier on return type has no effect".

m_spanningtree Fix rare desync when a KILL crosses a message that has the killed user's prefix and modifies global state

m_maphide Ignore unregistered users

Fixes #400 reported by @SaberUK

Merge pull request #397 from Robby-/insp20+cmode_b-limit

[2.0] cmode_b: Correction to the maxbans conditional

cmode_b: Correction to the maxbans conditional
It was possible to set one more ban beyond the configured limit.

m_messageflood Ask to be the last to handle OnUserPreMessage/OnUserPreNotice

Fixes #394 reported by @JDowny

m_banredirect Deny the join to the redirect channel if it has a redirecting ban set that matches the user

Fixes #395 reported by @patatje368

m_banredirect Remove unnecessary hooks, cleanup

- Remove prioritization before m_banexception, it doesn't even hook OnUserPreJoin
- Remove OnChannelDelete handler: nothing to do in that case, our extension item is automatically freed
- Remove empty OnRehash handler

Replace a GNUmake idiom with the portable equivalent.

Include <list> in base.h to prevent build errors on OpenBSD.

Fix the poll socket engine on BSD's other than FreeBSD.

OpenBSD does not have KERN_MAXFILESPERPROC.

m_ldapoper Fix typo spotted by @lordsith49

Fixes #392

Fix errors in socketengine_ports so it compiles on Solaris

Thanks to fraggeln for reporting and testing

Fix property list arguments.

Fixed m_ssl_gnutls build on Windows; gnutls.h uses ssize_t which is defined in gcrypt.h

Release 2.0.10

m_callerid Fix memory leak in CallerIDExtInfo::free()

Add a flag to Command that controls whether an empty last parameter is allowed or not

m_ident Reject invalid and error replies

Add IS_SERVER() and REG_ALL checks to (mostly oper only) commands taking a target nickname

If a SID was passed as the target user parameter or when it's an unregistered user reply with the "no such nick" (or the moral equivalent) message

Remove ServerLimits::Finalise(), it's completely wrong

Truncate <limits:identmax>+1 long idents in User::ChangeIdent()

Fix unsafe iteration in DelMode() when unloading m_permchannels

Fix #380 on BSD make.

Merge pull request #380 from SaberUK/insp20+fix-clang-warning

Fix invalid argument warnings on clang.

Fix invalid argument warnings on clang.

m_svshold Fix crash when called with 2 params, only triggerable by u-lines

m_cban, m_rline, m_shun Don't broadcast when user is a local user

spanningtree sends an ADDLINE in this case

Change a few signed counter variables to unsigned so they roll over later

Fix expired xlines being treated as live ones in XLineManager::AddLine()

This fixes ADDLINEs not being propagated and /gline etc. failing for already existing but expired xlines

Special thanks to @JDowny (TinMan) for his detailed report and cooperation that made this fix possible

Fixes #306 reported by @TurkDesk
Fixes #379 reported by @JDowny

Unbreak m_geoip

Fix thread handle leak in StartServiceThread()

Fix broken RetrieveLastError()

- FORMAT_MESSAGE_IGNORE_INSERTS flag wasn't used and we never pass insertion parameters, this could cause FormatMessage() to fail
- The return value of FormatMessage() wasn't checked, if it returned zero (failure) then RetrieveLastError() returned garbage if err[] wasn't initialized previously

Fix typos in ebdaf368e137fc933e648ee88a08a4f83e796f87

Properly shut down stdin

Replace hardcoded ints

m_remove Fix potential desync caused by committing the PART before getting acknowledgement from the target server

Fix DLLManager leak when an exception is thrown in init()
This also caused the dlopen()'d library to stay open forever

Throw an exception when attempting to register an extension with a name that already exists
Previously this silently failed

Register a few extensions that weren't registered

Register all commands, modes and extensions using AddService()

AddService() throws an exception if an item cannot be registered, modules no longer need to worry about AddMode() etc. failing

Dynamically determine the size of the eventlist[] passed to Attach()

m_sqlauth was attached to I_OnUserDisconnect but didn't provide a handler for it, remove

Attach to events and register services in init()

Merge pull request #376 from SaberUK/insp20+fix-clean

Fix the list of files to be removed by make.

Fix the list of files to be removed by make.

Make LocalUserList an std::list

Add a typedef for LocalUserList

m_cgiirc Fix wrong announcement when the host in WEBIRC is too long

m_cgiirc Don't apply resolved host or send notifications when the user is quitting (matched a line)

m_cgiirc Don't resolve IPs when <performance:nouserdns> is true

m_cgiirc De-duplicate code, recheck elines after ip/host changes
Fixes #350 reported by @MistrX

New functions:
- RecheckElineAndClass()
- ChangeIP() - decreases clone counts for a user, calls SetUserIP, increases clone counts after the new ip is applied
- HandleIdentOrPass() - common code extracted from CheckIdent() and CheckPass()

m_cgiirc Remove dead code, warn when a <cgihost> block has an invalid type value, misc. cleanup

m_dnsbl Add usednsbl option to connect classes
Don't touch users in classes where it's off

Reorder local user initialization steps

OnSetUserIP hook now runs after the user has a connect class and the g/k/z lines were checked (ip only, no host at that point)
Fixes #360 reported by @JDowny
Allows #336

cmd_rehash, m_spanningtree Fix REHASH messages duplicating at each hop
Fixes #375 reported by @JDowny

Merge pull request #374 from rburchell/ldapauth_vhosts

Ldapauth vhosts

ldapauth: register both extensions

ldapauth: RAII returned ldap message.

This ensures it is always freed, and coincidentally fixes freeing in the case of
LDAP errors.

ldapauth: fix memory leak

ldap_get_dn() results need to be freed. Use a RAII wrapper to do this.

Original code by Attila Molnar

ldapauth: Allow setting virtual hosts on identification with m_ldapauth.

<ldapauth:host>, when set, will be applied to users identifying with ldapauth.
The host can also take formatters from fields set on the DN of the user.

ldapauth: Add missing verbose logging to required attributes.

ldapauth: Rework required attributes code to use only one exit path.

This makes the upcoming patch to add optional virtual host support cleaner.

Change empty string assignments to .clear() or remove them entirely
Part 2 of ba5c0db795824c3fc1ad48ce332d7bdc440cb77f

Have FindUUID(const char*) call FindUUID(const std::string&), not the other way around

m_spanningtree Remove redundant params.size() checks

TreeSocket::ProcessConnectedLine() and CommandParser::CallHandler() only call the handler when there are enough parameters

Fix GnuTLS certificate generation on OS X.

Silence OpenSSL deprecation warnings on OS X.

m_noctcp Make extban 'C' work as intended

m_check, m_namesx, m_timedbans Minor cleanup

Simplify UserManager::AddLocalClone()/AddGlobalClone()

Remove TAXONOMY from opers.conf.example, it doesn't exist in 2.0
Fixes #369 reported by @DjGolia

m_ssl_gnutls Dynamically detect the number of certificates in the certfile
Remove the "certcount" setting, as it's no longer needed
When finished reading the certs, resize the buffer to the actual number of certs read

m_nokicks Remove u-line check, OnUserPreKick doesn't run in that case

m_customtitle Minor cleanup

- Remove TRANSLATE() as it makes no sense here (user parameter is not a nick)
- Compare strings with == instead of strcmp
- PassCompare uses strings, pass them as-is without c_str()

m_jumpserver Fix wrong nickname in the numeric when redirecting all users immediately, cleanup

- Fix typo
- Iterate string with iterator
- Move variables into the command handler that aren't used elsewhere

m_securelist Update description

m_remove Cleanup

- Command handler concatenates when maxparams is specified, don't do it ourselves
- String references to user and channel names are fine, no need for .c_str()

ServerConfig::ServerName is a std::string, don't call .c_str() on it when passing it to Channel::WriteChannelWithServ, etc.

m_remove, m_services_account Don't check whether nicks are u-lined, checking the server is enough

Change empty string assignments to .clear() or remove them entirely

m_globalload Fix wrong error message being sent when the module given to /gunloadmodule is not found

Remove usage of deprecated CallCommandHandler() and IsValidModuleCommand()

Remove calls to ListModeBase::OnCleanup (deprecated, empty function)

Fix m_ssl_gnutls under GCC on recent versions of GnuTLS.

Commas at the end of enumerator lists are valid in C99 but are not
valid in C++ before C++11. This causes a build error on GCC when
using -pedantic.

Merge pull request #365 from StevenVanAcker/unhook-uid

unhook --uid from --system in configure script

Merge pull request #366 from StevenVanAcker/autoconnect-reminder

added warning that autoconnect is now a separate tag

added warning that autoconnect is now a separate tag

unhook --uid from --system in configure script
Specifying --uid allows the ircd to run as a specific user, while
--system fixes the install directories to be spread all over the system.
Specifying --uid shouldn't imply --system. This fix allows a uid to be set
while not interfering with --prefix

Remove usage of the deprecated ConfigReader