diff options
| author | Roger Dingledine <arma@torproject.org> | 2003-09-26 10:03:50 +0000 |
|---|---|---|
| committer | Roger Dingledine <arma@torproject.org> | 2003-09-26 10:03:50 +0000 |
| commit | 9e5cafc395397426030e8098d64b8e25625863c5 (patch) | |
| tree | a92b2c34d62a5673746387e1b10e42788e1838ef /src/or/config.c | |
| parent | db2684149eb0b1528e0b545303ad4db69a0ed584 (diff) | |
| download | tor-9e5cafc395397426030e8098d64b8e25625863c5.tar.gz tor-9e5cafc395397426030e8098d64b8e25625863c5.zip | |
first pass: obey log convention
ERR is if something fatal just happened
WARNING is something bad happened, but we're still running. The bad thing
is either a bug in the code, an attack or buggy protocol/implementation
of the remote peer, etc. The operator should examine the bad thing and
try to correct it.
(No error or warning messages should be expected. I expect most people
to run on -l warning eventually.)
NOTICE is never ever used.
INFO means something happened (maybe bad, maybe ok), but there's nothing
you need to (or can) do about it.
DEBUG is for everything louder than INFO.
svn:r486
Diffstat (limited to 'src/or/config.c')
| -rw-r--r-- | src/or/config.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/or/config.c b/src/or/config.c index 31b7e0235b..348d8d7868 100644 --- a/src/or/config.c +++ b/src/or/config.c @@ -163,7 +163,7 @@ static int config_compare(struct config_line *c, char *key, int type, void *arg) case CONFIG_TYPE_BOOL: i = atoi(c->value); if (i != 0 && i != 1) { - log(LOG_ERR, "Boolean keyword '%s' expects 0 or 1", c->key); + log(LOG_WARNING, "Boolean keyword '%s' expects 0 or 1", c->key); return 0; } *(int *)arg = i; @@ -259,7 +259,7 @@ int getconfig(int argc, char **argv, or_options_t *options) { log(LOG_DEBUG,"Opening config file '%s'",fname); cf = config_open(fname); - if(!cf) { /* it's defined but not there. that's no good. */ + if(!cf) { log(LOG_ERR, "Unable to open configuration file '%s'.",fname); return -1; } |