Log "Can't read MOTD file" as "configuration error"

Now this error message is displayed in the console without debug prefix
when running the configuration test (--configtest).

Allow "Port = 0" in [Server] blocks

Port number 0 marks remote servers that try to connect to this
daemon, but where this daemon never tries to establis a connection
on its own: only incoming connections are allowed.

conf: fix 'Value of "..." is not a number!' for negative values

Don't use isdigit() function any more, because it only checks the
first character of the variable value and because it doesn't know
about the minus sign which is required e.g. for "Group = -1".

Don't read MOTD file twice

The MOTD file is read in Read_Config(), so don't read it when handling
the "MotdFile" configuration variable. Instead make sure that it is
initialized properly when (re-)reading the configuration.

Add [Features] section to ngircd-test{1|2}.conf

Fix commit 5a34bb203a:
It is not enough to strip the "No" prefix from "Ident" and "PAM",
but we have to introduce the new [Features] section to fix all
warning messages of ngIRCd.

Variables "Ident" and "PAM" in [Global] are completely wrong :-(

Enable WHOIS command to return information about services

Update testsuite configuration: strip No... prefixes

Add connection/socket information to some log messages

Implement channel mode 'O': "IRC operators only"

This channel mode is used on DALnet (bahamut), for example.

Remove ZeroConf variable from sample-ngircd.conf

Remove support for ZeroConf/Bonjour/Rendezvous service registration

TOPIC command: test for channel admin rights correctly

This enables other servers, services and IRC operators to change
channel topics, even when the client is not joined to this channel.

Now the handler for TOPIC behaves like the one for MODE.

Channel_CheckAdminRights(): test if client can admin a channel

This generic function tests if a client is allowed to do administrative
tasks to a specific channel:

 - servers and services are always truested ("allowed everything"),
 - channel operators are allowed,
 - IRC operarors are allowed if OperCanUseMode is set in the config.

conf: fix 'unknown section' FEATURES parse error

pointed out by Alex:
ngircd.conf, line 105: Unknown section "[Features]"!

IRC_TOPIC(): code cleanup

conf: move 'run-time-feature-disable' options to new FEATURE section

array: remove check for allocated == 0

allocated can only be zero if ->mem is NULL.

array: remove alignment of requested size

libc should know better than us.
Also, this helps debugging with tools like valgrind:
When you allocate an array of size x, and then erronoulsy
use x+1 valgrind cannot detect the bug because due to ALIGN_()
made by array.c we might have allocated more than size x...

conf: Warn if PAM=true when ngircd was built without PAM support

conf: add missing static qualifier

internal helper, so it should be static.

also, add UNUSED to 'Line'.

Update copyright notices for 2010 :)

And update the NEWS and ChangeLog file as well.

Make NoZeroConf option work with Howl

config: deprecate NoXX-Options

ngircd unfortunately uses several options using double-negation, e.g.

NoIdent = No, NoPam = No, etc.

This renames all options by dropping the "No" prefix, e.g.
"NoIdent = no" becomes "Ident = yes".

The old options will continue to work, but will cause a warning
message.

Also update man pages and default config.

To prevent silly
'Ident = yes' from appearing in  --configtest output in the
'ident support not compiled in and Ident Option not used' case,
make default value depend on feature availability.
If feature is available, enable by default, otherwise disable.

We might consider moving these options to a new
[Feature]

section, or something like that, because none of these options are
essential.

Another possible improvement:

'Ident = yes' option in ngircd.conf causes a warning if ngircd was
built without ident support.

This does not happen with e.g. zeroconf....

channel: fix confusing "adding to invite list" debug output

adding entries to ban list produced 'invite list' debug output...

Command throttling: introduce MAX_COMMANDS_SERVICE

New MAX_COMMANDS_SERVICE (currently set to MAX_COMMANDS_SERVER[10]),
so that services are handled like servers (and not regular users).

Don't throttle services and servers beeing registered

Xcode: correctly sort files

Don't assert() when serching a client for an invalid server token

This is only relevant when a trusted server on a server-server link
sends invalid commands.

ngIRCd release 17.1

--configtest: remember if MOTD is configured by file or phrase

Configuration variables "MotdFile" and "MotdPhrase" are mutually
exclusive; so don't display content in both of them when running
"ngircd --configtest": instead remember which one is beeing used.

Enhance log messages when establishing server links a little bit

Reset ID of outgoing server link on DNS error correctly

Not resetting the ID prevents the daemon from trying to re-establish
outgoing server links when the DNS resolver failed to resole a hostname.

Don't log critical (or worse) messages to stderr

stderr isn't redirected to the "error file" any more, so there is
no point in trying to log to it ...

Manual page ngircd(8): add SIGNALS section

Manual pages: update and simplyfy AUTHORS section

Remove "error file" when compiled with debug code enabled

The information written to the "error file" (/tmp/ngircd-<PID>.err) when
ngIRCd is compiled with debug code enabled isn't that usefule, so don't
create this file at all.

README: Updated list of implemented commands

add doc/README-Interix.txt and doc/Bopm.txt to distribution tarball

Merge branch 'numeric-329'

* numeric-329:
  New numeric 329: get channel creation time on "MODE #chan" commands
  Save channel creation time; new function Channel_CreationTime()

add doc/PAM.txt to distribution tarball

doc/PAM.txt was not included in the release tarball.
reported by Christoph Biedl.

New numeric 329: get channel creation time on "MODE #chan" commands

Save channel creation time; new function Channel_CreationTime()

ngIRCd release 17

Updated doc/Platforms.txt for upcoming release 17

contrib/platformtest.sh: make command name quoting consistent

contrib/ngircd-redhat.init: updated email address of Naoya Nakazawa

Fix up generation and distribution of sample-ngircd.conf

- Add generated sample-ngircd.conf to new .gitignore file,
- refactor Makefile.am to generate sample-ngircd.conf on "make all",
  to clean it up on "make clean", and to install it to the correct place.
- Make sure path names in sample-ngircd.conf are separated by "/".

doc: change path names in sample-ngircd.conf depending on sysconfdir

ngIRCd Release 17~rc3

Fix connect attempts to further IP addresses of outgoing server links

If a hostname resolves to more than one IP address (round-robin DNS,
IPv4 and IPv6) and an attempt to connect to the first address fails,
ngIRCd should try to connect to the 2nd address, 3rd address etc.

But because of a wrong variable used in the call to New_Server(),
the wrong server structure has been used in further connection attemps
which possibly lead to connection attempts to already connected servers.

Debian: Install default /etc/pam.d/ngircd allowing all logins

This is required for backwards compatibility when installing the -full
or -full-dbg package variant: PAM is enabled now but no configuration
present, so all login attempts would be denied ...

Creating /etc/pam.d/ngircd including "auth required pam_permit.so"
restores the old behaviour of allowing all connections.

Debian: update standards to 3.9.1; add libpam0g-dev dependency

Make contrib/platformtest.sh more portable

Mac OS X package ("make osxpkg"): generate PAM configuration

Xcode builds ("make xcode"): disable pam_fail_delay()

disable pam_fail_delay() only is available starting with Mac
OS X 10.6; but we use the 10.5 SDK for campatibility, so don't use
this function at all when building using Xcode.

Xcode: update project file, use 10.5.x SDK

This is required for universal 32 bit and 64 bit builds: now code
for ppc, i386, and x86_64 is generated (which requires 10.5 or newer).

Xcode builds ("make xcode"): detect version number correctly

ngIRCd release 17~rc2

Updated contrib/platformtest.sh (new version scheme)

 - handle version numbers generated by "git describe"
 - detect gcc compiler version correctly when "-std=xxx" is used

New doc/HowToRelease.txt file describing the release process

ZeroConf: include header files missing since commit a988bbc86a

Generate ngIRCd version number from GIT tag

Now the ngIRCd release/version number is deduced from the "current"
annotated GIT tag; see "git describe --help" for details. This is the
same scheme the Linux kernel uses and gives much more details version
numbers for interim releases and inofficial source archives generated
using "make dist".

Please note: the version number is only updated it the autogen.sh
script is run; so after pulling in and pushing out new commits, you
should run ./autogen.sh!

Make sourcecode compatible with ansi2knr again

This allows to compile ngIRCd using a pre-ANSI K&R C compiler again:
all source files are automatically converted by the included ansi2knr
program (of GNU automake/autoconf) before compiling them with the
K&R C compiler, but a few coding standards must be met.

Tested on Apple A/UX 3.x.
Regression testing on Linux and Mac OS X.

./configure: check if C compiler can compile ISO Standard C

This is required for enabling ansi2knr on systems that don't have an
ANSI C compiler installed (e.g. on A/UX with Apple standard C compiler).

./configure: check support for C prototypes again

Don't use PARAMS() macro for function implementations

The PARAMS() macro is only needed for function prototypes;
don't use it for the actual implementations.

Added m68k/apple/aux3.0.1 (gcc 2.7.2) to doc/Platforms.txt

Only try to set FD_CLOEXEC if this flag is defined

A/UX 3.x doesn't implement this constant, for example.

Only use "__attribute__ ((unused))" if GCC >=2.8 is used

At least GCC 2.7.2 doesn't support this attribute.

doc/Makefile.am: don¹t set docdir, automake handles it already

And elder make(1) programs don¹t like "x ?= y" ...

ngIRCd release 17~rc1

New configuration option "NoZeroConf" to disable ZeroConf registration

If ngIRCd is compiled to register its services using ZeroConf (e.g. using
Howl, Avahi or on Mac OS X) this parameter can be used to disable service
registration at runtime.

Xcode: only build current architecture in "Debug" target

doc/Platforms.txt: added NetBSD 5.0.2

Updated doc/Platforms.txt

Make sure sighandlers.h is listed in noinst_HEADERS

... because it must be included in the distribution archive :-)

const'ify ngt_SyslogFacilityName() function

This fixes the following gcc compiler warning:

tool.c: In function 'ngt_SyslogFacilityName':
tool.c:195: warning: return discards qualifiers from pointer target type

Debian packages: build "-full" and "-full-dbg" with support for PAM

New configuration option "SyslogFacility"

The new option "SyslogFacility" deines the syslog "facility" to which
ngIRCd should send log messages.

Possible values are system dependant, but most probably "auth", "daemon",
"user" and "local1" through "local7" are possible values; see syslog(3).
Default is "local5" for historical reasons.

New functions ngt_SyslogFacilityName() and ngt_SyslogFacilityID()

These both functions translate syslog facility names to ID numbers
and vice versa. On systems that don't define the facilitynames[] array
in syslog.h, we try to build one ourself.

Explicitly cast return value of read(2) to "int"

This fixes the following gcc warning, emitted by Xcode:

src/ngircd/sighandlers.c: In function 'Signal_Callback':
src/ngircd/sighandlers.c:239: warning: implicit conversion shortens 64-bit value into a 32-bit value

Add sighandlers.{c|h} to Xcode project

And update static Mac OS X config.h used by the Xcode project.

Don't call sigaction() if it is not available on the system

Fix signalpipe file descriptor leak on RESTART

Signals_Init() must only be called once.
This does not affect any ngircd release version.

Earlier version of this patch moved the io and sighandler
initialization before the while() loop, but as Alexander
Barton noticed that broke all systems without builtin select
support in io.c...

sighandlers.{c|h}: Code cleanup

 - declare signals_catch[] array not between the function implementations.
 - rename now local function NGIRCd_Rehash() to Rehash().
 - remove empty and therefore not used "catch SIGHUP; break;".

Make sighandlers.{c|h} compatible with ansi2knr

Updated ChangeLog to include signal handler changes

Bump version number to "17-dev"

Update ChangeLog and NEWS: include SIGUSR1/SIGUSR2 changes

Output connection status when dumping the internal server state

Reformat "server state" debug messages a little bit

Enable the daemon to dump its internal state in debug-mode.

This patch allows ngIRCd to dump its internal state (connected clients,
actual configuration) when compiled with --enable-debug. The daemon
catches two more signals:

 - SIGUSR1: toggle debug mode (on/off),
 - SIGUSR2: dump internal state to console/syslog.

signalhandlers: add fallback to deprecated sysv API

remove NGIRCd_SignalRehash

now that the main signal handling is done from the dispatcher
loop we can call NGIRCD_Rehash() directly.

the /REHASH handler can queue the Rehash() function for
execution by sending a SIGHUP.  It will be run when we
return back to the dispatch loop.

Add new 'delayed' signal handlers.

Allows to defer/queue signal processing for execution on the next
event dispatch call, i.e. we can perform any signal action in
normal, non-signal context.

Example uses:
- Reload everything on HUP without writing a global "SIGHUP_received"
  variable
- Dump status of internal Lists on SIGUSR1, etc.

io: add io_cloexec to set close-on-exec flag.

ng_ipaddr.h: include assert.h

We use assert() in this header, so we should include assert.h.

Conn_SyncServerStruct(): test all connections; and work case insensitive

Fix synchronization of established connections and configured server
structures after a configuration update:

 - Not only test servers that already have a connection, but also check
   and update configured servers to which a new connection is beeing
   established (SERVER_WAIT state).

 - And do the server name comparision case-insensitive.

Check_Servers(): skip servers already beeing connected

Let CheckServers() not only skip servers that already have a
connection, but also skip servers to which a new connection is
already beeing established (SERVER_WAIT state).

Check_Servers(): Code cleanup