Clarify directory and ORPort checking functions.

In order to make the OR and dir checking functions in router.c less confusing
we renamed some functions and splitted consider_testing_reachability() into
router_should_check_reachability() and router_do_reachability_checks(). Also we
improved the documentation.

Fixes #18918.

Signed-off-by: Fernando Fernandez Mancera <ffmancera@riseup.net>

oxford comma for-the-win

Add a "falls through" comment to make gcc happy.

Merge remote-tracking branch 'mikeperry/bug23101-mergeready-squashed'

Bug 23101: Pre-build HS-specific circuits (instead of general).

Prebuilt circs are 4 hops, since only server side HSDIR and intro circs
are 3 hops, and it is OK if those sometimes take longer to build.

Add new circuit purposes for hsdir activity.

This lets us control their path len and usage.

Implement layer 2 and layer 3 guard pinning via torrc.

Block circuit canibalization when HSRendezvousMiddleNodes is active.
Also make it apply to all HS circuits, not just rends.

Merge branch 'disable_signal_handlers'

Merge branch 'maint-0.3.2'

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge remote-tracking branch 'dgoulet/bug24895_032_02' into maint-0.3.2

Merge branch 'maint-0.3.0' into maint-0.3.1

"ours" merge to avoid maint-0.3.0's version of 24895.

Merge remote-tracking branch 'dgoulet/bug24895_031_02' into maint-0.3.1

Merge branch 'maint-0.2.9' into maint-0.3.0

 Conflicts:
        src/or/rendservice.c

Merge remote-tracking branch 'dgoulet/bug24895_029_02' into maint-0.2.9

Merge branch 'bug24895_031_02' into bug24895_032_02

Merge branch 'bug24895_029_02' into bug24895_031_02

hs: Use hs_service_max_rdv_failures consensus param, defaulting to 2

Merge branch 'maint-0.3.2'

Merge remote-tracking branch 'tlyu-github/travis-workaround-master'

Merge branch 'maint-0.3.1'

"ours" merge to avoid the 0.3.1 version of the 24863 fix.

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge remote-tracking branch 'tlyu-github/travis-workaround-031' into maint-0.3.1

Merge branch 'maint-0.3.0' into maint-0.3.1

"ours" merge to avoid 0.2.5 version of #24863 fix.

Merge branch 'maint-0.2.9' into maint-0.3.0

Merge branch 'maint-0.2.5' into maint-0.2.9

Improve documentation for signal code

Merge branch 'restart_debug_squashed'

Add a mostly disabled feature to debug restarting in-process

For 23847, we want Tor to be able to shut down and then restart in
the same process.  Here's a patch to make the Tor binary do that.
To test it, you need to build with --enable-restart-debugging, and
then you need to set the environment variable TOR_DEBUG_RESTART.
With this option, Tor will then run for 5 seconds, then restart
itself in-process without exiting.  This only happens once.

You can change the 5-second interval using
TOR_DEBUG_RESTART_AFTER_SECONDS.

Implements ticket 24583.

Don't call Libevent's event_base_free() on NULL.

It doesn't crash, but it produces a warning.

Fixes bug 24933; bugfix on 322abc030e53c7e84ca9f22a47b2965f2. Bug
not in any released Tor.

turn MAX_REND_FAILURES into a function

no actual changes in behavior

MAX_REND_FAILURES is 1, but we would try three times

Fix an "off by 2" error in counting rendezvous failures on the onion
service side.

While we thought we would stop the rendezvous attempt after one failed
circuit, we were actually making three circuit attempts before giving up.

Fixes bug 24895; bugfix on 0.0.6.

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.3.2'

Merge branch 'bug24826_031' into maint-0.3.1

Work around Travis CI ptrace regression

Travis CI deployed a change that disables ptrace capabilities in
container builds, which prevents LeakSanitizer on clang from working
properly.  Set "sudo: required" to force non-container builds for
clang and work around this.

Work around Travis CI ptrace regression

Travis CI deployed a change that disables ptrace capabilities in
container builds, which prevents LeakSanitizer on clang from working
properly.  Set "sudo: required" to force non-container builds for
clang and work around this.

Work around Travis CI ptrace regression

Travis CI deployed a change that disables ptrace capabilities in
container builds, which prevents LeakSanitizer on clang from working
properly.  Set "sudo: required" to force non-container builds for
clang and work around this.

Merge branch 'bug22798_029_squashed'

Merge branch 'maint-0.3.2'

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.3.2'

Merge remote-tracking branch 'arma/bug24894' into maint-0.3.2

update comment; ignore argument on non-win

Improve Windows performance with SIO_IDEAL_SEND_BACKLOG_QUERY.

Patch written by "Vort" on trac. Addresses ticket 22798.

Merge branch 'bug21074_029'

Add a cast to avoid a signed/unsigned comparison

fix a check-changes warning

Merge branch 'bug21074_029'

Merge branch 'maint-0.3.2'

Merge remote-tracking branch 'public/bug24652_032' into maint-0.3.2

Merge branch 'stack_em_up'

Repair a crash bug in src/test/bench

This was caused because of the new KeyDirectory and CacheDirectory
options not being set. Bugfix on 0.3.3.0-alpha; not in any released
Tor.

Improve fragile-hardening performance of consensus_split_lines.

For whatever reason, in my testing, using memchr() here improves
performance over strchr() by a great deal.

Fixes bug 24826; bugfix on 0.3.1.1-alpha.

Update scan-build.sh to work with the latest clangalyzer

Fix a (spurious) scan-build warning in test_introduce1_validation()

Since helper_create_introduce1_cell() checks "cell" for nullness,
scan-build is concerned that test_introduce1_validation()
dereferences it without checking it.  So, add a check.

Not backporting, since this is spurious, _and_ tests-only.

Remove false positives from channel_is_client()

Fix a set of false positives where relays would consider connections
to other relays as being client-only connections (and thus e.g.
deserving different link padding schemes) if those relays fell out
of the consensus briefly.

Now we look only at the initial handshake and whether the connection
authenticated as a relay.

Fixes bug 24898; bugfix on 0.3.1.1-alpha.

v3 onion services now obey MAX_REND_FAILURES

New-style (v3) onion services now obey the "max rendezvous circuit
attempts" logic.

Previously they would make as many rendezvous circuit attempts as they
could fit in the MAX_REND_TIMEOUT second window before giving up.

Fixes bug 24894; bugfix on 0.3.2.1-alpha.

Remove a needless changefile. (Needless because bug not released.)

Merge remote-tracking branch 'ffmancera/bug24861'

Fix minGW compatibility issue with zu format specifier.

Define TOR_PRIuSZ as minGW compiler doesn't support zu format specifier for
size_t type.

Fixes #24861 on ac9eebd.

Signed-off-by: Fernando Fernandez Mancera <ffernandezmancera@gmail.com>

Merge branch 'maint-0.3.2'

Reword requirement to set Myfamily correctly

Note contactinfo and myfamily as required in more places

Includes a sentence from cypherpunks; for ticket 24526.

Merge remote-tracking branch 'sebastian/myfamily'

Merge branch 'bug24582'

Reword requirement to set Myfamily correctly

On shutdown, mark openssl as uninitialized.

This causes openssl to get completely reinitialized on startup,
which is probably a good idea.

Make the warnings about restarting tor in-process more scary.

Merge branch 'restart_nocrash'

Merge branch 'simplify_transition_allowed'

Merge branch 'bug24526_squashed'

Note contactinfo and myfamily as required in more places

Includes a sentence from cypherpunks; for ticket 24526.

Add a blank line per dgoulet review.

Merge branch 'bug24584'

Compute flavor name before dereference, to avoid a STACK warning.

(See 24423)

Merge remote-tracking branch 'ffmancera/github/bug24573'

Merge branch 'bug24733_squashed_2'

Document the alignment limitation of tor_free()

Extract the raw_free() of ifc_buf into a new function.

Explain the problem more correctly.

Stop invoking undefined behaviour by using tor_free() on an unaligned pointer

... in get_interface_addresses_ioctl().

This pointer alignment issue exists on x86_64 macOS, but is unlikely to exist
elsewhere. (i386 macOS only requires 4-byte alignment, and other OSs have
8-byte ints.)

Fixes bug 24733; not in any released version of tor.

Merge remote-tracking branch 'ffmancera/github/bug24501'

Remove needless parentheses

Merge remote-tracking branch 'teor/Bug-23966'

ticket 22145 changes file

Merge remote-tracking branch 'teor/bug22145_032'

Merge branch 'maint-0.3.2'

"ours" merge to avoid version bump.

Increment version to 0.3.2.9-dev

Forward-port changelog and releasenotes for 0.3.2.9

Merge branch 'maint-0.3.2'

"ours" merge to avoid version bump.

Increment version to 0.3.2.9

Merge branch 'maint-0.3.2'

There was a merge conflict on fallback_dirs.inc, which we resolve in
favor of the newly generated version for #24801.

Merge branch 'maint-0.2.9' into maint-0.3.0

Merge branch 'maint-0.3.0' into maint-0.3.1

Merge branch 'maint-0.3.1' into maint-0.3.2

Add changes file for new fallback directory list.

Add 7 other fallback dirs from teor's tests

These are from "fallback_dirs_2018_01_06_2323_UTC_44aa1adf35_AU.inc"

Update the fallback_dirs.inc file: part 1

This takes the updated fallback_dirs_2018_01_06_CA.inc from
pastly's scan.

Merge remote-tracking branch 'arma/bug24555'

Merge branch 'maint-0.3.1' into maint-0.3.2