mirrors/ngircd
1
0
Fork 0
mirror of https://github.com/osmarks/ngircd.git synced 2024-12-12 09:50:29 +00:00
Code Issues Releases Wiki Activity

Remove limit on max number of predefined channels.

Browse Source 
This resolves Bugzilla Bug 68 ('Too many pre-defined channels configured.')
This commit is contained in:
Florian Westphal 2008-11-24 22:59:10 +01:00
parent 6500d443fc
commit a971047bc5
4 changed files with 89 additions and 91 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

69
src/ngircd/channel.c
View File

@ -97,55 +97,52 @@ Channel_InitPredefined( void )
{ {
/* Generate predefined persistent channels */ /* Generate predefined persistent channels */
CHANNEL *chan; CHANNEL *new_chan;
char *c; const struct Conf_Channel *conf_chan;
unsigned int i; const char *c;
size_t i, channel_count = array_length(&Conf_Channels, sizeof(*conf_chan));
for( i = 0; i < Conf_Channel_Count; i++ ) conf_chan = array_start(&Conf_Channels);
{
/* Check for Name configuration */
if( ! Conf_Channel[i].name[0] ) continue;
/* Check for invalid channel name */ assert(channel_count == 0 || conf_chan != NULL);
if( ! Channel_IsValidName( Conf_Channel[i].name ))
{ for (i = 0; i < channel_count; i++, conf_chan++) {
Log( LOG_ERR, "Can't create pre-defined channel: invalid name: \"%s\"!", Conf_Channel[i].name ); if (!conf_chan->name[0] || !Channel_IsValidName(conf_chan->name)) {
array_free(&Conf_Channel[i].topic); Log(LOG_ERR, "Can't create pre-defined channel: invalid name: \"%s\"",
conf_chan->name);
continue; continue;
} }
/* Check if the channel name is already in use */ new_chan = Channel_Search(conf_chan->name);
chan = Channel_Search( Conf_Channel[i].name ); if (new_chan) {
if( chan ) Log(LOG_INFO, "Can't create pre-defined channel \"%s\": name already in use.",
{ conf_chan->name);
Log( LOG_INFO, "Can't create pre-defined channel \"%s\": name already in use.", Conf_Channel[i].name );
array_free(&Conf_Channel[i].topic);
continue; continue;
} }
/* Create channel */ new_chan = Channel_Create(conf_chan->name);
chan = Channel_Create(Conf_Channel[i].name); if (!new_chan) {
if (chan) { Log(LOG_ERR, "Can't create pre-defined channel \"%s\"",
Channel_ModeAdd(chan, 'P'); conf_chan->name);
continue;
}
if (array_start(&Conf_Channel[i].topic) != NULL) Channel_ModeAdd(new_chan, 'P');
Channel_SetTopic(chan, NULL,
array_start(&Conf_Channel[i].topic));
array_free(&Conf_Channel[i].topic);
c = Conf_Channel[i].modes; if (conf_chan->topic[0])
Channel_SetTopic(new_chan, NULL, conf_chan->topic);
c = conf_chan->modes;
while (*c) while (*c)
Channel_ModeAdd(chan, *c++); Channel_ModeAdd(new_chan, *c++);
Channel_SetKey(chan, Conf_Channel[i].key); Channel_SetKey(new_chan, conf_chan->key);
Channel_SetMaxUsers(chan, Conf_Channel[i].maxusers); Channel_SetMaxUsers(new_chan, conf_chan->maxusers);
Log(LOG_INFO, "Created pre-defined channel \"%s\"",
Log(LOG_INFO, "Created pre-defined channel \"%s\".", conf_chan->name);
Conf_Channel[i].name );
}
else Log(LOG_ERR, "Can't create pre-defined channel \"%s\"!",
Conf_Channel[i].name );
} }
if (channel_count)
array_free(&Conf_Channels);
} /* Channel_InitPredefined */ } /* Channel_InitPredefined */

76
src/ngircd/conf.c
View File

@ -54,7 +54,7 @@ static bool Use_Log = true;
static CONF_SERVER New_Server; static CONF_SERVER New_Server;
static int New_Server_Idx; static int New_Server_Idx;
static size_t Conf_Channel_Count;
static void Set_Defaults PARAMS(( bool InitServers )); static void Set_Defaults PARAMS(( bool InitServers ));
static bool Read_Config PARAMS(( bool ngircd_starting )); static bool Read_Config PARAMS(( bool ngircd_starting ));
static bool Validate_Config PARAMS(( bool TestOnly, bool Rehash )); static bool Validate_Config PARAMS(( bool TestOnly, bool Rehash ));
@ -206,8 +206,9 @@ Conf_Test( void )
struct passwd *pwd; struct passwd *pwd;
struct group *grp; struct group *grp;
unsigned int i; unsigned int i;
char *topic;
bool config_valid; bool config_valid;
size_t predef_channel_count;
struct Conf_Channel *predef_chan;
Use_Log = false; Use_Log = false;
@ -299,18 +300,20 @@ Conf_Test( void )
printf( " Passive = %s\n\n", Conf_Server[i].flags & CONF_SFLAG_DISABLED ? "yes" : "no"); printf( " Passive = %s\n\n", Conf_Server[i].flags & CONF_SFLAG_DISABLED ? "yes" : "no");
} }
for( i = 0; i < Conf_Channel_Count; i++ ) { predef_channel_count = array_length(&Conf_Channels, sizeof(*predef_chan));
if( ! Conf_Channel[i].name[0] ) continue; predef_chan = array_start(&Conf_Channels);
for (i = 0; i < predef_channel_count; i++, predef_chan++) {
if (!predef_chan->name[0])
continue;
/* Valid "Channel" section */ /* Valid "Channel" section */
puts( "[CHANNEL]" ); puts( "[CHANNEL]" );
printf( " Name = %s\n", Conf_Channel[i].name ); printf(" Name = %s\n", predef_chan->name);
printf( " Modes = %s\n", Conf_Channel[i].modes ); printf(" Modes = %s\n", predef_chan->modes);
printf( " Key = %s\n", Conf_Channel[i].key ); printf(" Key = %s\n", predef_chan->key);
printf( " MaxUsers = %lu\n", Conf_Channel[i].maxusers ); printf(" MaxUsers = %lu\n", predef_chan->maxusers);
printf(" Topic = %s\n\n", predef_chan->topic);
topic = (char*)array_start(&Conf_Channel[i].topic);
printf( " Topic = %s\n\n", topic ? topic : "");
} }
return (config_valid ? 0 : 1); return (config_valid ? 0 : 1);
@ -657,19 +660,10 @@ Read_Config( bool ngircd_starting )
continue; continue;
} }
if (strcasecmp(section, "[CHANNEL]") == 0) { if (strcasecmp(section, "[CHANNEL]") == 0) {
if( Conf_Channel_Count + 1 > MAX_DEFCHANNELS ) {
Config_Error( LOG_ERR, "Too many pre-defined channels configured." );
} else {
/* Initialize new channel structure */
strcpy( Conf_Channel[Conf_Channel_Count].name, "" );
strcpy( Conf_Channel[Conf_Channel_Count].modes, "" );
strcpy( Conf_Channel[Conf_Channel_Count].key, "" );
Conf_Channel[Conf_Channel_Count].maxusers = 0;
array_free(&Conf_Channel[Conf_Channel_Count].topic);
Conf_Channel_Count++; Conf_Channel_Count++;
}
continue; continue;
} }
Config_Error( LOG_ERR, "%s, line %d: Unknown section \"%s\"!", NGIRCd_ConfFile, line, section ); Config_Error( LOG_ERR, "%s, line %d: Unknown section \"%s\"!", NGIRCd_ConfFile, line, section );
section[0] = 0x1; section[0] = 0x1;
} }
@ -1169,11 +1163,15 @@ Handle_SERVER( int Line, char *Var, char *Arg )
static bool static bool
Handle_Channelname(size_t chancount, const char *name) Handle_Channelname(struct Conf_Channel *new_chan, const char *name)
{ {
size_t size = sizeof( Conf_Channel[chancount].name ); size_t size = sizeof(new_chan->name);
char *dest = Conf_Channel[chancount].name; char *dest = new_chan->name;
/*
* channels must begin with &, +, or '#', if it is
* missing, add a '#'. This is only here for user convenience.
*/
if (*name && *name != '#') { if (*name && *name != '#') {
*dest = '#'; *dest = '#';
--size; --size;
@ -1187,45 +1185,51 @@ static void
Handle_CHANNEL(int Line, char *Var, char *Arg) Handle_CHANNEL(int Line, char *Var, char *Arg)
{ {
size_t len; size_t len;
size_t chancount = 0; size_t chancount;
struct Conf_Channel *chan;
assert( Line > 0 ); assert( Line > 0 );
assert( Var != NULL ); assert( Var != NULL );
assert( Arg != NULL ); assert( Arg != NULL );
if (Conf_Channel_Count > 0) assert(Conf_Channel_Count > 0);
chancount = Conf_Channel_Count - 1; chancount = Conf_Channel_Count - 1;
chan = array_alloc(&Conf_Channels, sizeof(*chan), chancount);
if (!chan) {
Config_Error(LOG_ERR, "Could not allocate memory for predefined channel (%d:%s = %s)", Line, Var, Arg);
return;
}
if (strcasecmp(Var, "Name") == 0) { if (strcasecmp(Var, "Name") == 0) {
if (!Handle_Channelname(chancount, Arg)) if (!Handle_Channelname(chan, Arg))
Config_Error_TooLong(Line, Var); Config_Error_TooLong(Line, Var);
return; return;
} }
if (strcasecmp(Var, "Modes") == 0) { if (strcasecmp(Var, "Modes") == 0) {
/* Initial modes */ /* Initial modes */
len = strlcpy( Conf_Channel[chancount].modes, Arg, sizeof( Conf_Channel[chancount].modes )); len = strlcpy(chan->modes, Arg, sizeof(chan->modes));
if (len >= sizeof( Conf_Channel[chancount].modes )) if (len >= sizeof(chan->modes))
Config_Error_TooLong( Line, Var ); Config_Error_TooLong( Line, Var );
return; return;
} }
if( strcasecmp( Var, "Topic" ) == 0 ) { if( strcasecmp( Var, "Topic" ) == 0 ) {
/* Initial topic */ /* Initial topic */
if (!array_copys( &Conf_Channel[chancount].topic, Arg)) len = strlcpy(chan->topic, Arg, sizeof(chan->topic));
if (len >= sizeof(chan->topic))
Config_Error_TooLong( Line, Var ); Config_Error_TooLong( Line, Var );
return; return;
} }
if( strcasecmp( Var, "Key" ) == 0 ) { if( strcasecmp( Var, "Key" ) == 0 ) {
/* Initial Channel Key (mode k) */ /* Initial Channel Key (mode k) */
len = strlcpy(Conf_Channel[chancount].key, Arg, sizeof(Conf_Channel[chancount].key)); len = strlcpy(chan->key, Arg, sizeof(chan->key));
if (len >= sizeof( Conf_Channel[chancount].key )) if (len >= sizeof(chan->key))
Config_Error_TooLong(Line, Var); Config_Error_TooLong(Line, Var);
return; return;
} }
if( strcasecmp( Var, "MaxUsers" ) == 0 ) { if( strcasecmp( Var, "MaxUsers" ) == 0 ) {
/* maximum user limit, mode l */ /* maximum user limit, mode l */
Conf_Channel[chancount].maxusers = (unsigned long) atol(Arg); chan->maxusers = (unsigned long) atol(Arg);
if (Conf_Channel[chancount].maxusers == 0) if (chan->maxusers == 0)
Config_Error_NaN(Line, Var); Config_Error_NaN(Line, Var);
return; return;
} }

11
src/ngircd/conf.h
View File

@ -67,14 +67,13 @@ struct SSLOptions {
#endif #endif
typedef struct _Conf_Channel struct Conf_Channel {
{
char name[CHANNEL_NAME_LEN]; /* Name of the channel */ char name[CHANNEL_NAME_LEN]; /* Name of the channel */
char modes[CHANNEL_MODE_LEN]; /* Initial channel modes */ char modes[CHANNEL_MODE_LEN]; /* Initial channel modes */
char key[CLIENT_PASS_LEN]; /* Channel key ("password", mode "k" ) */ char key[CLIENT_PASS_LEN]; /* Channel key ("password", mode "k" ) */
char topic[COMMAND_LEN]; /* Initial topic */
unsigned long maxusers; /* maximum usercount for this channel, mode "l" */ unsigned long maxusers; /* maximum usercount for this channel, mode "l" */
array topic; /* Initial topic */ };
} CONF_CHANNEL;
#define CONF_SFLAG_ONCE 1 /* Delete this entry after next disconnect */ #define CONF_SFLAG_ONCE 1 /* Delete this entry after next disconnect */
@ -132,8 +131,8 @@ GLOBAL unsigned int Conf_Oper_Count;
GLOBAL CONF_SERVER Conf_Server[MAX_SERVERS]; GLOBAL CONF_SERVER Conf_Server[MAX_SERVERS];
/* Pre-defined channels */ /* Pre-defined channels */
GLOBAL CONF_CHANNEL Conf_Channel[MAX_DEFCHANNELS]; GLOBAL array Conf_Channels;
GLOBAL unsigned int Conf_Channel_Count;
/* Pre-defined channels only */ /* Pre-defined channels only */
GLOBAL bool Conf_PredefChannelsOnly; GLOBAL bool Conf_PredefChannelsOnly;

2
src/ngircd/defines.h
View File

@ -36,8 +36,6 @@
#define MAX_SERVERS 16 /* Max. count of configurable servers */ #define MAX_SERVERS 16 /* Max. count of configurable servers */
#define MAX_DEFCHANNELS 16 /* Max. count of predefined channels */
#define MAX_SERVICES 8 /* Max. count of services */ #define MAX_SERVICES 8 /* Max. count of services */
#define MAX_WHOWAS 64 /* Max. number of WHOWAS items */ #define MAX_WHOWAS 64 /* Max. number of WHOWAS items */