]> arthur.barton.de Git - ngircd-alex.git/blobdiff - src/ngircd/client.c
- Count_MyServers() zaehlt nun wieder richtig, LUSERS sollte nun wieder die
[ngircd-alex.git] / src / ngircd / client.c
index 51338021d462acf64adfd134d1515e4ea0384488..b3b6118172f6bdcaf3906c9b41890f179354740d 100644 (file)
@@ -9,7 +9,7 @@
  * Naehere Informationen entnehmen Sie bitter der Datei COPYING. Eine Liste
  * der an ngIRCd beteiligten Autoren finden Sie in der Datei AUTHORS.
  *
- * $Id: client.c,v 1.43 2002/03/04 01:04:46 alex Exp $
+ * $Id: client.c,v 1.54 2002/04/14 13:54:51 alex Exp $
  *
  * client.c: Management aller Clients
  *
  * Ueber welchen IRC-Server die Verbindung nun tatsaechlich in das Netzwerk her-
  * gestellt wurde, muss der jeweiligen Struktur entnommen werden. Ist es dieser
  * Server gewesen, so existiert eine entsprechende CONNECTION-Struktur.
- *
- * $Log: client.c,v $
- * Revision 1.43  2002/03/04 01:04:46  alex
- * - neuen Clients mit Mode "a" wird nun auch der Default-Away-Text gesetzt.
- *
- * Revision 1.42  2002/03/03 17:17:01  alex
- * - strncpy() und vsnprintf() kopieren nun etwas "optimierter" (1 Byte weniger) :-)
- *
- * Revision 1.41  2002/03/02 01:35:50  alex
- * - Channel- und Nicknames werden nun ordentlich validiert.
- *
- * Revision 1.40  2002/02/27 23:23:53  alex
- * - Includes fuer einige Header bereinigt.
- *
- * Revision 1.39  2002/02/27 18:22:09  alex
- * - neue Funktion Client_SetAway() und Client_Away() implementiert.
- *
- * Revision 1.38  2002/02/27 14:47:53  alex
- * - Logging beim Abmelden von Clients (erneut) geaendert: nun ist's aber gut ;-)
- *
- * Revision 1.37  2002/02/17 19:02:49  alex
- * - Client_CheckNick() und Client_CheckID() lieferten u.U. falsche Ergebnisse.
- *
- * Revision 1.36  2002/02/06 16:49:41  alex
- * - neue Funktion Client_IsValidNick(), Nicknames werden besser validiert.
- *
- * Revision 1.35  2002/01/29 00:14:49  alex
- * - neue Funktion Client_TopServer(), Client_NewXXX() angepasst.
- *
- * Revision 1.34  2002/01/27 22:07:36  alex
- * - Client_GetFromID() besser dokumentiert, kleinere Aenderungen.
- *
- * Revision 1.33  2002/01/27 21:56:54  alex
- * - weitere Anpassungen an Chennals, v.a. ueber Server-Links.
- *
- * Revision 1.32  2002/01/27 18:27:12  alex
- * - Client_GetFromID() kommt nun auch mit Host-Masken zurecht.
- *
- * Revision 1.31  2002/01/21 00:08:50  alex
- * - wird ein Client entfernt, so wird er auch aus allen Channels geloescht.
- *
- * Revision 1.30  2002/01/18 15:32:01  alex
- * - bei Client_SetModes() wurde das NULL-Byte falsch gesetzt. Opsa.
- *
- * Revision 1.29  2002/01/16 22:10:35  alex
- * - neue Funktionen Client_xxxCount().
- *
- * Revision 1.28  2002/01/11 23:50:40  alex
- * - Hop-Count fuer den Server selber (0) wird korrekt initialisiert.
- *
- * Revision 1.27  2002/01/09 01:08:08  alex
- * - wird ein Server abgemeldet, so wird anderen Server ein SQUIT geschickt.
- *
- * Revision 1.26  2002/01/07 23:42:12  alex
- * - Es werden fuer alle Server eigene Token generiert,
- * - QUIT von einem Server fuer einen User wird an andere Server geforwarded,
- * - ebenso NICK-Befehle, die "fremde" User einfuehren.
- *
- * Revision 1.25  2002/01/07 15:31:00  alex
- * - Bei Log-Meldungen ueber Clients wird nun immer die "Client Mask" verwendet.
- *
- * Revision 1.24  2002/01/06 15:18:14  alex
- * - Loglevel und Meldungen nochmals geaendert. Level passen nun besser.
- *
- * Revision 1.23  2002/01/05 23:26:05  alex
- * - Vorbereitungen fuer Ident-Abfragen in Client-Strukturen.
- *
- * Revision 1.22  2002/01/05 20:08:17  alex
- * - neue Funktion Client_NextHop().
- *
- * Revision 1.21  2002/01/05 19:15:03  alex
- * - Fehlerpruefung bei select() in der "Hauptschleife" korrigiert.
- *
- * Revision 1.20  2002/01/04 17:57:08  alex
- * - Client_Destroy() an Server-Links angepasst.
- *
- * Revision 1.19  2002/01/04 01:21:22  alex
- * - Client-Strukturen koennen von anderen Modulen nun nur noch ueber die
- *   enstprechenden (zum Teil neuen) Funktionen angesprochen werden.
- *
- * Revision 1.18  2002/01/03 02:28:06  alex
- * - neue Funktion Client_CheckID(), diverse Aenderungen fuer Server-Links.
- *
- * Revision 1.17  2002/01/02 02:42:58  alex
- * - Copyright-Texte aktualisiert.
- *
- * Revision 1.16  2002/01/01 18:25:44  alex
- * - #include's fuer stdlib.h ergaenzt.
- *
- * Revision 1.15  2001/12/31 15:33:13  alex
- * - neuer Befehl NAMES, kleinere Bugfixes.
- * - Bug bei PING behoben: war zu restriktiv implementiert :-)
- *
- * Revision 1.14  2001/12/31 02:18:51  alex
- * - viele neue Befehle (WHOIS, ISON, OPER, DIE, RESTART),
- * - neuen Header "defines.h" mit (fast) allen Konstanten.
- * - Code Cleanups und viele "kleine" Aenderungen & Bugfixes.
- *
- * Revision 1.13  2001/12/30 19:26:11  alex
- * - Unterstuetzung fuer die Konfigurationsdatei eingebaut.
- *
- * Revision 1.12  2001/12/29 20:18:18  alex
- * - neue Funktion Client_SetHostname().
- *
- * Revision 1.11  2001/12/29 03:10:47  alex
- * - Client-Modes implementiert; Loglevel mal wieder angepasst.
- *
- * Revision 1.10  2001/12/27 19:13:47  alex
- * - neue Funktion Client_Search(), besseres Logging.
- *
- * Revision 1.9  2001/12/27 17:15:29  alex
- * - der eigene Hostname wird nun komplet (als FQDN) ermittelt.
- *
- * Revision 1.8  2001/12/27 16:54:51  alex
- * - neue Funktion Client_GetID(), liefert die "Client ID".
- *
- * Revision 1.7  2001/12/26 14:45:37  alex
- * - "Code Cleanups".
- *
- * Revision 1.6  2001/12/26 03:19:16  alex
- * - neue Funktion Client_Nick().
- *
- * Revision 1.5  2001/12/25 22:04:26  alex
- * - Aenderungen an den Debug- und Logging-Funktionen.
- *
- * Revision 1.4  2001/12/25 19:21:26  alex
- * - Client-Typ ("Status") besser unterteilt, My_Clients ist zudem nun global.
- *
- * Revision 1.3  2001/12/24 01:31:14  alex
- * - einige assert()'s eingestraeut.
- *
- * Revision 1.2  2001/12/23 22:04:37  alex
- * - einige neue Funktionen,
- * - CLIENT-Struktur erweitert.
- *
- * Revision 1.1  2001/12/14 08:13:43  alex
- * - neues Modul begonnen :-)
  */
 
 
 #define __client_c__
 
 
-#include <portab.h>
-#include "global.h"
+#include "portab.h"
 
-#include <imp.h>
+#include "imp.h"
 #include <assert.h>
 #include <unistd.h>
 #include <stdio.h>
 #include <string.h>
 #include <netdb.h>
 
-#include <exp.h>
+#include "exp.h"
 #include "client.h"
 
 #include <imp.h>
 #include "channel.h"
 #include "conf.h"
 #include "conn.h"
+#include "hash.h"
 #include "irc-write.h"
 #include "log.h"
 #include "messages.h"
@@ -223,8 +86,8 @@ GLOBAL VOID Client_Init( VOID )
        h = gethostbyname( This_Server->host );
        if( h ) strcpy( This_Server->host, h->h_name );
 
-       strcpy( This_Server->id, Conf_ServerName );
-       strcpy( This_Server->info, Conf_ServerInfo );
+       Client_SetID( This_Server, Conf_ServerName );
+       Client_SetInfo( This_Server, Conf_ServerInfo );
 
        My_Clients = This_Server;
 } /* Client_Init */
@@ -235,14 +98,14 @@ GLOBAL VOID Client_Exit( VOID )
        CLIENT *c, *next;
        INT cnt;
 
-       Client_Destroy( This_Server, "Server going down.", NULL );
+       Client_Destroy( This_Server, "Server going down.", NULL, FALSE );
        
        cnt = 0;
        c = My_Clients;
        while( c )
        {
                cnt++;
-               next = c->next;
+               next = (CLIENT *)c->next;
                free( c );
                c = next;
        }
@@ -306,19 +169,19 @@ GLOBAL CLIENT *Client_New( CONN_ID Idx, CLIENT *Introducer, CLIENT *TopServer, I
        if( strchr( client->modes, 'a' )) strcpy( client->away, DEFAULT_AWAY_MSG );
 
        /* Verketten */
-       client->next = My_Clients;
+       client->next = (POINTER *)My_Clients;
        My_Clients = client;
 
        return client;
 } /* Client_New */
 
 
-GLOBAL VOID Client_Destroy( CLIENT *Client, CHAR *LogMsg, CHAR *FwdMsg )
+GLOBAL VOID Client_Destroy( CLIENT *Client, CHAR *LogMsg, CHAR *FwdMsg, BOOLEAN SendQuit )
 {
        /* Client entfernen. */
        
        CLIENT *last, *c;
-       CHAR *txt;
+       CHAR msg[LINE_LEN], *txt;
 
        assert( Client != NULL );
 
@@ -326,41 +189,55 @@ GLOBAL VOID Client_Destroy( CLIENT *Client, CHAR *LogMsg, CHAR *FwdMsg )
        else txt = FwdMsg;
        if( ! txt ) txt = "Reason unknown.";
 
+       /* Netz-Split-Nachricht vorbereiten (noch nicht optimal) */
+       if( Client->type == CLIENT_SERVER ) sprintf( msg, "%s: lost server %s", This_Server->id, Client->id );
+
        last = NULL;
        c = My_Clients;
        while( c )
        {
                if(( Client->type == CLIENT_SERVER ) && ( c->introducer == Client ) && ( c != Client ))
                {
-                       Client_Destroy( c, LogMsg, FwdMsg );
+                       /* der Client, der geloescht wird ist ein Server. Der Client, den wir gerade
+                        * pruefen, ist ein Child von diesem und muss daher auch entfernt werden */
+                       Client_Destroy( c, NULL, msg, FALSE );
                        last = NULL;
                        c = My_Clients;
                        continue;
                }
                if( c == Client )
                {
+                       /* Wir haben den Client gefunden: entfernen */
                        if( last ) last->next = c->next;
-                       else My_Clients = c->next;
+                       else My_Clients = (CLIENT *)c->next;
 
                        if( c->type == CLIENT_USER )
                        {
                                if( c->conn_id != NONE )
                                {
-                                       /* Ein lokaler User. Alle andere Server informieren! */
+                                       /* Ein lokaler User */
                                        Log( LOG_NOTICE, "User \"%s\" unregistered (connection %d): %s", Client_Mask( c ), c->conn_id, txt );
 
-                                       if( FwdMsg ) IRC_WriteStrServersPrefix( NULL, c, "QUIT :%s", FwdMsg );
-                                       else IRC_WriteStrServersPrefix( NULL, c, "QUIT :" );
+                                       if( SendQuit )
+                                       {
+                                               /* Alle andere Server informieren! */
+                                               if( FwdMsg ) IRC_WriteStrServersPrefix( NULL, c, "QUIT :%s", FwdMsg );
+                                               else IRC_WriteStrServersPrefix( NULL, c, "QUIT :" );
+                                       }
                                }
                                else
                                {
-                                       /* Remote User. Andere Server informieren, ausser denen,
-                                        * die "in Richtung dem liegen", auf dem der User registriert
-                                        * ist. Von denen haben wir das QUIT ja wohl bekommen. */
+                                       /* Remote User */
                                        Log( LOG_DEBUG, "User \"%s\" unregistered: %s", Client_Mask( c ), txt );
-                                       
-                                       if( FwdMsg ) IRC_WriteStrServersPrefix( Client_NextHop( c ), c, "QUIT :%s", FwdMsg );
-                                       else IRC_WriteStrServersPrefix( Client_NextHop( c ), c, "QUIT :" );
+
+                                       if( SendQuit )
+                                       {
+                                               /* Andere Server informieren, ausser denen, die "in
+                                                * Richtung dem liegen", auf dem der User registriert
+                                                * ist. Von denen haben wir das QUIT ja wohl bekommen. */
+                                               if( FwdMsg ) IRC_WriteStrServersPrefix( Client_NextHop( c ), c, "QUIT :%s", FwdMsg );
+                                               else IRC_WriteStrServersPrefix( Client_NextHop( c ), c, "QUIT :" );
+                                       }
                                }
                                Channel_RemoveClient( c, FwdMsg ? FwdMsg : c->id );
                        }
@@ -368,8 +245,8 @@ GLOBAL VOID Client_Destroy( CLIENT *Client, CHAR *LogMsg, CHAR *FwdMsg )
                        {
                                if( c != This_Server )
                                {
-                                       if( c->conn_id != NONE ) Log( LOG_NOTICE, "Server \"%s\" unregistered (connection %d): %s", c->id, c->conn_id, txt );
-                                       else Log( LOG_NOTICE, "Server \"%s\" unregistered: %s", c->id, txt );
+                                       if( c->conn_id != NONE ) Log( LOG_NOTICE|LOG_snotice, "Server \"%s\" unregistered (connection %d): %s", c->id, c->conn_id, txt );
+                                       else Log( LOG_NOTICE|LOG_snotice, "Server \"%s\" unregistered: %s", c->id, txt );
                                }
 
                                /* andere Server informieren */
@@ -397,7 +274,7 @@ GLOBAL VOID Client_Destroy( CLIENT *Client, CHAR *LogMsg, CHAR *FwdMsg )
                        break;
                }
                last = c;
-               c = c->next;
+               c = (CLIENT *)c->next;
        }
 } /* Client_Destroy */
 
@@ -407,6 +284,8 @@ GLOBAL VOID Client_SetHostname( CLIENT *Client, CHAR *Hostname )
        /* Hostname eines Clients setzen */
        
        assert( Client != NULL );
+       assert( Hostname != NULL );
+       
        strncpy( Client->host, Hostname, CLIENT_HOST_LEN - 1 );
        Client->host[CLIENT_HOST_LEN - 1] = '\0';
 } /* Client_SetHostname */
@@ -414,11 +293,16 @@ GLOBAL VOID Client_SetHostname( CLIENT *Client, CHAR *Hostname )
 
 GLOBAL VOID Client_SetID( CLIENT *Client, CHAR *ID )
 {
-       /* Hostname eines Clients setzen */
+       /* Hostname eines Clients setzen, Hash-Wert berechnen */
 
        assert( Client != NULL );
+       assert( ID != NULL );
+       
        strncpy( Client->id, ID, CLIENT_ID_LEN - 1 );
        Client->id[CLIENT_ID_LEN - 1] = '\0';
+
+       /* Hash */
+       Client->hash = Hash( Client->id );
 } /* Client_SetID */
 
 
@@ -427,6 +311,8 @@ GLOBAL VOID Client_SetUser( CLIENT *Client, CHAR *User, BOOLEAN Idented )
        /* Username eines Clients setzen */
 
        assert( Client != NULL );
+       assert( User != NULL );
+       
        if( Idented ) strncpy( Client->user, User, CLIENT_USER_LEN - 1 );
        else
        {
@@ -442,6 +328,8 @@ GLOBAL VOID Client_SetInfo( CLIENT *Client, CHAR *Info )
        /* Hostname eines Clients setzen */
 
        assert( Client != NULL );
+       assert( Info != NULL );
+       
        strncpy( Client->info, Info, CLIENT_INFO_LEN - 1 );
        Client->info[CLIENT_INFO_LEN - 1] = '\0';
 } /* Client_SetInfo */
@@ -452,6 +340,8 @@ GLOBAL VOID Client_SetModes( CLIENT *Client, CHAR *Modes )
        /* Hostname eines Clients setzen */
 
        assert( Client != NULL );
+       assert( Modes != NULL );
+
        strncpy( Client->modes, Modes, CLIENT_MODE_LEN - 1 );
        Client->modes[CLIENT_MODE_LEN - 1] = '\0';
 } /* Client_SetModes */
@@ -462,6 +352,8 @@ GLOBAL VOID Client_SetPassword( CLIENT *Client, CHAR *Pwd )
        /* Von einem Client geliefertes Passwort */
 
        assert( Client != NULL );
+       assert( Pwd != NULL );
+       
        strncpy( Client->pwd, Pwd, CLIENT_PASS_LEN - 1 );
        Client->pwd[CLIENT_PASS_LEN - 1] = '\0';
 } /* Client_SetPassword */
@@ -515,6 +407,7 @@ GLOBAL VOID Client_SetToken( CLIENT *Client, INT Token )
 GLOBAL VOID Client_SetIntroducer( CLIENT *Client, CLIENT *Introducer )
 {
        assert( Client != NULL );
+       assert( Introducer != NULL );
        Client->introducer = Introducer;
 } /* Client_SetIntroducer */
 
@@ -585,36 +478,43 @@ GLOBAL CLIENT *Client_GetFromConn( CONN_ID Idx )
        while( c )
        {
                if( c->conn_id == Idx ) return c;
-               c = c->next;
+               c = (CLIENT *)c->next;
        }
        return NULL;
 } /* Client_GetFromConn */
 
 
-GLOBAL CLIENT *Client_GetFromID( CHAR *Nick )
+GLOBAL CLIENT *Client_Search( CHAR *Nick )
 {
        /* Client-Struktur, die den entsprechenden Nick hat, liefern.
         * Wird keine gefunden, so wird NULL geliefert. */
 
-       CHAR n[CLIENT_ID_LEN], *ptr;
+       CHAR search_id[CLIENT_ID_LEN], *ptr;
        CLIENT *c = NULL;
+       UINT32 search_hash;
 
        assert( Nick != NULL );
 
        /* Nick kopieren und ggf. Host-Mask abschneiden */
-       strncpy( n, Nick, CLIENT_ID_LEN - 1 );
-       n[CLIENT_ID_LEN - 1] = '\0';
-       ptr = strchr( n, '!' );
+       strncpy( search_id, Nick, CLIENT_ID_LEN - 1 );
+       search_id[CLIENT_ID_LEN - 1] = '\0';
+       ptr = strchr( search_id, '!' );
        if( ptr ) *ptr = '\0';
 
+       search_hash = Hash( search_id );
+
        c = My_Clients;
        while( c )
        {
-               if( strcasecmp( c->id, n ) == 0 ) return c;
-               c = c->next;
+               if( c->hash == search_hash )
+               {
+                       /* lt. Hash-Wert: Treffer! */
+                       if( strcasecmp( c->id, search_id ) == 0 ) return c;
+               }
+               c = (CLIENT *)c->next;
        }
        return NULL;
-} /* Client_GetFromID */
+} /* Client_Search */
 
 
 GLOBAL CLIENT *Client_GetFromToken( CLIENT *Client, INT Token )
@@ -632,7 +532,7 @@ GLOBAL CLIENT *Client_GetFromToken( CLIENT *Client, INT Token )
        while( c )
        {
                if(( c->type == CLIENT_SERVER ) && ( c->introducer == Client ) && ( c->token == Token )) return c;
-               c = c->next;
+               c = (CLIENT *)c->next;
        }
        return NULL;
 } /* Client_GetFromToken */
@@ -656,6 +556,10 @@ GLOBAL CHAR *Client_ID( CLIENT *Client )
 {
        assert( Client != NULL );
 
+#ifdef DEBUG
+       if( Client->type == CLIENT_USER ) assert( strlen( Client->id ) < CLIENT_NICK_LEN );
+#endif
+                                                  
        if( Client->id[0] ) return Client->id;
        else return "*";
 } /* Client_ID */
@@ -671,7 +575,7 @@ GLOBAL CHAR *Client_Info( CLIENT *Client )
 GLOBAL CHAR *Client_User( CLIENT *Client )
 {
        assert( Client != NULL );
-       if( Client->user ) return Client->user;
+       if( Client->user[0] ) return Client->user;
        else return "~";
 } /* Client_User */
 
@@ -785,8 +689,6 @@ GLOBAL BOOLEAN Client_CheckNick( CLIENT *Client, CHAR *Nick )
 {
        /* Nick ueberpruefen */
 
-       CLIENT *c;
-       
        assert( Client != NULL );
        assert( Nick != NULL );
        
@@ -798,16 +700,11 @@ GLOBAL BOOLEAN Client_CheckNick( CLIENT *Client, CHAR *Nick )
        }
 
        /* Nick bereits vergeben? */
-       c = My_Clients;
-       while( c )
+       if( Client_Search( Nick ))
        {
-               if( strcasecmp( c->id, Nick ) == 0 )
-               {
-                       /* den Nick gibt es bereits */
-                       IRC_WriteStrClient( Client, ERR_NICKNAMEINUSE_MSG, Client_ID( Client ), Nick );
-                       return FALSE;
-               }
-               c = c->next;
+               /* den Nick gibt es bereits */
+               IRC_WriteStrClient( Client, ERR_NICKNAMEINUSE_MSG, Client_ID( Client ), Nick );
+               return FALSE;
        }
 
        return TRUE;
@@ -844,32 +741,13 @@ GLOBAL BOOLEAN Client_CheckID( CLIENT *Client, CHAR *ID )
                        Conn_Close( Client->conn_id, str, str, TRUE );
                        return FALSE;
                }
-               c = c->next;
+               c = (CLIENT *)c->next;
        }
 
        return TRUE;
 } /* Client_CheckID */
 
 
-GLOBAL CLIENT *Client_Search( CHAR *ID )
-{
-       /* Client suchen, auf den ID passt */
-
-       CLIENT *c;
-
-       assert( ID != NULL );
-
-       c = My_Clients;
-       while( c )
-       {
-               if( strcasecmp( c->id, ID ) == 0 ) return c;
-               c = c->next;
-       }
-       
-       return NULL;
-} /* Client_Search */
-
-
 GLOBAL CLIENT *Client_First( VOID )
 {
        /* Ersten Client liefern. */
@@ -884,7 +762,7 @@ GLOBAL CLIENT *Client_Next( CLIENT *c )
         * so wird NULL geliefert. */
 
        assert( c != NULL );
-       return c->next;
+       return (CLIENT *)c->next;
 } /* Client_Next */
 
 
@@ -920,7 +798,17 @@ GLOBAL INT Client_MyServiceCount( VOID )
 
 GLOBAL INT Client_MyServerCount( VOID )
 {
-       return MyCount( CLIENT_SERVER );
+       CLIENT *c;
+       INT cnt;
+
+       cnt = 0;
+       c = My_Clients;
+       while( c )
+       {
+               if(( c->type == CLIENT_SERVER ) && ( c->hops == 1 )) cnt++;
+               c = (CLIENT *)c->next;
+       }
+       return cnt;
 } /* Client_MyServerCount */
 
 
@@ -934,7 +822,7 @@ GLOBAL INT Client_OperCount( VOID )
        while( c )
        {
                if( c && ( c->type == CLIENT_USER ) && ( strchr( c->modes, 'o' ))) cnt++;
-               c = c->next;
+               c = (CLIENT *)c->next;
        }
        return cnt;
 } /* Client_OperCount */
@@ -950,7 +838,7 @@ GLOBAL INT Client_UnknownCount( VOID )
        while( c )
        {
                if( c && ( c->type != CLIENT_USER ) && ( c->type != CLIENT_SERVICE ) && ( c->type != CLIENT_SERVER )) cnt++;
-               c = c->next;
+               c = (CLIENT *)c->next;
        }
        return cnt;
 } /* Client_UnknownCount */
@@ -960,7 +848,7 @@ GLOBAL BOOLEAN Client_IsValidNick( CHAR *Nick )
 {
        /* Ist der Nick gueltig? */
 
-       CHAR *ptr, goodchars[] = ";0123456789";
+       CHAR *ptr, goodchars[] = ";0123456789-";
        
        assert( Nick != NULL );
 
@@ -989,8 +877,8 @@ LOCAL INT Count( CLIENT_TYPE Type )
        c = My_Clients;
        while( c )
        {
-               if( c && ( c->type == Type )) cnt++;
-               c = c->next;
+               if( c->type == Type ) cnt++;
+               c = (CLIENT *)c->next;
        }
        return cnt;
 } /* Count */
@@ -1005,8 +893,8 @@ LOCAL INT MyCount( CLIENT_TYPE Type )
        c = My_Clients;
        while( c )
        {
-               if( c && ( c->introducer == This_Server ) && ( c->type == Type )) cnt++;
-               c = c->next;
+               if(( c->introducer == This_Server ) && ( c->type == Type )) cnt++;
+               c = (CLIENT *)c->next;
        }
        return cnt;
 } /* MyCount */
@@ -1026,6 +914,7 @@ LOCAL CLIENT *New_Client_Struct( VOID )
        }
 
        c->next = NULL;
+       c->hash = 0;
        c->type = CLIENT_UNKNOWN;
        c->conn_id = NONE;
        c->introducer = NULL;
@@ -1062,7 +951,7 @@ LOCAL VOID Generate_MyToken( CLIENT *Client )
                        c = My_Clients;
                        continue;
                }
-               else c = c->next;
+               else c = (CLIENT *)c->next;
        }
        Client->mytoken = token;
        Log( LOG_DEBUG, "Assigned token %d to server \"%s\".", token, Client->id );