Merge remote-tracking branch 'arma/bug19003-try2' into maint-0.2.8

This commit is contained in:
Nick Mathewson 2016-05-12 11:09:33 -04:00
commit ce6f2d1c4d
3 changed files with 22 additions and 4 deletions

5
changes/bug19003 Normal file
View File

@ -0,0 +1,5 @@
o Minor bugfixes (small networks):
- Allow directories in small networks to bootstrap by
skipping DirPort checks when the consensus has no exits.
Resolves #19003, bugfix on #18050 in 0.2.8.1-alpha.
Patch by teor.

View File

@ -1674,7 +1674,11 @@ circuit_launch(uint8_t purpose, int flags)
return circuit_launch_by_extend_info(purpose, NULL, flags); return circuit_launch_by_extend_info(purpose, NULL, flags);
} }
/* DOCDOC */ /* Do we have enough descriptors to build paths?
* If need_exit is true, return 1 if we can build exit paths.
* (We need at least one Exit in the consensus to build exit paths.)
* If need_exit is false, return 1 if we can build internal paths.
*/
static int static int
have_enough_path_info(int need_exit) have_enough_path_info(int need_exit)
{ {

View File

@ -1549,8 +1549,10 @@ proxy_mode(const or_options_t *options)
* and * and
* - We have ORPort set * - We have ORPort set
* and * and
* - We believe both our ORPort and DirPort (if present) are reachable from * - We believe our ORPort and DirPort (if present) are reachable from
* the outside; or * the outside; or
* - We believe our ORPort is reachable from the outside, and we can't
* check our DirPort because the consensus has no exits; or
* - We are an authoritative directory server. * - We are an authoritative directory server.
*/ */
static int static int
@ -1568,8 +1570,15 @@ decide_if_publishable_server(void)
return 1; return 1;
if (!router_get_advertised_or_port(options)) if (!router_get_advertised_or_port(options))
return 0; return 0;
if (!check_whether_orport_reachable())
return check_whether_orport_reachable() && check_whether_dirport_reachable(); return 0;
if (router_have_consensus_path() == CONSENSUS_PATH_INTERNAL) {
/* All set: there are no exits in the consensus (maybe this is a tiny
* test network), so we can't check our DirPort reachability. */
return 1;
} else {
return check_whether_dirport_reachable();
}
} }
/** Initiate server descriptor upload as reasonable (if server is publishable, /** Initiate server descriptor upload as reasonable (if server is publishable,