From afb1c79879d520d52307eddb7e46f551a2a6d75d Mon Sep 17 00:00:00 2001 From: Roger Dingledine Date: Fri, 18 Nov 2005 02:47:09 +0000 Subject: [PATCH] Leave options->DirServers alone -- if the user didn't specify any, just add the default ones directly to the trusted dirserver list. This fixes a bug where people running controllers would setconf or the equivalent, and Tor would start yelling at them about setting their own DirServer lines. svn:r5418 --- src/or/config.c | 45 +++++++++++++++++++++++---------------------- src/or/or.h | 6 +++--- src/or/routerlist.c | 2 +- 3 files changed, 27 insertions(+), 26 deletions(-) diff --git a/src/or/config.c b/src/or/config.c index e44ab8584a..b5f6df73f5 100644 --- a/src/or/config.c +++ b/src/or/config.c @@ -430,6 +430,19 @@ safe_str(const char *address) return address; } +/** Add the default directory servers directly into the trusted dir list. */ +static void +add_default_trusted_dirservers(void) +{ + const char *dirservers[] = { +"moria1 v1 18.244.0.188:9031 FFCB 46DB 1339 DA84 674C 70D7 CB58 6434 C437 0441", +"moria2 v1 18.244.0.114:80 719B E45D E224 B607 C537 07D0 E214 3E2D 423E 74CF", +"tor26 v1 86.59.21.38:80 847B 1F85 0344 D787 6491 A548 92F9 0493 4E4E B85D" }; + parse_dir_server_line(dirservers[0], 0); + parse_dir_server_line(dirservers[1], 0); + parse_dir_server_line(dirservers[2], 0); +} + /** Fetch the active option list, and take actions based on it. All of the * things we do should survive being done repeatedly. If present, * old_options contains the previous value of the options. @@ -541,12 +554,16 @@ options_act(or_options_t *old_options) int running_tor = options->command == CMD_RUN_TOR; clear_trusted_dir_servers(); - for (cl = options->DirServers; cl; cl = cl->next) { - if (parse_dir_server_line(cl->value, 0)<0) { - err(LD_BUG, - "Bug: Previously validated DirServer line could not be added!"); - return -1; + if (options->DirServers) { + for (cl = options->DirServers; cl; cl = cl->next) { + if (parse_dir_server_line(cl->value, 0)<0) { + err(LD_BUG, + "Bug: Previously validated DirServer line could not be added!"); + return -1; + } } + } else { + add_default_trusted_dirservers(); } if (running_tor && rend_config_services(options, 0)<0) { @@ -1343,20 +1360,6 @@ option_reset(config_format_t *fmt, or_options_t *options, } } -/** Set options->DirServers to contain the default directory - * servers. */ -static void -add_default_trusted_dirservers(or_options_t *options) -{ - config_line_append(&options->DirServers, "DirServer", - "moria1 v1 18.244.0.188:9031 FFCB 46DB 1339 DA84 674C 70D7 CB58 6434 C437 0441"); - config_line_append(&options->DirServers, "DirServer", - "moria2 v1 18.244.0.114:80 719B E45D E224 B607 C537 07D0 E214 3E2D 423E 74CF"); - config_line_append(&options->DirServers, "DirServer", - "tor26 v1 86.59.21.38:80 847B 1F85 0344 D787 6491 A548 92F9 0493 4E4E B85D"); -// "tor.noreply.org:9030 847B 1F85 0344 D787 6491 A548 92F9 0493 4E4E B85D"); -} - /** Print a usage message for tor. */ static void print_usage(void) @@ -2076,9 +2079,7 @@ options_validate(or_options_t *options) result = -1; } - if (!options->DirServers) { - add_default_trusted_dirservers(options); - } else { + if (options->DirServers) { COMPLAIN("You have used DirServer to specify directory authorities in your configuration. This is potentially dangerous: it can make you look different from all other Tor users, and hurt your anonymity. Even if you've specified the same authorities as Tor uses by default, the defaults could change in the future. Be sure you know what you're doing."); for (cl = options->DirServers; cl; cl = cl->next) { if (parse_dir_server_line(cl->value, 1)<0) diff --git a/src/or/or.h b/src/or/or.h index b262a99732..07cb7e9c36 100644 --- a/src/or/or.h +++ b/src/or/or.h @@ -1142,7 +1142,7 @@ typedef struct config_line_t { struct config_line_t *next; } config_line_t; -/** Configuration options for a Tor process */ +/** Configuration options for a Tor process. */ typedef struct { uint32_t _magic; @@ -1154,10 +1154,10 @@ typedef struct { const char *command_arg; /**< Argument for command-line option. */ config_line_t *OldLogOptions; /**< List of configuration lines - * for logfiles, old style. */ + * for logfiles, old style. */ config_line_t *Logs; /**< New-style list of configuration lines - * for logs */ + * for logs */ char *DebugLogFile; /**< Where to send verbose log messages. */ char *DataDirectory; /**< OR only: where to store long-term data. */ diff --git a/src/or/routerlist.c b/src/or/routerlist.c index 655c1cdfe4..c6b4a1c8e4 100644 --- a/src/or/routerlist.c +++ b/src/or/routerlist.c @@ -2413,7 +2413,7 @@ add_trusted_dir_server(const char *nickname, const char *address, } } else { if (tor_lookup_hostname(address, &a)) { - warn(LD_CONFIG, "Unable to lookup address for directory server at %s", + warn(LD_CONFIG, "Unable to lookup address for directory server at '%s'", address); return; }