conn.c: Code cleanup

Reformat and restructure some code, move all SSL related code into
one single #ifdef block, for example.

Xcode: remove imp.h and exp.h from project file

These files have been removed from ngIRCd in commit 259c314d,
"Remove imp.h and exp.h header files".

Re-add #include's for header files of the C file itself

This partially reverts commit b130b35f4, "Update #include's: remove
unused and add missing ones", but fixes the following compiler and
analyzer warnings of Apple Xcode 5:

 "Semantic issue: No previous prototype for function 'yyy'"

Update copyright notices of recently changed files

Update #include's: remove unused and add missing ones

The "deheader" tool (<http://www.catb.org/~esr/deheader/>) has been
used to find unused #include directives as well as missing ones.

Tested on:

- A/UX 3.1.1
- ArchLinux (2014-03-17)
- Debian GNU/Hurd
- Debian GNU/Linux 6.0.9
- Debian GNU/Linux 7.4
- Fedora 20
- FreeBSD 9.2
- OpenBSD 4.8
- OpenBSD 5.1
- OS X 10.9
- Solaris 11

portab.h: Restructure and clean up code

- Don't use HAVE_SYS_TYPES_H, <sys/types.h> is a required header now.
- Streamline check for boolean data types.
- Better indent #ifdef constructs.

configure[.ng]: <sys/types.h> is a required header file

ngIRCd already includes <sys/types.h> in a lot of places without
checking for its existence (for example in "ngircd.c", "io.c", ...),
therefore make it a required header file.

io.c: <sys/time.h> is only needed when using select()

Only include the <sys/time.h> header when using the select() IO API,
it isn't required otherwise.

Remove imp.h and exp.h header files

These include files don't have a function any more, remove them.

strdup.c: Code cleanup

Don't define SSL_Want{Read|Write}() when SSL is disabled

Not even call the "dummy" functions.

Code cleanup, remove blank lines

Fix two K&R C portability issues

Fix the following two errors emitted by the Apple K&R C compiler
on Apple A/UX:

"./class.c", line 47: no automatic aggregate initialization
"./class.c", line 47: illegal lhs of assignment operator

"./conf.c", line 1052: syntax error

Tested on A/UX 3.1.1.

Merge pull request #6 from norrs/pam_server_password_fix

Validate server password when PAM is disabled.

Use server password when PAM is compiled in but disabled

CHARCONV: Fix handling conversion errors

Don't overwrite already converted text!

Streamline punctuation of log messages

See commit d38d153f for details.

Services.txt: Update information for Anope 2.x

IRC_INVITE(): Code cleanup

Streamline punctuation of log messages

See commit d38d153f for details.

Correctly use cloaked IRC masks on "INVITE nickname"

The cloaked IRC mask of a user is his visible mask, so the daemon has
to use it for generating the "one time" entries for the invite list of
the given channel.

Without this patch, ngIRCd records the real IRC mask which will never
match while the target client is "+x", and even worse, will disclose
the real mask on "MODE #channel +I" commands :-/

Bug reported by Cahata on #ngircd, thanks!

Use correct sender as target for ISUPPORT replies on "VERSION"

This fixes commit 2e168c78 ("Return ISUPPORT(005) numerics on
'VERSION'") and make sure, that the 005 numerics are correctly
routed back to the sender.

Return ISUPPORT(005) numerics on "VERSION"

This is how ircd-seven, Charybdis, Hybrid, and InspIRCd behave,
for example.

configure: Only link "contrib/Debian" if it exists

This isn't the case on "VPATH builds", for example.

Merge pull request #5 from grawity/accountname-whois

Show account name in WHOIS

Show account name in WHOIS

This uses the same numeric as Charybdis and ircu families.

ngircd.conf.5: Update manual page

Merge branch 'no-range-matching' of git://arthur.barton.de/ngircd-alex

* 'no-range-matching' of git://arthur.barton.de/ngircd-alex:
  Remove "range matching" functionality

Use $(MKDIR_P) instead of $(mkinstalldirs) in Makefile's

And test for "mkdir -p" using AC_PROG_MKDIR_P in "configure".

Fix configure script and "make check" for TCP Wrappers

Add missing #include's and static variables.

Problem spotted on OpenBSD.

Add libwrap at the end of the configure run

If libwrap becomes added earlier, other tests may fail because of not all
external variables required by libwrap are available when linking: for
example allow_severity and deny_severity.

This patch adds generic support for the LDFLAGS_END and LIBS_END variables
(CFLAGS_END has been implemented already).

Problem spotted on OpenBSD.

Remove "range matching" functionality

Don't support "range marching" in our pattern matching code using
the "[...]" syntax, because [ and ] are valid characters in nick
names and one has to quote them currently using the "\" character,
which is quite unexpected. For example:

  Nick "te[st" => "MODE #channel +b te\[st"

And remove quoting altogether, too, because "*" and "?" don't need
to be quoted because these characters are not allowed in IRC masks,
nicks, and hostnames.

Reported by "hifi" (Toni Spets) on IRC, thanks!

platformtest.sh: Allow using separate source and build trees

Now you can call platformtest.sh using its complete path name from an
other directory which is then used for building.

See <http://www.gnu.org/software/automake/manual/html_node/VPATH-Builds.html>

Please not that the build system itself is still (re-)genrated in the
original source tree. This can be avoided by passing the "-x" switch to
the platformtest.sh script.

platformtest.sh: Show only up to 10 bytes of version information

platformtest.sh: New option "-x", don't renegerate build system

Client_SetHostname(): Don't use strlen() to test for value

Test suite: explicitely enable glibc memory checking

Send "fake '*' key" in "MODE -k" replies

According to RFC 2812 3.2.3 "Channel mode message" and the examples
there, it looks like clients should use "MODE -k <key>" to unset channel
keys; and that's how other servers and services behave and do expect it.

(But please note that this is NOT the case for "MODE -l"!)

In the end, it doesn't make sense to specify a key when UNsetting it at
all, and different services behave diffrently when clients do not send
the currently set key to unset it - some ignore such calls, for example!

But this implementation is quite relaxed, it accepts any key when
unsetting channel mode "k" and even accepts no key at all. But the reply
will always include an "*" character for every "-k" parameter.

Add "config.cache" to .gitignore file

platformtest.sh: don't use "test -e", it isn't portable

Makefile.am: don't use "test -e", it isn't portable

Support non-standard vsnprintf() return code

C99 states that vsnprintf() "returns the number of characters that
would have been printed if the n were unlimited"; but according to the
Linux manual page "glibc until 2.0.6 would return -1 when the output
was truncated" -- so we have to handle both cases ...

Partially revert bc098794: don't require autoconf 2.67

GNU autoconf 2.61 is still sufficient to generate the configure script, even
after updating configure.{ng|in}, so don't require a newer version.

doc/Platforms.txt: Update some systems to ngIRCd 21

- update i686/pc/cygwin, gcc 4.8.2
- update x86_64/apple/darwin13.0.0, Apple clang 5.0
- update x86_64/unknown/freebsd8.4, gcc 4.2.1
- update x86_64/unknown/freebsd9.1, gcc 4.2.1
- update x86_64/unknown/linux-gnu, gcc 4.8.2
- update x86_64/unknown/linux-gnu, Open64
- update x86_64/unknown/openbsd4.8, gcc 4.2.1
- update x86_64/unknown/openbsd5.1, gcc 4.2.1

Update configure.ng for autoconf 2.67

We already require GNU autoconf 2.67 for generating our build system (at
least this is what the INSTALL document states), so update the build system
accordingly and implement all changes that autoupdate(1) suggests:

 - Update AC_PREREQ and AC_INIT
 - Use AC_LINK_IFELSE, AC_RUN_IFELSE, and AC_COMPILE_IFELSE
 - Remove AC_TYPE_SIGNAL (we don't use RETSIGTYPE)

Update Copyright notices for 2014

Merge branch 'bug167-WebircIPAnoDNS' of git://arthur.barton.de/ngircd-alex

* 'bug167-WebircIPAnoDNS' of git://arthur.barton.de/ngircd-alex:
  WEBIRC: Don't respect hostname when DNS is disabled

Merge branch 'bug165-005-NETWORK' of git://arthur.barton.de/ngircd-alex

* 'bug165-005-NETWORK' of git://arthur.barton.de/ngircd-alex:
  Implement new configuration option "Network"

portabtest: Add checks for strdup(), strndup(), and strtok_r()

portabtest: Actually test functions

Test functions snprintf(), strlcpy(), strlcat(), vsnprintf() for
correctness, not only existance (which was quite useless, because
if they weren't available, the program could not have been linked
at all ...).

portabtest: Only use one exit code to indicate errors

portabtest: Code cleanup

Fix permanent k/gline

Reported by Toni Spets (hifi - at - jnz - dot - fi).

Implement new configuration option "Network"

The new configuration variable "Network" is used to set the (completely
optional) "network name", to which this instance of the daemon belongs.
When set, this name is used in the ISUPPORT(005) numeric which is sent to
all clients connecting to the server after logging in.

Closes bug #165.

WEBIRC: Don't respect hostname when DNS is disabled

When DNS lookups are disabled, don't set the hostname received by the
WEBIRC command, but use the IP address instead.

Reported by Toni Spets <toni.spets@iki.fi>, thanks!

Closes bug #167.

OS X has a working getaddrinfo() implementation

Check for working getaddrinfo() function

At least AIX 4.3.3 and 5.1 have a broken implementation of getaddrinfo()
which doesn't handle "0" as numeric service correctly. This patch adds
a configure check for this case and changes all calling functions to only
use getaddrinfo() if it "works".

See <http://www.stacken.kth.se/lists/heimdal-discuss/2004-05/msg00059.html>

Fix spelling

doc/Platforms.txt: Add and update more systems

- add i386/unknown/netbsdelf1.5.2, egcs-1.1.2
- update i386/unknown/openbsd5.3, gcc 4.2.1
- add i386/unknown/openbsd5.4, gcc 4.2.1
- add x86_64/unknown/linux-gnu, nwcc 0.8.2
- update x86_64/unknown/linux-gnu, tcc 0.9.25

Thanks to Götz Hoffart <goetz@hoffart.de>!

doc/Platforms.txt: Add and update more systems

- add i386/unknown/openbsd3.5, gcc 2.95.3
- add x86_64/unknown/linux-gnu, Sun C 5.12
- update mipsel/unknown/linux-gnu, gcc 4.4.5

Thanks to Götz Hoffart <goetz@hoffart.de>!

doc/Platforms.txt: add DragonflyBSD 3.4 on x86_64

Thanks to Götz Hoffart <goetz@hoffart.de>!

doc/Platforms.txt: add OpenBSD 5.4 on HPPA

configure: clean up function definitions

Only use unsetenv() when it is available

AIX 4.3 dosn't support it, for example.

conf-ssl.h: make code compatible with pre-ANSI C compilers

vsnprintf.c: make code compatible with ansi2knr tool

Remove unused vsnprintf.c test function

Add comments around some IRC_SetPenalty() calls

Rework the penalty handling slightly

Increase the penalty for a command before checking its arguments.  This
makes the handling more consistent and allow us to move more penalties to
Handle_Request().

Move IRC_SetPenalty() to Handle_Request() when possible

This centralizes the penalty handling. It also exposes some commands that
lack it for our attention.

Introduce a macro to define the command list

This will pave the way to other changes and simplifies the entries somewhat.

When sending an error use the IRC_WriteErrClient() variant

This ensures that all errors have a 2 second penalty.

On bad /oper set the penalty to 10 seconds

This helps against brute-force attempts.

Remove unneded IRC_SetPenalty() calls

IRC_WriteErrClient() already calls IRC_SetPenalty().  While here convert
some IRC_SetPenalty() + IRC_WriteStrClient() to IRC_WriteErrClient().

Use sizeof() intead of the explicit size

Remove wrong #ifdef in Option_String()

This fixes the following error when compiling without zlib support:

  irc.c: In function ‘Option_String’:
  irc.c:487: error: ‘options’ undeclared (first use in this function)

Reported by "der_baer" on #ngircd.

NEWS/ChangeLog: Clean up version numbers and release dates

ngIRCd Release 21

doc/Platforms.txt: add Mac OS X 10.6.8

Update doc/Platforms.txt

ngircd.init: Make sure no stale PID file is left over

platformtest.sh/Platforms.txt: allow user names up to 8 characters

Check and call arc4random_stir() if present

FreeBSD prior to 10.0 does not automatically stir on fork(). Same with
current NetBSD. If arc4random_stir() is present assume is needed and
call it instead of srand().

Debian: Fix sed(1) rules adjusting "ngircd-full" package

Error introduced by last commit :-/

Debian: Fix default "HelpFile" file name in ngircd.conf

The "full" package variants must use "/usr/share/doc/ngircd-full/Commands.txt"
and "/usr/share/doc/ngircd-full-dbg/Commands.txt" respectively.

ngIRCd Release 21~rc2

platformtest.sh: Detect clang compilers

Add support for longer config lines

With the introduction of CipherList we could have longer config lines.
Handle up to 1024 bytes and warn if the line will be truncated.

Report the correct file on error

doc/Platforms.txt: Add Open64 and tcc C compilers on Linux

Thanks to Götz Hoffart!

platformtest.sh: Detect tcc compiler

Add support for arc4random

If arc4random is present it will be used over the srand/rand interface.
This fixes some warnings in OpenBSD-current.

Fix another strcat warning missed in commit 4c5b43

platformtest.sh: Clean up GIT source tree, when possible

platformtest.sh: Detect Apple LLVM (clang) compiler

Update (date of) manual pages

ChnageLog file: even more spelling fixes ...

INSTALL file: Update "Upgrade Information"

Fix spelling in NEWS and ChangeLog files

ngircd.c, main(): use strlcat() instead of strcat()

This fixes the following warning on OpenBSD 5.3:

 ngircd.o(.text+0xeb4): In function `main':
  src/ngircd/ngircd.c:300: warning: strcat() is almost always misused,
                                    please use strlcat()

Thanks to Götz Hoffart for reporting this!