Fix potential buffer-overflow from malformed RSFNC request.

[solanum.git] / src / newconf.c

diff --git a/src/newconf.c b/src/newconf.c
index 8f8dd380d32f4767e3a9c916b0e42bfadf3c60f7..95013c53601f7d7b7b054bd22eaef693d5423867 100644 (file)
--- a/src/newconf.c
+++ b/src/newconf.c
@@ -2098,15 +2098,15 @@ static struct ConfEntry conf_admin_table[] =
 
 static struct ConfEntry conf_log_table[] =
 {
-       { "fname_userlog",      CF_QSTRING, NULL, MAXPATHLEN, &ConfigFileEntry.fname_userlog    },
-       { "fname_fuserlog",     CF_QSTRING, NULL, MAXPATHLEN, &ConfigFileEntry.fname_fuserlog   },
-       { "fname_operlog",      CF_QSTRING, NULL, MAXPATHLEN, &ConfigFileEntry.fname_operlog    },
-       { "fname_foperlog",     CF_QSTRING, NULL, MAXPATHLEN, &ConfigFileEntry.fname_foperlog   },
-       { "fname_serverlog",    CF_QSTRING, NULL, MAXPATHLEN, &ConfigFileEntry.fname_serverlog  },
-       { "fname_killlog",      CF_QSTRING, NULL, MAXPATHLEN, &ConfigFileEntry.fname_killlog    },
-       { "fname_klinelog",     CF_QSTRING, NULL, MAXPATHLEN, &ConfigFileEntry.fname_klinelog   },
-       { "fname_operspylog",   CF_QSTRING, NULL, MAXPATHLEN, &ConfigFileEntry.fname_operspylog },
-       { "fname_ioerrorlog",   CF_QSTRING, NULL, MAXPATHLEN, &ConfigFileEntry.fname_ioerrorlog },
+       { "fname_userlog",      CF_QSTRING, NULL, PATH_MAX, &ConfigFileEntry.fname_userlog      },
+       { "fname_fuserlog",     CF_QSTRING, NULL, PATH_MAX, &ConfigFileEntry.fname_fuserlog     },
+       { "fname_operlog",      CF_QSTRING, NULL, PATH_MAX, &ConfigFileEntry.fname_operlog      },
+       { "fname_foperlog",     CF_QSTRING, NULL, PATH_MAX, &ConfigFileEntry.fname_foperlog     },
+       { "fname_serverlog",    CF_QSTRING, NULL, PATH_MAX, &ConfigFileEntry.fname_serverlog    },
+       { "fname_killlog",      CF_QSTRING, NULL, PATH_MAX, &ConfigFileEntry.fname_killlog      },
+       { "fname_klinelog",     CF_QSTRING, NULL, PATH_MAX, &ConfigFileEntry.fname_klinelog     },
+       { "fname_operspylog",   CF_QSTRING, NULL, PATH_MAX, &ConfigFileEntry.fname_operspylog   },
+       { "fname_ioerrorlog",   CF_QSTRING, NULL, PATH_MAX, &ConfigFileEntry.fname_ioerrorlog },
        { "\0",                 0,          NULL, 0,          NULL }
 };
 
@@ -2192,7 +2192,7 @@ static struct ConfEntry conf_general_table[] =
        { "default_operstring", CF_QSTRING, NULL, REALLEN,    &ConfigFileEntry.default_operstring },
        { "default_adminstring",CF_QSTRING, NULL, REALLEN,    &ConfigFileEntry.default_adminstring },
        { "servicestring",      CF_QSTRING, NULL, REALLEN,    &ConfigFileEntry.servicestring },
-       { "egdpool_path",       CF_QSTRING, NULL, MAXPATHLEN, &ConfigFileEntry.egdpool_path },
+       { "egdpool_path",       CF_QSTRING, NULL, PATH_MAX, &ConfigFileEntry.egdpool_path },
        { "kline_reason",       CF_QSTRING, NULL, REALLEN, &ConfigFileEntry.kline_reason },
        { "identify_service",   CF_QSTRING, NULL, REALLEN, &ConfigFileEntry.identifyservice },
        { "identify_command",   CF_QSTRING, NULL, REALLEN, &ConfigFileEntry.identifycommand },
@@ -2275,8 +2275,8 @@ static struct ConfEntry conf_channel_table[] =
        { "only_ascii_channels", CF_YESNO, NULL, 0, &ConfigChannel.only_ascii_channels },
        { "use_except",         CF_YESNO, NULL, 0, &ConfigChannel.use_except            },
        { "use_invex",          CF_YESNO, NULL, 0, &ConfigChannel.use_invex             },
-       { "use_knock",          CF_YESNO, NULL, 0, &ConfigChannel.use_knock             },
        { "use_forward",        CF_YESNO, NULL, 0, &ConfigChannel.use_forward           },
+       { "use_knock",          CF_YESNO, NULL, 0, &ConfigChannel.use_knock             },
        { "resv_forcepart",     CF_YESNO, NULL, 0, &ConfigChannel.resv_forcepart        },
        { "channel_target_change", CF_YESNO, NULL, 0, &ConfigChannel.channel_target_change      },
        { "disable_local_channels", CF_YESNO, NULL, 0, &ConfigChannel.disable_local_channels },