Avoid buffer overflow with 64-bit long.

Fix Darwin / clang harder

Clang apparently doesn't like "static inline", so just remove "inline".

Fix compilation under Darwin

AC_C_INLINE with -Werror caused the configure check to fail and #define
inline away. This caused <ctype.h> to define various functions like
isspace() in multiple translation units, leading to link errors because those
symbols were defined in multiple files.

Fix more Coverity warnings

string_buffer_append_substring: Allocate space for NUL terminator

slab-read: Close the log file after we finish reading it

sar: Plug some memory leaks

parse_record_int: Do not leak memory on ABORT() calls

proto-p10: Fix Coverity warnings

irc_p10_ntop: Check ii < 8 before reading ip->in6[ii].
irc_numeric: Add missing va_end(arg_list).
cmd_burst: Check that the numnick is at least 4 digits long.
cmd_num_gline: Remove redundant check for (argc > 5).

proto-common: Fix Coverity warnings

cmd_stats: Fix format string types.
mod_chanmode_apply: Move free(bn), solely to make Coverity happy.

opserv: Fix Coverity warnings

opserv_join_check: Don't opserv_alert() if we don't have an OpServ.
add_user_alert: Ignore alerts with no discriminators.
opserv_discrim_search: Handle the case when there are multiple channels
  and the user is not in the later channels.

nickserv.c: Fix Coverity warnings

nickserv_modify_handle_flags: Don't send_message(NULL, ...).
nickserv_define_func: Fix what looks like a typo.

modcmd: Fix Coverity warnings

svccmd_invoke_argv: new_argv must be static for us to use it as argv.
cmd_command: Make sure we never run past the end of HANDLE_FLAGS[].
modcmd_describe_command: Don't overrun buffers if MSG_NONE is too large.
cmd_joiner: cmdname must be static for us to use it in argv.
modcmd_saxdb_write_command: If the command has flags but don't know any of
  them, write "???" instead.
modcmd_load_bots: Don't overrun bot hostnames.
service_make_alias: Check for an empty alias.
import_aliases_db: Plug a memory leak.

sockcheck: Fix Coverity warnings

sockcheck_readable: Die if we get a bad template.
sockcheck_queue_address: Make sure Coverity knows we won't overrun
  sci->hostname[].
sockcheck_create_response: The "&& end" was spurious.
sockcheck_add_test: Plug some memory leaks.

uplink_insert(): Require a hostname to enable an uplink

log: Warn if a log name is too long

helpfile: Avoid division by zero

chanserv: Fix Coverity warnings

bad_channel_ban: It doesn't like deref'ing *victimCount after checking if
  it was null.  We only care about the output parameters on success.
eject_user: Our earlier code assumes ban is not null, so don't check it.
cmd_set: buf has to be static so that we can save it in argv[].
chanserv_conf_read: Don't leak change.
chanserv_read_suspended: Don't try to strdup(NULL).

mod-iauth_loc: Use correct size for user.ident and user.hostname.

proto-p10: Simplify assignments to make_numeric's outbuf

mod-helpserv: Reduce size of some strftime() buffers.

gcc 8.3 and glibc 2.29 complain about these possibly overflowing later
snprintf() calls.

sar_getaddr_request: Avoid truncation to full_name.

Pacify gcc 7's -Wimplicit-fallthrough.

mod-iauth_loc: Tell users if their account is suspended.

nickserv: Add flag to remotely oper up clients.

This requires ircd support, e.g. u2.10.12.15 with a config block like:
  UWorld { oper = "srvx.example.org"; };

Move do_expandos() to tools.c/common.h.

iauth_loc: Do not dereference null handle_info pointer.

nickserv: Remove extraneous byte from struct handle_info.

MD5_CRYPT_LENGTH already includes the null terminator byte.

mod-blacklist: Fix typo in comment.

sha256: Output using big-endian byte ordering.

Most test vectors on the Internet use big-endian ordering.
cryptpass_sha256() uses Z85 encoding for big-endian SHA-256 output.

tools: Add Z85 (Ascii85 variant) encoder and decoder.

cmd_opmode: Support OPMODE <user> +o.

sha256: New hash function for passwords.

blacklist: Fix potential memory leak (#15)

Use realloc() instead. In the event we get multiple TXT
records, we just reallocate on top of that instead of leaking
the last result.

blacklist: Fix null terminator being improperly placed

Also avoid undefined behavior

Makefile: Fix HAS_GIT even more.

Out-of-tree builds would fail.

proto-p10: Fail BURST on invalid channel names.

Makefile.am: Fix default target name.

configure.ac: Only set HAS_GIT if gitdir exists.

Makefile: Pass "make distcheck".

Makefile: Remove mention of non-existent UPGRADE file.

proto-p10: Avoid errors triggered by maintainer mode.

help: Use modes that actually exist in csearch

nickserv: Allow renaming handles to new casings

unsplit_string(): Be even more graceful.  *cough*

unsplit_string(): Gracefully handle max == 0.

proto-p10: Fix burst syntax for channels with bans but no users.

Remove references to StrictOp, AutoOp, Voice

Remove these options from set_shows

Conflicts [resolved by Michael Poole]:
srvx.conf.example

opserv: Add ?csearch "mode" criterion.

proto-p10: Support cmd_rping().

This requires a new helper function, GetServerG().

mod-iauth_loc: Quash a compiler warning.

tools: Quash a compiler warning.

configure.in: Rename to configure.ac to make autoreconf happy.

mod-iauth_loc: Login failures should use AGAIN, not NO.

gitignore: Ignore config.h.in~

Merge pull request #9 from staticfox/mem_leak

opserv_cdiscrim_create(): Fix memory leak

Merge pull request #10 from staticfox/syntax

opserv_discrim_create(): wrap argc - 1 in parentheses

Merge pull request #11 from staticfox/listlogins

nickserv: Add LISTLOGINS

nickserv: Add LISTLOGINS

opserv_discrim_create(): wrap argc - 1 in parentheses

This is mostly to match the syntax of opserv_cdiscrim_create(),
but it does help provide clarity.

opserv_cdiscrim_create(): Fix memory leak

If opserv_cdiscrim_create()'s (used in CSEARCH) parameters are
not met, it returns NULL. However, the issue is that it should be
going to the fail statement rather than simply returning NULL.
This creates the issue that discrim is never freed, where fail frees
discrim and returns null in one shot.

Merge pull request #7 from blindirc/helpserv-pickup-helpfile

Improve helpserv pickup help entry

Change PICKUP's help entry to reflect its functionality

Merge pull request #5 from blindirc/checkemail-doc

Document nickserv.checkemail and nickserv.checkpass

Added in help for checkpass

Added in help documentation for checkemail

Merge pull request #3 from blindirc/gitignore

Ignore object files when committing

Ignore object files when committing

Fix persistence lines in helpserv set output

Fix invalid-interval message in HelpServ

Bound check for intervals in mod-helpserv

This prevents integer overflow in helpserv bots configuration.
This integer overflow can lead to a srvx crash and thus to an
impossible restart without fixing the configuration manually.

See CVE-2014-5508

Add a login-on-connect module using XQUERY/XRESPONSE.

Support ircu's XQUERY and XRESPONSE.

Fix the modules list for module names ending in c.

The backslashes in the regular expression \.c\$ were not quoted enough.

Enable silent rules for automake (requires automake 1.10 or later).

Move m4 support files into m4/ directory.

automake 1.11 (or something else new in Ubuntu 11.10) complains if
this is not done.

Fix which userData is checked for deny-nonchannel memo sending.

Fixes SourceForge bug #3308832.

src/mod-memoserv.c (memoserv_can_send): Require the recipient to have
    joined a channel for their membership there to count as common
    with the sender.  (This was the original intent.)

Do not show "[#channel (500,)]" in a !myaccess list.

Fixes SourceForge bug #3072634.

src/chanserv.c (cmd_myaccess): Record where the comma was, and remove
    it if nothing was added after it.

Allow !trim (for users) to work for helpers.

Fixes SourceForge bug #3081638.

src/chanserv.c (cmd_trim_users): Use GetChannelUser() to check user
  access, so that overrides work.

Avoid adding expire_channels() to the timeq on !expire.

Fixes SourceForge bug #3000577.

src/chanserv.c (expire_channels): Only (re-)add this function to timeq
    if the data argument is NULL.
  (cmd_expire): Pass a non-NULL pointer when calling expire_channels().

Re-fix compilation of src/modcmd.c.

(I'm not sure why it looked like this compiled before.)

src/modcmd.c (check_alias_args): strtoul has an attribute that makes
  gcc warn if its result is not used, so store (but ignore) its result.

Fix compilation warnings from gcc 4.6.

src/chanserv.c (zoot_list): Delete this function.
  (cmd_invite): Remove never-read uData variable.
  (ban_read_helper): Remove never-read bData variable.
src/helpfile.c (vsend_message): Remove never-read expand_pos variable.
src/ioset.c (ioset_buffered_read): Remove never-read fdnum variable.
src/log.c (log_entry_search): Remove never-read last variable.
src/mod-helpserv.c (cmd_add_user): Remove never-read new_user variable.
src/modcmd.c (check_alias_args): Remove never-read tmp variable.
src/nickserv.c (cmd_reclaim): Remove never-read hi variable.
src/proto-p10.c (cmd_topic): Remove never-read chan_ts variable.
  (AddLocalUser): Remove never-read timestamp variable.

Document the ChanServ toy commands.

src/chanserv.help: Add documentation for !8ball, !d, !huggle, !ping,
  !unf and !wut.

Fix MIN_ARGC for ChanServ's !8ball and !d commands.

src/chanserv.c (init_chanserv): These commands need at least one argument.

Fix typo in #if test for localtime_r().

Fix the name and type of helpserv_bot.persist_types[].

This array really contains persistence lengths.  gcc 4.5.2 complains
about the mismatched comparisons.

src/mod-helpserv.c (struct helpserv_bot): Rename persist_types to
    persist_lengths.  Change its type to enum persistence_length.

Update IP glob matching to work more accurately with IPv6.

When given an IP like 2001::1234:5678:1:2:3:4, srvx (properly) makes
a glob like 2001:0:4137:*.  However, it wouldn't realize that the
glob actually matches the IP, because it used string glob matching.

src/tools.c (user_matches_glob): Use a parsed glob to match against
  the user's IP address.

Complain properly about unterminated comments in config files.

sylar noticed that unterminated comments would often cause an
infinite loop in the config parser.

src/recdb.c (fail_codes): Add UNTERMINATED_COMMENT value.
  (parse_skip_ws): Abort with that on EOF inside a comment.
  (failure_reason): Handle UNTERMINATED_COMMENT.

Fix available byte count returned by ioq_grow().

ioq_grow() needs to use the same -1 that ioq_put_avail() uses when
the "get" pointer is at byte 0.

src/ioset.c (ioq_grow): Reduce the reported available-byte count by 1.

Fix endianness of IPv6 addresses in generate_hostmask().

src/proto-common.c (generate_hostmask): The IPv6 address parts are
    network-endian, so convert them before printing them.

Fix G-line handling and ?gtrace time display.

src/gline.c (gline_comparator): Use lifetime rather than expiration time to
    sort G-lines.
  (gline_add): Only update expiration time and reason if the new
     modification time is newer than the previous one.  Never update
     issuer (ircu doesn't track it).
src/opserv.c (gtrace_print_func): Display G-lines with no expiration time
  correctly.

Clean up the implementations of mod_chanmode_format().

src/proto-bahamut.c (mod_chanmode_format): Use a second buffer for the
    arguments that get associated with some modes, and append it to
    outbuff at the end.

src/proto-p10.c (mod_chanmode_format): Likewise.

Silently ignore +z when users use it from ChanServ.  Fixes SF#2970686.

src/proto-p10.c (mod_chanmode_parse): Just ignore +z when MCP_REGISTERED
  is set (which only happens from ChanServ).
src/opserv.c (cmd_clearmodes): Do not automatically clear +z.

Simplify G-line code somewhat.  Generate better output for ?gtrace print.

A hub with a negative clock skew will cause adverse results (like the
issued or lastmod times being in the network's future).  That will be
fixed in ircu.

src/gline.c (gline_for_p): Rewrite to just compare the G-line pointer.
  (gline_add): Use that instead.  Move "issued" back in time if the
    lastmod time is earlier than we know the G-line was issued.
src/opserv.c (OSMSG_GTRACE_FOREVER): New format string.
  (OSMSG_GTRACE_EXPIRED): Likewise.
  (gtrace_print_func): Use the appropriate OSMSG_GTRACE_* string.

Implement remote /time query functionality.

src/proto-p10.c (cmd_time): New function.
  (init_parse): Register it.

Support G-line lifetimes.

This involves using the old "expiration" as when the G-line becomes
inactive, and adding a new "lifetime" that is the latest of all the
expiration times that the G-line has ever had.  For convenience,
gline_add() will compare its new "lifetime" argument with its calculated
expiration time -- so callers can safely pass 0 as the lifetime.

src/gline.h (struct gline): Add the lifetime field.  Add Doxygen comments
    for future reference.
  (struct gline_discrim): Add bounds for the lifetime field.
  (gline_add): Add lifetime argument.

src/gline.c (KEY_LIFETIME): New database key.
  (delete_gline_for_p): Delete.
  (gline_expire): Use lifetime to determine whether a G-line expired.
  (gline_remove): Simplify greatly, because we don't actually remove
    G-lines any more (we just mark them immediately expired).
  (gline_add): Add parameter for lifetime, and use it.
  (gline_add_record): Parse lifetime field if it is present.  Simplify
    the parsing of the issued field.
  (gline_write_entry): Write the lifetime.
  (gline_discrim_create): Use ULONG_MAX rather than INT_MAX for
    maximum timestamps.  Parse "lifetime" arguments.
  (gline_discrim_match): Check the lifetime bounds.

src/mod-blacklist.c (dnsbl_hit): Pass 0 lifetime for gline_add().
  (blacklist_check_user): Likewise.

src/mod-sockcheck.c (sockcheck_issue_gline): Likewise.

src/opserv.c (OSMSG_GTRACE_FORMAT): Add lifetime field.
  (opserv_block): Pass 0 lifetime for gline_add().
  (cmd_gline): Likewise.
  (opserv_new_user_check): Likewise.
  (gtrace_print_func): Format the lifetime.

src/proto-p10.c (irc_gline): Include the lifetime in the correct place.
  (irc_ungline): Include the current timestamp (last modified time).
  (cmd_num_gline): When the lifetime argument is given, parse it.
  (cmd_gline): Likewise.  Also implement the normal ircu check for
    expiration as an absolute versus relative time.

Fix various mode-related bugs (including SF#2964782 and SF#2966959).

src/chanserv.c (cmd_mode): Use MCP_NO_APASS, and unparse the mode
    change for display.
  (chan_opt_usergreeting): Use MCP_NO_APASS here too.
  (chanserv_conf_read): Likewise.
  (chanserv_channel_read): Likewise.

src/opserv.c (cmd_kick): Do not let users try to kick services.

src/proto.h (MCP_NO_APASS): New flag.

src/proto-p10.c (irc_user_mode_chars): Allow displaying +x.
  (mod_usermode): Treat +h as setting +x also.
  (mod_chanmode_parse): Reject +A and +U when passed MCP_NO_APASS.

Fix parsing of (P10) MODE <nickname> +<mode> <parameters>.

src/proto-p10.c (cmd_mode): Unsplit the mode and any trailing arguments.

Remove empty +z channels when the +z is removed.

src/proto-p10.c (dead_channels): New variable.
  (parse_line): Release any dead channels after each command.
  (mod_chanmode_parse): Queue -z'ed channels to be released.

Do not send infolines when a normal user joins a +D channel.

src/chanserv.c (handle_join): Suppress infolines for users whose JOIN is
    not going to be broadcast.