]> arthur.barton.de Git - ngircd-alex.git/blobdiff - src/ngircd/irc-info.c
Streamline effect of "MorePrivacy" option (WHOIS, LIST)
[ngircd-alex.git] / src / ngircd / irc-info.c
index b4c951f3a63345c92daa755c1bf88ab48bc4f555..1bbaf57b3b53a35e905be647c5f075690ade38ac 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * ngIRCd -- The Next Generation IRC Daemon
- * Copyright (c)2001-2012 Alexander Barton (alex@barton.de) and Contributors.
+ * Copyright (c)2001-2014 Alexander Barton (alex@barton.de) and Contributors.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * IRC info commands
  */
 
-#include "imp.h"
 #include <assert.h>
-#include <errno.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <strings.h>
+#include <time.h>
 
 #include "ngircd.h"
 #include "conn-func.h"
 #include "channel.h"
 #include "class.h"
 #include "conf.h"
-#include "defines.h"
 #include "lists.h"
-#include "log.h"
 #include "messages.h"
 #include "match.h"
-#include "tool.h"
 #include "parse.h"
 #include "irc.h"
+#include "irc-macros.h"
 #include "irc-write.h"
 #include "client-cap.h"
+#include "op.h"
 
-#include "exp.h"
 #include "irc-info.h"
 
+/* Local functions */
 
+static unsigned int
+t_diff(time_t *t, const time_t d)
+{
+       time_t diff, remain;
+
+       diff = *t / d;
+       remain = diff * d;
+       *t -= remain;
+
+       return (unsigned int)diff;
+}
+
+static unsigned int
+uptime_days(time_t *now)
+{
+       return t_diff(now, 60 * 60 * 24);
+}
+
+static unsigned int
+uptime_hrs(time_t *now)
+{
+       return t_diff(now, 60 * 60);
+}
+
+static unsigned int
+uptime_mins(time_t *now)
+{
+       return t_diff(now, 60);
+}
+
+static bool
+write_whoreply(CLIENT *Client, CLIENT *c, const char *channelname, const char *flags)
+{
+       return IRC_WriteStrClient(Client, RPL_WHOREPLY_MSG, Client_ID(Client),
+                                 channelname, Client_User(c),
+                                 Client_HostnameDisplayed(c),
+                                 Client_ID(Client_Introducer(c)), Client_ID(c),
+                                 flags, Client_Hops(c), Client_Info(c));
+}
+
+/**
+ * Return channel user mode prefix(es).
+ *
+ * @param Client The client requesting the mode prefixes.
+ * @param chan_user_modes String with channel user modes.
+ * @param str String buffer to which the prefix(es) will be appended.
+ * @param len Size of "str" buffer.
+ * @return Pointer to "str".
+ */
+static char *
+who_flags_qualifier(CLIENT *Client, const char *chan_user_modes,
+                   char *str, size_t len)
+{
+       assert(Client != NULL);
+
+       if (Client_Cap(Client) & CLIENT_CAP_MULTI_PREFIX) {
+               if (strchr(chan_user_modes, 'q'))
+                       strlcat(str, "~", len);
+               if (strchr(chan_user_modes, 'a'))
+                       strlcat(str, "&", len);
+               if (strchr(chan_user_modes, 'o'))
+                       strlcat(str, "@", len);
+               if (strchr(chan_user_modes, 'h'))
+                       strlcat(str, "%", len);
+               if (strchr(chan_user_modes, 'v'))
+                       strlcat(str, "+", len);
+
+               return str;
+       }
+
+       if (strchr(chan_user_modes, 'q'))
+               strlcat(str, "~", len);
+       else if (strchr(chan_user_modes, 'a'))
+               strlcat(str, "&", len);
+       else if (strchr(chan_user_modes, 'o'))
+               strlcat(str, "@", len);
+       else if (strchr(chan_user_modes, 'h'))
+               strlcat(str, "%", len);
+       else if (strchr(chan_user_modes, 'v'))
+               strlcat(str, "+", len);
+
+       return str;
+}
+
+/**
+ * Send WHO reply for a "channel target" ("WHO #channel").
+ *
+ * @param Client Client requesting the information.
+ * @param Chan Channel being requested.
+ * @param OnlyOps Only display IRC operators.
+ * @return CONNECTED or DISCONNECTED.
+ */
+static bool
+IRC_WHO_Channel(CLIENT *Client, CHANNEL *Chan, bool OnlyOps)
+{
+       bool is_visible, is_member, is_ircop;
+       CL2CHAN *cl2chan;
+       char flags[10];
+       CLIENT *c;
+       int count = 0;
+
+       assert( Client != NULL );
+       assert( Chan != NULL );
+
+       is_member = Channel_IsMemberOf(Chan, Client);
+
+       /* Secret channel? */
+       if (!is_member && Channel_HasMode(Chan, 's'))
+               return IRC_WriteStrClient(Client, RPL_ENDOFWHO_MSG,
+                                         Client_ID(Client), Channel_Name(Chan));
+
+       cl2chan = Channel_FirstMember(Chan);
+       for (; cl2chan ; cl2chan = Channel_NextMember(Chan, cl2chan)) {
+               c = Channel_GetClient(cl2chan);
+
+               is_ircop = Client_HasMode(c, 'o');
+               if (OnlyOps && !is_ircop)
+                       continue;
+
+               is_visible = !Client_HasMode(c, 'i');
+               if (is_member || is_visible) {
+                       memset(flags, 0, sizeof(flags));
+
+                       if (Client_HasMode(c, 'a'))
+                               flags[0] = 'G'; /* away */
+                       else
+                               flags[0] = 'H';
+
+                       if (is_ircop)
+                               flags[1] = '*';
+
+                       who_flags_qualifier(Client, Channel_UserModes(Chan, c),
+                                           flags, sizeof(flags));
+
+                       if (!write_whoreply(Client, c, Channel_Name(Chan),
+                                           flags))
+                               return DISCONNECTED;
+                       count++;
+               }
+       }
+
+       /* If there are a lot of clients, increase the penalty a bit */
+       if (count > MAX_RPL_WHO)
+               IRC_SetPenalty(Client, 1);
+
+       return IRC_WriteStrClient(Client, RPL_ENDOFWHO_MSG, Client_ID(Client),
+                                 Channel_Name(Chan));
+}
+
+/**
+ * Send WHO reply for a "mask target" ("WHO m*sk").
+ *
+ * @param Client Client requesting the information.
+ * @param Mask Mask being requested or NULL for "all" clients.
+ * @param OnlyOps Only display IRC operators.
+ * @return CONNECTED or DISCONNECTED.
+ */
+static bool
+IRC_WHO_Mask(CLIENT *Client, char *Mask, bool OnlyOps)
+{
+       CLIENT *c;
+       CL2CHAN *cl2chan;
+       CHANNEL *chan;
+       bool client_match, is_visible;
+       char flags[3];
+       int count = 0;
+
+       assert (Client != NULL);
+
+       if (Mask)
+               ngt_LowerStr(Mask);
+
+       IRC_SetPenalty(Client, 3);
+       for (c = Client_First(); c != NULL; c = Client_Next(c)) {
+               if (Client_Type(c) != CLIENT_USER)
+                       continue;
+
+               if (OnlyOps && !Client_HasMode(c, 'o'))
+                       continue;
+
+               if (Mask) {
+                       /* Match pattern against user host/server/name/nick */
+                       client_match = MatchCaseInsensitive(Mask,
+                                                           Client_Hostname(c));
+                       if (!client_match)
+                               client_match = MatchCaseInsensitive(Mask,
+                                                                   Client_ID(Client_Introducer(c)));
+                       if (!client_match)
+                               client_match = MatchCaseInsensitive(Mask,
+                                                                   Client_Info(c));
+                       if (!client_match)
+                               client_match = MatchCaseInsensitive(Mask,
+                                                                   Client_ID(c));
+                       if (!client_match)
+                               continue;       /* no match: skip this client */
+               }
+
+               is_visible = !Client_HasMode(c, 'i');
+
+               /* Target client is invisible, but mask matches exactly? */
+               if (!is_visible && Mask && strcasecmp(Client_ID(c), Mask) == 0)
+                       is_visible = true;
+
+               /* Target still invisible, but are both on the same channel? */
+               if (!is_visible) {
+                       cl2chan = Channel_FirstChannelOf(Client);
+                       while (cl2chan && !is_visible) {
+                               chan = Channel_GetChannel(cl2chan);
+                               if (Channel_IsMemberOf(chan, c))
+                                       is_visible = true;
+                               cl2chan = Channel_NextChannelOf(Client, cl2chan);
+                       }
+               }
+
+               if (!is_visible)        /* target user is not visible */
+                       continue;
+
+               if (IRC_CheckListTooBig(Client, count, MAX_RPL_WHO, "WHO"))
+                       break;
+
+               memset(flags, 0, sizeof(flags));
+
+               if (Client_HasMode(c, 'a'))
+                       flags[0] = 'G'; /* away */
+               else
+                       flags[0] = 'H';
+
+               if (Client_HasMode(c, 'o'))
+                       flags[1] = '*';
+
+               if (!write_whoreply(Client, c, "*", flags))
+                       return DISCONNECTED;
+               count++;
+       }
+
+       return IRC_WriteStrClient(Client, RPL_ENDOFWHO_MSG, Client_ID(Client),
+                                 Mask ? Mask : "*");
+}
+
+/**
+ * Generate WHOIS reply of one actual client.
+ *
+ * @param Client The client from which this command has been received.
+ * @param from The client requesting the information ("originator").
+ * @param c The client of which information should be returned.
+ * @return CONNECTED or DISCONNECTED.
+ */
+static bool
+IRC_WHOIS_SendReply(CLIENT *Client, CLIENT *from, CLIENT *c)
+{
+       char str[COMMAND_LEN];
+       CL2CHAN *cl2chan;
+       CHANNEL *chan;
+
+       assert(Client != NULL);
+       assert(from != NULL);
+       assert(c != NULL);
+
+       /* Nick, user, hostname and client info */
+       if (!IRC_WriteStrClient(from, RPL_WHOISUSER_MSG, Client_ID(from),
+                               Client_ID(c), Client_User(c),
+                               Client_HostnameDisplayed(c), Client_Info(c)))
+               return DISCONNECTED;
+
+       /* Server */
+       if (!IRC_WriteStrClient(from, RPL_WHOISSERVER_MSG, Client_ID(from),
+                               Client_ID(c), Client_ID(Client_Introducer(c)),
+                               Client_Info(Client_Introducer(c))))
+               return DISCONNECTED;
+
+       /* Channels, show only if client has no +I or if from is oper */
+       if(!(Client_HasMode(c, 'I')) || Client_HasMode(from, 'o'))  {
+               snprintf(str, sizeof(str), RPL_WHOISCHANNELS_MSG,
+                        Client_ID(from), Client_ID(c));
+               cl2chan = Channel_FirstChannelOf(c);
+               while (cl2chan) {
+                       chan = Channel_GetChannel(cl2chan);
+                       assert(chan != NULL);
+
+                       /* next */
+                       cl2chan = Channel_NextChannelOf(c, cl2chan);
+
+                       /* Secret channel? */
+                       if (Channel_HasMode(chan, 's')
+                           && !Channel_IsMemberOf(chan, Client))
+                               continue;
+
+                       /* Local channel and request is not from a user? */
+                       if (Client_Type(Client) == CLIENT_SERVER
+                           && Channel_IsLocal(chan))
+                               continue;
+
+                       /* Concatenate channel names */
+                       if (str[strlen(str) - 1] != ':')
+                               strlcat(str, " ", sizeof(str));
+
+                       who_flags_qualifier(Client, Channel_UserModes(chan, c),
+                                           str, sizeof(str));
+                       strlcat(str, Channel_Name(chan), sizeof(str));
+
+                       if (strlen(str) > (COMMAND_LEN - CHANNEL_NAME_LEN - 4)) {
+                               /* Line becomes too long: send it! */
+                               if (!IRC_WriteStrClient(Client, "%s", str))
+                                       return DISCONNECTED;
+                               snprintf(str, sizeof(str), RPL_WHOISCHANNELS_MSG,
+                                        Client_ID(from), Client_ID(c));
+                       }
+               }
+               if(str[strlen(str) - 1] != ':') {
+                       /* There is data left to send: */
+                       if (!IRC_WriteStrClient(Client, "%s", str))
+                               return DISCONNECTED;
+               }
+       }
+
+       /* IRC-Services? */
+       if (Client_Type(c) == CLIENT_SERVICE &&
+           !IRC_WriteStrClient(from, RPL_WHOISSERVICE_MSG,
+                               Client_ID(from), Client_ID(c)))
+               return DISCONNECTED;
+
+       /* IRC-Operator? */
+       if (Client_Type(c) != CLIENT_SERVICE &&
+           Client_HasMode(c, 'o') &&
+           !IRC_WriteStrClient(from, RPL_WHOISOPERATOR_MSG,
+                               Client_ID(from), Client_ID(c)))
+               return DISCONNECTED;
+
+       /* IRC-Bot? */
+       if (Client_HasMode(c, 'B') &&
+           !IRC_WriteStrClient(from, RPL_WHOISBOT_MSG,
+                               Client_ID(from), Client_ID(c)))
+               return DISCONNECTED;
+
+       /* Connected using SSL? */
+       if (Conn_UsesSSL(Client_Conn(c))) {
+               if (!IRC_WriteStrClient(from, RPL_WHOISSSL_MSG, Client_ID(from),
+                                       Client_ID(c)))
+                       return DISCONNECTED;
+
+               /* Certificate fingerprint? */
+               if (Conn_GetCertFp(Client_Conn(c)) &&
+                   from == c &&
+                   !IRC_WriteStrClient(from, RPL_WHOISCERTFP_MSG,
+                                       Client_ID(from), Client_ID(c),
+                                       Conn_GetCertFp(Client_Conn(c))))
+                       return DISCONNECTED;
+       }
+
+       /* Registered nickname? */
+       if (Client_HasMode(c, 'R') &&
+           !IRC_WriteStrClient(from, RPL_WHOISREGNICK_MSG,
+                               Client_ID(from), Client_ID(c)))
+               return DISCONNECTED;
+
+       /* Account name metadata? */
+       if (Client_AccountName(c) &&
+           !IRC_WriteStrClient(from, RPL_WHOISLOGGEDIN_MSG,
+                               Client_ID(from), Client_ID(c),
+                               Client_AccountName(c)))
+               return DISCONNECTED;
+
+       /* Local client and requester is the user itself or an IRC Op? */
+       if (Client_Conn(c) > NONE &&
+           (from == c || Client_HasMode(from, 'o'))) {
+               /* Client hostname */
+               if (!IRC_WriteStrClient(from, RPL_WHOISHOST_MSG,
+                                       Client_ID(from), Client_ID(c),
+                                       Client_Hostname(c), Client_IPAText(c)))
+                       return DISCONNECTED;
+               /* Client modes */
+               if (!IRC_WriteStrClient(from, RPL_WHOISMODES_MSG,
+                                       Client_ID(from), Client_ID(c), Client_Modes(c)))
+                       return DISCONNECTED;
+       }
+
+       /* Idle and signon time (local clients only!) */
+       if (!Conf_MorePrivacy && Client_Conn(c) > NONE &&
+           !IRC_WriteStrClient(from, RPL_WHOISIDLE_MSG,
+                               Client_ID(from), Client_ID(c),
+                               (unsigned long)Conn_GetIdle(Client_Conn(c)),
+                               (unsigned long)Conn_GetSignon(Client_Conn(c))))
+               return DISCONNECTED;
+
+       /* Away? */
+       if (Client_HasMode(c, 'a') &&
+           !IRC_WriteStrClient(from, RPL_AWAY_MSG,
+                               Client_ID(from), Client_ID(c), Client_Away(c)))
+               return DISCONNECTED;
+
+       return CONNECTED;
+}
+
+static bool
+WHOWAS_EntryWrite(CLIENT *prefix, WHOWAS *entry)
+{
+       char t_str[60];
+
+       (void)strftime(t_str, sizeof(t_str), "%a %b %d %H:%M:%S %Y",
+                      localtime(&entry->time));
+
+       if (!IRC_WriteStrClient(prefix, RPL_WHOWASUSER_MSG, Client_ID(prefix),
+                               entry->id, entry->user, entry->host, entry->info))
+               return DISCONNECTED;
+
+       return IRC_WriteStrClient(prefix, RPL_WHOISSERVER_MSG, Client_ID(prefix),
+                                 entry->id, entry->server, t_str);
+}
+
+#ifdef SSL_SUPPORT
+static bool
+Show_MOTD_SSLInfo(CLIENT *Client)
+{
+       char buf[COMMAND_LEN];
+       char c_str[128];
+
+       if (Conn_GetCipherInfo(Client_Conn(Client), c_str, sizeof(c_str))) {
+               snprintf(buf, sizeof(buf), "Connected using Cipher %s", c_str);
+               if (!IRC_WriteStrClient(Client, RPL_MOTD_MSG,
+                                       Client_ID(Client), buf))
+                       return false;
+       }
+
+       if (Conn_GetCertFp(Client_Conn(Client))) {
+               snprintf(buf, sizeof(buf),
+                        "Your client certificate fingerprint is: %s",
+                        Conn_GetCertFp(Client_Conn(Client)));
+               if (!IRC_WriteStrClient(Client, RPL_MOTD_MSG,
+                                       Client_ID(Client), buf))
+                       return false;
+       }
+
+       return true;
+}
+#else
+static bool
+Show_MOTD_SSLInfo(UNUSED CLIENT *c)
+{
+       return true;
+}
+#endif
+
+/* Global functions */
+
+/**
+ * Handler for the IRC command "ADMIN".
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
+ */
 GLOBAL bool
 IRC_ADMIN(CLIENT *Client, REQUEST *Req )
 {
@@ -53,41 +502,38 @@ IRC_ADMIN(CLIENT *Client, REQUEST *Req )
        assert( Client != NULL );
        assert( Req != NULL );
 
-       if(( Req->argc > 1 )) return IRC_WriteStrClient( Client, ERR_NEEDMOREPARAMS_MSG, Client_ID( Client ), Req->command );
-
-       /* find target ... */
-       if( Req->argc == 1 ) target = Client_Search( Req->argv[0] );
-       else target = Client_ThisServer( );
-
-       /* find Prefix */
-       if( Client_Type( Client ) == CLIENT_SERVER ) prefix = Client_Search( Req->prefix );
-       else prefix = Client;
-       if( ! prefix ) return IRC_WriteStrClient( Client, ERR_NOSUCHNICK_MSG, Client_ID( Client ), Req->prefix );
-
-       /* forwad message to another server? */
-       if( target != Client_ThisServer( ))
-       {
-               if(( ! target ) || ( Client_Type( target ) != CLIENT_SERVER )) return IRC_WriteStrClient( prefix, ERR_NOSUCHSERVER_MSG, Client_ID( prefix ), Req->argv[0] );
+       _IRC_GET_SENDER_OR_RETURN_(prefix, Req, Client)
+       _IRC_GET_TARGET_SERVER_OR_RETURN_(target, Req, 0, prefix)
 
-               /* forward */
-               IRC_WriteStrClientPrefix( target, prefix, "ADMIN %s", Req->argv[0] );
+       /* Forward? */
+       if(target != Client_ThisServer()) {
+               IRC_WriteStrClientPrefix(target, prefix,
+                                        "ADMIN %s", Client_ID(target));
                return CONNECTED;
        }
 
-       /* mit Versionsinfo antworten */
-       if( ! IRC_WriteStrClient( Client, RPL_ADMINME_MSG, Client_ID( prefix ), Conf_ServerName )) return DISCONNECTED;
-       if( ! IRC_WriteStrClient( Client, RPL_ADMINLOC1_MSG, Client_ID( prefix ), Conf_ServerAdmin1 )) return DISCONNECTED;
-       if( ! IRC_WriteStrClient( Client, RPL_ADMINLOC2_MSG, Client_ID( prefix ), Conf_ServerAdmin2 )) return DISCONNECTED;
-       if( ! IRC_WriteStrClient( Client, RPL_ADMINEMAIL_MSG, Client_ID( prefix ), Conf_ServerAdminMail )) return DISCONNECTED;
+       if (!IRC_WriteStrClient(Client, RPL_ADMINME_MSG, Client_ID(prefix),
+                               Conf_ServerName))
+               return DISCONNECTED;
+       if (!IRC_WriteStrClient(Client, RPL_ADMINLOC1_MSG, Client_ID(prefix),
+                               Conf_ServerAdmin1))
+               return DISCONNECTED;
+       if (!IRC_WriteStrClient(Client, RPL_ADMINLOC2_MSG, Client_ID(prefix),
+                               Conf_ServerAdmin2))
+               return DISCONNECTED;
+       if (!IRC_WriteStrClient(Client, RPL_ADMINEMAIL_MSG, Client_ID(prefix),
+                               Conf_ServerAdminMail))
+               return DISCONNECTED;
 
-       IRC_SetPenalty( Client, 1 );
        return CONNECTED;
 } /* IRC_ADMIN */
 
-
 /**
  * Handler for the IRC command "INFO".
- * See RFC 2812 section 3.4.10.
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
  */
 GLOBAL bool
 IRC_INFO(CLIENT * Client, REQUEST * Req)
@@ -98,38 +544,13 @@ IRC_INFO(CLIENT * Client, REQUEST * Req)
        assert(Client != NULL);
        assert(Req != NULL);
 
-       /* Wrong number of parameters? */
-       if (Req->argc > 1)
-               return IRC_WriteStrClient(Client, ERR_NEEDMOREPARAMS_MSG,
-                                         Client_ID(Client), Req->command);
+       _IRC_GET_SENDER_OR_RETURN_(prefix, Req, Client)
+       _IRC_GET_TARGET_SERVER_OR_RETURN_(target, Req, 0, prefix)
 
-       /* Determine prefix */
-       if (Client_Type(Client) == CLIENT_SERVER)
-               prefix = Client_Search(Req->prefix);
-       else
-               prefix = Client;
-       if (!prefix)
-               return IRC_WriteStrClient(Client, ERR_NOSUCHNICK_MSG,
-                                         Client_ID(Client), Req->prefix);
-
-       /* Look for a target */
-       if (Req->argc > 0)
-               target = Client_Search(Req->argv[0]);
-       else
-               target = Client_ThisServer();
-
-       /* Make sure that the target is a server */
-       if (target && Client_Type(target) != CLIENT_SERVER)
-               target = Client_Introducer(target);
-
-       if (!target)
-               return IRC_WriteStrClient(prefix, ERR_NOSUCHSERVER_MSG,
-                                         Client_ID(prefix), Req->argv[0]);
-
-       /* Pass on to another server? */
+       /* Forward? */
        if (target != Client_ThisServer()) {
                IRC_WriteStrClientPrefix(target, prefix, "INFO %s",
-                                        Req->argv[0]);
+                                        Client_ID(target));
                return CONNECTED;
        }
 
@@ -151,16 +572,12 @@ IRC_INFO(CLIENT * Client, REQUEST * Req)
        if (!IRC_WriteStrClient(Client, RPL_ENDOFINFO_MSG, Client_ID(prefix)))
                return DISCONNECTED;
 
-       IRC_SetPenalty(Client, 2);
        return CONNECTED;
 } /* IRC_INFO */
 
-
 /**
  * Handler for the IRC "ISON" command.
  *
- * See RFC 2812, 4.9 "Ison message".
- *
  * @param Client The client from which this command has been received.
  * @param Req Request structure with prefix and all parameters.
  * @return CONNECTED or DISCONNECTED.
@@ -176,11 +593,6 @@ IRC_ISON( CLIENT *Client, REQUEST *Req )
        assert(Client != NULL);
        assert(Req != NULL);
 
-       /* Bad number of arguments? */
-       if (Req->argc < 1)
-               return IRC_WriteStrClient(Client, ERR_NEEDMOREPARAMS_MSG,
-                                         Client_ID(Client), Req->command);
-
        strlcpy(rpl, RPL_ISON_MSG, sizeof rpl);
        for (i = 0; i < Req->argc; i++) {
                /* "All" ircd even parse ":<x> <y> ..." arguments and split
@@ -201,12 +613,9 @@ IRC_ISON( CLIENT *Client, REQUEST *Req )
        return IRC_WriteStrClient(Client, rpl, Client_ID(Client));
 } /* IRC_ISON */
 
-
 /**
  * Handler for the IRC "LINKS" command.
  *
- * See RFC 2812, 3.4.5 "Links message".
- *
  * @param Client The client from which this command has been received.
  * @param Req Request structure with prefix and all parameters.
  * @return CONNECTED or DISCONNECTED.
@@ -220,11 +629,7 @@ IRC_LINKS(CLIENT *Client, REQUEST *Req)
        assert(Client != NULL);
        assert(Req != NULL);
 
-       IRC_SetPenalty(Client, 1);
-
-       if (Req->argc > 2)
-               return IRC_WriteStrClient(Client, ERR_NEEDMOREPARAMS_MSG,
-                                         Client_ID(Client), Req->command);
+       _IRC_GET_SENDER_OR_RETURN_(from, Req, Client)
 
        /* Get pointer to server mask or "*", if none given */
        if (Req->argc > 0)
@@ -232,26 +637,15 @@ IRC_LINKS(CLIENT *Client, REQUEST *Req)
        else
                mask = "*";
 
-       if (Client_Type(Client) == CLIENT_SERVER)
-               from = Client_Search(Req->prefix);
-       else
-               from = Client;
-       if (!from)
-               return IRC_WriteStrClient(Client, ERR_NOSUCHNICK_MSG,
-                                         Client_ID(Client), Req->prefix);
-
        /* Forward? */
        if (Req->argc == 2) {
-               target = Client_Search(Req->argv[0]);
-               if (! target || Client_Type(target) != CLIENT_SERVER)
-                       return IRC_WriteStrClient(from, ERR_NOSUCHSERVER_MSG,
-                                                 Client_ID(from),
-                                                 Req->argv[0] );
-               else
-                       if (target != Client_ThisServer())
-                               return IRC_WriteStrClientPrefix(target, from,
-                                               "LINKS %s %s", Req->argv[0],
-                                               Req->argv[1]);
+               _IRC_GET_TARGET_SERVER_OR_RETURN_(target, Req, 0, from)
+               if (target != Client_ThisServer()) {
+                       IRC_WriteStrClientPrefix(target, from,
+                                       "LINKS %s %s", Client_ID(target),
+                                       Req->argv[1]);
+                       return CONNECTED;
+               }
        }
 
        c = Client_First();
@@ -272,7 +666,13 @@ IRC_LINKS(CLIENT *Client, REQUEST *Req)
                                  Client_ID(from), mask);
 } /* IRC_LINKS */
 
-
+/**
+ * Handler for the IRC "LUSERS" command.
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
+ */
 GLOBAL bool
 IRC_LUSERS( CLIENT *Client, REQUEST *Req )
 {
@@ -281,37 +681,26 @@ IRC_LUSERS( CLIENT *Client, REQUEST *Req )
        assert( Client != NULL );
        assert( Req != NULL );
 
-       if(( Req->argc > 2 )) return IRC_WriteStrClient( Client, ERR_NEEDMOREPARAMS_MSG, Client_ID( Client ), Req->command );
-
-       /* Absender ermitteln */
-       if( Client_Type( Client ) == CLIENT_SERVER ) from = Client_Search( Req->prefix );
-       else from = Client;
-       if( ! from ) return IRC_WriteStrClient( Client, ERR_NOSUCHNICK_MSG, Client_ID( Client ), Req->prefix );
+       _IRC_GET_SENDER_OR_RETURN_(from, Req, Client)
+       _IRC_GET_TARGET_SERVER_OR_RETURN_(target, Req, 1, from)
 
-       /* An anderen Server forwarden? */
-       if( Req->argc == 2 )
-       {
-               target = Client_Search( Req->argv[1] );
-               if(( ! target ) || ( Client_Type( target ) != CLIENT_SERVER )) return IRC_WriteStrClient( from, ERR_NOSUCHSERVER_MSG, Client_ID( from ), Req->argv[1] );
-               else if( target != Client_ThisServer( )) return IRC_WriteStrClientPrefix( target, from, "LUSERS %s %s", Req->argv[0], Req->argv[1] );
+       /* Forward? */
+       if (target != Client_ThisServer()) {
+               IRC_WriteStrClientPrefix(target, from,
+                                        "LUSERS %s %s", Req->argv[0],
+                                        Client_ID(target));
+               return CONNECTED;
        }
 
-       /* Wer ist der Absender? */
-       if( Client_Type( Client ) == CLIENT_SERVER ) target = Client_Search( Req->prefix );
-       else target = Client;
-       if( ! target ) return IRC_WriteStrClient( Client, ERR_NOSUCHNICK_MSG, Client_ID( Client ), Req->prefix );
-
-       IRC_Send_LUSERS( target );
-
-       IRC_SetPenalty( target, 1 );
-       return CONNECTED;
+       return IRC_Send_LUSERS(from);
 } /* IRC_LUSERS */
 
-
 /**
  * Handler for the IRC command "SERVLIST".
- * List registered services, see RFC 2811, section 3.5.1: the syntax is
- * "SERVLIST [<mask> [<type>]]".
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
  */
 GLOBAL bool
 IRC_SERVLIST(CLIENT *Client, REQUEST *Req)
@@ -321,10 +710,6 @@ IRC_SERVLIST(CLIENT *Client, REQUEST *Req)
        assert(Client != NULL);
        assert(Req != NULL);
 
-       if (Req->argc > 2)
-               return IRC_WriteStrClient(Client, ERR_NEEDMOREPARAMS_MSG,
-                                         Client_ID(Client), Req->command);
-
        if (Req->argc < 2 || strcmp(Req->argv[1], "0") == 0) {
                for (c = Client_First(); c!= NULL; c = Client_Next(c)) {
                        if (Client_Type(c) != CLIENT_SERVICE)
@@ -345,7 +730,13 @@ IRC_SERVLIST(CLIENT *Client, REQUEST *Req)
                                  Req->argc > 1 ? Req->argv[1] : "0");
 } /* IRC_SERVLIST */
 
-
+/**
+ * Handler for the IRC command "MOTD".
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
+ */
 GLOBAL bool
 IRC_MOTD( CLIENT *Client, REQUEST *Req )
 {
@@ -354,31 +745,26 @@ IRC_MOTD( CLIENT *Client, REQUEST *Req )
        assert( Client != NULL );
        assert( Req != NULL );
 
-       if( Req->argc > 1 ) return IRC_WriteStrClient( Client, ERR_NEEDMOREPARAMS_MSG, Client_ID( Client ), Req->command );
+       _IRC_GET_SENDER_OR_RETURN_(from, Req, Client)
+       _IRC_GET_TARGET_SERVER_OR_RETURN_(target, Req, 0, from)
 
-       /* From aus Prefix ermitteln */
-       if( Client_Type( Client ) == CLIENT_SERVER ) from = Client_Search( Req->prefix );
-       else from = Client;
-       if( ! from ) return IRC_WriteStrClient( Client, ERR_NOSUCHNICK_MSG, Client_ID( Client ), Req->prefix );
-
-       if( Req->argc == 1 )
-       {
-               /* forward? */
-               target = Client_Search( Req->argv[0] );
-               if(( ! target ) || ( Client_Type( target ) != CLIENT_SERVER )) return IRC_WriteStrClient( from, ERR_NOSUCHSERVER_MSG, Client_ID( from ), Req->argv[0] );
-
-               if( target != Client_ThisServer( ))
-               {
-                       /* Ok, anderer Server ist das Ziel: forwarden */
-                       return IRC_WriteStrClientPrefix( target, from, "MOTD %s", Req->argv[0] );
-               }
+       /* Forward? */
+       if (target != Client_ThisServer()) {
+               IRC_WriteStrClientPrefix(target, from, "MOTD %s",
+                                        Client_ID(target));
+               return CONNECTED;
        }
 
-       IRC_SetPenalty( from, 3 );
-       return IRC_Show_MOTD( from );
+       return IRC_Show_MOTD(from);
 } /* IRC_MOTD */
 
-
+/**
+ * Handler for the IRC command "NAMES".
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
+ */
 GLOBAL bool
 IRC_NAMES( CLIENT *Client, REQUEST *Req )
 {
@@ -389,121 +775,73 @@ IRC_NAMES( CLIENT *Client, REQUEST *Req )
        assert( Client != NULL );
        assert( Req != NULL );
 
-       if( Req->argc > 2 ) return IRC_WriteStrClient( Client, ERR_NEEDMOREPARAMS_MSG, Client_ID( Client ), Req->command );
+       _IRC_GET_SENDER_OR_RETURN_(from, Req, Client)
+       _IRC_GET_TARGET_SERVER_OR_RETURN_(target, Req, 1, from)
 
-       /* use prefix to determine "From" */
-       if( Client_Type( Client ) == CLIENT_SERVER ) from = Client_Search( Req->prefix );
-       else from = Client;
-       if( ! from ) return IRC_WriteStrClient( Client, ERR_NOSUCHNICK_MSG, Client_ID( Client ), Req->prefix );
-
-       if( Req->argc == 2 )
-       {
-               /* forward to another server? */
-               target = Client_Search( Req->argv[1] );
-               if(( ! target ) || ( Client_Type( target ) != CLIENT_SERVER )) return IRC_WriteStrClient( from, ERR_NOSUCHSERVER_MSG, Client_ID( from ), Req->argv[1] );
-
-               if( target != Client_ThisServer( )) {
-                       /* target is another server, forward */
-                       return IRC_WriteStrClientPrefix( target, from, "NAMES %s :%s", Req->argv[0], Req->argv[1] );
-               }
+       /* Forward? */
+       if (target != Client_ThisServer()) {
+               IRC_WriteStrClientPrefix(target, from, "NAMES %s :%s",
+                                        Req->argv[0], Client_ID(target));
+               return CONNECTED;
        }
 
-       if( Req->argc > 0 )
-       {
-               /* bestimmte Channels durchgehen */
-               ptr = strtok( Req->argv[0], "," );
-               while( ptr )
-               {
-                       chan = Channel_Search( ptr );
-                       if( chan )
-                       {
-                               /* print name */
-                               if( ! IRC_Send_NAMES( from, chan )) return DISCONNECTED;
-                       }
-                       if( ! IRC_WriteStrClient( from, RPL_ENDOFNAMES_MSG, Client_ID( from ), ptr )) return DISCONNECTED;
-
-                       /* get next channel name */
+       if (Req->argc > 0) {
+               /* Return NAMES list for specific channels */
+               ptr = strtok(Req->argv[0], ",");
+               while(ptr) {
+                       chan = Channel_Search(ptr);
+                       if (chan && !IRC_Send_NAMES(from, chan))
+                               return DISCONNECTED;
+                       if (!IRC_WriteStrClient(from, RPL_ENDOFNAMES_MSG,
+                                               Client_ID(from), ptr))
+                               return DISCONNECTED;
                        ptr = strtok( NULL, "," );
                }
                return CONNECTED;
        }
 
-       chan = Channel_First( );
-       while( chan )
-       {
-               if( ! IRC_Send_NAMES( from, chan )) return DISCONNECTED;
-
-               chan = Channel_Next( chan );
+       chan = Channel_First();
+       while (chan) {
+               if (!IRC_Send_NAMES(from, chan))
+                       return DISCONNECTED;
+               chan = Channel_Next(chan);
        }
 
        /* Now print all clients which are not in any channel */
-       c = Client_First( );
-       snprintf( rpl, sizeof( rpl ), RPL_NAMREPLY_MSG, Client_ID( from ), "*", "*" );
-       while( c )
-       {
-               if(( Client_Type( c ) == CLIENT_USER ) && ( Channel_FirstChannelOf( c ) == NULL ) && ( ! strchr( Client_Modes( c ), 'i' )))
-               {
-                       /* its a user, concatenate ... */
-                       if( rpl[strlen( rpl ) - 1] != ':' ) strlcat( rpl, " ", sizeof( rpl ));
-                       strlcat( rpl, Client_ID( c ), sizeof( rpl ));
-
-                       if( strlen( rpl ) > ( LINE_LEN - CLIENT_NICK_LEN - 4 ))
-                       {
-                               /* Line is gwoing too long, send now */
-                               if( ! IRC_WriteStrClient( from, "%s", rpl )) return DISCONNECTED;
-                               snprintf( rpl, sizeof( rpl ), RPL_NAMREPLY_MSG, Client_ID( from ), "*", "*" );
-                       }
-               }
-
-               c = Client_Next( c );
-       }
-       if( rpl[strlen( rpl ) - 1] != ':')
-       {
-               if( ! IRC_WriteStrClient( from, "%s", rpl )) return DISCONNECTED;
-       }
-
-       IRC_SetPenalty( from, 1 );
-       return IRC_WriteStrClient( from, RPL_ENDOFNAMES_MSG, Client_ID( from ), "*" );
-} /* IRC_NAMES */
-
-
-static unsigned int
-t_diff(time_t *t, const time_t d)
-{
-       time_t diff, remain;
-
-       diff = *t / d;
-       remain = diff * d;
-       *t -= remain;
-
-       return (unsigned int)diff;
-}
-
-
-static unsigned int
-uptime_days(time_t *now)
-{
-       return t_diff(now, 60 * 60 * 24);
-}
-
-
-static unsigned int
-uptime_hrs(time_t *now)
-{
-       return t_diff(now, 60 * 60);
-}
-
+       c = Client_First();
+       snprintf(rpl, sizeof(rpl), RPL_NAMREPLY_MSG, Client_ID(from), "*", "*");
+       while (c) {
+               if (Client_Type(c) == CLIENT_USER
+                   && Channel_FirstChannelOf(c) == NULL
+                   && !Client_HasMode(c, 'i'))
+               {
+                       /* its a user, concatenate ... */
+                       if (rpl[strlen(rpl) - 1] != ':')
+                               strlcat(rpl, " ", sizeof(rpl));
+                       strlcat(rpl, Client_ID(c), sizeof(rpl));
 
-static unsigned int
-uptime_mins(time_t *now)
-{
-        return t_diff(now, 60);
-}
+                       if (strlen(rpl) > COMMAND_LEN - CLIENT_NICK_LEN - 4) {
+                               /* Line is gwoing too long, send now */
+                               if (!IRC_WriteStrClient(from, "%s", rpl))
+                                       return DISCONNECTED;
+                               snprintf(rpl, sizeof(rpl), RPL_NAMREPLY_MSG,
+                                        Client_ID(from), "*", "*");
+                       }
+               }
+               c = Client_Next(c);
+       }
+       if (rpl[strlen(rpl) - 1] != ':' && !IRC_WriteStrClient(from, "%s", rpl))
+               return DISCONNECTED;
 
+       return IRC_WriteStrClient(from, RPL_ENDOFNAMES_MSG, Client_ID(from), "*");
+} /* IRC_NAMES */
 
 /**
  * Handler for the IRC command "STATS".
- * See RFC 2812 section 3.4.4.
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
  */
 GLOBAL bool
 IRC_STATS( CLIENT *Client, REQUEST *Req )
@@ -516,36 +854,19 @@ IRC_STATS( CLIENT *Client, REQUEST *Req )
        unsigned int days, hrs, mins;
        struct list_head *list;
        struct list_elem *list_item;
+       bool more_links = false;
 
        assert(Client != NULL);
        assert(Req != NULL);
 
-       if (Req->argc > 2)
-               return IRC_WriteStrClient(Client, ERR_NEEDMOREPARAMS_MSG,
-                                         Client_ID(Client), Req->command);
-
-       /* use prefix to determine "From" */
-       if (Client_Type(Client) == CLIENT_SERVER)
-               from = Client_Search(Req->prefix);
-       else
-               from = Client;
-
-       if (!from)
-               return IRC_WriteStrClient(Client, ERR_NOSUCHNICK_MSG,
-                                         Client_ID(Client), Req->prefix);
-
-       if (Req->argc == 2) {
-               /* forward to another server? */
-               target = Client_Search(Req->argv[1]);
-               if ((!target) || (Client_Type(target) != CLIENT_SERVER))
-                       return IRC_WriteStrClient(from, ERR_NOSUCHSERVER_MSG,
-                                                Client_ID(from), Req->argv[1]);
+       _IRC_GET_SENDER_OR_RETURN_(from, Req, Client)
+       _IRC_GET_TARGET_SERVER_OR_RETURN_(target, Req, 1, from)
 
-               if (target != Client_ThisServer()) {
-                       /* forward to another server */
-                       return IRC_WriteStrClientPrefix(target, from,
-                                    "STATS %s %s", Req->argv[0], Req->argv[1]);
-               }
+       /* Forward? */
+       if (target != Client_ThisServer()) {
+               IRC_WriteStrClientPrefix(target, from, "STATS %s %s",
+                                        Req->argv[0], Client_ID(target));
+               return CONNECTED;
        }
 
        if (Req->argc > 0)
@@ -559,7 +880,7 @@ IRC_STATS( CLIENT *Client, REQUEST *Req )
        case 'k':       /* Server-local bans ("K-Lines") */
        case 'K':
                if (!Client_HasMode(from, 'o'))
-                   return IRC_WriteStrClient(from, ERR_NOPRIVILEGES_MSG,
+                   return IRC_WriteErrClient(from, ERR_NOPRIVILEGES_MSG,
                                              Client_ID(from));
                if (query == 'g' || query == 'G')
                        list = Class_GetList(CLASS_GLINE);
@@ -576,16 +897,20 @@ IRC_STATS( CLIENT *Client, REQUEST *Req )
                                list_item = Lists_GetNext(list_item);
                        }
                break;
+       case 'L':       /* Link status (servers and user links) */
+               if (!Op_Check(from, Req))
+                       return Op_NoPrivileges(from, Req);
+               more_links = true;
+
        case 'l':       /* Link status (servers and own link) */
-       case 'L':
                time_now = time(NULL);
                for (con = Conn_First(); con != NONE; con = Conn_Next(con)) {
                        cl = Conn_GetClient(con);
                        if (!cl)
                                continue;
-                       if ((Client_Type(cl) == CLIENT_SERVER)
-                           || (cl == Client)) {
-                               /* Server link or our own connection */
+                       if (Client_Type(cl) == CLIENT_SERVER ||
+                           cl == Client ||
+                           (more_links && Client_Type(cl) == CLIENT_USER)) {
 #ifdef ZLIB
                                if (Conn_Options(con) & CONN_ZIP) {
                                        if (!IRC_WriteStrClient
@@ -637,25 +962,33 @@ IRC_STATS( CLIENT *Client, REQUEST *Req )
                break;
        }
 
-       IRC_SetPenalty(from, 2);
        return IRC_WriteStrClient(from, RPL_ENDOFSTATS_MSG,
                                  Client_ID(from), query);
 } /* IRC_STATS */
 
-
 /**
  * Handler for the IRC command "SUMMON".
- * See RFC 2812 section 4.5. ngIRCd doesn't implement this functionality and
- * therefore answers with ERR_SUMMONDISABLED.
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
  */
 GLOBAL bool
 IRC_SUMMON(CLIENT * Client, UNUSED REQUEST * Req)
 {
-       return IRC_WriteStrClient(Client, ERR_SUMMONDISABLED_MSG,
+       assert(Client != NULL);
+
+       return IRC_WriteErrClient(Client, ERR_SUMMONDISABLED_MSG,
                                  Client_ID(Client));
 } /* IRC_SUMMON */
 
-
+/**
+ * Handler for the IRC command "TIME".
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
+ */
 GLOBAL bool
 IRC_TIME( CLIENT *Client, REQUEST *Req )
 {
@@ -663,35 +996,31 @@ IRC_TIME( CLIENT *Client, REQUEST *Req )
        char t_str[64];
        time_t t;
 
-       assert( Client != NULL );
-       assert( Req != NULL );
-
-       if( Req->argc > 1 ) return IRC_WriteStrClient( Client, ERR_NEEDMOREPARAMS_MSG, Client_ID( Client ), Req->command );
-
-       if( Client_Type( Client ) == CLIENT_SERVER ) from = Client_Search( Req->prefix );
-       else from = Client;
-       if( ! from ) return IRC_WriteStrClient( Client, ERR_NOSUCHNICK_MSG, Client_ID( Client ), Req->prefix );
+       assert(Client != NULL);
+       assert(Req != NULL);
 
-       if( Req->argc == 1 )
-       {
-               target = Client_Search( Req->argv[0] );
-               if(( ! target ) || ( Client_Type( target ) != CLIENT_SERVER )) return IRC_WriteStrClient( Client, ERR_NOSUCHSERVER_MSG, Client_ID( Client ), Req->argv[0] );
+       _IRC_GET_SENDER_OR_RETURN_(from, Req, Client)
+       _IRC_GET_TARGET_SERVER_OR_RETURN_(target, Req, 0, from)
 
-               if( target != Client_ThisServer( ))
-               {
-                       return IRC_WriteStrClientPrefix( target, from, "TIME %s", Req->argv[0] );
-               }
+       /* Forward? */
+       if (target != Client_ThisServer()) {
+               IRC_WriteStrClientPrefix(target, from, "TIME %s",
+                                        Client_ID(target));
+               return CONNECTED;
        }
 
        t = time( NULL );
-       (void)strftime( t_str, 60, "%A %B %d %Y -- %H:%M %Z", localtime( &t ));
-       return IRC_WriteStrClient( from, RPL_TIME_MSG, Client_ID( from ), Client_ID( Client_ThisServer( )), t_str );
+       (void)strftime(t_str, 60, "%A %B %d %Y -- %H:%M %Z", localtime(&t));
+       return IRC_WriteStrClient(from, RPL_TIME_MSG, Client_ID(from),
+                                 Client_ID(Client_ThisServer()), t_str);
 } /* IRC_TIME */
 
-
 /**
  * Handler for the IRC command "USERHOST".
- * See RFC 2812 section 4.8.
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
  */
 GLOBAL bool
 IRC_USERHOST(CLIENT *Client, REQUEST *Req)
@@ -703,10 +1032,6 @@ IRC_USERHOST(CLIENT *Client, REQUEST *Req)
        assert(Client != NULL);
        assert(Req != NULL);
 
-       if ((Req->argc < 1))
-               return IRC_WriteStrClient(Client, ERR_NEEDMOREPARAMS_MSG,
-                                         Client_ID(Client), Req->command);
-
        if (Req->argc > 5)
                max = 5;
        else
@@ -715,288 +1040,87 @@ IRC_USERHOST(CLIENT *Client, REQUEST *Req)
        strlcpy(rpl, RPL_USERHOST_MSG, sizeof rpl);
        for (i = 0; i < max; i++) {
                c = Client_Search(Req->argv[i]);
-               if (c && (Client_Type(c) == CLIENT_USER)) {
-                       /* This Nick is "online" */
-                       strlcat(rpl, Client_ID(c), sizeof(rpl));
-                       if (Client_HasMode(c, 'o'))
-                               strlcat(rpl, "*", sizeof(rpl));
-                       strlcat(rpl, "=", sizeof(rpl));
-                       if (Client_HasMode(c, 'a'))
-                               strlcat(rpl, "-", sizeof(rpl));
-                       else
-                               strlcat(rpl, "+", sizeof(rpl));
-                       strlcat(rpl, Client_User(c), sizeof(rpl));
-                       strlcat(rpl, "@", sizeof(rpl));
-                       strlcat(rpl, Client_HostnameCloaked(c), sizeof(rpl));
-                       strlcat(rpl, " ", sizeof(rpl));
-               }
-       }
-       ngt_TrimLastChr(rpl, ' ');
-
-       return IRC_WriteStrClient(Client, rpl, Client_ID(Client));
-} /* IRC_USERHOST */
-
-
-/**
- * Handler for the IRC command "USERS".
- * See RFC 2812 section 4.6. As suggested there the command is disabled.
- */
-GLOBAL bool
-IRC_USERS(CLIENT * Client, UNUSED REQUEST * Req)
-{
-       return IRC_WriteStrClient(Client, ERR_USERSDISABLED_MSG,
-                                 Client_ID(Client));
-} /* IRC_USERS */
-
-
-GLOBAL bool
-IRC_VERSION( CLIENT *Client, REQUEST *Req )
-{
-       CLIENT *target, *prefix;
-
-       assert( Client != NULL );
-       assert( Req != NULL );
-
-       if(( Req->argc > 1 )) return IRC_WriteStrClient( Client, ERR_NEEDMOREPARAMS_MSG, Client_ID( Client ), Req->command );
-
-       /* Ziel suchen */
-       if( Req->argc == 1 ) target = Client_Search( Req->argv[0] );
-       else target = Client_ThisServer( );
-
-       /* Prefix ermitteln */
-       if( Client_Type( Client ) == CLIENT_SERVER ) prefix = Client_Search( Req->prefix );
-       else prefix = Client;
-       if( ! prefix ) return IRC_WriteStrClient( Client, ERR_NOSUCHNICK_MSG, Client_ID( Client ), Req->prefix );
-
-       /* An anderen Server weiterleiten? */
-       if( target != Client_ThisServer( ))
-       {
-               if(( ! target ) || ( Client_Type( target ) != CLIENT_SERVER )) return IRC_WriteStrClient( prefix, ERR_NOSUCHSERVER_MSG, Client_ID( prefix ), Req->argv[0] );
-
-               /* forwarden */
-               IRC_WriteStrClientPrefix( target, prefix, "VERSION %s", Req->argv[0] );
-               return CONNECTED;
-       }
-
-       /* send version information */
-       IRC_SetPenalty(Client, 1);
-       return IRC_WriteStrClient(Client, RPL_VERSION_MSG, Client_ID(prefix),
-                                 PACKAGE_NAME, PACKAGE_VERSION,
-                                 NGIRCd_DebugLevel, Conf_ServerName,
-                                 NGIRCd_VersionAddition);
-} /* IRC_VERSION */
-
-
-static bool
-write_whoreply(CLIENT *Client, CLIENT *c, const char *channelname, const char *flags)
-{
-       return IRC_WriteStrClient(Client, RPL_WHOREPLY_MSG, Client_ID(Client),
-                                 channelname, Client_User(c),
-                                 Client_HostnameCloaked(c),
-                                 Client_ID(Client_Introducer(c)), Client_ID(c),
-                                 flags, Client_Hops(c), Client_Info(c));
-}
-
-
-static const char *
-who_flags_status(const char *client_modes)
-{
-       if (strchr(client_modes, 'a'))
-               return "G"; /* away */
-       return "H";
-}
-
-
-static char *
-who_flags_qualifier(CLIENT *Client, const char *chan_user_modes, char *str, size_t len)
-{
-       assert(Client != NULL);
-
-       if (Client_Cap(Client) & CLIENT_CAP_MULTI_PREFIX) {
-               if (strchr(chan_user_modes, 'q'))
-                       strlcat(str, "~", len);
-               if (strchr(chan_user_modes, 'a'))
-                       strlcat(str, "&", len);
-               if (strchr(chan_user_modes, 'o'))
-                       strlcat(str, "@", len);
-               if (strchr(chan_user_modes, 'h'))
-                       strlcat(str, "%", len);
-               if (strchr(chan_user_modes, 'v'))
-                       strlcat(str, "+", len);
-
-               return str;
-       }
-
-       if (strchr(chan_user_modes, 'q'))
-               strlcat(str, "~", len);
-       else if (strchr(chan_user_modes, 'a'))
-               strlcat(str, "&", len);
-       else if (strchr(chan_user_modes, 'o'))
-               strlcat(str, "@", len);
-       else if (strchr(chan_user_modes, 'h'))
-               strlcat(str, "%", len);
-       else if (strchr(chan_user_modes, 'v'))
-               strlcat(str, "+", len);
-
-       return str;
-}
-
-
-/**
- * Send WHO reply for a "channel target" ("WHO #channel").
- *
- * @param Client Client requesting the information.
- * @param Chan Channel being requested.
- * @param OnlyOps Only display IRC operators.
- * @return CONNECTED or DISCONNECTED.
- */
-static bool
-IRC_WHO_Channel(CLIENT *Client, CHANNEL *Chan, bool OnlyOps)
-{
-       bool is_visible, is_member, is_ircop;
-       CL2CHAN *cl2chan;
-       const char *client_modes;
-       const char *chan_user_modes;
-       char flags[10];
-       CLIENT *c;
-       int count = 0;
-
-       assert( Client != NULL );
-       assert( Chan != NULL );
-
-       IRC_SetPenalty(Client, 1);
-
-       is_member = Channel_IsMemberOf(Chan, Client);
-
-       /* Secret channel? */
-       if (!is_member && strchr(Channel_Modes(Chan), 's'))
-               return IRC_WriteStrClient(Client, RPL_ENDOFWHO_MSG,
-                                         Client_ID(Client), Channel_Name(Chan));
-
-       cl2chan = Channel_FirstMember(Chan);
-       for (; cl2chan ; cl2chan = Channel_NextMember(Chan, cl2chan)) {
-               c = Channel_GetClient(cl2chan);
-
-               client_modes = Client_Modes(c);
-               is_ircop = strchr(client_modes, 'o') != NULL;
-               if (OnlyOps && !is_ircop)
-                       continue;
-
-               is_visible = strchr(client_modes, 'i') == NULL;
-               if (is_member || is_visible) {
-                       strcpy(flags, who_flags_status(client_modes));
-                       if (is_ircop)
-                               strlcat(flags, "*", sizeof(flags));
-
-                       chan_user_modes = Channel_UserModes(Chan, c);
-                       who_flags_qualifier(c, chan_user_modes, flags, sizeof(flags));
-
-                       if (!write_whoreply(Client, c, Channel_Name(Chan),
-                                           flags))
-                               return DISCONNECTED;
-                       count++;
-               }
-       }
-
-       /* If there are a lot of clients, augment penalty a bit */
-       if (count > MAX_RPL_WHO)
-               IRC_SetPenalty(Client, 1);
-
-       return IRC_WriteStrClient(Client, RPL_ENDOFWHO_MSG, Client_ID(Client),
-                                 Channel_Name(Chan));
-}
-
-
-/**
- * Send WHO reply for a "mask target" ("WHO m*sk").
- *
- * @param Client Client requesting the information.
- * @param Mask Mask being requested or NULL for "all" clients.
- * @param OnlyOps Only display IRC operators.
- * @return CONNECTED or DISCONNECTED.
- */
-static bool
-IRC_WHO_Mask(CLIENT *Client, char *Mask, bool OnlyOps)
-{
-       CLIENT *c;
-       CL2CHAN *cl2chan;
-       CHANNEL *chan;
-       bool client_match, is_visible;
-       char flags[4];
-       int count = 0;
-
-       assert (Client != NULL);
-
-       if (Mask)
-               ngt_LowerStr(Mask);
-
-       IRC_SetPenalty(Client, 3);
-       for (c = Client_First(); c != NULL; c = Client_Next(c)) {
-               if (Client_Type(c) != CLIENT_USER)
-                       continue;
-
-               if (OnlyOps && !Client_HasMode(c, 'o'))
-                       continue;
-
-               if (Mask) {
-                       /* Match pattern against user host/server/name/nick */
-                       client_match = MatchCaseInsensitive(Mask,
-                                               Client_Hostname(c));
-                       if (!client_match)
-                               client_match = MatchCaseInsensitive(Mask,
-                                               Client_ID(Client_Introducer(c)));
-                       if (!client_match)
-                               client_match = MatchCaseInsensitive(Mask,
-                                               Client_Info(c));
-                       if (!client_match)
-                               client_match = MatchCaseInsensitive(Mask,
-                                               Client_ID(c));
-                       if (!client_match)
-                               continue;       /* no match: skip this client */
+               if (c && (Client_Type(c) == CLIENT_USER)) {
+                       /* This Nick is "online" */
+                       strlcat(rpl, Client_ID(c), sizeof(rpl));
+                       if (Client_HasMode(c, 'o'))
+                               strlcat(rpl, "*", sizeof(rpl));
+                       strlcat(rpl, "=", sizeof(rpl));
+                       if (Client_HasMode(c, 'a'))
+                               strlcat(rpl, "-", sizeof(rpl));
+                       else
+                               strlcat(rpl, "+", sizeof(rpl));
+                       strlcat(rpl, Client_User(c), sizeof(rpl));
+                       strlcat(rpl, "@", sizeof(rpl));
+                       strlcat(rpl, Client_HostnameDisplayed(c), sizeof(rpl));
+                       strlcat(rpl, " ", sizeof(rpl));
                }
+       }
+       ngt_TrimLastChr(rpl, ' ');
 
-               is_visible = !Client_HasMode(c, 'i');
+       return IRC_WriteStrClient(Client, rpl, Client_ID(Client));
+} /* IRC_USERHOST */
 
-               /* Target client is invisible, but mask matches exactly? */
-               if (!is_visible && Mask && strcasecmp(Client_ID(c), Mask) == 0)
-                       is_visible = true;
+/**
+ * Handler for the IRC command "USERS".
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
+ */
+GLOBAL bool
+IRC_USERS(CLIENT * Client, UNUSED REQUEST * Req)
+{
+       assert(Client != NULL);
 
-               /* Target still invisible, but are both on the same channel? */
-               if (!is_visible) {
-                       cl2chan = Channel_FirstChannelOf(Client);
-                       while (cl2chan && !is_visible) {
-                               chan = Channel_GetChannel(cl2chan);
-                               if (Channel_IsMemberOf(chan, c))
-                                       is_visible = true;
-                               cl2chan = Channel_NextChannelOf(Client, cl2chan);
-                       }
-               }
+       return IRC_WriteErrClient(Client, ERR_USERSDISABLED_MSG,
+                                 Client_ID(Client));
+} /* IRC_USERS */
 
-               if (!is_visible)        /* target user is not visible */
-                       continue;
+/**
+ * Handler for the IRC command "VERSION".
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
+ */
+GLOBAL bool
+IRC_VERSION( CLIENT *Client, REQUEST *Req )
+{
+       CLIENT *target, *prefix;
 
-               if (IRC_CheckListTooBig(Client, count, MAX_RPL_WHO, "WHO"))
-                       break;
+       assert( Client != NULL );
+       assert( Req != NULL );
 
-               strcpy(flags, who_flags_status(Client_Modes(c)));
-               if (strchr(Client_Modes(c), 'o'))
-                       strlcat(flags, "*", sizeof(flags));
+       _IRC_GET_SENDER_OR_RETURN_(prefix, Req, Client)
+       _IRC_GET_TARGET_SERVER_OR_RETURN_(target, Req, 0, prefix)
 
-               if (!write_whoreply(Client, c, "*", flags))
-                       return DISCONNECTED;
-               count++;
+       /* Forward? */
+       if (target != Client_ThisServer()) {
+               IRC_WriteStrClientPrefix(target, prefix, "VERSION %s",
+                                        Client_ID(target));
+               return CONNECTED;
        }
 
-       return IRC_WriteStrClient(Client, RPL_ENDOFWHO_MSG, Client_ID(Client),
-                                 Mask ? Mask : "*");
-}
+       /* send version information */
+       if (!IRC_WriteStrClient(Client, RPL_VERSION_MSG, Client_ID(prefix),
+                               PACKAGE_NAME, PACKAGE_VERSION,
+                               NGIRCd_DebugLevel, Conf_ServerName,
+                               NGIRCd_VersionAddition))
+               return DISCONNECTED;
+
+#ifndef STRICT_RFC
+       /* send RPL_ISUPPORT(005) numerics */
+       if (!IRC_Send_ISUPPORT(prefix))
+               return DISCONNECTED;
+#endif
 
+       return CONNECTED;
+} /* IRC_VERSION */
 
 /**
  * Handler for the IRC "WHO" command.
  *
- * See RFC 2812, 3.6.1 "Who query".
- *
  * @param Client The client from which this command has been received.
  * @param Req Request structure with prefix and all parameters.
  * @return CONNECTED or DISCONNECTED.
@@ -1010,25 +1134,20 @@ IRC_WHO(CLIENT *Client, REQUEST *Req)
        assert (Client != NULL);
        assert (Req != NULL);
 
-       if (Req->argc > 2)
-               return IRC_WriteStrClient(Client, ERR_NEEDMOREPARAMS_MSG,
-                                         Client_ID(Client), Req->command);
-
        only_ops = false;
-
        if (Req->argc == 2) {
                if (strcmp(Req->argv[1], "o") == 0)
                        only_ops = true;
 #ifdef STRICT_RFC
-               else
-                       return IRC_WriteStrClient(Client,
+               else {
+                       return IRC_WriteErrClient(Client,
                                                  ERR_NEEDMOREPARAMS_MSG,
                                                  Client_ID(Client),
                                                  Req->command);
+               }
 #endif
        }
 
-       IRC_SetPenalty(Client, 1);
        if (Req->argc >= 1) {
                /* Channel or mask given */
                chan = Channel_Search(Req->argv[0]);
@@ -1044,141 +1163,16 @@ IRC_WHO(CLIENT *Client, REQUEST *Req)
        }
 
        /* No channel or (valid) mask given */
-       IRC_SetPenalty(Client, 2);
        return IRC_WHO_Mask(Client, NULL, only_ops);
 } /* IRC_WHO */
 
-
 /**
- * Generate WHOIS reply of one actual client.
+ * Handler for the IRC "WHOIS" command.
  *
  * @param Client The client from which this command has been received.
- * @param from The client requesting the information ("originator").
- * @param c The client of which information should be returned.
+ * @param Req Request structure with prefix and all parameters.
  * @return CONNECTED or DISCONNECTED.
  */
-static bool
-IRC_WHOIS_SendReply(CLIENT *Client, CLIENT *from, CLIENT *c)
-{
-       char str[LINE_LEN + 1];
-       CL2CHAN *cl2chan;
-       CHANNEL *chan;
-
-       assert(Client != NULL);
-       assert(from != NULL);
-       assert(c != NULL);
-
-       /* Nick, user, hostname and client info */
-       if (!IRC_WriteStrClient(from, RPL_WHOISUSER_MSG, Client_ID(from),
-                               Client_ID(c), Client_User(c),
-                               Client_HostnameCloaked(c), Client_Info(c)))
-               return DISCONNECTED;
-
-       /* Server */
-       if (!IRC_WriteStrClient(from, RPL_WHOISSERVER_MSG, Client_ID(from),
-                               Client_ID(c), Client_ID(Client_Introducer(c)),
-                               Client_Info(Client_Introducer(c))))
-               return DISCONNECTED;
-
-       /* Channels */
-       snprintf(str, sizeof(str), RPL_WHOISCHANNELS_MSG,
-                Client_ID(from), Client_ID(c));
-       cl2chan = Channel_FirstChannelOf(c);
-       while (cl2chan) {
-               chan = Channel_GetChannel(cl2chan);
-               assert(chan != NULL);
-
-               /* next */
-               cl2chan = Channel_NextChannelOf(c, cl2chan);
-
-               /* Secret channel? */
-               if (strchr(Channel_Modes(chan), 's')
-                   && !Channel_IsMemberOf(chan, Client))
-                       continue;
-
-               /* Local channel and request is not from a user? */
-               if (Client_Type(Client) == CLIENT_SERVER
-                   && Channel_IsLocal(chan))
-                       continue;
-
-               /* Concatenate channel names */
-               if (str[strlen(str) - 1] != ':')
-                       strlcat(str, " ", sizeof(str));
-
-               who_flags_qualifier(c, Channel_UserModes(chan, c), str, sizeof(str));
-               strlcat(str, Channel_Name(chan), sizeof(str));
-
-               if (strlen(str) > (LINE_LEN - CHANNEL_NAME_LEN - 4)) {
-                       /* Line becomes too long: send it! */
-                       if (!IRC_WriteStrClient(Client, "%s", str))
-                               return DISCONNECTED;
-                       snprintf(str, sizeof(str), RPL_WHOISCHANNELS_MSG,
-                                Client_ID(from), Client_ID(c));
-               }
-       }
-       if(str[strlen(str) - 1] != ':') {
-               /* There is data left to send: */
-               if (!IRC_WriteStrClient(Client, "%s", str))
-                       return DISCONNECTED;
-       }
-
-       /* IRC-Operator? */
-       if (Client_HasMode(c, 'o') &&
-           !IRC_WriteStrClient(from, RPL_WHOISOPERATOR_MSG,
-                               Client_ID(from), Client_ID(c)))
-               return DISCONNECTED;
-
-       /* IRC-Bot? */
-       if (Client_HasMode(c, 'B') &&
-           !IRC_WriteStrClient(from, RPL_WHOISBOT_MSG,
-                               Client_ID(from), Client_ID(c)))
-               return DISCONNECTED;
-
-       /* Connected using SSL? */
-       if (Conn_UsesSSL(Client_Conn(c)) &&
-           !IRC_WriteStrClient(from, RPL_WHOISSSL_MSG, Client_ID(from),
-                               Client_ID(c)))
-               return DISCONNECTED;
-
-       /* Registered nick name? */
-       if (Client_HasMode(c, 'R') &&
-           !IRC_WriteStrClient(from, RPL_WHOISREGNICK_MSG,
-                               Client_ID(from), Client_ID(c)))
-               return DISCONNECTED;
-
-       if (Client_Conn(c) > NONE && (Client_OperByMe(from) || from == c) &&
-           !IRC_WriteStrClient(from, RPL_WHOISHOST_MSG, Client_ID(from),
-                               Client_ID(c), Client_Hostname(c),
-                               Conn_GetIPAInfo(Client_Conn(c))))
-               return DISCONNECTED;
-
-       /* Idle and signon time (local clients only!) */
-       if (!Conf_MorePrivacy && Client_Conn(c) > NONE &&
-           !IRC_WriteStrClient(from, RPL_WHOISIDLE_MSG,
-                               Client_ID(from), Client_ID(c),
-                               (unsigned long)Conn_GetIdle(Client_Conn(c)),
-                               (unsigned long)Conn_GetSignon(Client_Conn(c))))
-               return DISCONNECTED;
-
-       /* Away? */
-       if (Client_HasMode(c, 'a') &&
-           !IRC_WriteStrClient(from, RPL_AWAY_MSG,
-                               Client_ID(from), Client_ID(c), Client_Away(c)))
-               return DISCONNECTED;
-
-       return CONNECTED;
-} /* IRC_WHOIS_SendReply */
-
-
-/**
- * Handler for the IRC "WHOIS" command.
- *
- * See RFC 2812, 3.6.2 "Whois query".
- *
- * @param Client       The client from which this command has been received.
- * @param Req          Request structure with prefix and all parameters.
- * @return             CONNECTED or DISCONNECTED.
- */
 GLOBAL bool
 IRC_WHOIS( CLIENT *Client, REQUEST *Req )
 {
@@ -1191,40 +1185,28 @@ IRC_WHOIS( CLIENT *Client, REQUEST *Req )
        assert( Client != NULL );
        assert( Req != NULL );
 
-       /* Bad number of parameters? */
-       if (Req->argc < 1 || Req->argc > 2)
-               return IRC_WriteStrClient(Client, ERR_NEEDMOREPARAMS_MSG,
-                                         Client_ID(Client), Req->command);
+       /* Wrong number of parameters? */
+       if (Req->argc < 1)
+               return IRC_WriteErrClient(Client, ERR_NONICKNAMEGIVEN_MSG,
+                                         Client_ID(Client));
 
-       /* Search sender of the WHOIS */
-       if (Client_Type(Client) == CLIENT_SERVER) {
-               from = Client_Search(Req->prefix);
-       } else {
-               IRC_SetPenalty(Client, 1);
-               from = Client;
-       }
-       if (!from)
-               return IRC_WriteStrClient(Client, ERR_NOSUCHNICK_MSG,
-                                         Client_ID(Client), Req->prefix);
+       _IRC_ARGC_LE_OR_RETURN_(Client, Req, 2)
+       _IRC_GET_SENDER_OR_RETURN_(from, Req, Client)
 
        /* Get target server for this command */
        if (Req->argc > 1) {
-               /* Search the target server, which can be specified as a
-                * nick name on that server as well: */
-               target = Client_Search(Req->argv[0]);
-               if (!target)
-                       return IRC_WriteStrClient(from, ERR_NOSUCHSERVER_MSG,
-                                               Client_ID(from), Req->argv[0]);
+               _IRC_GET_TARGET_SERVER_OR_RETURN_(target, Req, 0, Client)
        } else
                target = Client_ThisServer();
+
        assert(target != NULL);
 
-       /* Forward to other server? */
-       if (Client_NextHop(target) != Client_ThisServer() &&
-           Client_Type(Client_NextHop(target)) == CLIENT_SERVER)
-               return IRC_WriteStrClientPrefix(target, from,
-                                               "WHOIS %s :%s",
-                                               Req->argv[0], Req->argv[1]);
+       /* Forward? */
+       if (target != Client_ThisServer()) {
+               IRC_WriteStrClientPrefix(target, from, "WHOIS %s :%s",
+                                        Req->argv[0], Req->argv[1]);
+               return CONNECTED;
+       }
 
        is_remote = Client_Conn(from) < 0;
        strlcpy(mask, Req->argv[Req->argc - 1], sizeof(mask));
@@ -1243,11 +1225,12 @@ IRC_WHOIS( CLIENT *Client, REQUEST *Req )
                 */
                if (!has_wildcards || is_remote) {
                        c = Client_Search(query);
-                       if (c && Client_Type(c) == CLIENT_USER) {
+                       if (c && (Client_Type(c) == CLIENT_USER
+                                 || Client_Type(c) == CLIENT_SERVICE)) {
                                if (!IRC_WHOIS_SendReply(Client, from, c))
                                        return DISCONNECTED;
                        } else {
-                               if (!IRC_WriteStrClient(Client,
+                               if (!IRC_WriteErrClient(Client,
                                                        ERR_NOSUCHNICK_MSG,
                                                        Client_ID(Client),
                                                        query))
@@ -1257,12 +1240,13 @@ IRC_WHOIS( CLIENT *Client, REQUEST *Req )
                }
                if (got_wildcard) {
                        /* we already handled one wildcard query */
-                       if (!IRC_WriteStrClient(Client, ERR_NOSUCHNICK_MSG,
+                       if (!IRC_WriteErrClient(Client, ERR_NOSUCHNICK_MSG,
                             Client_ID(Client), query))
                                return DISCONNECTED;
                        continue;
                }
                got_wildcard = true;
+               /* Increase penalty for wildcard queries */
                IRC_SetPenalty(Client, 3);
 
                for (c = Client_First(); c; c = Client_Next(c)) {
@@ -1281,7 +1265,7 @@ IRC_WHOIS( CLIENT *Client, REQUEST *Req )
                }
 
                if (match_count == 0)
-                       IRC_WriteStrClient(Client, ERR_NOSUCHNICK_MSG,
+                       IRC_WriteErrClient(Client, ERR_NOSUCHNICK_MSG,
                                           Client_ID(Client),
                                           Req->argv[Req->argc - 1]);
        }
@@ -1289,27 +1273,12 @@ IRC_WHOIS( CLIENT *Client, REQUEST *Req )
                                  Client_ID(from), Req->argv[Req->argc - 1]);
 } /* IRC_WHOIS */
 
-
-static bool
-WHOWAS_EntryWrite(CLIENT *prefix, WHOWAS *entry)
-{
-       char t_str[60];
-
-       (void)strftime(t_str, sizeof(t_str), "%a %b %d %H:%M:%S %Y",
-                                       localtime(&entry->time));
-
-       if (!IRC_WriteStrClient(prefix, RPL_WHOWASUSER_MSG, Client_ID(prefix),
-                       entry->id, entry->user, entry->host, entry->info))
-                               return DISCONNECTED;
-
-       return IRC_WriteStrClient(prefix, RPL_WHOISSERVER_MSG, Client_ID(prefix),
-                 entry->id, entry->server, t_str);
-}
-
 /**
- * IRC "WHOWAS" function.
- * This function implements the IRC command "WHOWHAS". It handles local
- * requests and request that should be forwarded to other servers.
+ * Handler for the IRC "WHOWAS" command.
+ *
+ * @param Client The client from which this command has been received.
+ * @param Req Request structure with prefix and all parameters.
+ * @return CONNECTED or DISCONNECTED.
  */
 GLOBAL bool
 IRC_WHOWAS( CLIENT *Client, REQUEST *Req )
@@ -1323,43 +1292,24 @@ IRC_WHOWAS( CLIENT *Client, REQUEST *Req )
        assert( Client != NULL );
        assert( Req != NULL );
 
-       /* Do not reveal any info on disconnected users? */
-       if (Conf_MorePrivacy)
-               return CONNECTED;
-
        /* Wrong number of parameters? */
-       if (Req->argc > 3)
-               return IRC_WriteStrClient(Client, ERR_NEEDMOREPARAMS_MSG,
-                                       Client_ID(Client), Req->command);
        if (Req->argc < 1)
-               return IRC_WriteStrClient(Client, ERR_NONICKNAMEGIVEN_MSG, Client_ID(Client));
-
-       /* Search target */
-       if (Req->argc == 3)
-               target = Client_Search(Req->argv[2]);
-       else
-               target = Client_ThisServer();
+               return IRC_WriteErrClient(Client, ERR_NONICKNAMEGIVEN_MSG,
+                                         Client_ID(Client));
 
-       /* Get prefix */
-       if (Client_Type(Client) == CLIENT_SERVER)
-               prefix = Client_Search(Req->prefix);
-       else
-               prefix = Client;
+       _IRC_ARGC_LE_OR_RETURN_(Client, Req, 3)
+       _IRC_GET_SENDER_OR_RETURN_(prefix, Req, Client)
+       _IRC_GET_TARGET_SERVER_OR_RETURN_(target, Req, 2, prefix)
 
-       if (!prefix)
-               return IRC_WriteStrClient(Client, ERR_NOSUCHNICK_MSG,
-                                               Client_ID(Client), Req->prefix);
+       /* Do not reveal any info on disconnected users? */
+       if (Conf_MorePrivacy)
+               return CONNECTED;
 
-       /* Forward to other server? */
+       /* Forward? */
        if (target != Client_ThisServer()) {
-               if (!target || (Client_Type(target) != CLIENT_SERVER))
-                       return IRC_WriteStrClient(prefix, ERR_NOSUCHSERVER_MSG,
-                                       Client_ID(prefix), Req->argv[2]);
-
-               /* Forward */
-               IRC_WriteStrClientPrefix( target, prefix, "WHOWAS %s %s %s",
-                                         Req->argv[0], Req->argv[1],
-                                         Req->argv[2] );
+               IRC_WriteStrClientPrefix(target, prefix, "WHOWAS %s %s %s",
+                                        Req->argv[0], Req->argv[1],
+                                        Client_ID(target));
                return CONNECTED;
        }
 
@@ -1403,14 +1353,14 @@ IRC_WHOWAS( CLIENT *Client, REQUEST *Req )
                                break;
                } while (i != last);
 
-               if (nc == 0 && !IRC_WriteStrClient(prefix, ERR_WASNOSUCHNICK_MSG,
+               if (nc == 0 && !IRC_WriteErrClient(prefix, ERR_WASNOSUCHNICK_MSG,
                                                Client_ID(prefix), nick))
                        return DISCONNECTED;
        }
-       return IRC_WriteStrClient(prefix, RPL_ENDOFWHOWAS_MSG, Client_ID(prefix), Req->argv[0]);
+       return IRC_WriteStrClient(prefix, RPL_ENDOFWHOWAS_MSG,
+                                 Client_ID(prefix), Req->argv[0]);
 } /* IRC_WHOWAS */
 
-
 /**
  * Send LUSERS reply to a client.
  *
@@ -1427,7 +1377,7 @@ IRC_Send_LUSERS(CLIENT *Client)
 
        assert(Client != NULL);
 
-       /* Users, services and serevers in the network */
+       /* Users, services and servers in the network */
        if (!IRC_WriteStrClient(Client, RPL_LUSERCLIENT_MSG, Client_ID(Client),
                                Client_UserCount(), Client_ServiceCount(),
                                Client_ServerCount()))
@@ -1483,54 +1433,6 @@ IRC_Send_LUSERS(CLIENT *Client)
        return CONNECTED;
 } /* IRC_Send_LUSERS */
 
-
-static bool
-Show_MOTD_Start(CLIENT *Client)
-{
-       return IRC_WriteStrClient(Client, RPL_MOTDSTART_MSG,
-               Client_ID( Client ), Client_ID( Client_ThisServer( )));
-}
-
-static bool
-Show_MOTD_Sendline(CLIENT *Client, const char *msg)
-{
-       return IRC_WriteStrClient(Client, RPL_MOTD_MSG, Client_ID( Client ), msg);
-}
-
-static bool
-Show_MOTD_End(CLIENT *Client)
-{
-       if (!IRC_WriteStrClient(Client, RPL_ENDOFMOTD_MSG, Client_ID(Client)))
-               return DISCONNECTED;
-
-       if (*Conf_CloakHost)
-               return IRC_WriteStrClient(Client, RPL_HOSTHIDDEN_MSG,
-                                         Client_ID(Client),
-                                         Client_Hostname(Client));
-
-       return CONNECTED;
-}
-
-#ifdef SSL_SUPPORT
-static bool Show_MOTD_SSLInfo(CLIENT *Client)
-{
-       bool ret = true;
-       char buf[COMMAND_LEN] = "Connected using Cipher ";
-
-       if (!Conn_GetCipherInfo(Client_Conn(Client), buf + 23, sizeof buf - 23))
-               return true;
-
-       if (!Show_MOTD_Sendline(Client, buf))
-               ret = false;
-
-       return ret;
-}
-#else
-static inline bool
-Show_MOTD_SSLInfo(UNUSED CLIENT *c)
-{ return true; }
-#endif
-
 GLOBAL bool
 IRC_Show_MOTD( CLIENT *Client )
 {
@@ -1541,9 +1443,10 @@ IRC_Show_MOTD( CLIENT *Client )
 
        len_tot = array_bytes(&Conf_Motd);
        if (len_tot == 0 && !Conn_UsesSSL(Client_Conn(Client)))
-               return IRC_WriteStrClient(Client, ERR_NOMOTD_MSG, Client_ID(Client));
+               return IRC_WriteErrClient(Client, ERR_NOMOTD_MSG, Client_ID(Client));
 
-       if (!Show_MOTD_Start(Client))
+       if (!IRC_WriteStrClient(Client, RPL_MOTDSTART_MSG, Client_ID(Client),
+                               Client_ID(Client_ThisServer())))
                return DISCONNECTED;
 
        line = array_start(&Conf_Motd);
@@ -1553,16 +1456,24 @@ IRC_Show_MOTD( CLIENT *Client )
                assert(len_tot >= len_str);
                len_tot -= len_str;
 
-               if (!Show_MOTD_Sendline(Client, line))
+               if (!IRC_WriteStrClient(Client, RPL_MOTD_MSG, Client_ID(Client), line))
                        return DISCONNECTED;
                line += len_str;
        }
 
        if (!Show_MOTD_SSLInfo(Client))
                return DISCONNECTED;
-       return Show_MOTD_End(Client);
-} /* IRC_Show_MOTD */
 
+       if (!IRC_WriteStrClient(Client, RPL_ENDOFMOTD_MSG, Client_ID(Client)))
+               return DISCONNECTED;
+
+       if (*Conf_CloakHost)
+               return IRC_WriteStrClient(Client, RPL_HOSTHIDDEN_MSG,
+                                         Client_ID(Client),
+                                         Client_Hostname(Client));
+
+       return CONNECTED;
+} /* IRC_Show_MOTD */
 
 /**
  * Send NAMES reply for a specific client and channel.
@@ -1575,7 +1486,7 @@ GLOBAL bool
 IRC_Send_NAMES(CLIENT * Client, CHANNEL * Chan)
 {
        bool is_visible, is_member;
-       char str[LINE_LEN + 1];
+       char str[COMMAND_LEN];
        CL2CHAN *cl2chan;
        CLIENT *cl;
 
@@ -1592,7 +1503,7 @@ IRC_Send_NAMES(CLIENT * Client, CHANNEL * Chan)
                return CONNECTED;
 
        /* Secret channel? */
-       if (!is_member && strchr(Channel_Modes(Chan), 's'))
+       if (!is_member && Channel_HasMode(Chan, 's'))
                return CONNECTED;
 
        snprintf(str, sizeof(str), RPL_NAMREPLY_MSG, Client_ID(Client), "=",
@@ -1601,7 +1512,7 @@ IRC_Send_NAMES(CLIENT * Client, CHANNEL * Chan)
        while (cl2chan) {
                cl = Channel_GetClient(cl2chan);
 
-               if (strchr(Client_Modes(cl), 'i'))
+               if (Client_HasMode(cl, 'i'))
                        is_visible = false;
                else
                        is_visible = true;
@@ -1610,10 +1521,11 @@ IRC_Send_NAMES(CLIENT * Client, CHANNEL * Chan)
                        if (str[strlen(str) - 1] != ':')
                                strlcat(str, " ", sizeof(str));
 
-                       who_flags_qualifier(cl, Channel_UserModes(Chan, cl), str, sizeof(str));
+                       who_flags_qualifier(Client, Channel_UserModes(Chan, cl),
+                                           str, sizeof(str));
                        strlcat(str, Client_ID(cl), sizeof(str));
 
-                       if (strlen(str) > (LINE_LEN - CLIENT_NICK_LEN - 4)) {
+                       if (strlen(str) > (COMMAND_LEN - CLIENT_NICK_LEN - 4)) {
                                if (!IRC_WriteStrClient(Client, "%s", str))
                                        return DISCONNECTED;
                                snprintf(str, sizeof(str), RPL_NAMREPLY_MSG,
@@ -1632,7 +1544,6 @@ IRC_Send_NAMES(CLIENT * Client, CHANNEL * Chan)
        return CONNECTED;
 } /* IRC_Send_NAMES */
 
-
 /**
  * Send the ISUPPORT numeric (005).
  * This numeric indicates the features that are supported by this server.
@@ -1641,8 +1552,12 @@ IRC_Send_NAMES(CLIENT * Client, CHANNEL * Chan)
 GLOBAL bool
 IRC_Send_ISUPPORT(CLIENT * Client)
 {
+       if (Conf_Network[0] && !IRC_WriteStrClient(Client, RPL_ISUPPORTNET_MSG,
+                                                  Client_ID(Client),
+                                                  Conf_Network))
+               return DISCONNECTED;
        if (!IRC_WriteStrClient(Client, RPL_ISUPPORT1_MSG, Client_ID(Client),
-                               Conf_MaxJoins))
+                               CHANTYPES, CHANTYPES, Conf_MaxJoins))
                return DISCONNECTED;
        return IRC_WriteStrClient(Client, RPL_ISUPPORT2_MSG, Client_ID(Client),
                                  CHANNEL_NAME_LEN - 1, Conf_MaxNickLength - 1,
@@ -1651,5 +1566,4 @@ IRC_Send_ISUPPORT(CLIENT * Client)
                                  MAX_HNDL_CHANNEL_LISTS);
 } /* IRC_Send_ISUPPORT */
 
-
 /* -eof- */