X-Git-Url: https://arthur.barton.de/cgi-bin/gitweb.cgi?p=ngircd-alex.git;a=blobdiff_plain;f=src%2Fngircd%2Fclient.c;h=917b230eb74edcc9be73d8518476d6cd1edb1a18;hp=07fd6481bb3a55c3f5848bd12294ce0385095075;hb=dce77559fa6a0ad31b306615600957f024618587;hpb=c48501245e730ce2b33547f716d297bb0b805e77 diff --git a/src/ngircd/client.c b/src/ngircd/client.c index 07fd6481..917b230e 100644 --- a/src/ngircd/client.c +++ b/src/ngircd/client.c @@ -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.39 2002/02/27 18:22:09 alex Exp $ + * $Id: client.c,v 1.46 2002/03/10 22:03:20 alex Exp $ * * client.c: Management aller Clients * @@ -21,6 +21,27 @@ * Server gewesen, so existiert eine entsprechende CONNECTION-Struktur. * * $Log: client.c,v $ + * Revision 1.46 2002/03/10 22:03:20 alex + * - Netz-Splits werden nun als soche ausgegeben. + * + * Revision 1.45 2002/03/10 17:15:20 alex + * - der Bindestrich ("-") ist nun auch in Nicknames erlaubt. + * + * Revision 1.44 2002/03/06 14:30:43 alex + * - ein paar assert()-Tests ergaenzt. + * + * 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. * @@ -169,7 +190,7 @@ #include "channel.h" #include "conf.h" #include "conn.h" -#include "irc.h" +#include "irc-write.h" #include "log.h" #include "messages.h" @@ -290,6 +311,9 @@ GLOBAL CLIENT *Client_New( CONN_ID Idx, CLIENT *Introducer, CLIENT *TopServer, I if( Modes ) Client_SetModes( client, Modes ); if( Type == CLIENT_SERVER ) Generate_MyToken( client ); + /* ist der User away? */ + if( strchr( client->modes, 'a' )) strcpy( client->away, DEFAULT_AWAY_MSG ); + /* Verketten */ client->next = My_Clients; My_Clients = client; @@ -303,7 +327,7 @@ GLOBAL VOID Client_Destroy( CLIENT *Client, CHAR *LogMsg, CHAR *FwdMsg ) /* Client entfernen. */ CLIENT *last, *c; - CHAR *txt; + CHAR msg[LINE_LEN], *txt; assert( Client != NULL ); @@ -311,19 +335,28 @@ GLOBAL VOID Client_Destroy( CLIENT *Client, CHAR *LogMsg, CHAR *FwdMsg ) else txt = FwdMsg; if( ! txt ) txt = "Reason unknown."; + if( Client->type == CLIENT_SERVER ) + { + /* Netz-Split-Nachricht vorbereiten */ + sprintf( msg, "%s | %s", Client_ID( Client ), Client_ID( Client_TopServer( Client ) ? Client_TopServer( Client ) : Client_ThisServer( ))); + } + 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); 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; @@ -392,7 +425,9 @@ GLOBAL VOID Client_SetHostname( CLIENT *Client, CHAR *Hostname ) /* Hostname eines Clients setzen */ assert( Client != NULL ); - strncpy( Client->host, Hostname, CLIENT_HOST_LEN ); + assert( Hostname != NULL ); + + strncpy( Client->host, Hostname, CLIENT_HOST_LEN - 1 ); Client->host[CLIENT_HOST_LEN - 1] = '\0'; } /* Client_SetHostname */ @@ -402,7 +437,9 @@ GLOBAL VOID Client_SetID( CLIENT *Client, CHAR *ID ) /* Hostname eines Clients setzen */ assert( Client != NULL ); - strncpy( Client->id, ID, CLIENT_ID_LEN ); + assert( ID != NULL ); + + strncpy( Client->id, ID, CLIENT_ID_LEN - 1 ); Client->id[CLIENT_ID_LEN - 1] = '\0'; } /* Client_SetID */ @@ -412,11 +449,13 @@ GLOBAL VOID Client_SetUser( CLIENT *Client, CHAR *User, BOOLEAN Idented ) /* Username eines Clients setzen */ assert( Client != NULL ); - if( Idented ) strncpy( Client->user, User, CLIENT_USER_LEN ); + assert( User != NULL ); + + if( Idented ) strncpy( Client->user, User, CLIENT_USER_LEN - 1 ); else { Client->user[0] = '~'; - strncpy( Client->user + 1, User, CLIENT_USER_LEN - 1 ); + strncpy( Client->user + 1, User, CLIENT_USER_LEN - 2 ); } Client->user[CLIENT_USER_LEN - 1] = '\0'; } /* Client_SetUser */ @@ -427,7 +466,9 @@ GLOBAL VOID Client_SetInfo( CLIENT *Client, CHAR *Info ) /* Hostname eines Clients setzen */ assert( Client != NULL ); - strncpy( Client->info, Info, CLIENT_INFO_LEN ); + assert( Info != NULL ); + + strncpy( Client->info, Info, CLIENT_INFO_LEN - 1 ); Client->info[CLIENT_INFO_LEN - 1] = '\0'; } /* Client_SetInfo */ @@ -437,7 +478,9 @@ GLOBAL VOID Client_SetModes( CLIENT *Client, CHAR *Modes ) /* Hostname eines Clients setzen */ assert( Client != NULL ); - strncpy( Client->modes, Modes, CLIENT_MODE_LEN ); + assert( Modes != NULL ); + + strncpy( Client->modes, Modes, CLIENT_MODE_LEN - 1 ); Client->modes[CLIENT_MODE_LEN - 1] = '\0'; } /* Client_SetModes */ @@ -447,7 +490,9 @@ GLOBAL VOID Client_SetPassword( CLIENT *Client, CHAR *Pwd ) /* Von einem Client geliefertes Passwort */ assert( Client != NULL ); - strncpy( Client->pwd, Pwd, CLIENT_PASS_LEN ); + assert( Pwd != NULL ); + + strncpy( Client->pwd, Pwd, CLIENT_PASS_LEN - 1 ); Client->pwd[CLIENT_PASS_LEN - 1] = '\0'; } /* Client_SetPassword */ @@ -461,7 +506,7 @@ GLOBAL VOID Client_SetAway( CLIENT *Client, CHAR *Txt ) if( Txt ) { /* Client AWAY setzen */ - strncpy( Client->away, Txt, CLIENT_AWAY_LEN ); + strncpy( Client->away, Txt, CLIENT_AWAY_LEN - 1 ); Client->away[CLIENT_AWAY_LEN - 1] = '\0'; Client_ModeAdd( Client, 'a' ); Log( LOG_DEBUG, "User \"%s\" is away: %s", Client_Mask( Client ), Txt ); @@ -500,6 +545,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 */ @@ -578,17 +624,17 @@ GLOBAL CLIENT *Client_GetFromConn( CONN_ID Idx ) GLOBAL CLIENT *Client_GetFromID( CHAR *Nick ) { - /* Client-Struktur, die den entsprechenden Nick hat, - * liefern. Wird keine gefunden, so wird NULL geliefert. */ + /* Client-Struktur, die den entsprechenden Nick hat, liefern. + * Wird keine gefunden, so wird NULL geliefert. */ - CHAR n[CLIENT_ID_LEN + 1], *ptr; + CHAR n[CLIENT_ID_LEN], *ptr; CLIENT *c = NULL; assert( Nick != NULL ); /* Nick kopieren und ggf. Host-Mask abschneiden */ - strncpy( n, Nick, CLIENT_ID_LEN ); - n[CLIENT_ID_LEN] = '\0'; + strncpy( n, Nick, CLIENT_ID_LEN - 1 ); + n[CLIENT_ID_LEN - 1] = '\0'; ptr = strchr( n, '!' ); if( ptr ) *ptr = '\0'; @@ -641,6 +687,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 */ @@ -944,11 +994,23 @@ GLOBAL INT Client_UnknownCount( VOID ) GLOBAL BOOLEAN Client_IsValidNick( CHAR *Nick ) { /* Ist der Nick gueltig? */ + + CHAR *ptr, goodchars[] = ";0123456789-"; assert( Nick != NULL ); if( Nick[0] == '#' ) return FALSE; - if( strlen( Nick ) > CLIENT_NICK_LEN ) return FALSE; + if( strchr( goodchars, Nick[0] )) return FALSE; + if( strlen( Nick ) >= CLIENT_NICK_LEN ) return FALSE; + + ptr = Nick; + while( *ptr ) + { + if(( *ptr < 'A' ) && ( ! strchr( goodchars, *ptr ))) return FALSE; + if(( *ptr > '}' ) && ( ! strchr( goodchars, *ptr ))) return FALSE; + ptr++; + } + return TRUE; } /* Client_IsValidNick */