]> arthur.barton.de Git - ngircd-alex.git/blobdiff - src/ngircd/conf.c
New option to scrub incoming CTCP commands
[ngircd-alex.git] / src / ngircd / conf.c
index a00049eef97cc29630a6d22af8e556f7efd61a2c..6bd224f3c11ea58189d081beb361d97783bf721f 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * ngIRCd -- The Next Generation IRC Daemon
- * Copyright (c)2001-2010 Alexander Barton (alex@barton.de)
+ * Copyright (c)2001-2011 Alexander Barton (alex@barton.de) and Contributors.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -64,7 +64,8 @@ static bool Read_Config PARAMS(( bool ngircd_starting ));
 static bool Validate_Config PARAMS(( bool TestOnly, bool Rehash ));
 
 static void Handle_GLOBAL PARAMS(( int Line, char *Var, char *Arg ));
-static void Handle_FEATURES PARAMS(( int Line, char *Var, char *Arg ));
+static void Handle_LIMITS PARAMS(( int Line, char *Var, char *Arg ));
+static void Handle_OPTIONS PARAMS(( int Line, char *Var, char *Arg ));
 static void Handle_OPERATOR PARAMS(( int Line, char *Var, char *Arg ));
 static void Handle_SERVER PARAMS(( int Line, char *Var, char *Arg ));
 static void Handle_CHANNEL PARAMS(( int Line, char *Var, char *Arg ));
@@ -72,19 +73,27 @@ static void Handle_CHANNEL PARAMS(( int Line, char *Var, char *Arg ));
 static void Config_Error PARAMS(( const int Level, const char *Format, ... ));
 
 static void Config_Error_NaN PARAMS(( const int LINE, const char *Value ));
+static void Config_Error_Section PARAMS(( const int Line, const char *Item,
+                                         const char *Section ));
 static void Config_Error_TooLong PARAMS(( const int LINE, const char *Value ));
 
 static void Init_Server_Struct PARAMS(( CONF_SERVER *Server ));
 
+
 #ifdef WANT_IPV6
 #define DEFAULT_LISTEN_ADDRSTR "::,0.0.0.0"
 #else
 #define DEFAULT_LISTEN_ADDRSTR "0.0.0.0"
 #endif
 
+
 #ifdef SSL_SUPPORT
+
 struct SSLOptions Conf_SSLOptions;
 
+/**
+ * Initialize SSL configuration.
+ */
 static void
 ConfSSL_Init(void)
 {
@@ -99,59 +108,48 @@ ConfSSL_Init(void)
        array_free_wipe(&Conf_SSLOptions.KeyFilePassword);
 }
 
-static bool
-ssl_print_configvar(const char *name, const char *file)
+/**
+ * Make sure that a configured file is readable.
+ *
+ * Currently, this function is only used for SSL-related options ...
+ *
+ * @param Var Configuration variable
+ * @param Filename Configured filename
+ */
+static void
+CheckFileReadable(const char *Var, const char *Filename)
 {
        FILE *fp;
 
-       if (!file) {
-               printf("  %s =\n", name);
-               return true;
-       }
-
-       fp = fopen(file, "r");
+       fp = fopen(Filename, "r");
        if (fp)
                fclose(fp);
        else
-               fprintf(stderr, "ERROR: %s \"%s\": %s\n",
-                       name, file, strerror(errno));
-
-       printf("  %s = %s\n", name, file);
-       return fp != NULL;
+               Config_Error(LOG_ERR, "Can't read \"%s\" (\"%s\"): %s",
+                            Filename, Var, strerror(errno));
 }
 
-static bool
-ConfSSL_Puts(void)
-{
-       bool ret;
-
-       ret = ssl_print_configvar("SSLKeyFile", Conf_SSLOptions.KeyFile);
-
-       if (!ssl_print_configvar("SSLCertFile", Conf_SSLOptions.CertFile))
-               ret = false;
-
-       if (!ssl_print_configvar("SSLDHFile", Conf_SSLOptions.DHFile))
-               ret = false;
-
-       if (array_bytes(&Conf_SSLOptions.KeyFilePassword))
-               puts("  SSLKeyFilePassword = <secret>");
-
-       array_free_wipe(&Conf_SSLOptions.KeyFilePassword);
-
-       return ret;
-}
 #endif
 
+
+/**
+ * Duplicate string and warn on errors.
+ *
+ * @returns Pointer to string on success, NULL otherwise.
+ */
 static char *
 strdup_warn(const char *str)
 {
        char *ptr = strdup(str);
        if (!ptr)
-               Config_Error(LOG_ERR, "Could not allocate mem for string: %s", str);
+               Config_Error(LOG_ERR,
+                            "Could not allocate memory for string: %s", str);
        return ptr;
 }
 
-
+/**
+ * Output a comma separated list of ports (integer values).
+ */
 static void
 ports_puts(array *a)
 {
@@ -169,7 +167,9 @@ ports_puts(array *a)
        putc('\n', stdout);
 }
 
-
+/**
+ * Parse a comma separated string into an array of port numbers (integers).
+ */
 static void
 ports_parse(array *a, int Line, char *Arg)
 {
@@ -179,8 +179,6 @@ ports_parse(array *a, int Line, char *Arg)
 
        array_trunc(a);
 
-       /* Ports on that the server should listen. More port numbers
-        * must be separated by "," */
        ptr = strtok( Arg, "," );
        while (ptr) {
                ngt_TrimStr(ptr);
@@ -199,15 +197,21 @@ ports_parse(array *a, int Line, char *Arg)
        }
 }
 
-
+/**
+ * Initialize configuration module.
+ */
 GLOBAL void
 Conf_Init( void )
 {
        Read_Config( true );
        Validate_Config(false, false);
-} /* Config_Init */
-
+}
 
+/**
+ * "Rehash" (reload) server configuration.
+ *
+ * @returns true if configuration has been re-read, false on errors.
+ */
 GLOBAL bool
 Conf_Rehash( void )
 {
@@ -218,9 +222,11 @@ Conf_Rehash( void )
        /* Update CLIENT structure of local server */
        Client_SetInfo(Client_ThisServer(), Conf_ServerInfo);
        return true;
-} /* Config_Rehash */
-
+}
 
+/**
+ * Output a boolean value as "yes/no" string.
+ */
 static const char*
 yesno_to_str(int boolean_value)
 {
@@ -229,7 +235,9 @@ yesno_to_str(int boolean_value)
        return "no";
 }
 
-
+/**
+ * Free all IRC operator configuration structures.
+ */
 static void
 opers_free(void)
 {
@@ -245,6 +253,9 @@ opers_free(void)
        array_free(&Conf_Opers);
 }
 
+/**
+ * Output all IRC operator configuration structures.
+ */
 static void
 opers_puts(void)
 {
@@ -264,12 +275,18 @@ opers_puts(void)
        }
 }
 
-
+/**
+ * Read configuration, validate and output it.
+ *
+ * This function waits for a keypress of the user when stdin/stdout are valid
+ * tty's ("you can read our nice message and we can read in your keypress").
+ *
+ * @return     0 on succes, 1 on failure(s); therefore the result code can
+ *             directly be used by exit() when running "ngircd --configtest".
+ */
 GLOBAL int
 Conf_Test( void )
 {
-       /* Read configuration, validate and output it. */
-
        struct passwd *pwd;
        struct group *grp;
        unsigned int i;
@@ -284,25 +301,20 @@ Conf_Test( void )
 
        config_valid = Validate_Config(true, false);
 
-       /* If stdin and stdout ("you can read our nice message and we can
-        * read in your keypress") are valid tty's, wait for a key: */
-       if( isatty( fileno( stdin )) && isatty( fileno( stdout ))) {
-               puts( "OK, press enter to see a dump of your service configuration ..." );
-               getchar( );
-       } else {
-               puts( "Ok, dump of your server configuration follows:\n" );
-       }
+       /* Valid tty? */
+       if(isatty(fileno(stdin)) && isatty(fileno(stdout))) {
+               puts("OK, press enter to see a dump of your server configuration ...");
+               getchar();
+       } else
+               puts("Ok, dump of your server configuration follows:\n");
 
-       puts( "[GLOBAL]" );
+       puts("[GLOBAL]");
        printf("  Name = %s\n", Conf_ServerName);
-       printf("  Info = %s\n", Conf_ServerInfo);
-#ifndef PAM
-       printf("  Password = %s\n", Conf_ServerPwd);
-#endif
-       printf("  WebircPassword = %s\n", Conf_WebircPwd);
        printf("  AdminInfo1 = %s\n", Conf_ServerAdmin1);
        printf("  AdminInfo2 = %s\n", Conf_ServerAdmin2);
        printf("  AdminEMail = %s\n", Conf_ServerAdminMail);
+       printf("  Info = %s\n", Conf_ServerInfo);
+       printf("  Listen = %s\n", Conf_ListenAddress);
        if (Using_MotdFile) {
                printf("  MotdFile = %s\n", Conf_MotdFile);
                printf("  MotdPhrase =\n");
@@ -311,58 +323,75 @@ Conf_Test( void )
                printf("  MotdPhrase = %s\n", array_bytes(&Conf_Motd)
                       ? (const char*) array_start(&Conf_Motd) : "");
        }
-       printf("  ChrootDir = %s\n", Conf_Chroot);
+#ifndef PAM
+       printf("  Password = %s\n", Conf_ServerPwd);
+#endif
        printf("  PidFile = %s\n", Conf_PidFile);
-       printf("  Listen = %s\n", Conf_ListenAddress);
-       fputs("  Ports = ", stdout);
+       printf("  Ports = ");
        ports_puts(&Conf_ListenPorts);
-#ifdef SSL_SUPPORT
-       fputs("  SSLPorts = ", stdout);
-       ports_puts(&Conf_SSLOptions.ListenPorts);
-       if (!ConfSSL_Puts())
-               config_valid = false;
-#endif
-
-       pwd = getpwuid(Conf_UID);
-       if (pwd)
-               printf("  ServerUID = %s\n", pwd->pw_name);
-       else
-               printf("  ServerUID = %ld\n", (long)Conf_UID);
        grp = getgrgid(Conf_GID);
        if (grp)
                printf("  ServerGID = %s\n", grp->gr_name);
        else
                printf("  ServerGID = %ld\n", (long)Conf_GID);
-#ifdef SYSLOG
-       printf("  SyslogFacility = %s\n",
-              ngt_SyslogFacilityName(Conf_SyslogFacility));
-#endif
-       printf("  PingTimeout = %d\n", Conf_PingTimeout);
-       printf("  PongTimeout = %d\n", Conf_PongTimeout);
+       pwd = getpwuid(Conf_UID);
+       if (pwd)
+               printf("  ServerUID = %s\n", pwd->pw_name);
+       else
+               printf("  ServerUID = %ld\n", (long)Conf_UID);
+       puts("");
+
+       puts("[LIMITS]");
        printf("  ConnectRetry = %d\n", Conf_ConnectRetry);
-       printf("  OperCanUseMode = %s\n", yesno_to_str(Conf_OperCanMode));
-       printf("  OperServerMode = %s\n", yesno_to_str(Conf_OperServerMode));
-       printf("  AllowRemoteOper = %s\n", yesno_to_str(Conf_AllowRemoteOper));
-       printf("  PredefChannelsOnly = %s\n", yesno_to_str(Conf_PredefChannelsOnly));
-#ifdef WANT_IPV6
-       printf("  ConnectIPv4 = %s\n", yesno_to_str(Conf_ConnectIPv6));
-       printf("  ConnectIPv6 = %s\n", yesno_to_str(Conf_ConnectIPv4));
-#endif
        printf("  MaxConnections = %ld\n", Conf_MaxConnections);
        printf("  MaxConnectionsIP = %d\n", Conf_MaxConnectionsIP);
        printf("  MaxJoins = %d\n", Conf_MaxJoins > 0 ? Conf_MaxJoins : -1);
        printf("  MaxNickLength = %u\n", Conf_MaxNickLength - 1);
-       printf("  NoticeAuth = %s\n", yesno_to_str(Conf_NoticeAuth));
+       printf("  PingTimeout = %d\n", Conf_PingTimeout);
+       printf("  PongTimeout = %d\n", Conf_PongTimeout);
+       puts("");
+
+       puts("[OPTIONS]");
+       printf("  AllowRemoteOper = %s\n", yesno_to_str(Conf_AllowRemoteOper));
+       printf("  ChrootDir = %s\n", Conf_Chroot);
        printf("  CloakHost = %s\n", Conf_CloakHost);
        printf("  CloakUserToNick = %s\n", yesno_to_str(Conf_CloakUserToNick));
-#ifndef STRICT_RFC
-       printf("  RequireAuthPing = %s\n", yesno_to_str(Conf_AuthPing));
+#ifdef WANT_IPV6
+       printf("  ConnectIPv4 = %s\n", yesno_to_str(Conf_ConnectIPv6));
+       printf("  ConnectIPv6 = %s\n", yesno_to_str(Conf_ConnectIPv4));
 #endif
-
-       printf("\n[FEATURES]\n");
        printf("  DNS = %s\n", yesno_to_str(Conf_DNS));
+#ifdef IDENT
        printf("  Ident = %s\n", yesno_to_str(Conf_Ident));
+#endif
+       printf("  NoticeAuth = %s\n", yesno_to_str(Conf_NoticeAuth));
+       printf("  OperCanUseMode = %s\n", yesno_to_str(Conf_OperCanMode));
+       printf("  OperServerMode = %s\n", yesno_to_str(Conf_OperServerMode));
+#ifdef PAM
        printf("  PAM = %s\n", yesno_to_str(Conf_PAM));
+#endif
+       printf("  PredefChannelsOnly = %s\n", yesno_to_str(Conf_PredefChannelsOnly));
+#ifndef STRICT_RFC
+       printf("  RequireAuthPing = %s\n", yesno_to_str(Conf_AuthPing));
+#endif
+       printf("  ScrubCTCP = %s\n", yesno_to_str(Conf_ScrubCTCP));
+#ifdef SSL_SUPPORT
+       printf("  SSLCertFile = %s\n", Conf_SSLOptions.CertFile);
+       printf("  SSLDHFile = %s\n", Conf_SSLOptions.DHFile);
+       printf("  SSLKeyFile = %s\n", Conf_SSLOptions.KeyFile);
+       if (array_bytes(&Conf_SSLOptions.KeyFilePassword))
+               puts("  SSLKeyFilePassword = <secret>");
+       else
+               puts("  SSLKeyFilePassword = ");
+       array_free_wipe(&Conf_SSLOptions.KeyFilePassword);
+       printf("  SSLPorts = ");
+       ports_puts(&Conf_SSLOptions.ListenPorts);
+#endif
+#ifdef SYSLOG
+       printf("  SyslogFacility = %s\n",
+              ngt_SyslogFacilityName(Conf_SyslogFacility));
+#endif
+       printf("  WebircPassword = %s\n", Conf_WebircPwd);
        puts("");
 
        opers_puts();
@@ -403,17 +432,19 @@ Conf_Test( void )
        }
 
        return (config_valid ? 0 : 1);
-} /* Conf_Test */
-
+}
 
+/**
+ * Remove connection information from configured server.
+ *
+ * If the server is set as "once", delete it from our configuration;
+ * otherwise set the time for the next connection attempt.
+ *
+ * Non-server connections will be silently ignored.
+ */
 GLOBAL void
 Conf_UnsetServer( CONN_ID Idx )
 {
-       /* Set next time for next connection attempt, if this is a server
-        * link that is (still) configured here. If the server is set as
-        * "once", delete it from our configuration.
-        * Non-Server-Connections will be silently ignored. */
-
        int i;
        time_t t;
 
@@ -439,26 +470,26 @@ Conf_UnsetServer( CONN_ID Idx )
                                Conf_Server[i].lasttry = t;
                }
        }
-} /* Conf_UnsetServer */
-
+}
 
+/**
+ * Set connection information for specified configured server.
+ */
 GLOBAL void
 Conf_SetServer( int ConfServer, CONN_ID Idx )
 {
-       /* Set connection for specified configured server */
-
        assert( ConfServer > NONE );
        assert( Idx > NONE );
 
        Conf_Server[ConfServer].conn_id = Idx;
-} /* Conf_SetServer */
-
+}
 
+/**
+ * Get index of server in configuration structure.
+ */
 GLOBAL int
 Conf_GetServer( CONN_ID Idx )
 {
-       /* Get index of server in configuration structure */
-
        int i = 0;
 
        assert( Idx > NONE );
@@ -467,18 +498,20 @@ Conf_GetServer( CONN_ID Idx )
                if( Conf_Server[i].conn_id == Idx ) return i;
        }
        return NONE;
-} /* Conf_GetServer */
-
+}
 
+/**
+ * Enable a server by name and adjust its port number.
+ *
+ * @returns    true if a server has been enabled and now has a valid port
+ *             number and host name for outgoing connections.
+ */
 GLOBAL bool
 Conf_EnableServer( const char *Name, UINT16 Port )
 {
-       /* Enable specified server and adjust port */
-
        int i;
 
        assert( Name != NULL );
-
        for( i = 0; i < MAX_SERVERS; i++ ) {
                if( strcasecmp( Conf_Server[i].name, Name ) == 0 ) {
                        /* Gotcha! Set port and enable server: */
@@ -488,53 +521,74 @@ Conf_EnableServer( const char *Name, UINT16 Port )
                }
        }
        return false;
-} /* Conf_EnableServer */
-
+}
 
+/**
+ * Enable a server by name.
+ *
+ * The server is only usable as outgoing server, if it has set a valid port
+ * number for outgoing connections!
+ * If not, you have to use Conf_EnableServer() function to make it available.
+ *
+ * @returns    true if a server has been enabled; false otherwise.
+ */
 GLOBAL bool
 Conf_EnablePassiveServer(const char *Name)
 {
-       /* Enable specified server */
        int i;
 
        assert( Name != NULL );
        for (i = 0; i < MAX_SERVERS; i++) {
-               if ((strcasecmp( Conf_Server[i].name, Name ) == 0) && (Conf_Server[i].port > 0)) {
+               if ((strcasecmp( Conf_Server[i].name, Name ) == 0)
+                   && (Conf_Server[i].port > 0)) {
                        /* BINGO! Enable server */
                        Conf_Server[i].flags &= ~CONF_SFLAG_DISABLED;
                        return true;
                }
        }
        return false;
-} /* Conf_EnablePassiveServer */
-
+}
 
+/**
+ * Disable a server by name.
+ * An already established connection will be disconnected.
+ *
+ * @returns    true if a server was found and has been disabled.
+ */
 GLOBAL bool
 Conf_DisableServer( const char *Name )
 {
-       /* Enable specified server and adjust port */
-
        int i;
 
        assert( Name != NULL );
-
        for( i = 0; i < MAX_SERVERS; i++ ) {
                if( strcasecmp( Conf_Server[i].name, Name ) == 0 ) {
                        /* Gotcha! Disable and disconnect server: */
                        Conf_Server[i].flags |= CONF_SFLAG_DISABLED;
-                       if( Conf_Server[i].conn_id > NONE ) Conn_Close( Conf_Server[i].conn_id, NULL, "Server link terminated on operator request", true);
+                       if( Conf_Server[i].conn_id > NONE )
+                               Conn_Close(Conf_Server[i].conn_id, NULL,
+                                          "Server link terminated on operator request",
+                                          true);
                        return true;
                }
        }
        return false;
-} /* Conf_DisableServer */
-
+}
 
+/**
+ * Add a new remote server to our configuration.
+ *
+ * @param Name         Name of the new server.
+ * @param Port         Port number to connect to or 0 for incoming connections.
+ * @param Host         Host name to connect to.
+ * @param MyPwd                Password that will be sent to the peer.
+ * @param PeerPwd      Password that must be received from the peer.
+ * @returns            true if the new server has been added; false otherwise.
+ */
 GLOBAL bool
-Conf_AddServer( const char *Name, UINT16 Port, const char *Host, const char *MyPwd, const char *PeerPwd )
+Conf_AddServer(const char *Name, UINT16 Port, const char *Host,
+              const char *MyPwd, const char *PeerPwd)
 {
-       /* Add new server to configuration */
-
        int i;
 
        assert( Name != NULL );
@@ -558,35 +612,19 @@ Conf_AddServer( const char *Name, UINT16 Port, const char *Host, const char *MyP
        Conf_Server[i].flags = CONF_SFLAG_ONCE;
 
        return true;
-} /* Conf_AddServer */
-
+}
 
 /**
- * Check if the given nick name is an service
+ * Check if the given nick name is an service.
+ *
+ * @returns true if the given nick name belongs to an "IRC service".
  */
 GLOBAL bool
 Conf_IsService(int ConfServer, const char *Nick)
 {
        return MatchCaseInsensitive(Conf_Server[ConfServer].svs_mask, Nick);
-} /* Conf_IsService */
-
-
-static void
-Set_Defaults_Optional(void)
-{
-#ifdef IDENTAUTH
-       Conf_Ident = true;
-#else
-       Conf_Ident = false;
-#endif
-#ifdef PAM
-       Conf_PAM = true;
-#else
-       Conf_PAM = false;
-#endif
 }
 
-
 /**
  * Initialize configuration settings with their default values.
  */
@@ -595,51 +633,62 @@ Set_Defaults(bool InitServers)
 {
        int i;
 
+       /* Global */
        strcpy(Conf_ServerName, "");
-       snprintf(Conf_ServerInfo, sizeof Conf_ServerInfo, "%s %s",
-                PACKAGE_NAME, PACKAGE_VERSION);
-       strcpy(Conf_ServerPwd, "");
-
        strcpy(Conf_ServerAdmin1, "");
        strcpy(Conf_ServerAdmin2, "");
        strcpy(Conf_ServerAdminMail, "");
-
+       snprintf(Conf_ServerInfo, sizeof Conf_ServerInfo, "%s %s",
+                PACKAGE_NAME, PACKAGE_VERSION);
+       free(Conf_ListenAddress);
+       Conf_ListenAddress = NULL;
+       array_free(&Conf_Motd);
        strlcpy(Conf_MotdFile, SYSCONFDIR, sizeof(Conf_MotdFile));
        strlcat(Conf_MotdFile, MOTD_FILE, sizeof(Conf_MotdFile));
-
-       Conf_UID = Conf_GID = 0;
-       strlcpy(Conf_Chroot, CHROOT_DIR, sizeof(Conf_Chroot));
+       strcpy(Conf_ServerPwd, "");
        strlcpy(Conf_PidFile, PID_FILE, sizeof(Conf_PidFile));
+       Conf_UID = Conf_GID = 0;
 
-       free(Conf_ListenAddress);
-       Conf_ListenAddress = NULL;
-
-       Conf_PingTimeout = 120;
-       Conf_PongTimeout = 20;
+       /* Limits */
        Conf_ConnectRetry = 60;
-       Conf_DNS = true;
-       Conf_NoticeAuth = false;
-
-       Conf_Oper_Count = 0;
-       Conf_Channel_Count = 0;
-
-       Conf_OperCanMode = false;
-       Conf_OperServerMode = false;
-       Conf_AllowRemoteOper = false;
-       Conf_PredefChannelsOnly = false;
-
-       Conf_ConnectIPv4 = true;
-       Conf_ConnectIPv6 = true;
-
        Conf_MaxConnections = 0;
        Conf_MaxConnectionsIP = 5;
        Conf_MaxJoins = 10;
        Conf_MaxNickLength = CLIENT_NICK_LEN_DEFAULT;
+       Conf_PingTimeout = 120;
+       Conf_PongTimeout = 20;
 
+       /* Options */
+       Conf_AllowRemoteOper = false;
+#ifndef STRICT_RFC
+       Conf_AuthPing = false;
+#endif
+       strlcpy(Conf_Chroot, CHROOT_DIR, sizeof(Conf_Chroot));
        strcpy(Conf_CloakHost, "");
        Conf_CloakUserToNick = false;
-
+       Conf_ConnectIPv4 = true;
+#ifdef WANT_IPV6
+       Conf_ConnectIPv6 = true;
+#else
+       Conf_ConnectIPv6 = false;
+#endif
+       Conf_DNS = true;
+#ifdef IDENTAUTH
+       Conf_Ident = true;
+#else
+       Conf_Ident = false;
+#endif
+       Conf_NoticeAuth = false;
+       Conf_OperCanMode = false;
+       Conf_OperServerMode = false;
+#ifdef PAM
+       Conf_PAM = true;
+#else
+       Conf_PAM = false;
+#endif
+       Conf_PredefChannelsOnly = false;
 #ifdef SYSLOG
+       Conf_ScrubCTCP = false;
 #ifdef LOG_LOCAL5
        Conf_SyslogFacility = LOG_LOCAL5;
 #else
@@ -647,23 +696,22 @@ Set_Defaults(bool InitServers)
 #endif
 #endif
 
-#ifndef STRICT_RFC
-       Conf_AuthPing = false;
-#endif
-
-       Set_Defaults_Optional();
+       /* Initialize IRC operators and channels */
+       Conf_Oper_Count = 0;
+       Conf_Channel_Count = 0;
 
        /* Initialize server configuration structures */
        if (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 */
-
-
+/**
+ * Get number of configured listening ports.
+ *
+ * @returns The number of ports (IPv4+IPv6) on which the server should listen.
+ */
 static bool
 no_listenports(void)
 {
@@ -674,6 +722,11 @@ no_listenports(void)
        return cnt == 0;
 }
 
+/**
+ * Read MOTD ("message of the day") file.
+ *
+ * @param filename     Name of the file to read.
+ */
 static void
 Read_Motd(const char *filename)
 {
@@ -705,11 +758,19 @@ Read_Motd(const char *filename)
        fclose(fp);
 }
 
+/**
+ * Read ngIRCd configuration file.
+ *
+ * Please note that this function uses exit(1) on fatal errors and therefore
+ * can result in ngIRCd terminating!
+ *
+ * @param ngircd_starting      Flag indicating if ngIRCd is starting or not.
+ * @returns                    true when the configuration file has been read
+ *                             successfully; false otherwise.
+ */
 static bool
 Read_Config( bool ngircd_starting )
 {
-       /* Read configuration file. */
-
        char section[LINE_LEN], str[LINE_LEN], *var, *arg, *ptr;
        const UINT16 defaultport = 6667;
        int line, i, n;
@@ -783,8 +844,9 @@ Read_Config( bool ngircd_starting )
                /* Is this the beginning of a new section? */
                if(( str[0] == '[' ) && ( str[strlen( str ) - 1] == ']' )) {
                        strlcpy( section, str, sizeof( section ));
-                       if (strcasecmp( section, "[GLOBAL]" ) == 0 ||
-                           strcasecmp( section, "[FEATURES]") == 0)
+                       if (strcasecmp(section, "[GLOBAL]") == 0 ||
+                           strcasecmp(section, "[LIMITS]") == 0 ||
+                           strcasecmp(section, "[OPTIONS]") == 0)
                                continue;
 
                        if( strcasecmp( section, "[SERVER]" ) == 0 ) {
@@ -820,7 +882,9 @@ Read_Config( bool ngircd_starting )
                                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;
                }
                if( section[0] == 0x1 ) continue;
@@ -835,12 +899,22 @@ Read_Config( bool ngircd_starting )
                var = str; ngt_TrimStr( var );
                arg = ptr + 1; ngt_TrimStr( arg );
 
-               if( strcasecmp( section, "[GLOBAL]" ) == 0 ) Handle_GLOBAL( line, var, arg );
-               else if( strcasecmp( section, "[FEATURES]" ) == 0 ) Handle_FEATURES( line, var, arg );
-               else if( strcasecmp( section, "[OPERATOR]" ) == 0 ) Handle_OPERATOR( line, var, arg );
-               else if( strcasecmp( section, "[SERVER]" ) == 0 ) Handle_SERVER( line, var, arg );
-               else if( strcasecmp( section, "[CHANNEL]" ) == 0 ) Handle_CHANNEL( line, var, arg );
-               else Config_Error( LOG_ERR, "%s, line %d: Variable \"%s\" outside section!", NGIRCd_ConfFile, line, var );
+               if(strcasecmp(section, "[GLOBAL]") == 0)
+                       Handle_GLOBAL(line, var, arg);
+               else if(strcasecmp(section, "[LIMITS]") == 0)
+                       Handle_LIMITS(line, var, arg);
+               else if(strcasecmp(section, "[OPTIONS]") == 0)
+                       Handle_OPTIONS(line, var, arg);
+               else if(strcasecmp(section, "[OPERATOR]") == 0)
+                       Handle_OPERATOR(line, var, arg);
+               else if(strcasecmp(section, "[SERVER]") == 0)
+                       Handle_SERVER(line, var, arg);
+               else if(strcasecmp(section, "[CHANNEL]") == 0)
+                       Handle_CHANNEL(line, var, arg);
+               else
+                       Config_Error(LOG_ERR,
+                                    "%s, line %d: Variable \"%s\" outside section!",
+                                    NGIRCd_ConfFile, line, var);
        }
 
        /* Close configuration file */
@@ -874,10 +948,23 @@ Read_Config( bool ngircd_starting )
        /* No MOTD phrase configured? (re)try motd file. */
        if (array_bytes(&Conf_Motd) == 0)
                Read_Motd(Conf_MotdFile);
-       return true;
-} /* Read_Config */
 
+#ifdef SSL_SUPPORT
+       /* Make sure that all SSL-related files are readable */
+       CheckFileReadable("SSLCertFile", Conf_SSLOptions.CertFile);
+       CheckFileReadable("SSLDHFile", Conf_SSLOptions.DHFile);
+       CheckFileReadable("SSLKeyFile", Conf_SSLOptions.KeyFile);
+#endif
 
+       return true;
+}
+
+/**
+ * Check whether an string argument is true or false.
+ *
+ * @param Arg  Input string.
+ * @returns    true if string has been parsed as "yes"/"true"/"on".
+ */
 static bool
 Check_ArgIsTrue( const char *Arg )
 {
@@ -886,9 +973,15 @@ Check_ArgIsTrue( const char *Arg )
        if( atoi( Arg ) != 0 ) return true;
 
        return false;
-} /* Check_ArgIsTrue */
-
+}
 
+/**
+ * Handle setting of "MaxNickLength".
+ *
+ * @param Line Line number in configuration file.
+ * @raram Arg  Input string.
+ * @returns    New configured maximum nick name length.
+ */
 static unsigned int
 Handle_MaxNickLength(int Line, const char *Arg)
 {
@@ -908,9 +1001,11 @@ Handle_MaxNickLength(int Line, const char *Arg)
                return 2;
        }
        return new;
-} /* Handle_MaxNickLength */
-
+}
 
+/**
+ * Output a warning messages if IDENT is configured but not compiled in.
+ */
 static void
 WarnIdent(int UNUSED Line)
 {
@@ -918,28 +1013,57 @@ WarnIdent(int UNUSED Line)
        if (Conf_Ident) {
                /* user has enabled ident lookups explicitly, but ... */
                Config_Error(LOG_WARNING,
-                       "%s: line %d: %s=True, but ngircd was built without support",
-                       NGIRCd_ConfFile, Line, "Ident");
+                       "%s: line %d: \"Ident = yes\", but ngircd was built without IDENT support!",
+                       NGIRCd_ConfFile, Line);
        }
 #endif
 }
 
+/**
+ * Output a warning messages if IPv6 is configured but not compiled in.
+ */
+static void
+WarnIPv6(int UNUSED Line)
+{
+#ifndef WANT_IPV6
+       if (Conf_ConnectIPv6) {
+               /* user has enabled IPv6 explicitly, but ... */
+               Config_Error(LOG_WARNING,
+                       "%s: line %d: \"ConnectIPv6 = yes\", but ngircd was built without IPv6 support!",
+                       NGIRCd_ConfFile, Line);
+       }
+#endif
+}
+
+/**
+ * Output a warning messages if PAM is configured but not compiled in.
+ */
 static void
 WarnPAM(int UNUSED Line)
 {
 #ifndef PAM
        if (Conf_PAM) {
                Config_Error(LOG_WARNING,
-                       "%s: line %d: %s=True, but ngircd was built without support",
-                       NGIRCd_ConfFile, Line, "PAM");
+                       "%s: line %d: \"PAM = yes\", but ngircd was built without PAM support!",
+                       NGIRCd_ConfFile, Line);
        }
 #endif
 }
 
+/**
+ * Handle legacy "NoXXX" options in [GLOBAL] section.
+ *
+ * TODO: This function and support for "NoXXX" could be removed starting
+ * with ngIRCd release 19 (one release after marking it "deprecated").
+ *
+ * @param Var  Variable name.
+ * @param Arg  Argument string.
+ * @returns    true if a NoXXX option has been processed; false otherwise.
+ */
 static bool
 CheckLegacyNoOption(const char *Var, const char *Arg)
 {
-       if( strcasecmp( Var, "NoDNS" ) == 0 ) {
+       if(strcasecmp(Var, "NoDNS") == 0) {
                Conf_DNS = !Check_ArgIsTrue( Arg );
                return true;
        }
@@ -954,6 +1078,62 @@ CheckLegacyNoOption(const char *Var, const char *Arg)
        return false;
 }
 
+/**
+ * Handle deprecated legacy options in [GLOBAL] section.
+ *
+ * TODO: This function and support for these options in the [Global] section
+ * could be removed starting with ngIRCd release 19 (one release after
+ * marking it "deprecated").
+ *
+ * @param Var  Variable name.
+ * @param Arg  Argument string.
+ * @returns    true if a legacy option has been processed; false otherwise.
+ */
+static const char*
+CheckLegacyGlobalOption(int Line, char *Var, char *Arg)
+{
+       if (strcasecmp(Var, "AllowRemoteOper") == 0
+           || strcasecmp(Var, "ChrootDir") == 0
+           || strcasecmp(Var, "ConnectIPv4") == 0
+           || strcasecmp(Var, "ConnectIPv6") == 0
+           || strcasecmp(Var, "OperCanUseMode") == 0
+           || strcasecmp(Var, "OperServerMode") == 0
+           || strcasecmp(Var, "PredefChannelsOnly") == 0
+#ifdef SSL_SUPPORT
+           || strcasecmp(Var, "SSLCertFile") == 0
+           || strcasecmp(Var, "SSLDHFile") == 0
+           || strcasecmp(Var, "SSLKeyFile") == 0
+           || strcasecmp(Var, "SSLKeyFilePassword") == 0
+           || strcasecmp(Var, "SSLPorts") == 0
+#endif
+           || strcasecmp(Var, "SyslogFacility") == 0
+           || strcasecmp(Var, "WebircPassword") == 0) {
+               Handle_OPTIONS(Line, Var, Arg);
+               return "[Options]";
+       }
+       if (strcasecmp(Var, "ConnectRetry") == 0
+           || strcasecmp(Var, "MaxConnections") == 0
+           || strcasecmp(Var, "MaxConnectionsIP") == 0
+           || strcasecmp(Var, "MaxJoins") == 0
+           || strcasecmp(Var, "MaxNickLength") == 0
+           || strcasecmp(Var, "PingTimeout") == 0
+           || strcasecmp(Var, "PongTimeout") == 0) {
+               Handle_LIMITS(Line, Var, Arg);
+               return "[Limits]";
+       }
+
+       return NULL;
+}
+
+/**
+ * Strip "no" prefix of a string.
+ *
+ * TODO: This function and support for "NoXXX" should be removed starting
+ * with ngIRCd release 19! (One release after marking it "deprecated").
+ *
+ * @param str  Pointer to input string starting with "no".
+ * @returns    New pointer to string without "no" prefix.
+ */
 static const char *
 NoNo(const char *str)
 {
@@ -961,351 +1141,384 @@ NoNo(const char *str)
        return str + 2;
 }
 
+/**
+ * Invert "boolean" string.
+ *
+ * TODO: This function and support for "NoXXX" should be removed starting
+ * with ngIRCd release 19! (One release after marking it "deprecated").
+ *
+ * @param arg  "Boolean" input string.
+ * @returns    Pointer to inverted "boolean string".
+ */
 static const char *
 InvertArg(const char *arg)
 {
        return yesno_to_str(!Check_ArgIsTrue(arg));
 }
 
+/**
+ * Handle variable in [Global] configuration section.
+ *
+ * @param Line Line numer in configuration file.
+ * @param Var  Variable name.
+ * @param Arg  Variable argument.
+ */
 static void
 Handle_GLOBAL( int Line, char *Var, char *Arg )
 {
        struct passwd *pwd;
        struct group *grp;
        size_t len;
-       
-       assert( Line > 0 );
-       assert( Var != NULL );
-       assert( Arg != NULL );
-       
-       if( strcasecmp( Var, "Name" ) == 0 ) {
-               /* Server name */
-               len = strlcpy( Conf_ServerName, Arg, sizeof( Conf_ServerName ));
-               if (len >= sizeof( Conf_ServerName ))
-                       Config_Error_TooLong( Line, Var );
-               return;
-       }
-       if( strcasecmp( Var, "CloakHost" ) == 0 ) {
-               /* Client hostname */
-               len = strlcpy( Conf_CloakHost, Arg, sizeof( Conf_CloakHost ));
-               if (len >= sizeof( Conf_CloakHost ))
-                       Config_Error_TooLong( Line, Var );
-               return;
-       }
-       if( strcasecmp( Var, "CloakUserToNick" ) == 0 ) {
-               /* Use client nick name as user name */
-               Conf_CloakUserToNick = Check_ArgIsTrue( Arg );
-               return;
-       }
-       if( strcasecmp( Var, "Info" ) == 0 ) {
-               /* Info text of server */
-               len = strlcpy( Conf_ServerInfo, Arg, sizeof( Conf_ServerInfo ));
-               if (len >= sizeof( Conf_ServerInfo ))
-                       Config_Error_TooLong ( Line, Var );
-               return;
-       }
-       if( strcasecmp( Var, "Password" ) == 0 ) {
-               /* Global server password */
-               len = strlcpy( Conf_ServerPwd, Arg, sizeof( Conf_ServerPwd ));
-               if (len >= sizeof( Conf_ServerPwd ))
-                       Config_Error_TooLong( Line, Var );
+       const char *section;
+
+       assert(Line > 0);
+       assert(Var != NULL);
+       assert(Arg != NULL);
+
+       if (strcasecmp(Var, "Name") == 0) {
+               len = strlcpy(Conf_ServerName, Arg, sizeof(Conf_ServerName));
+               if (len >= sizeof(Conf_ServerName))
+                       Config_Error_TooLong(Line, Var);
                return;
        }
-       if (strcasecmp(Var, "WebircPassword") == 0) {
-               /* Password required for WEBIRC command */
-               len = strlcpy(Conf_WebircPwd, Arg, sizeof(Conf_WebircPwd));
-               if (len >= sizeof(Conf_WebircPwd))
+       if (strcasecmp(Var, "AdminInfo1") == 0) {
+               len = strlcpy(Conf_ServerAdmin1, Arg, sizeof(Conf_ServerAdmin1));
+               if (len >= sizeof(Conf_ServerAdmin1))
                        Config_Error_TooLong(Line, Var);
                return;
        }
-       if( strcasecmp( Var, "AdminInfo1" ) == 0 ) {
-               /* Administrative info #1 */
-               len = strlcpy( Conf_ServerAdmin1, Arg, sizeof( Conf_ServerAdmin1 ));
-               if (len >= sizeof( Conf_ServerAdmin1 ))
-                       Config_Error_TooLong ( Line, Var );
+       if (strcasecmp(Var, "AdminInfo2") == 0) {
+               len = strlcpy(Conf_ServerAdmin2, Arg, sizeof(Conf_ServerAdmin2));
+               if (len >= sizeof(Conf_ServerAdmin2))
+                       Config_Error_TooLong(Line, Var);
                return;
        }
-       if( strcasecmp( Var, "AdminInfo2" ) == 0 ) {
-               /* Administrative info #2 */
-               len = strlcpy( Conf_ServerAdmin2, Arg, sizeof( Conf_ServerAdmin2 ));
-               if (len >= sizeof( Conf_ServerAdmin2 ))
-                       Config_Error_TooLong ( Line, Var );
+       if (strcasecmp(Var, "AdminEMail") == 0) {
+               len = strlcpy(Conf_ServerAdminMail, Arg,
+                       sizeof(Conf_ServerAdminMail));
+               if (len >= sizeof(Conf_ServerAdminMail))
+                       Config_Error_TooLong(Line, Var);
                return;
        }
-       if( strcasecmp( Var, "AdminEMail" ) == 0 ) {
-               /* Administrative email contact */
-               len = strlcpy( Conf_ServerAdminMail, Arg, sizeof( Conf_ServerAdminMail ));
-               if (len >= sizeof( Conf_ServerAdminMail ))
-                       Config_Error_TooLong( Line, Var );
+       if (strcasecmp(Var, "Info") == 0) {
+               len = strlcpy(Conf_ServerInfo, Arg, sizeof(Conf_ServerInfo));
+               if (len >= sizeof(Conf_ServerInfo))
+                       Config_Error_TooLong(Line, Var);
                return;
        }
-
-       if( strcasecmp( Var, "Ports" ) == 0 ) {
-               ports_parse(&Conf_ListenPorts, Line, Arg);
+       if (strcasecmp(Var, "Listen") == 0) {
+               if (Conf_ListenAddress) {
+                       Config_Error(LOG_ERR,
+                                    "Multiple Listen= options, ignoring: %s",
+                                    Arg);
+                       return;
+               }
+               Conf_ListenAddress = strdup_warn(Arg);
+               /* If allocation fails, we're in trouble: we cannot ignore the
+                * error -- otherwise ngircd would listen on all interfaces. */
+               if (!Conf_ListenAddress) {
+                       Config_Error(LOG_ALERT,
+                                    "%s exiting due to fatal errors!",
+                                    PACKAGE_NAME);
+                       exit(1);
+               }
                return;
        }
-       if( strcasecmp( Var, "MotdFile" ) == 0 ) {
-               len = strlcpy( Conf_MotdFile, Arg, sizeof( Conf_MotdFile ));
-               if (len >= sizeof( Conf_MotdFile ))
-                       Config_Error_TooLong( Line, Var );
+       if (strcasecmp(Var, "MotdFile") == 0) {
+               len = strlcpy(Conf_MotdFile, Arg, sizeof(Conf_MotdFile));
+               if (len >= sizeof(Conf_MotdFile))
+                       Config_Error_TooLong(Line, Var);
                return;
        }
-       if( strcasecmp( Var, "MotdPhrase" ) == 0 ) {
-               /* "Message of the day" phrase (instead of file) */
+       if (strcasecmp(Var, "MotdPhrase") == 0) {
                len = strlen(Arg);
                if (len == 0)
                        return;
                if (len >= LINE_LEN) {
-                       Config_Error_TooLong( Line, Var );
+                       Config_Error_TooLong(Line, Var);
                        return;
                }
                if (!array_copyb(&Conf_Motd, Arg, len + 1))
-                       Config_Error(LOG_WARNING, "%s, line %d: Could not append MotdPhrase: %s",
-                                                       NGIRCd_ConfFile, Line, strerror(errno));
+                       Config_Error(LOG_WARNING,
+                                    "%s, line %d: Could not append MotdPhrase: %s",
+                                    NGIRCd_ConfFile, Line, strerror(errno));
                Using_MotdFile = false;
                return;
        }
-       if( strcasecmp( Var, "ChrootDir" ) == 0 ) {
-               /* directory for chroot() */
-               len = strlcpy( Conf_Chroot, Arg, sizeof( Conf_Chroot ));
-               if (len >= sizeof( Conf_Chroot ))
-                       Config_Error_TooLong( Line, Var );
+       if(strcasecmp(Var, "Password") == 0) {
+               len = strlcpy(Conf_ServerPwd, Arg, sizeof(Conf_ServerPwd));
+               if (len >= sizeof(Conf_ServerPwd))
+                       Config_Error_TooLong(Line, Var);
                return;
        }
-       if ( strcasecmp( Var, "PidFile" ) == 0 ) {
-               /* name of pidfile */
-               len = strlcpy( Conf_PidFile, Arg, sizeof( Conf_PidFile ));
-               if (len >= sizeof( Conf_PidFile ))
-                       Config_Error_TooLong( Line, Var );
+       if (strcasecmp(Var, "PidFile") == 0) {
+               len = strlcpy(Conf_PidFile, Arg, sizeof(Conf_PidFile));
+               if (len >= sizeof(Conf_PidFile))
+                       Config_Error_TooLong(Line, Var);
                return;
        }
-       if( strcasecmp( Var, "ServerUID" ) == 0 ) {
-               /* UID the daemon should switch to */
-               pwd = getpwnam( Arg );
-               if( pwd ) Conf_UID = pwd->pw_uid;
-               else {
-                       Conf_UID = (unsigned int)atoi( Arg );
-                       if (!Conf_UID && strcmp(Arg, "0"))
-                               Config_Error_NaN(Line, Var);
-               }
+       if (strcasecmp(Var, "Ports") == 0) {
+               ports_parse(&Conf_ListenPorts, Line, Arg);
                return;
        }
-       if( strcasecmp( Var, "ServerGID" ) == 0 ) {
-               /* GID the daemon should use */
-               grp = getgrnam( Arg );
-               if( grp ) Conf_GID = grp->gr_gid;
+       if (strcasecmp(Var, "ServerGID") == 0) {
+               grp = getgrnam(Arg);
+               if (grp)
+                       Conf_GID = grp->gr_gid;
                else {
                        Conf_GID = (unsigned int)atoi(Arg);
                        if (!Conf_GID && strcmp(Arg, "0"))
-                               Config_Error_NaN( Line, Var );
-               }
-               return;
-       }
-       if( strcasecmp( Var, "PingTimeout" ) == 0 ) {
-               /* PING timeout */
-               Conf_PingTimeout = atoi( Arg );
-               if( Conf_PingTimeout < 5 ) {
-                       Config_Error( LOG_WARNING, "%s, line %d: Value of \"PingTimeout\" too low!",
-                                                                       NGIRCd_ConfFile, Line );
-                       Conf_PingTimeout = 5;
-               }
-               return;
-       }
-       if( strcasecmp( Var, "PongTimeout" ) == 0 ) {
-               /* PONG timeout */
-               Conf_PongTimeout = atoi( Arg );
-               if( Conf_PongTimeout < 5 ) {
-                       Config_Error( LOG_WARNING, "%s, line %d: Value of \"PongTimeout\" too low!",
-                                                                       NGIRCd_ConfFile, Line );
-                       Conf_PongTimeout = 5;
+                               Config_Error_NaN(Line, Var);
                }
                return;
        }
-       if( strcasecmp( Var, "ConnectRetry" ) == 0 ) {
-               /* Seconds between connection attempts to other servers */
-               Conf_ConnectRetry = atoi( Arg );
-               if( Conf_ConnectRetry < 5 ) {
-                       Config_Error( LOG_WARNING, "%s, line %d: Value of \"ConnectRetry\" too low!",
-                                                                       NGIRCd_ConfFile, Line );
-                       Conf_ConnectRetry = 5;
+       if (strcasecmp(Var, "ServerUID") == 0) {
+               pwd = getpwnam(Arg);
+               if (pwd)
+                       Conf_UID = pwd->pw_uid;
+               else {
+                       Conf_UID = (unsigned int)atoi(Arg);
+                       if (!Conf_UID && strcmp(Arg, "0"))
+                               Config_Error_NaN(Line, Var);
                }
                return;
        }
-       if( strcasecmp( Var, "PredefChannelsOnly" ) == 0 ) {
-               /* Should we only allow pre-defined-channels? (i.e. users cannot create their own channels) */
-               Conf_PredefChannelsOnly = Check_ArgIsTrue( Arg );
-               return;
-       }
 
        if (CheckLegacyNoOption(Var, Arg)) {
-               Config_Error(LOG_WARNING, "%s, line %d: \"No\"-Prefix has been removed, use \"%s = %s\" in [FEATURES] section instead",
-                                       NGIRCd_ConfFile, Line, NoNo(Var), InvertArg(Arg));
+               /* TODO: This function and support for "NoXXX" could be
+                * be removed starting with ngIRCd release 19 (one release
+                * after marking it "deprecated"). */
+               Config_Error(LOG_WARNING,
+                            "%s, line %d (section \"Global\"): \"No\"-Prefix is deprecated, use \"%s = %s\" in [Options] section!",
+                            NGIRCd_ConfFile, Line, NoNo(Var), InvertArg(Arg));
                if (strcasecmp(Var, "NoIdent") == 0)
                        WarnIdent(Line);
                else if (strcasecmp(Var, "NoPam") == 0)
                        WarnPAM(Line);
                return;
        }
-#ifdef WANT_IPV6
-       /* the default setting for all the WANT_IPV6 special options is 'true' */
-       if( strcasecmp( Var, "ConnectIPv6" ) == 0 ) {
-               /* connect to other hosts using ipv6, if they have an AAAA record? */
-               Conf_ConnectIPv6 = Check_ArgIsTrue( Arg );
-               return;
-       }
-       if( strcasecmp( Var, "ConnectIPv4" ) == 0 ) {
-               /* connect to other hosts using ipv4.
-                * again, this can be used for ipv6-only setups */
-               Conf_ConnectIPv4 = Check_ArgIsTrue( Arg );
-               return;
-       }
-#endif
-       if( strcasecmp( Var, "OperCanUseMode" ) == 0 ) {
-               /* Are IRC operators allowed to use MODE in channels they aren't Op in? */
-               Conf_OperCanMode = Check_ArgIsTrue( Arg );
-               return;
-       }
-       if( strcasecmp( Var, "OperServerMode" ) == 0 ) {
-               /* Mask IRC operator as if coming from the server? (ircd-irc2 compat hack) */
-               Conf_OperServerMode = Check_ArgIsTrue( Arg );
+       if ((section = CheckLegacyGlobalOption(Line, Var, Arg))) {
+               /** TODO: This function and support for these options in the
+                * [Global] section could be removed starting with ngIRCd
+                * release 19 (one release after marking it "deprecated"). */
+               Config_Error(LOG_WARNING,
+                            "%s, line %d (section \"Global\"): \"%s\" is deprecated here, move it to %s!",
+                            NGIRCd_ConfFile, Line, Var, section);
                return;
        }
-       if(strcasecmp(Var, "AllowRemoteOper") == 0) {
-               /* Are remote IRC operators allowed to control this server? */
-               Conf_AllowRemoteOper = Check_ArgIsTrue(Arg);
+
+       Config_Error_Section(Line, Var, "Global");
+}
+
+/**
+ * Handle variable in [Limits] configuration section.
+ *
+ * @param Line Line numer in configuration file.
+ * @param Var  Variable name.
+ * @param Arg  Variable argument.
+ */
+static void
+Handle_LIMITS(int Line, char *Var, char *Arg)
+{
+       assert(Line > 0);
+       assert(Var != NULL);
+       assert(Arg != NULL);
+
+       if (strcasecmp(Var, "ConnectRetry") == 0) {
+               Conf_ConnectRetry = atoi(Arg);
+               if (Conf_ConnectRetry < 5) {
+                       Config_Error(LOG_WARNING,
+                                    "%s, line %d: Value of \"ConnectRetry\" too low!",
+                                    NGIRCd_ConfFile, Line);
+                       Conf_ConnectRetry = 5;
+               }
                return;
        }
-       if( strcasecmp( Var, "MaxConnections" ) == 0 ) {
-               /* Maximum number of connections. 0 -> "no limit". */
-               Conf_MaxConnections = atol( Arg );
+       if (strcasecmp(Var, "MaxConnections") == 0) {
+               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" */
-               Conf_MaxConnectionsIP = atoi( Arg );
+       if (strcasecmp(Var, "MaxConnectionsIP") == 0) {
+               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". */
-               Conf_MaxJoins = atoi( Arg );
+       if (strcasecmp(Var, "MaxJoins") == 0) {
+               Conf_MaxJoins = atoi(Arg);
                if (!Conf_MaxJoins && strcmp(Arg, "0"))
                        Config_Error_NaN(Line, Var);
                return;
        }
-       if( strcasecmp( Var, "MaxNickLength" ) == 0 ) {
-               /* Maximum length of a nick name; must be same on all servers
-                * within the IRC network! */
+       if (strcasecmp(Var, "MaxNickLength") == 0) {
                Conf_MaxNickLength = Handle_MaxNickLength(Line, Arg);
                return;
        }
-       if(strcasecmp(Var, "NoticeAuth") == 0) {
-               /* send NOTICE AUTH messages to clients on connect */
-               Conf_NoticeAuth = Check_ArgIsTrue(Arg);
+       if (strcasecmp(Var, "PingTimeout") == 0) {
+               Conf_PingTimeout = atoi(Arg);
+               if (Conf_PingTimeout < 5) {
+                       Config_Error(LOG_WARNING,
+                                    "%s, line %d: Value of \"PingTimeout\" too low!",
+                                    NGIRCd_ConfFile, Line);
+                       Conf_PingTimeout = 5;
+               }
                return;
        }
-
-       if( strcasecmp( Var, "Listen" ) == 0 ) {
-               /* IP-Address to bind sockets */
-               if (Conf_ListenAddress) {
-                       Config_Error(LOG_ERR, "Multiple Listen= options, ignoring: %s", Arg);
-                       return;
-               }
-               Conf_ListenAddress = strdup_warn(Arg);
-               /*
-                * if allocation fails, we're in trouble:
-                * we cannot ignore the error -- otherwise ngircd
-                * would listen on all interfaces.
-                */
-               if (!Conf_ListenAddress) {
-                       Config_Error(LOG_ALERT, "%s exiting due to fatal errors!", PACKAGE_NAME);
-                       exit(1);
+       if (strcasecmp(Var, "PongTimeout") == 0) {
+               Conf_PongTimeout = atoi(Arg);
+               if (Conf_PongTimeout < 5) {
+                       Config_Error(LOG_WARNING,
+                                    "%s, line %d: Value of \"PongTimeout\" too low!",
+                                    NGIRCd_ConfFile, Line);
+                       Conf_PongTimeout = 5;
                }
                return;
        }
 
-#ifdef SSL_SUPPORT
-       if( strcasecmp( Var, "SSLPorts" ) == 0 ) {
-               ports_parse(&Conf_SSLOptions.ListenPorts, Line, Arg);
+       Config_Error_Section(Line, Var, "Limits");
+}
+
+/**
+ * Handle variable in [Options] configuration section.
+ *
+ * @param Line Line numer in configuration file.
+ * @param Var  Variable name.
+ * @param Arg  Variable argument.
+ */
+static void
+Handle_OPTIONS(int Line, char *Var, char *Arg)
+{
+       size_t len;
+
+       assert(Line > 0);
+       assert(Var != NULL);
+       assert(Arg != NULL);
+
+       if (strcasecmp(Var, "AllowRemoteOper") == 0) {
+               Conf_AllowRemoteOper = Check_ArgIsTrue(Arg);
                return;
        }
-
-       if( strcasecmp( Var, "SSLKeyFile" ) == 0 ) {
-               assert(Conf_SSLOptions.KeyFile == NULL );
-               Conf_SSLOptions.KeyFile = strdup_warn(Arg);
+       if (strcasecmp(Var, "ChrootDir") == 0) {
+               len = strlcpy(Conf_Chroot, Arg, sizeof(Conf_Chroot));
+               if (len >= sizeof(Conf_Chroot))
+                       Config_Error_TooLong(Line, Var);
                return;
        }
-       if( strcasecmp( Var, "SSLCertFile" ) == 0 ) {
-               assert(Conf_SSLOptions.CertFile == NULL );
-               Conf_SSLOptions.CertFile = strdup_warn(Arg);
+       if (strcasecmp(Var, "CloakHost") == 0) {
+               len = strlcpy(Conf_CloakHost, Arg, sizeof(Conf_CloakHost));
+               if (len >= sizeof(Conf_CloakHost))
+                       Config_Error_TooLong(Line, Var);
                return;
        }
-
-       if( strcasecmp( Var, "SSLKeyFilePassword" ) == 0 ) {
-               assert(array_bytes(&Conf_SSLOptions.KeyFilePassword) == 0);
-               if (!array_copys(&Conf_SSLOptions.KeyFilePassword, Arg))
-                       Config_Error( LOG_ERR, "%s, line %d (section \"Global\"): Could not copy %s: %s!",
-                                                               NGIRCd_ConfFile, Line, Var, strerror(errno));
+       if (strcasecmp(Var, "CloakUserToNick") == 0) {
+               Conf_CloakUserToNick = Check_ArgIsTrue(Arg);
                return;
        }
-       if( strcasecmp( Var, "SSLDHFile" ) == 0 ) {
-               assert(Conf_SSLOptions.DHFile == NULL);
-               Conf_SSLOptions.DHFile = strdup_warn( Arg );
-                return;
-        }
-#endif
-#ifdef SYSLOG
-       if (strcasecmp(Var, "SyslogFacility") == 0) {
-               Conf_SyslogFacility = ngt_SyslogFacilityID(Arg,
-                                                          Conf_SyslogFacility);
+       if (strcasecmp(Var, "ConnectIPv6") == 0) {
+               Conf_ConnectIPv6 = Check_ArgIsTrue(Arg);
+               WarnIPv6(Line);
                return;
        }
-#endif
-#ifndef STRICT_RFC
-       if (strcasecmp(Var, "RequireAuthPing") == 0 ) {
-               /* Require new clients to do an "autheticatin PING-PONG" */
-               Conf_AuthPing = Check_ArgIsTrue(Arg);
+       if (strcasecmp(Var, "ConnectIPv4") == 0) {
+               Conf_ConnectIPv4 = Check_ArgIsTrue(Arg);
                return;
        }
-#endif
-       Config_Error(LOG_ERR, "%s, line %d (section \"Global\"): Unknown variable \"%s\"!",
-                                                               NGIRCd_ConfFile, Line, Var);
-} /* Handle_GLOBAL */
-
-
-static void
-Handle_FEATURES(int Line, char *Var, char *Arg)
-{
-       assert( Line > 0 );
-       assert( Var != NULL );
-       assert( Arg != NULL );
-
-       if( strcasecmp( Var, "DNS" ) == 0 ) {
-               /* do reverse dns lookups when clients connect? */
-               Conf_DNS = Check_ArgIsTrue( Arg );
+       if (strcasecmp(Var, "DNS") == 0) {
+               Conf_DNS = Check_ArgIsTrue(Arg);
                return;
        }
        if (strcasecmp(Var, "Ident") == 0) {
-               /* do IDENT lookups when clients connect? */
                Conf_Ident = Check_ArgIsTrue(Arg);
                WarnIdent(Line);
                return;
        }
-       if(strcasecmp(Var, "PAM") == 0) {
-               /* use PAM library to authenticate users */
+       if (strcasecmp(Var, "NoticeAuth") == 0) {
+               Conf_NoticeAuth = Check_ArgIsTrue(Arg);
+               return;
+       }
+       if (strcasecmp(Var, "OperCanUseMode") == 0) {
+               Conf_OperCanMode = Check_ArgIsTrue(Arg);
+               return;
+       }
+       if (strcasecmp(Var, "OperServerMode") == 0) {
+               Conf_OperServerMode = Check_ArgIsTrue(Arg);
+               return;
+       }
+       if (strcasecmp(Var, "PAM") == 0) {
                Conf_PAM = Check_ArgIsTrue(Arg);
                WarnPAM(Line);
                return;
        }
+       if (strcasecmp(Var, "PredefChannelsOnly") == 0) {
+               Conf_PredefChannelsOnly = Check_ArgIsTrue(Arg);
+               return;
+       }
+#ifndef STRICT_RFC
+       if (strcasecmp(Var, "RequireAuthPing") == 0) {
+               Conf_AuthPing = Check_ArgIsTrue(Arg);
+               return;
+       }
+#endif
+       if (strcasecmp(Var, "ScrubCTCP") == 0) {
+               Conf_ScrubCTCP = Check_ArgIsTrue(Arg);
+               return;
+       }
+#ifdef SSL_SUPPORT
+       if (strcasecmp(Var, "SSLCertFile") == 0) {
+               assert(Conf_SSLOptions.CertFile == NULL);
+               Conf_SSLOptions.CertFile = strdup_warn(Arg);
+               return;
+       }
+       if (strcasecmp(Var, "SSLDHFile") == 0) {
+               assert(Conf_SSLOptions.DHFile == NULL);
+               Conf_SSLOptions.DHFile = strdup_warn(Arg);
+               return;
+       }
+       if (strcasecmp(Var, "SSLKeyFile") == 0) {
+               assert(Conf_SSLOptions.KeyFile == NULL);
+               Conf_SSLOptions.KeyFile = strdup_warn(Arg);
+               return;
+       }
+       if (strcasecmp(Var, "SSLKeyFilePassword") == 0) {
+               assert(array_bytes(&Conf_SSLOptions.KeyFilePassword) == 0);
+               if (!array_copys(&Conf_SSLOptions.KeyFilePassword, Arg))
+                       Config_Error(LOG_ERR,
+                                    "%s, line %d (section \"Global\"): Could not copy %s: %s!",
+                                    NGIRCd_ConfFile, Line, Var,
+                                    strerror(errno));
+               return;
+       }
+       if (strcasecmp(Var, "SSLPorts") == 0) {
+               ports_parse(&Conf_SSLOptions.ListenPorts, Line, Arg);
+               return;
+       }
+#endif
+#ifdef SYSLOG
+       if (strcasecmp(Var, "SyslogFacility") == 0) {
+               Conf_SyslogFacility = ngt_SyslogFacilityID(Arg,
+                                                          Conf_SyslogFacility);
+               return;
+       }
+#endif
+       if (strcasecmp(Var, "WebircPassword") == 0) {
+               len = strlcpy(Conf_WebircPwd, Arg, sizeof(Conf_WebircPwd));
+               if (len >= sizeof(Conf_WebircPwd))
+                       Config_Error_TooLong(Line, Var);
+               return;
+       }
 
-       Config_Error(LOG_ERR,
-                    "%s, line %d (section \"Features\"): Unknown variable \"%s\"!",
-                    NGIRCd_ConfFile, Line, Var);
+       Config_Error_Section(Line, Var, "Options");
 }
 
+/**
+ * Handle variable in [Operator] configuration section.
+ *
+ * @param Line Line numer in configuration file.
+ * @param Var  Variable name.
+ * @param Arg  Variable argument.
+ */
 static void
 Handle_OPERATOR( int Line, char *Var, char *Arg )
 {
@@ -1343,17 +1556,23 @@ Handle_OPERATOR( int Line, char *Var, char *Arg )
                op->mask = strdup_warn( Arg );
                return;
        }
-       Config_Error( LOG_ERR, "%s, line %d (section \"Operator\"): Unknown variable \"%s\"!",
-                                                               NGIRCd_ConfFile, Line, Var );
-} /* Handle_OPERATOR */
 
+       Config_Error_Section(Line, Var, "Operator");
+}
 
+/**
+ * Handle variable in [Server] configuration section.
+ *
+ * @param Line Line numer in configuration file.
+ * @param Var  Variable name.
+ * @param Arg  Variable argument.
+ */
 static void
 Handle_SERVER( int Line, char *Var, char *Arg )
 {
        long port;
        size_t len;
-       
+
        assert( Line > 0 );
        assert( Var != NULL );
        assert( Arg != NULL );
@@ -1439,11 +1658,19 @@ Handle_SERVER( int Line, char *Var, char *Arg )
                return;
        }
 
-       Config_Error( LOG_ERR, "%s, line %d (section \"Server\"): Unknown variable \"%s\"!",
-                                                               NGIRCd_ConfFile, Line, Var );
-} /* Handle_SERVER */
-
+       Config_Error_Section(Line, Var, "Server");
+}
 
+/**
+ * Copy channel name into channel structure.
+ *
+ * If the channel name is not valid because of a missing prefix ('#', '&'),
+ * a default prefix of '#' will be added.
+ *
+ * @param new_chan     New already allocated channel structure.
+ * @param name         Name of the new channel.
+ * @returns            true on success, false otherwise.
+ */
 static bool
 Handle_Channelname(struct Conf_Channel *new_chan, const char *name)
 {
@@ -1462,7 +1689,13 @@ Handle_Channelname(struct Conf_Channel *new_chan, const char *name)
        return size > strlcpy(dest, name, size);
 }
 
-
+/**
+ * Handle variable in [Channel] configuration section.
+ *
+ * @param Line Line numer in configuration file.
+ * @param Var  Variable name.
+ * @param Arg  Variable argument.
+ */
 static void
 Handle_CHANNEL(int Line, char *Var, char *Arg)
 {
@@ -1523,11 +1756,19 @@ Handle_CHANNEL(int Line, char *Var, char *Arg)
                return;
        }
 
-       Config_Error( LOG_ERR, "%s, line %d (section \"Channel\"): Unknown variable \"%s\"!",
-                                                               NGIRCd_ConfFile, Line, Var );
-} /* Handle_CHANNEL */
-
+       Config_Error_Section(Line, Var, "Channel");
+}
 
+/**
+ * Validate server configuration.
+ *
+ * Please note that this function uses exit(1) on fatal errors and therefore
+ * can result in ngIRCd terminating!
+ *
+ * @param Configtest   true if the daemon has been called with "--configtest".
+ * @param Rehash       true if re-reading configuration on runtime.
+ * @returns            true if configuration is valid.
+ */
 static bool
 Validate_Config(bool Configtest, bool Rehash)
 {
@@ -1625,16 +1866,40 @@ Validate_Config(bool Configtest, bool Rehash)
 #endif
 
        return config_valid;
-} /* Validate_Config */
-
+}
 
+/**
+ * Output "line too long" warning.
+ *
+ * @param Line Line number in configuration file.
+ * @param Item Affected variable name.
+ */
 static void
 Config_Error_TooLong ( const int Line, const char *Item )
 {
        Config_Error( LOG_WARNING, "%s, line %d: Value of \"%s\" too long!", NGIRCd_ConfFile, Line, Item );
 }
 
+/**
+ * Output "unknown variable" warning.
+ *
+ * @param Line         Line number in configuration file.
+ * @param Item         Affected variable name.
+ * @param Section      Section name.
+ */
+static void
+Config_Error_Section(const int Line, const char *Item, const char *Section)
+{
+       Config_Error(LOG_ERR, "%s, line %d (section \"%s\"): Unknown variable \"%s\"!",
+                    NGIRCd_ConfFile, Line, Section, Item);
+}
 
+/**
+ * Output "not a number" warning.
+ *
+ * @param Line Line number in configuration file.
+ * @param Item Affected variable name.
+ */
 static void
 Config_Error_NaN( const int Line, const char *Item )
 {
@@ -1642,7 +1907,16 @@ Config_Error_NaN( const int Line, const char *Item )
                                                NGIRCd_ConfFile, Line, Item );
 }
 
-
+/**
+ * Output configuration error to console and/or logfile.
+ *
+ * On runtime, the normal log functions of the daemon are used. But when
+ * testing the configuration ("--configtest"), all messages go directly
+ * to the console.
+ *
+ * @param Level                Severity level of the message.
+ * @param Format       Format string; see printf() function.
+ */
 #ifdef PROTOTYPES
 static void Config_Error( const int Level, const char *Format, ... )
 #else
@@ -1652,8 +1926,6 @@ const char *Format;
 va_dcl
 #endif
 {
-       /* Error! Write to console and/or logfile. */
-
        char msg[MAX_LOG_MSG_LEN];
        va_list ap;
 
@@ -1666,17 +1938,21 @@ va_dcl
 #endif
        vsnprintf( msg, MAX_LOG_MSG_LEN, Format, ap );
        va_end( ap );
-       
-       /* During "normal operations" the log functions of the daemon should
-        * be used, but during testing of the configuration file, all messages
-        * should go directly to the console: */
-       if (Use_Log) Log( Level, "%s", msg );
-       else puts( msg );
-} /* Config_Error */
 
+       if (!Use_Log) {
+               if (Level <= LOG_WARNING)
+                       printf(" - %s\n", msg);
+               else
+                       puts(msg);
+       } else
+               Log(Level, "%s", msg);
+}
 
 #ifdef DEBUG
 
+/**
+ * Dump internal state of the "configuration module".
+ */
 GLOBAL void
 Conf_DebugDump(void)
 {
@@ -1693,16 +1969,18 @@ Conf_DebugDump(void)
                    Conf_Server[i].group, Conf_Server[i].flags,
                    Conf_Server[i].conn_id);
        }
-} /* Conf_DebugDump */
+}
 
 #endif
 
-
+/**
+ * Initialize server configuration structur to default values.
+ *
+ * @param Server       Pointer to server structure to initialize.
+ */
 static void
 Init_Server_Struct( CONF_SERVER *Server )
 {
-       /* Initialize server configuration structur to default values */
-
        assert( Server != NULL );
 
        memset( Server, 0, sizeof (CONF_SERVER) );
@@ -1715,7 +1993,6 @@ Init_Server_Struct( CONF_SERVER *Server )
        Proc_InitStruct(&Server->res_stat);
        Server->conn_id = NONE;
        memset(&Server->bind_addr, 0, sizeof(&Server->bind_addr));
-} /* Init_Server_Struct */
-
+}
 
 /* -eof- */