relay: Set some output arguments in stubs

Fixes part of bug 33674; not in any released version of tor.

Merge branch 'maint-0.4.3'

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

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

Merge branch 'maint-0.4.3'

relay/dirauth: Set some output arguments in stubs

And document how some functions set their output arguments.

Fixes bug 33674; bugfix on 0.4.3.1-alpha.

Set *have_low_ports_out from stub port_parse_ports_relay().

Previously we just ignored this option, which would leave it unset,
and cause an assertion failure later on when running with the User
option.

Fixes bug 33668; bugfix on 0.4.3.1-alpha.

Merge branch 'ticket33678_043' into ticket33678_master

Don't take the practracker disable changes from 0.4.3.

practracker: Disable practracker in git hooks

Disable our coding standards best practices tracker in our git hooks.

0.4.3 branches only.

Closes ticket 33678.

Merge branch 'maint-0.4.3'

dirauth: Remove a duplicate macro definition

Obviously correct changes to already-reviewed code.

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.

Merge branch 'maint-0.4.3'

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.