Add Doxygen @file documentation to each source and header file

[ngircd.git] / src / ngircd / conf.c

diff --git a/src/ngircd/conf.c b/src/ngircd/conf.c
index 32846faa052002f068b41cf628ebc15d71bfb60a..c2dde4b79e3a6f6747cdbef3ce60a999571636c8 100644 (file)
--- a/src/ngircd/conf.c
+++ b/src/ngircd/conf.c
@@ -7,13 +7,16 @@
  * 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.
- *
- * Configuration management (reading, parsing & validation)
  */
 
 
 #include "portab.h"
 
+/**
+ * @file
+ * Configuration management (reading, parsing & validation)
+ */
+
 #include "imp.h"
 #include <assert.h>
 #include <errno.h>
@@ -641,6 +644,9 @@ Set_Defaults(bool InitServers)
                for (i = 0; i < MAX_SERVERS;
                     Init_Server_Struct(&Conf_Server[i++]));
        }
+
+       /* Free MOTD; this is important when reloading the configuration */
+       array_free(&Conf_Motd);
 } /* Set_Defaults */
 
 
@@ -665,7 +671,7 @@ Read_Motd(const char *filename)
 
        fp = fopen(filename, "r");
        if (!fp) {
-               Log(LOG_WARNING, "Can't read MOTD file \"%s\": %s",
+               Config_Error(LOG_WARNING, "Can't read MOTD file \"%s\": %s",
                                        filename, strerror(errno));
                return;
        }
@@ -1016,7 +1022,6 @@ Handle_GLOBAL( int Line, char *Var, char *Arg )
                len = strlcpy( Conf_MotdFile, Arg, sizeof( Conf_MotdFile ));
                if (len >= sizeof( Conf_MotdFile ))
                        Config_Error_TooLong( Line, Var );
-               Read_Motd(Arg);
                return;
        }
        if( strcasecmp( Var, "MotdPhrase" ) == 0 ) {
@@ -1053,11 +1058,9 @@ Handle_GLOBAL( int Line, char *Var, char *Arg )
                pwd = getpwnam( Arg );
                if( pwd ) Conf_UID = pwd->pw_uid;
                else {
-#ifdef HAVE_ISDIGIT
-                       if( ! isdigit( (int)*Arg )) Config_Error_NaN( Line, Var );
-                       else
-#endif
                        Conf_UID = (unsigned int)atoi( Arg );
+                       if (!Conf_UID && strcmp(Arg, "0"))
+                               Config_Error_NaN(Line, Var);
                }
                return;
        }
@@ -1066,11 +1069,9 @@ Handle_GLOBAL( int Line, char *Var, char *Arg )
                grp = getgrnam( Arg );
                if( grp ) Conf_GID = grp->gr_gid;
                else {
-#ifdef HAVE_ISDIGIT
-                       if( ! isdigit( (int)*Arg )) Config_Error_NaN( Line, Var );
-                       else
-#endif
-                       Conf_GID = (unsigned int)atoi( Arg );
+                       Conf_GID = (unsigned int)atoi(Arg);
+                       if (!Conf_GID && strcmp(Arg, "0"))
+                               Config_Error_NaN( Line, Var );
                }
                return;
        }
@@ -1151,29 +1152,23 @@ Handle_GLOBAL( int Line, char *Var, char *Arg )
        }
        if( strcasecmp( Var, "MaxConnections" ) == 0 ) {
                /* Maximum number of connections. 0 -> "no limit". */
-#ifdef HAVE_ISDIGIT
-               if( ! isdigit( (int)*Arg )) Config_Error_NaN( Line, Var);
-               else
-#endif
                Conf_MaxConnections = atol( Arg );
+               if (!Conf_MaxConnections && strcmp(Arg, "0"))
+                       Config_Error_NaN(Line, Var);
                return;
        }
        if( strcasecmp( Var, "MaxConnectionsIP" ) == 0 ) {
                /* Maximum number of simultaneous connections from one IP. 0 -> "no limit" */
-#ifdef HAVE_ISDIGIT
-               if( ! isdigit( (int)*Arg )) Config_Error_NaN( Line, Var );
-               else
-#endif
                Conf_MaxConnectionsIP = atoi( Arg );
+               if (!Conf_MaxConnectionsIP && strcmp(Arg, "0"))
+                       Config_Error_NaN(Line, Var);
                return;
        }
        if( strcasecmp( Var, "MaxJoins" ) == 0 ) {
                /* Maximum number of channels a user can join. 0 -> "no limit". */
-#ifdef HAVE_ISDIGIT
-               if( ! isdigit( (int)*Arg )) Config_Error_NaN( Line, Var );
-               else
-#endif
                Conf_MaxJoins = atoi( Arg );
+               if (!Conf_MaxJoins && strcmp(Arg, "0"))
+                       Config_Error_NaN(Line, Var);
                return;
        }
        if( strcasecmp( Var, "MaxNickLength" ) == 0 ) {
@@ -1369,11 +1364,12 @@ Handle_SERVER( int Line, char *Var, char *Arg )
        if( strcasecmp( Var, "Port" ) == 0 ) {
                /* Port to which this server should connect */
                port = atol( Arg );
-               if( port > 0 && port < 0xFFFF )
+               if (port >= 0 && port < 0xFFFF)
                        New_Server.port = (UINT16)port;
                else
-                       Config_Error( LOG_ERR, "%s, line %d (section \"Server\"): Illegal port number %ld!",
-                                                                               NGIRCd_ConfFile, Line, port );
+                       Config_Error(LOG_ERR,
+                               "%s, line %d (section \"Server\"): Illegal port number %ld!",
+                               NGIRCd_ConfFile, Line, port );
                return;
        }
 #ifdef SSL_SUPPORT
@@ -1384,12 +1380,9 @@ Handle_SERVER( int Line, char *Var, char *Arg )
 #endif
        if( strcasecmp( Var, "Group" ) == 0 ) {
                /* Server group */
-#ifdef HAVE_ISDIGIT
-               if( ! isdigit( (int)*Arg ))
-                       Config_Error_NaN( Line, Var );
-               else
-#endif
                New_Server.group = atoi( Arg );
+               if (!New_Server.group && strcmp(Arg, "0"))
+                       Config_Error_NaN(Line, Var);
                return;
        }
        if( strcasecmp( Var, "Passive" ) == 0 ) {
@@ -1477,7 +1470,7 @@ Handle_CHANNEL(int Line, char *Var, char *Arg)
        if( strcasecmp( Var, "MaxUsers" ) == 0 ) {
                /* maximum user limit, mode l */
                chan->maxusers = (unsigned long) atol(Arg);
-               if (chan->maxusers == 0)
+               if (!chan->maxusers && strcmp(Arg, "0"))
                        Config_Error_NaN(Line, Var);
                return;
        }