Merge branch 'maint-0.2.2'

This commit is contained in:
Roger Dingledine 2010-10-01 14:12:11 -04:00
commit 165aaf560a
2 changed files with 12 additions and 1 deletions

6
changes/bug1534 Normal file
View File

@ -0,0 +1,6 @@
o Minor bugfixes:
- When we're trying to guess whether we know our IP address as
a relay, we would log various ways that we failed to guess
our address, but never log that we ended up guessing it
successfully. Now add a log line to help confused and anxious
relay operators. Bugfix on 0.1.2.1-alpha; fixes bug 1534.

View File

@ -1257,6 +1257,8 @@ static int router_guess_address_from_dir_headers(uint32_t *guess);
int int
router_pick_published_address(or_options_t *options, uint32_t *addr) router_pick_published_address(or_options_t *options, uint32_t *addr)
{ {
char buf[INET_NTOA_BUF_LEN];
struct in_addr a;
if (resolve_my_address(LOG_INFO, options, addr, NULL) < 0) { if (resolve_my_address(LOG_INFO, options, addr, NULL) < 0) {
log_info(LD_CONFIG, "Could not determine our address locally. " log_info(LD_CONFIG, "Could not determine our address locally. "
"Checking if directory headers provide any hints."); "Checking if directory headers provide any hints.");
@ -1266,6 +1268,9 @@ router_pick_published_address(or_options_t *options, uint32_t *addr)
return -1; return -1;
} }
} }
a.s_addr = htonl(*addr);
tor_inet_ntoa(&a, buf, sizeof(buf));
log_info(LD_CONFIG,"Success: chose address '%s'.", buf);
return 0; return 0;
} }
@ -1289,7 +1294,7 @@ router_rebuild_descriptor(int force)
if (router_pick_published_address(options, &addr) < 0) { if (router_pick_published_address(options, &addr) < 0) {
/* Stop trying to rebuild our descriptor every second. We'll /* Stop trying to rebuild our descriptor every second. We'll
* learn that it's time to try again when server_has_changed_ip() * learn that it's time to try again when ip_address_changed()
* marks it dirty. */ * marks it dirty. */
desc_clean_since = time(NULL); desc_clean_since = time(NULL);
return -1; return -1;