]> arthur.barton.de Git - ngircd-alex.git/blobdiff - src/ngircd/parse.c
Introduce option to configure the maximum nick name lenth in ngircd.conf
[ngircd-alex.git] / src / ngircd / parse.c
index a2adf07b4ec7054844ba50f78f5af6dc47b85024..5109169b72a068b8ff4655adda8bcb6637c093b1 100644 (file)
 /*
  * ngIRCd -- The Next Generation IRC Daemon
- * Copyright (c)2001 by Alexander Barton (alex@barton.de)
+ * Copyright (c)2001,2002 by Alexander Barton (alex@barton.de)
  *
- * Dieses Programm ist freie Software. Sie koennen es unter den Bedingungen
- * der GNU General Public License (GPL), wie von der Free Software Foundation
- * herausgegeben, weitergeben und/oder modifizieren, entweder unter Version 2
- * der Lizenz oder (wenn Sie es wuenschen) jeder spaeteren Version.
- * Naehere Informationen entnehmen Sie bitter der Datei COPYING. Eine Liste
- * der an comBase beteiligten Autoren finden Sie in der Datei AUTHORS.
- *
- * $Id: parse.c,v 1.8 2001/12/29 03:08:19 alex Exp $
- *
- * parse.c: Parsen der Client-Anfragen
- *
- * $Log: parse.c,v $
- * Revision 1.8  2001/12/29 03:08:19  alex
- * - Fuehrende und folgende Leerzeichen etc. in Requests werden geloescht.
- * - Logmeldungen (mal wieder) ein wenig angepasst.
- *
- * Revision 1.7  2001/12/27 19:13:21  alex
- * - neue Befehle NOTICE und PRIVMSG.
- * - Debug-Logging ein wenig reduziert.
- *
- * Revision 1.6  2001/12/26 14:45:37  alex
- * - "Code Cleanups".
- *
- * Revision 1.5  2001/12/26 03:23:03  alex
- * - PING/PONG-Befehle implementiert.
- *
- * Revision 1.4  2001/12/25 22:04:26  alex
- * - Aenderungen an den Debug- und Logging-Funktionen.
- *
- * Revision 1.3  2001/12/25 19:18:36  alex
- * - Gross- und Kleinschreibung der IRC-Befehle wird ignoriert.
- * - bessere Debug-Ausgaben.
- *
- * Revision 1.2  2001/12/23 21:56:47  alex
- * - bessere Debug-Ausgaben,
- * - Bug im Parameter-Parser behoben (bei "langem" Parameter)
- * - erste IRC-Befehle werden erkannt :-)
- *
- * Revision 1.1  2001/12/21 23:53:16  alex
- * - Modul zum Parsen von Client-Requests begonnen.
+ * 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
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ * Please read the file COPYING, README and AUTHORS for more information.
  */
 
 
-#include <portab.h>
-#include "global.h"
+#include "portab.h"
+
+static char UNUSED id[] = "$Id: parse.c,v 1.69 2007/11/21 12:16:36 alex Exp $";
+
+/**
+ * @file
+ * IRC command parser and validator.
+ */
 
-#include <imp.h>
+#include "imp.h"
 #include <assert.h>
+#include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
+#include <strings.h>
 
+#include "ngircd.h"
+#include "defines.h"
+#include "conn-func.h"
 #include "client.h"
-#include "conn.h"
-#include "irc.h"
+#include "channel.h"
 #include "log.h"
 #include "messages.h"
 #include "tool.h"
 
-#include <exp.h>
+#include "exp.h"
 #include "parse.h"
 
+#include "imp.h"
+#include "irc.h"
+#include "irc-channel.h"
+#include "irc-info.h"
+#include "irc-login.h"
+#include "irc-mode.h"
+#include "irc-op.h"
+#include "irc-oper.h"
+#include "irc-server.h"
+#include "irc-write.h"
+#include "numeric.h"
 
-LOCAL VOID Init_Request( REQUEST *Req );
+#include "exp.h"
 
-LOCAL BOOLEAN Parse_Error( CONN_ID Idx, CHAR *Error );
 
-LOCAL BOOLEAN Validate_Prefix( REQUEST *Req );
-LOCAL BOOLEAN Validate_Command( REQUEST *Req );
-LOCAL BOOLEAN Validate_Args( REQUEST *Req );
+COMMAND My_Commands[] =
+{
+       { "ADMIN", IRC_ADMIN, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "AWAY", IRC_AWAY, CLIENT_USER, 0, 0, 0 },
+       { "CONNECT", IRC_CONNECT, CLIENT_USER, 0, 0, 0 },
+       { "DIE", IRC_DIE, CLIENT_USER, 0, 0, 0 },
+       { "DISCONNECT", IRC_DISCONNECT, CLIENT_USER, 0, 0, 0 },
+       { "ERROR", IRC_ERROR, 0xFFFF, 0, 0, 0 },
+       { "HELP", IRC_HELP, CLIENT_USER, 0, 0, 0 },
+       { "INVITE", IRC_INVITE, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "ISON", IRC_ISON, CLIENT_USER, 0, 0, 0 },
+       { "JOIN", IRC_JOIN, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "KICK", IRC_KICK, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "KILL", IRC_KILL, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "LINKS", IRC_LINKS, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "LIST", IRC_LIST, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "LUSERS", IRC_LUSERS, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "MODE", IRC_MODE, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "MOTD", IRC_MOTD, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "NAMES", IRC_NAMES, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "NICK", IRC_NICK, 0xFFFF, 0, 0, 0 },
+       { "NJOIN", IRC_NJOIN, CLIENT_SERVER, 0, 0, 0 },
+       { "NOTICE", IRC_NOTICE, 0xFFFF, 0, 0, 0 },
+       { "OPER", IRC_OPER, CLIENT_USER, 0, 0, 0 },
+       { "PART", IRC_PART, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "PASS", IRC_PASS, 0xFFFF, 0, 0, 0 },
+       { "PING", IRC_PING, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "PONG", IRC_PONG, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "PRIVMSG", IRC_PRIVMSG, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "QUIT", IRC_QUIT, 0xFFFF, 0, 0, 0 },
+       { "REHASH", IRC_REHASH, CLIENT_USER, 0, 0, 0 },
+       { "RESTART", IRC_RESTART, CLIENT_USER, 0, 0, 0 },
+       { "SERVER", IRC_SERVER, 0xFFFF, 0, 0, 0 },
+       { "SQUIT", IRC_SQUIT, CLIENT_SERVER, 0, 0, 0 },
+       { "STATS", IRC_STATS, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "TIME", IRC_TIME, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "TOPIC", IRC_TOPIC, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "TRACE", IRC_TRACE, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "USER", IRC_USER, 0xFFFF, 0, 0, 0 },
+       { "USERHOST", IRC_USERHOST, CLIENT_USER, 0, 0, 0 },
+       { "VERSION", IRC_VERSION, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "WALLOPS", IRC_WALLOPS, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "WHO", IRC_WHO, CLIENT_USER, 0, 0, 0 },
+       { "WHOIS", IRC_WHOIS, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+       { "WHOWAS", IRC_WHOWAS, CLIENT_USER|CLIENT_SERVER, 0, 0, 0 },
+#ifdef IRCPLUS
+       { "CHANINFO", IRC_CHANINFO, CLIENT_SERVER, 0, 0, 0 },
+#endif
+       { NULL, NULL, 0x0, 0, 0, 0 } /* Ende-Marke */
+};
 
-LOCAL BOOLEAN Handle_Request( CONN_ID Idx, REQUEST *Req );
+NUMERIC My_Numerics[] =
+{
+       { 005, IRC_Num_ISUPPORT },
+       { 376, IRC_Num_ENDOFMOTD },
+       { 0, NULL } /* end marker */
+};
 
 
-GLOBAL VOID Parse_Init( VOID )
-{
-} /* Parse_Init */
+static void Init_Request PARAMS(( REQUEST *Req ));
 
+static bool Validate_Prefix PARAMS(( CONN_ID Idx, REQUEST *Req, bool *Closed ));
+static bool Validate_Command PARAMS(( CONN_ID Idx, REQUEST *Req, bool *Closed ));
+static bool Validate_Args PARAMS(( CONN_ID Idx, REQUEST *Req, bool *Closed ));
 
-GLOBAL VOID Parse_Exit( VOID )
-{
-} /* Parse_Exit */
+static bool Handle_Request PARAMS(( CONN_ID Idx, REQUEST *Req ));
 
 
-GLOBAL BOOLEAN Parse_Request( CONN_ID Idx, CHAR *Request )
+/**
+ * Return the pointer to the global "IRC command structure".
+ * This structure, an array of type "COMMAND" describes all the IRC commands
+ * implemented by ngIRCd and how to handle them.
+ * @return Pointer to the global command structure.
+ */
+GLOBAL COMMAND *
+Parse_GetCommandStruct( void )
 {
-       /* Client-Request parsen. Bei einem schwerwiegenden Fehler wird
-        * die Verbindung geschlossen und FALSE geliefert.
-        * Der Aufbau gueltiger Requests ist in RFC 2812, 2.3 definiert. */
+       return My_Commands;
+} /* Parse_GetCommandStruct */
+
 
+/**
+ * Parse a command ("request") received from a client.
+ * 
+ * This function is called after the connection layer received a valid CR+LF
+ * terminated line of text: we asume that this is a valid IRC command and
+ * try to do something useful with it :-)
+ *
+ * All errors are reported to the client from which the command has been
+ * received, and if the error is fatal this connection is closed down.
+ *
+ * This function is able to parse the syntax as described in RFC 2812,
+ * section 2.3.
+ *
+ * @param Idx Index of the connection from which the command has been received.
+ * @param Request NULL terminated line of text (the "command").
+ * @return true on success (valid command or "regular" error), false if a
+ *     fatal error occured and the connection has been shut down.
+ */
+GLOBAL bool
+Parse_Request( CONN_ID Idx, char *Request )
+{
        REQUEST req;
-       CHAR *start, *ptr;
+       char *start, *ptr;
+       bool closed;
 
        assert( Idx >= 0 );
        assert( Request != NULL );
 
 #ifdef SNIFFER
-       Log( LOG_DEBUG, " <- connection %d: '%s'.", Idx, Request );
+       if( NGIRCd_Sniffer ) Log( LOG_DEBUG, " <- connection %d: '%s'.", Idx, Request );
 #endif
-       
+
        Init_Request( &req );
 
        /* Fuehrendes und folgendes "Geraffel" verwerfen */
@@ -112,21 +177,34 @@ GLOBAL BOOLEAN Parse_Request( CONN_ID Idx, CHAR *Request )
                /* Prefix vorhanden */
                req.prefix = Request + 1;
                ptr = strchr( Request, ' ' );
-               if( ! ptr ) return Parse_Error( Idx, "Invalid prefix (command missing!?)" );
+               if( ! ptr )
+               {
+                       Log( LOG_DEBUG, "Connection %d: Parse error: prefix without command!?", Idx );
+                       return Conn_WriteStr( Idx, "ERROR :Prefix without command!?" );
+               }
                *ptr = '\0';
+#ifndef STRICT_RFC
+               /* multiple Leerzeichen als Trenner zwischen
+                * Prefix und Befehl ignorieren */
+               while( *(ptr + 1) == ' ' ) ptr++;
+#endif
                start = ptr + 1;
        }
        else start = Request;
 
-       if( ! Validate_Prefix( &req )) return Parse_Error( Idx, "Invalid prefix");
-
        /* Befehl */
        ptr = strchr( start, ' ' );
-       if( ptr ) *ptr = '\0';
+       if( ptr )
+       {
+               *ptr = '\0';
+#ifndef STRICT_RFC
+               /* multiple Leerzeichen als Trenner vor
+                * Parametern ignorieren */
+               while( *(ptr + 1) == ' ' ) ptr++;
+#endif
+       }
        req.command = start;
 
-       if( ! Validate_Command( &req )) return Parse_Error( Idx, "Invalid command" );
-
        /* Argumente, Parameter */
        if( ptr )
        {
@@ -144,31 +222,47 @@ GLOBAL BOOLEAN Parse_Request( CONN_ID Idx, CHAR *Request )
                        {
                                req.argv[req.argc] = start;
                                ptr = strchr( start, ' ' );
-                               if( ptr ) *ptr = '\0';
+                               if( ptr )
+                               {
+                                       *ptr = '\0';
+#ifndef STRICT_RFC
+                                       /* multiple Leerzeichen als
+                                        * Parametertrenner ignorieren */
+                                       while( *(ptr + 1) == ' ' ) ptr++;
+#endif
+                               }
                        }
-                       
+
                        req.argc++;
 
                        if( start[0] == ':' ) break;
                        if( req.argc > 14 ) break;
-                       
+
                        if( ptr ) start = ptr + 1;
                        else start = NULL;
                }
        }
-       
-       if( ! Validate_Args( &req )) return Parse_Error( Idx, "Invalid argument(s)" );
-       
+
+       /* Daten validieren */
+       if( ! Validate_Prefix( Idx, &req, &closed )) return ! closed;
+       if( ! Validate_Command( Idx, &req, &closed )) return ! closed;
+       if( ! Validate_Args( Idx, &req, &closed )) return ! closed;
+
        return Handle_Request( Idx, &req );
 } /* Parse_Request */
 
 
-LOCAL VOID Init_Request( REQUEST *Req )
+/**
+ * Initialize request structure.
+ * @param Req Request structure to be initialized.
+ */
+static void
+Init_Request( REQUEST *Req )
 {
        /* Neue Request-Struktur initialisieren */
 
-       INT i;
-       
+       int i;
+
        assert( Req != NULL );
 
        Req->prefix = NULL;
@@ -178,73 +272,217 @@ LOCAL VOID Init_Request( REQUEST *Req )
 } /* Init_Request */
 
 
-LOCAL BOOLEAN Parse_Error( CONN_ID Idx, CHAR *Error )
+static bool
+Validate_Prefix( CONN_ID Idx, REQUEST *Req, bool *Closed )
 {
-       /* Fehler beim Parsen. Fehlermeldung an den Client schicken.
-        * TRUE: Connection wurde durch diese Funktion nicht geschlossen,
-        * FALSE: Connection wurde terminiert. */
-       
-       CHAR msg[256];
-       
+       CLIENT *client, *c;
+
        assert( Idx >= 0 );
-       assert( Error != NULL );
+       assert( Req != NULL );
 
-       sprintf( msg, "Parse error: %s!", Error );
-       return Conn_WriteStr( Idx, msg );
-} /* Parse_Error */
+       *Closed = false;
 
+       /* ist ueberhaupt ein Prefix vorhanden? */
+       if( ! Req->prefix ) return true;
 
-LOCAL BOOLEAN Validate_Prefix( REQUEST *Req )
-{
-       assert( Req != NULL );
-       return TRUE;
+       /* Client-Struktur der Connection ermitteln */
+       client = Conn_GetClient( Idx );
+       assert( client != NULL );
+
+       /* nur validieren, wenn bereits registrierte Verbindung */
+       if(( Client_Type( client ) != CLIENT_USER ) && ( Client_Type( client ) != CLIENT_SERVER ) && ( Client_Type( client ) != CLIENT_SERVICE ))
+       {
+               /* noch nicht registrierte Verbindung.
+                * Das Prefix wird ignoriert. */
+               Req->prefix = NULL;
+               return true;
+       }
+
+       /* pruefen, ob der im Prefix angegebene Client bekannt ist */
+       c = Client_Search( Req->prefix );
+       if( ! c )
+       {
+               /* im Prefix angegebener Client ist nicht bekannt */
+               Log( LOG_ERR, "Invalid prefix \"%s\", client not known (connection %d, command %s)!?", Req->prefix, Idx, Req->command );
+               if( ! Conn_WriteStr( Idx, "ERROR :Invalid prefix \"%s\", client not known!?", Req->prefix )) *Closed = true;
+               return false;
+       }
+
+       /* pruefen, ob der Client mit dem angegebenen Prefix in Richtung
+        * des Senders liegt, d.h. sicherstellen, dass das Prefix nicht
+        * gefaelscht ist */
+       if( Client_NextHop( c ) != client )
+       {
+               /* das angegebene Prefix ist aus dieser Richtung, also
+                * aus der gegebenen Connection, ungueltig! */
+               Log( LOG_ERR, "Spoofed prefix \"%s\" from \"%s\" (connection %d, command %s)!", Req->prefix, Client_Mask( Conn_GetClient( Idx )), Idx, Req->command );
+               Conn_Close( Idx, NULL, "Spoofed prefix", true);
+               *Closed = true;
+               return false;
+       }
+
+       return true;
 } /* Validate_Prefix */
 
 
-LOCAL BOOLEAN Validate_Command( REQUEST *Req )
+static bool
+Validate_Command( UNUSED CONN_ID Idx, UNUSED REQUEST *Req, bool *Closed )
 {
+       assert( Idx >= 0 );
        assert( Req != NULL );
-       return TRUE;
+       *Closed = false;
+
+       return true;
 } /* Validate_Comman */
 
 
-LOCAL BOOLEAN Validate_Args( REQUEST *Req )
+static bool
+Validate_Args( UNUSED CONN_ID Idx, UNUSED REQUEST *Req, bool *Closed )
 {
+       assert( Idx >= 0 );
        assert( Req != NULL );
-       return TRUE;
+       *Closed = false;
+
+       return true;
 } /* Validate_Args */
 
 
-LOCAL BOOLEAN Handle_Request( CONN_ID Idx, REQUEST *Req )
+static bool
+Handle_Request( CONN_ID Idx, REQUEST *Req )
 {
        /* Client-Request verarbeiten. Bei einem schwerwiegenden Fehler
-        * wird die Verbindung geschlossen und FALSE geliefert. */
+        * wird die Verbindung geschlossen und false geliefert. */
 
-       CLIENT *client;
+       CLIENT *client, *target, *prefix;
+       char str[LINE_LEN];
+       bool result;
+       COMMAND *cmd;
+       NUMERIC *num;
+       int i;
 
        assert( Idx >= 0 );
        assert( Req != NULL );
        assert( Req->command != NULL );
 
-       client = Client_GetFromConn( Idx );
+       client = Conn_GetClient( Idx );
        assert( client != NULL );
 
-       if( strcasecmp( Req->command, "PASS" ) == 0 ) return IRC_PASS( client, Req );
-       else if( strcasecmp( Req->command, "NICK" ) == 0 ) return IRC_NICK( client, Req );
-       else if( strcasecmp( Req->command, "USER" ) == 0 ) return IRC_USER( client, Req );
-       else if( strcasecmp( Req->command, "QUIT" ) == 0 ) return IRC_QUIT( client, Req );
-       else if( strcasecmp( Req->command, "PING" ) == 0 ) return IRC_PING( client, Req );
-       else if( strcasecmp( Req->command, "PONG" ) == 0 ) return IRC_PONG( client, Req );
-       else if( strcasecmp( Req->command, "MOTD" ) == 0 ) return IRC_MOTD( client, Req );
-       else if( strcasecmp( Req->command, "PRIVMSG" ) == 0 ) return IRC_PRIVMSG( client, Req );
-       else if( strcasecmp( Req->command, "NOTICE" ) == 0 ) return IRC_NOTICE( client, Req );
-       else if( strcasecmp( Req->command, "MODE" ) == 0 ) return IRC_MODE( client, Req );
+       /* Numeric? */
+       if ((Client_Type(client) == CLIENT_SERVER ||
+            Client_Type(client) == CLIENT_UNKNOWNSERVER)
+           && strlen(Req->command) == 3 && atoi(Req->command) > 1) {
+               /* Command is a status code ("numeric") from an other server */
+
+               /* Determine target */
+               if (Req->argc > 0)
+                       target = Client_Search( Req->argv[0] );
+               else
+                       target = NULL;
+               if (!target) {
+                       /* Status code without target!? */
+                       if (Req->argc > 0)
+                               Log(LOG_WARNING,
+                                   "Unknown target for status code %s: \"%s\"",
+                                   Req->command, Req->argv[0]);
+                       else
+                               Log(LOG_WARNING,
+                                   "Unknown target for status code %s!",
+                                   Req->command);
+                       return true;
+               }
+               if (target == Client_ThisServer()) {
+                       /* This server is the target of the numeric */
+                       i = atoi(Req->command);
+
+                       num = My_Numerics;
+                       while (num->numeric > 0) {
+                               if (i != num->numeric) {
+                                       num++;
+                                       continue;
+                               }
+                               result = (num->function)(client, Req);
+                               return result;
+                       }
+                       
+                       LogDebug("Ignored status code %s from \"%s\".",
+                                Req->command, Client_ID(client));
+                       return true;
+               }
+
+               /* Determine source */
+               if( ! Req->prefix[0] )
+               {
+                       /* Oops, no prefix!? */
+                       Log( LOG_WARNING, "Got status code %s from \"%s\" without prefix!?", Req->command, Client_ID( client ));
+                       return true;
+               }
+               else prefix = Client_Search( Req->prefix );
+               if( ! prefix )
+               {
+                       /* Oops, unknown prefix!? */
+                       Log( LOG_WARNING, "Got status code %s from unknown source: \"%s\"", Req->command, Req->prefix );
+                       return true;
+               }
+
+               /* Forward status code */
+               strlcpy( str, Req->command, sizeof( str ));
+               for( i = 0; i < Req->argc; i++ )
+               {
+                       if( i < Req->argc - 1 ) strlcat( str, " ", sizeof( str ));
+                       else strlcat( str, " :", sizeof( str ));
+                       strlcat( str, Req->argv[i], sizeof( str ));
+               }
+               return IRC_WriteStrClientPrefix( target, prefix, "%s", str );
+       }
+
+       cmd = My_Commands;
+       while( cmd->name )
+       {
+               /* Befehl suchen */
+               if( strcasecmp( Req->command, cmd->name ) != 0 )
+               {
+                       cmd++; continue;
+               }
+
+               if( Client_Type( client ) & cmd->type )
+               {
+                       /* Command is allowed for this client: call it and count produced bytes */
+                       Conn_ResetWCounter( );
+                       result = (cmd->function)( client, Req );
+                       cmd->bytes += Conn_WCounter( );
+
+                       /* Adjust counters */
+                       if( Client_Type( client ) != CLIENT_SERVER ) cmd->lcount++;
+                       else cmd->rcount++;
+
+                       return result;
+               }
+               else
+               {
+                       /* Befehl ist fuer diesen Client-Typ nicht erlaubt! */
+                       return IRC_WriteStrClient( client, ERR_NOTREGISTERED_MSG, Client_ID( client ));
+               }
+       }
+
+       if( Client_Type( client ) != CLIENT_USER &&
+           Client_Type( client ) != CLIENT_SERVER &&
+           Client_Type( client ) != CLIENT_SERVICE )
+               return true;
        
-       /* Unbekannter Befehl */
-       IRC_WriteStrClient( client, This_Server, ERR_UNKNOWNCOMMAND_MSG, Client_Name( client ), Req->command );
-       Log( LOG_DEBUG, "User \"%s!%s@%s\": Unknown command \"%s\", %d %s,%s prefix.", client->nick, client->user, client->host, Req->command, Req->argc, Req->argc == 1 ? "parameter" : "parameters", Req->prefix ? "" : " no" );
+       /* Unknown command and registered connection: generate error: */
+       Log( LOG_DEBUG, "Connection %d: Unknown command \"%s\", %d %s,%s prefix.",
+                       Client_Conn( client ), Req->command, Req->argc,
+                       Req->argc == 1 ? "parameter" : "parameters",
+                       Req->prefix ? "" : " no" );
+
+       if (Client_Type(client) != CLIENT_SERVER) {
+               result = IRC_WriteStrClient(client, ERR_UNKNOWNCOMMAND_MSG,
+                               Client_ID(client), Req->command);
+               Conn_SetPenalty(Idx, 1);
+               return result;
+       }
 
-       return TRUE;
+       return true;
 } /* Handle_Request */