Don't allow empty channel names ("#") in strict RFC mode.

[ngircd-alex.git] / src / ngircd / channel.c

diff --git a/src/ngircd/channel.c b/src/ngircd/channel.c
index 5caf7d0ce267fa68598cbca60c67414b29b15f03..aec6aa2da9e9a6b686535040e46432099c424fd0 100644 (file)
--- a/src/ngircd/channel.c
+++ b/src/ngircd/channel.c
@@ -183,30 +183,34 @@ Channel_Join( CLIENT *Client, char *Name )
 {
        CHANNEL *chan;
 
-       assert( Client != NULL );
-       assert( Name != NULL );
+       assert(Client != NULL);
+       assert(Name != NULL);
 
        /* Check that the channel name is valid */
-       if( ! Channel_IsValidName( Name )) {
-               IRC_WriteStrClient( Client, ERR_NOSUCHCHANNEL_MSG, Client_ID( Client ), Name );
+       if (! Channel_IsValidName(Name)) {
+               IRC_WriteStrClient(Client, ERR_NOSUCHCHANNEL_MSG,
+                                  Client_ID(Client), Name);
                return false;
        }
 
-       chan = Channel_Search( Name );
-       if( chan ) {
+       chan = Channel_Search(Name);
+       if(chan) {
                /* Check if the client is already in the channel */
-               if( Get_Cl2Chan( chan, Client )) return false;
-       }
-       else
-       {
-               /* If the specified channel doesn't exist, the channel is created */
-               chan = Channel_Create( Name );
-               if (!chan) return false;
+               if (Get_Cl2Chan(chan, Client))
+                       return false;
+       } else {
+               /* If the specified channel does not exist, the channel
+                * is now created */
+               chan = Channel_Create(Name);
+               if (!chan)
+                       return false;
        }
 
        /* Add user to Channel */
-       if( ! Add_Client( chan, Client )) return false;
-       else return true;
+       if (! Add_Client(chan, Client))
+               return false;
+
+       return true;
 } /* Channel_Join */
 
 
@@ -419,7 +423,7 @@ Channel_Next( CHANNEL *Chan )
 GLOBAL CHANNEL *
 Channel_Search( const char *Name )
 {
-       /* Channel-Struktur suchen */
+       /* Search channel structure */
 
        CHANNEL *c;
        UINT32 search_hash;
@@ -432,7 +436,7 @@ Channel_Search( const char *Name )
        {
                if( search_hash == c->hash )
                {
-                       /* lt. Hash-Wert: Treffer! */
+                       /* hash hit */
                        if( strcasecmp( Name, c->name ) == 0 ) return c;
                }
                c = c->next;
@@ -496,6 +500,10 @@ Channel_IsValidName( const char *Name )
 {
        assert( Name != NULL );
 
+#ifdef STRICT_RFC
+       if (strlen(Name) <= 1)
+               return false;
+#endif
        if (strchr("+#", Name[0]) == NULL)
                return false;
        if (strlen(Name) >= CHANNEL_NAME_LEN)
@@ -839,7 +847,7 @@ Add_Client( CHANNEL *Chan, CLIENT *Client )
        assert( Chan != NULL );
        assert( Client != NULL );
 
-       /* neue CL2CHAN-Struktur anlegen */
+       /* Create new CL2CHAN structure */
        cl2chan = (CL2CHAN *)malloc( sizeof( CL2CHAN ));
        if( ! cl2chan )
        {
@@ -850,7 +858,7 @@ Add_Client( CHANNEL *Chan, CLIENT *Client )
        cl2chan->client = Client;
        strcpy( cl2chan->modes, "" );
 
-       /* Verketten */
+       /* concatenate */
        cl2chan->next = My_Cl2Chan;
        My_Cl2Chan = cl2chan;
 
@@ -884,7 +892,7 @@ Remove_Client( int Type, CHANNEL *Chan, CLIENT *Client, CLIENT *Origin, const ch
        c = cl2chan->channel;
        assert( c != NULL );
 
-       /* Aus Verkettung loesen und freigeben */
+       /* maintain cl2chan list */
        if( last_cl2chan ) last_cl2chan->next = cl2chan->next;
        else My_Cl2Chan = cl2chan->next;
        free( cl2chan );
@@ -930,7 +938,7 @@ Remove_Client( int Type, CHANNEL *Chan, CLIENT *Client, CLIENT *Origin, const ch
                        }
        }
 
-       /* Wenn Channel nun leer und nicht pre-defined: loeschen */
+       /* When channel is empty and is not pre-defined, delete */
        if( ! strchr( Channel_Modes( Chan ), 'P' ))
        {
                if( ! Get_First_Cl2Chan( NULL, Chan )) Delete_Channel( Chan );
@@ -1030,7 +1038,7 @@ Get_Next_Cl2Chan( CL2CHAN *Start, CLIENT *Client, CHANNEL *Channel )
 static bool
 Delete_Channel( CHANNEL *Chan )
 {
-       /* Channel-Struktur loeschen */
+       /* delete channel structure */
 
        CHANNEL *chan, *last_chan;
 
@@ -1046,11 +1054,11 @@ Delete_Channel( CHANNEL *Chan )
 
        Log( LOG_DEBUG, "Freed channel structure for \"%s\".", Chan->name );
 
-       /* Invite- und Ban-Lists aufraeumen */
+       /* free invite and ban lists */
        Lists_Free( &chan->list_bans );
        Lists_Free( &chan->list_invites );
 
-       /* Neu verketten und freigeben */
+       /* maintain channel list */
        if( last_chan ) last_chan->next = chan->next;
        else My_Channels = chan->next;
        free( chan );