test/circuitbuild: Refactor test case array

Avoid repeating test names.

Part of 33633.

relay: End circuitbuild logs with "."

Consistent logs make testing easier.

Part of 33633.

relay: Check for NULL arguments in circuitbuild

Part of 33633.

relay: Make circuitbuild functions STATIC

Allow the circuitbuild_relay functions to be accessed by the unit tests.

Part of 33633.

relay: Remove a redundant function return value

Part of 33633.

relay: Split out opening a connection for an extend

Part of 33633.

relay: Refactor some long lines from circuit_extend()

Part of 33633.

relay: Split link specifier checks from circuit_extend()

Part of 33633.

relay: Split state checks out of circuit_extend()

Part of 33633.

relay: Improve the comments on onionskin_answer()

Part of 33633.

relay: Protocol warn when a client gets an extend

circuit_extend() may be called when a client receives an extend cell,
even if the relay module is disabled.

Log a protocol warning when the relay module is disabled.

Part of 33633.

Move LOG_PROTOCOL_WARN to app/config.h

LOG_PROTOCOL_WARN was in core/or.h, but the function it depends on is in
app/config.h. Put them in the same header, to reduce dependencies.

Part of 33633.

relay: Split out relay-only circuit building

Move the relay-only circuit building functions into a new file.

Part of 33633.

relay: Move inform_testing_rechability() to relay

Move inform_testing_rechability() to the relay module, and disable it
when the relay module is disabled.

Part of 33633.

relay: Re-order selftest header

The selftest header declares functions, and stubs for when the relay
module is disabled. Put the functions and stubs in the same order.

Part of 33633.

Merge branch 'maint-0.4.3'

Merge branch 'maint-0.4.2' into maint-0.4.3

Merge branch 'maint-0.4.1' into maint-0.4.2

Merge branch 'ticket33643_skip_035' into ticket33643_skip_041

Appveyor: disable crypto/openssl_version

Add a TOR_SKIP_TESTCASES environment variable for suppressing tests.

For example, "TOR_SKIP_TESTCASES=crypto/.. ./src/test/test" will run
the tests and suppress all the "crypto/" tests.  You could get the
same effect by running "./src/test/test :crypto/..", but that can be
harder to arrange from CI.

Part of a fix/workaround for 33643.

Fix erroneous spaces in circuitmux_ewma.c

Merge branch 'tor-github/pr/1778'

Remove changes files that have appeared in 0.4.3.3-alpha

(findMergedChanges.pl is the tool to do this.)

Merge remote-tracking branch 'tor-github/pr/1807/head'

Merge branch 'tor-github/pr/1792'

changes: Add changes file for ticket 33458

Signed-off-by: David Goulet <dgoulet@torproject.org>

hs-v3: Improve accessor semantic of client cached object

Add an inline helper function that indicates if the cached object contains a
decrypted descriptor or not.

The descriptor object is NULL if tor is unable to decrypt it (lacking client
authorization) and some actions need to be done only when we have a decrypted
object.

This improves code semantic.

Fixes #33458

Signed-off-by: David Goulet <dgoulet@torproject.org>

Merge branch 'maint-0.4.3'

doc: Fix a manual page typo

And consistently talk about extra-info document publishing.

Obviously correct documentation changes.

coverity: Silence spurious unreachable warning

Closes bug 33641; not in any released version of tor.

Merge branch 'maint-0.4.3'

practracker: Regenerate exceptions file

Accept the current state of the tor codebase, including all the
files and functions that have increased technical debt over the
last few months.

Accepts all the current practracker warnings, so that reviewers
can focus on new warnings.

Closes ticket 33635.

Merge remote-tracking branch 'tor-github/pr/1806' into maint-0.4.3

fix typo in comment

doc: Fix a man page typo

Obviously correct changes to already-reviewed code.

Merge branch 'maint-0.4.3'

"ours" to avoid version bump.

Merge branch 'maint-0.4.2' into maint-0.4.3

"ours" to avoid version bump.

Merge branch 'maint-0.4.1' into maint-0.4.2

"ours" to avoid version bump.

Merge branch 'maint-0.3.5' into maint-0.4.1

"ours" to avoid version bump.

Bump version to 0.4.2.7-dev

Bump version to 0.4.1.9-dev

Bump version to 0.3.5.10-dev

Copy changelogs for today's releases to master.

Merge branch 'maint-0.4.3'

Merge branch 'maint-0.4.2' into maint-0.4.3

Merge branch 'maint-0.4.1' into maint-0.4.2

Merge branch 'maint-0.3.5' into maint-0.4.1

Port rsa_private_key_too_long() to work on OpenSSL 1.1.0.

Merge branch 'maint-0.4.3'

Merge branch 'maint-0.4.2' into maint-0.4.3

Merge branch 'maint-0.4.1' into maint-0.4.2

Merge branch 'maint-0.3.5' into maint-0.4.1

Merge branch 'trove_2020_002_041' into maint-0.4.1

Merge branch 'trove_2020_002_035' into maint-0.3.5

Merge branch 'maint-0.4.3'

Merge branch 'maint-0.4.2' into maint-0.4.3

Merge branch 'maint-0.4.1' into maint-0.4.2

Merge branch 'trove_2020_004_041_v2' into maint-0.4.1

Merge branch 'maint-0.4.3'

Split a wide line.

Merge branch 'maint-0.4.3'

Add unittest for TROVE-2020-003.

This unittest currently fails on purpose (to demonstrate the bug) but it will
stop failing after the next commit (the bugfix).

Merge branch 'maint-0.4.3'

Merge branch 'maint-0.4.1' into maint-0.4.2

Merge branch 'maint-0.3.5' into maint-0.4.1

Merge branch 'maint-0.4.2' into maint-0.4.3

Fix TROVE-2020-003.

Given that ed25519 public key validity checks are usually not needed
and (so far) they are only necessary for onion addesses in the Tor
protocol, we decided to fix this specific bug instance without
modifying the rest of the codebase (see below for other fix
approaches).

In our minimal fix we check that the pubkey in
hs_service_add_ephemeral() is valid and error out otherwise.

Trivial bugfixes found during TROVE investigation.

Merge branch 'trove_2020_002_035' into trove_2020_002_041

Use >= consistently with max_bits.

Add off-by-one checks for key length.

Extract key length check into a new function, and check more fields.

In the openssl that I have, it should be safe to only check the size
of n.  But if I'm wrong, or if other openssls work differently, we
should check whether any of the fields are too large.

Issue spotted by Teor.

sendme: Emit version 1 by default

Closes #33623

Signed-off-by: David Goulet <dgoulet@torproject.org>

Merge remote-tracking branch 'tor-github/pr/1784'

Run "practracker.py --regen-overbroad"

We've successfully resolved a bunch of technical debt over the last
few months. Let's remove the allowances for that debt from the
practracker exceptions.

Merge branch 'pr1777_squashed'

Conditionally disable part of hs_intropoint/introduce1_validation

This makes it not fail when ALL_BUGS_ARE_FATAL is enabled. Fixes bug 33546.

Conditionally disable part of dir/param_voting_lookup that expects soft asserts to happen

Try fixing Windows build to work with ALL_BUGS_ARE_FATAL

Documentation fixups

Use --enable-all-bugs-are-fatal on hardened Appveyor build

Don't create new CI jobs; use --enable-all-bugs-are-fatal with some of the existing jobs

Add --enable-all-bugs-are-fatal option to configure

This option enables the ALL_BUGS_ARE_FATAL macro.

And use --enable-all-bugs-are-fatal in the Travis configuration.

Run "make autostyle"

circpad_setup_machine_on_circ(): exit early on error.

This function does a nonfatal assertion to make sure that a machine
is not registered twice, but Tobias Pulls found a case where it
happens.  Instead, make the function exit early so that it doesn't
cause a remotely triggered memory leak.

Fixes bug 33619; bugfix on 0.4.0.1-alpha.  This is also tracked as
TROVE-2020-004.

Merge branch 'maint-0.4.3'

Merge branch 'ticket32672_042_squashed_w_test' into maint-0.4.3

Adjust unit tests for patch for 32672 (rejecting old version)

Patch by Neel Chauhan.

Merge branch 'ticket32672_041_squashed' into ticket32672_042_squashed_w_test

Reject 0.2.9 and 0.4.0 in dirserv_rejects_tor_version()

Merge branch 'maint-0.4.3'

Merge branch 'maint-0.4.2' into maint-0.4.3

Merge branch 'maint-0.4.1' into maint-0.4.2

Merge branch 'maint-0.3.5' into maint-0.4.1

Travis: Produce detailed chutney diagnostics

When a Travis chutney job fails, use chutney's new "diagnostics.sh" tool
to produce detailed diagnostic output.

Closes ticket 32792.

Merge branch 'trove_2020_002_035' into trove_2020_002_041

Fix memory leak in crypto_pk_asn1_decode_private.

(Deep, deep thanks to Taylor for reminding me to test this!)

Add a test for crypto_pk_asn1_decode_private maxbits.

Revise TROVE-2020-002 fix to work on older OpenSSL versions.

Although OpenSSL before 1.1.1 is no longer supported, it's possible
that somebody is still using it with 0.3.5, so we probably shouldn't
break it with this fix.