[or-cvs] fix an edge case in parsing config options (thanks weasel)
Roger Dingledine
arma at seul.org
Tue Feb 1 00:05:59 UTC 2005
Update of /home2/or/cvsroot/tor/src/or
In directory moria.mit.edu:/home2/arma/work/onion/cvs/tor/src/or
Modified Files:
config.c
Log Message:
fix an edge case in parsing config options (thanks weasel)
Index: config.c
===================================================================
RCS file: /home2/or/cvsroot/tor/src/or/config.c,v
retrieving revision 1.305
retrieving revision 1.306
diff -u -d -r1.305 -r1.306
--- config.c 28 Jan 2005 08:53:47 -0000 1.305
+++ config.c 1 Feb 2005 00:05:57 -0000 1.306
@@ -508,6 +508,9 @@
static config_var_t *config_find_option(const char *key)
{
int i;
+ size_t keylen = strlen(key);
+ if(!keylen)
+ return NULL; /* if they say "--" on the commandline, it's not an option */
/* First, check for an exact (case-insensitive) match */
for (i=0; config_vars[i].name; ++i) {
if (!strcasecmp(key, config_vars[i].name))
@@ -515,7 +518,7 @@
}
/* If none, check for an abbreviated match */
for (i=0; config_vars[i].name; ++i) {
- if (!strncasecmp(key, config_vars[i].name, strlen(key))) {
+ if (!strncasecmp(key, config_vars[i].name, keylen)) {
log_fn(LOG_WARN, "The abbreviation '%s' is deprecated. "
"Tell Nick and Roger to make it official, or just use '%s' instead",
key, config_vars[i].name);
More information about the tor-commits
mailing list