Use tor_addr_from_getsockname() in several places

I'm leaving the getsockname code in transproxy alone, since it is
comparatively isolated, rather platform-specific, and hard to test.

Implements 18105.

Add a new tor_addr_from_getsockname()

We use this pattern all over, and this should simplify matters a
bit.  Part of 18105.

Merge remote-tracking branch 'ahf-oniongit/bugs/25026'

Bump version to 0.3.3.1-alpha-dev

Reflow changelog

We do not put a colon here.

bump version to 0.3.3.1-alpha

Proofreading step by seborn (thx)

finish writing the blurb

More editing on the ChangeLog

Update HiddenServiceVersion man-page entry to only accept either 2 or 3.

This patch updates the HiddenServiceVersion man-page entry to only
accept either 2 or 3 as argument and not a list of multiple versions.

See: https://bugs.torproject.org/25026

spellcheck the changelog

re-sort and re-flow the changelog

Light changelog relabeling and editing

Start on a changelog for 0.3.1.1-alpha

Remove changes files that also appeared in 0.3.2.9

Merge branch 'maint-0.3.2'

Fix a memory leak in build_unopened_fourhop

This is a unit-test-only leak, but let's fix it anyway so it doesn't
hide real bugs.

Bug not in any released version of Tor.

Fix a memory leak in scheduler/loop_kist

Fixes bug 25005.

Add missing static keywords

crypto_openssl_header_version_str and crypto_openssl_version_str in
crypto_openssl_mgt.c should be static.

openssl_mutexes code belongs in openssl_mgt.c

add a missing "compat_openssl.h"

perhaps this was the missing include?

Add a missing include for openssl 1.0.2

Merge remote-tracking branch 'mikeperry/bug24946'

Merge remote-tracking branch 'asn/bug24896'

Merge remote-tracking branch 'ffmancera/bug24658-openssl'

Add onion service activity information to our heartbeat logs.

Move the openssl namespace back into .c files.

As we're trying not to have all the other modules in Tor, we moved the openssl
namespace includes back into crypto.c and crypto_openssl_mgt.c files.

Follows #24658.

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

Bug 24946: Fix a warning message caused by a missed purpose check.

Also fix three other checks (found by inspection of all
CIRCUIT_PURPOSE_C_GENERAL occurrences).

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

Add crypto_openssl_mgt.[ch] for compiling dependencies.

Included crypto_openssl_mgt.[ch] into the appropiate files in order to resolve
compiling and dependencies issues.

Follows #24658.

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

Tweaks into functions and variables in crypto_openssl_mgt.[ch]

Renamed free_openssl() to crypto_openssl_free_all(). Also we made variables and
functions static again.

Follows #24658.

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

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.

Rename crypto_openssl.[ch] to crypto_openssl_mgt.[ch]

Rename crypto_openssl.[ch] to crypto_openssl_mgt.[ch] because it is possible we
need crypto_openssl.[ch] in the future.

Follows #24658.

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

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.