Conn_Write(): Make sure there is a client when detecting its type

The assert(client != NULL) got triggered during our tests, so there is
an error path that resulted in the connection being still established
(sock >= 0) but the client structure already freed.

So Conn_Write() should handle it!

Proc_Close(): Only close socket if it is still valid

It could be invalid when calling Proc_Close() a 2nd time, for exmaple,
which could happen when we hit a timeout doing IDENT requests :-(

WHOIS command: make sure matching is case-insensitive

And make sure that RPL_ENDOFWHOIS replies with the unmodified mask
like it has been received from the client.

WHOIS command: don't anser queries for IRC servers

Thanks to Cahata for spotting this!

WHOIS command: make sure the reply ends with RPL_ENDOFWHOIS

Up to now, each reply for itself ended in RPL_ENDOFWHOIS and queries
for unknown nick names lacked the RPL_ENDOFWHOIS -- both is wrong.

README: update features list, borrow from list on our website

LINKS command: support <mask> parameter

The <mask> can be used to limit the servers shown in the listing.

IRC_LINKS(): Code cleanup; more documentation

Add 1 second penalty for every further target on PRIVMSG/NOTICE

This reduces the possibility of flooding channels with commands like
"PRIVMSG/NOTICE #a,#n,#c,... :message" a little bit.

Problem noticed by Cahata -- thanks!

Conn_SetPenalty(): Add new "penalty time" on each function call

Until now, the penalty time has only been set when longer as the
already set one, so it didn't accumulate.

And add documentation for and clean up code in Conn_SetPenalty() and
Conn_ResetPenalty() functions.

Enhance log messages when setting user and group

NGIRCd_getNobodyID(): Code cleanup

Display correct error message when "Server{UID|GID}" is invalid

This partly closes bug #118. ngIRCd still starts up even when
Server{UID|GID} is invalid: then the daemon falls back to "nobody"
when running with root(0) privileges (as before).

NGIRCd_Init(): Code cleanup

main(): Code cleanup

README: point to included COPYING file, not gnu.org

Update Copyright notices for 2012

lists: don't crash if reason ptr is NULL

commit 15fec92ed75c3de0b32c40d005e93e3f61aef77e
(Update list item, if it already exists) can make ngircd
crash because 'Reason' can be NULL, as reported by
Cahata on the ngircd mailing list.

Doesn't affect any released ngircd versions.

Also, make sure that we do not pass NULL as arguments
to a '%s' printf-like function.

Don't stop join handling on faulty channel, skip it

When JOIN is received with more than one channel name, don't stop
processing on the first error (e.g. bad name, wrong channel key, ...)
but report an error and continue with the other given channel names.

Reported by Cahata -- thanks!

IRC_JOIN(): Code cleanup

ISON command: reply with correct upper-/lowercase nick names

Reported by Cahata -- thanks!

IRC_ISON(): Code cleanup

Remove unused "bool have_arg" from IRC_WHO()

This fixes:

 irc-info.c: In function ‘IRC_WHO’:
 irc-info:936:18: warning: variable ‘have_arg’ set but not used

Update NEWS and ChangeLog for next ngIRCd release

ngircd.conf.5: reword description of "Ports" variable

New configuration option "PAMIsOptional"

When "PAMIsOptional" is set, clients not sending a password are still
allowed to connect: they won't become "identified" and keep the "~"
character prepended to their supplied user name.

PAM: don't use global password buffer for conv struct

Use the pointer of the password of the client directly.
Eventually we can get rid of the global password again ...

doc/Modes.txt: document channel mode "r"

And make clear, that user mode "r" and channel mode "r" are not set by
ngIRCd itself but by IRC services.

Fixed handling of WHO commands

This fixes two bugs:
 - "WHO <nick>" returned nothing at all if the user was "+i"
   (reported by Cahata, thanks).
 - "WHO <nick|nickmask>" returned channel names instead of "*"
   when the user was member of a (visible) channel.

Clean up code and add documentation as well.

Remove unused "time_t now" from Lists_Check()

This fixes:

 lists.c: In function ‘Lists_Check’:
 lists.c:330:9: warning: variable ‘now’ set but not used

Fixed some spelling errors in documentation and code comments

Thanks to Christoph Biedl!

contrib/Debian/control: Update and complete "Build-Depends"

Update our Debian package descriptions with "official" ones

See Debian Bug #648241 for details.

Fixed typo in two error messages

Thanks to Christoph Biedl!

LUSERS reply: only count "visible" channels

Rename Channel_Count() to Channel_CountVisible() and only count channels
that are visible to the requesting client, so the existence of secret
channels is no longer revealed by using LUSERS.

Reported by Cahata -- thanks!

IRC_Send_LUSERS(): Code cleanup

Don't stop mode handling on unknown modes; skip it

Unknown user and channel modes no longer stop the mode parser, but are
simply ignored. Therefore modes after the unknown one are now handled.

This is how ircd2.10/ircd2.11/ircd-seven behave, at least.

Reported by Cahata -- thanks!

IRC_xLINE(): output an error message for unexpected "lines"

This fixes:

 irc-oper.c: In function ‘IRC_xLINE’:
 irc-oper.c:429: warning: ‘class’ may be used uninitialized in this function
 irc-oper.c:430: warning: ‘class_c’ may be used uninitialized in this function

README: Update list of implemented commands

Update list item, if it already exists

This updates the "validity" (timeout) as well as the "reason" text,
if given.

Lists_CheckDupeMask(): return pointer to already existing item

The old behavior of returning true/false is compatible to this change,
so there are no other code changes required.

Log better error messages when rejecting clients

Synchronize G-Lines on server login

Op_Check(): always accept commands from a remote server itself

Class_GetList() now retuns a pointer to list_head structure

Check G-Line and K-Line lists after authenticating clients

New functions Lists_Expire() and Class_Expire()

GLINE and KLINE lists are cleared automatically now, not when they are
checked. So "STATS g" and "STATS k" no longer show expired entries :-)

Implement IRC_xLINE(): handler for "GLINE" and "KLINE" commands

Op_Check(): return client that initiated the request or NULL

The old behavior of returning true/false is compatible to this change,
so there are no other code changes required.

irc-oper.c: code cleanup; more documentation

Implement IRC "STATS g" and "STATS k" command

New function Class_GetList()

lists.{c|h}: code cleanup; more documentation

List and class handling: add optional "reason" text

Adjust Lists_Add() and Class_AddMask() accordingly, implement
Lists_GetReason() and Lists_GetValidity().

Add new class.{c|h} module to Xcode project

Add new class.{c|h} to project

Implement Class_{AddMask|DeleteMask|IsMember}() functions.

Lists: change "only once" property into "valid until"

The old "only once" true/false behavior is still supported, so there
are no other code changes required.

Xcode: update project file for Xcode 4.2

define HAVE_GAI_STRERROR for Mac OS X Xcode builds

On Mac OS X 10.7 Lion, this fixes

  In file included from .../contrib/MacOSX/../../src/ngircd/client.c:28:
   /Developer/SDKs/MacOSX10.7.sdk/usr/include/netdb.h:272:13:
   error: expected identifier or '('

Configuration: get rid of Conf_Oper_Count and Conf_Channel_Count

Count elements dynamically when needed.

./configure: Fix logic and quoting of poll() detection code

This fixes commit 8e193df ...

Suppress 'Can't create pre-defined channel: invalid name: ""' messages

Skip predefined channel structures that have configured no name,
like the "--configtest" does.

Only use poll() when poll.h exists as well

Not only check for poll(), make sure poll.h exists as well

This fixes building ngIRCd on Debian GNU/Linux 1.3 "Bo" :-)

whois-test: handle local hostname = "localhost.localdomain"

Use the pattern "localhost*" for valid local hostnames.

sample-ngircd.conf: show correct default for "PAM" variable

The default of "PAM" is "yes" when ngIRCd has been configured to use it,
so show the correct default value in the sample configuration file.

Closes #119.

Update GPL 2 license text to current version

See <http://www.gnu.org/licenses/old-licenses/gpl-2.0.txt>.

Test for gai_strerror()

If gai_strerror() isn't available, use a macro that simply returns
a static error message (regardless of the real error code).

For example, GNU libc 2.0.7 doesn't implement gai_strerror().

Only use AI_NUMERICHOST if it is #define'd

It isn't using GNU libc 2.0.7, for example ...

defines.h: fix comment: "lenth" -> "length"

Reported by Christoph Biedl in #ngircd. Thanks!

Init_Server_Struct(): correctly zero Server->bind_addr

Don't use the size of the pointer, use the size of the variable!

Clean up and fix comments of Check_ArgIsTrue()

Thanks to kaFux for pointing this out!
And fix code formatting as well ...

Update doc/GIT.txt

Only close "unrelated" sockets in forked child processes

This fixes the problem that ngIRCd can't do any IDENT lookups because
of the socket has already been closed in the child process.

The bug has been introduced starting with ngIRCd 17 ... :-(
(commit ID 6ebb31ab35e)

Added doc/Modes.txt: document modes supported by ngIRCd

Implemented user mode "R" and channel mode "R"

 - User mode "R": indicates that the nick name of this user is "registered".
   This mode isn't handled by ngIRCd itself, but must be set and unset by
   IRC services like Anope.

 - Channel mode "R": only registered users (having the user mode "R" set)
   are allowed to join this channel.

Use Proc_Close() to remove no longer unused pipes to child processes

This removes spurious (but harmless) debug messages.

New function Proc_Close() to shutdown pipes to child processes

Introduce DEBUG_BUFFER, rework some debug messages

DEBUG_BUFFER is off by default and therefore disables these messages:
 - "Handle_Write() called for connection XX, YY bytes pending ..."
 - "Connection XX: ZZ bytes left in read buffer."

Testsuite: bind to loopback (127.0.0.1) interface only

doc/Platforms.txt: ngIRCd 18 on Nexenta works

Thanks to Götz Hoffart for testing!

New 2nd message "Nickname too long" for error code 432

Client_CheckNick(), Client_IsValidNick(): code cleanup

Merge branch 'ServerMode'

* ServerMode:
  Handle channel user modes 'a', 'h', and 'q' from remote servers
  Handle unknown channel modes on server links
  Handle unknown user modes on server links
  IRC_MODE(), Client_Mode(): code cleanup [2/2]
  Enlarge client user mode buffer, reduce client flags buffer
  Infom clients when other servers change their user modes
  IRC_MODE(), Client_Mode(): code cleanup [1/2]

Merge branch 'bug113-SrvPrefix'

* bug113-SrvPrefix:
  Slightly change (and document!) IRC_KILL() calling convention
  Spoofed prefixes: close connection on non-server links only

Xcode: update project file to Xcode 3.2 or newer

Xcode requires Mac OS X 10.6 or newer; Xcode 4 supports this project
format as well, so effectively you can use Mac OS X 10.6.x or 10.7.x
for building ngIRCd with the Apple Xcode IDE.

Xcode: Mac OS X config.h: support 10.5 as well as 10.6/10.7 SDK

Xcode: exclude more Xcode 4 specific directories in ".gitignore"

Cast getpid() and time() results for srand() input

This fixes:

src/ngircd/ngircd.c:596: warning: implicit conversion
 shortens 64-bit value into a 32-bit value

(i686-apple-darwin11-llvm-gcc-4.2)

Xcode: update and add missing files to project

IRC_QUIT(): disconnect directly linked servers sending QUIT

Without this patch, the server becomes removed from the network and
the client structures, but the connection isn't shut down at all ...

contrib/ngindent: detect "gindent" as GNU indent

Handle channel user modes 'a', 'h', and 'q' from remote servers

These channel user modes aren't used for anything at the moment, but
ngIRCd knows that these three modes are "channel user modes" and not
"channel modes", that is that these modes take an "nick name" argument.

Like unknown user and channel modes, these modes are saved and forwarded,
but ignored otherwise.

Handle unknown channel modes on server links

Handle unknown user modes on server links

IRC_MODE(), Client_Mode(): code cleanup [2/2]

Enlarge client user mode buffer, reduce client flags buffer

We have to enlage our user mode buffer, so we can handle even unknown
user modes in the future; and reduce the client flags buffer, because
I can't imagine why we ever would need ~100 flags!?

Now we support up to 15 user modes (was: 8) and up to 15 flags (was: 99).

So in the end, we even save 99-15+8-15=77 bytes for each client structure!

Infom clients when other servers change their user modes

doc/Platforms.txt: re-add mipsel/unknown/linux-gnu with gcc 4.1.2

Updated doc/Platforms.txt: mipsel/unknown/linux-gnu