Merge branch 'maint-0.3.0'

Improve descriptor checks in the new guard algorithm.

- Make sure we check at least two guards for descriptor before making
  circuits. We typically use the first primary guard for circuits, but
  it can also happen that we use the second primary guard (e.g. if we
  pick our first primary guard as an exit), so we should make sure we
  have descriptors for both of them.

- Remove BUG() from the guard_has_descriptor() check since we now know
  that this can happen in rare but legitimate situations as well, and we
  should just move to the next guard in that case.

Merge branch 'maint-0.3.0'

Merge branch 'maint-0.2.9' into maint-0.3.0

"Ours" merge to avoid version bump

Bump to 0.2.9.10-dev

Port 0.2.9.10 changelog and releasenotes to master

Merge branch 'maint-0.3.0'

Remove the space before the interpreter name in test-network.sh

Apparently some unixes don't like this.
Part of #21581.

Merge branch 'maint-0.3.0'

Remove a spurious test-network-all warning that triggers when sh is not bash

Part of 21581.

Restore support for test-network.sh on BSD and other systems without bash

(But use bash if it's available.)
This is a workaround until we remove bash-specific code in 19699.

Fixes bug 21581; bugfix on 21562, not in any released version of tor.

Merge remote-tracking branch 'teor/bug21510'

Merge remote-tracking branch 'teor/test21470-029'

Merge remote-tracking branch 'teor/bug21507-029'

Merge remote-tracking branch 'public/bug21407'

Merge branch 'maint-0.3.0'

This is an "ours" merge to avoid a version bump.

bump version to 0.3.0.4-rc

Merge branch 'maint-0.2.9' into maint-0.3.0

This is an "ours" merge to avoid a version bump.

bump version to 0.2.9.10

Merge branch 'maint-0.2.8' into maint-0.2.9

This is an "ours" merge to avoid taking a version bump

Merge branch 'maint-0.2.7-redux' into maint-0.2.8

This is an "ours" merge to avoid taking a version bump, and to
avoid replaying the post-0.2.7.6 history of "maint-0.2.7-redux" onto maint-0.2.8, which already included the relevant changes.

bump version to 0.2.8.13

Bump version to 0.2.7.7

Merge branch 'maint-0.2.6' into maint-0.2.7-redux

"ours" merge to avoid version bumps

bump to 0.2.6.11

Merge branch 'maint-0.2.5' into maint-0.2.6

"ours" merge to avoid version bumps

Bump version to 0.2.5.13

Merge branch 'maint-0.2.4' into maint-0.2.5

"ours" merge to avoid bumping version

Bump to 0.2.4.28

Update changelog sorting rules to know about dirauth changes

Remove changes files from master for items in <=0.3.0.4-rc

Merge branch 'maint-0.3.0'

Merge remote-tracking branch 'public/bug21007_case2_030' into maint-0.3.0

Merge branch 'maint-0.3.0'

Merge branch 'bug21027_v2_squashed' into maint-0.3.0

Change approach to preventing duplicate guards.

Previously I'd made a bad assumption in the implementation of
prop271 in 0.3.0.1-alpha: I'd assumed that there couldn't be two
guards with the same identity.  That's true for non-bridges, but in
the bridge case, we allow two bridges to have the same ID if they
have different addr:port combinations -- in order to have the same
bridge ID running multiple PTs.

Fortunately, this assumption wasn't deeply ingrained: we stop
enforcing the "one guard per ID" rule in the bridge case, and
instead enforce "one guard per <id,addr,port>".

We also needed to tweak our implementation of
get_bridge_info_for_guard, since it made the same incorrect
assumption.

Fixes bug 21027; bugfix on 0.3.0.1-alpha.

Merge branch 'disable_memory_sentinels_squashed'

Code to disable memory sentinels for fuzzing

This feature makes it possible to turn off memory sentinels (like
those used for safety in buffers.c and memarea.c) when fuzzing, so
that we can catch bugs that they would otherwise prevent.

Merge branch 'maint-0.3.0'

Merge remote-tracking branch 'teor/feature21570-030' into maint-0.3.0

Merge branch 'maint-0.3.0'

Merge branch 'bug21369_check_029_squashed' into maint-0.3.0

Merge branch 'maint-0.3.0'

Merge branch 'bug21420_029_squashed' into maint-0.3.0

Revise the logic for picking the start time for link certs

Since 0.2.4.11-alpha (in 0196647970a91d) we've tried to randomize
the start time to up to some time in the past.  But unfortunately we
allowed the start time to be in the future as well, which isn't
really legit.

The new behavior lets the start time be be up to
MAX(cert_lifetime-2days, 0) in the past, but never in the future.

Fixes bug 21420; bugfix on 0.2.4.11-alpha.

Merge branch 'maint-0.3.0'

Merge remote-tracking branch 'teor/bug20711' into maint-0.3.0

Fix a wide line

Fix a "directive within macro arguments" warning

Merge remote-tracking branch 'ahf/bugs/21206'

Use bash in src/test/test-network.sh

This ensures we reliably call chutney's newer tools/test-network.sh when
available.

Fixes bug 21562; bugfix on tor-0.2.9.1-alpha.

Reserve a test-network.sh exit status of 2 for unknown arguments

Part of 21570.

Log tor warnings during 'make test-network-all'

Requires the chutney changes from 21572.
(Otherwise, asks users to upgrade their chutney.)

Implements 21570.

Add one other BUG check to try to fix/solve 21369.

Teor thinks that this connection_dirserv_add_dir_bytes_to_outbuf()
might be the problem, if the "remaining" calculation underflows.  So
I'm adding a couple of checks there, and improving the casts.

Try to check for (and prevent) buffer size INT_MAX overflow better.

Possible fix or diagnostic for 21369.

Add a quiet mode to test-network.sh

Requires the equivalent chutney changes in 21571.

Part of 21570.

Merge branch 'maint-0.3.0'

Merge remote-tracking branch 'public/bug21472_030' into maint-0.3.0

Merge branch 'maint-0.3.0'

Whitespace fix.

Merge branch 'maint-0.3.0'

hs: Fix bad use of sizeof() when encoding ESTABLISH_INTRO legacy cell

When encoding a legacy ESTABLISH_INTRO cell, we were using the sizeof() on a
pointer instead of using the real size of the destination buffer leading to an
overflow passing an enormous value to the signing digest function.
Fortunately, that value was only used to make sure the destination buffer
length was big enough for the key size and in this case it always was because
of the overflow.

Fixes #21553

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

Also allow C_MEASURE_TIMEOUT circuits to lack guard state.

Fixes a case of 21007; bugfix on 0.3.0.1-alpha when prop271 was
implemented. Found by toralf.

Small fixes to fuzzing documentation.

Make display of captured unit test log messages consistent

There was a missing space and an extra colon.

Fixes bug 21510; bugfix on 0.2.9.3-alpha.

Changes file for 21470

Reject versions that have non-numeric prefixes

Unit tests #21507.
Part of #21470.

Add unit tests that ensure out of range versions are rejected

Unit tests for #21278.
Part of #21470.

Add unit tests for version parsing integer size inconsistencies

Unit tests for #21450.
Part of #21470.

Add unit tests for current tor version git tags

Related to #21470

Add unit tests for the current range of tor version status tags

Related to #21470.

Reject Tor versions that contain non-numeric prefixes

strto* and _atoi64 accept +, -, and various whitespace before numeric
characters. And permitted whitespace is different between POSIX and Windows.

Fixes bug 21507 and part of 21508; bugfix on 0.0.8pre1.

Merge branch 'maint-0.2.6' into maint-0.2.7-redux

Merge branch 'maint-0.3.0'

Remove a redundant check in ..transition_affects_guards()

scan-build found that we we checking UseEntryGuards twice.

Fixes bug 21492.

Merge branch 'maint-0.3.0'

Merge branch 'maint-0.2.9' into maint-0.3.0

Check for micro < 0, rather than checking "minor" twice.

Bug found with clang scan-build.  Fixes bug on f63e06d3dc6757d.
Bug not present in any released Tor.

Fix memleak in test_getinfo_helper_onion.

Fix on fc58c37e334fec71a20. Not in any released tor

The UseCreateFast consensus parameter now defaults to 0.

You can still override it with FastFirstHopPK.  But that's
deprecated.

Closes ticket 21407.

Save number of sent/received RELAY_DATA cells for directory connections.

This patch makes us store the number of sent and received RELAY_DATA
cells used for directory connections. We log the numbers after we have
received an EOF in connection_dir_client_reached_eof() from the
directory server.

Merge remote-tracking branch 'meejah/ticket-21329-onions-current'

Bump master to 0.3.1.0-alpha-dev

hs: Avoid a strlen(NULL) if descriptor is not found in cache

Instead of returning 404 error code, this led to a NULL pointer being used and
thus a crash of tor.

Fixes #21471

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

Merge branch 'maint-0.2.9'

whoops; make 21450 compile

Merge branch 'bug21447'

Merge branch 'maint-0.2.9'

Limit version numbers to 0...INT32_MAX.

Closes 21450; patch from teor.

Merge branch 'maint-0.2.9'

Merge branch 'maint-0.2.8' into maint-0.2.9

Merge branch 'maint-0.2.7' into maint-0.2.8

Merge branch 'maint-0.2.6' into maint-0.2.7

Merge branch 'maint-0.2.5' into maint-0.2.6

Merge branch 'maint-0.2.8' of git-rw.torproject.org:/tor into maint-0.2.8

Merge branch 'maint-0.2.7' of git-rw.torproject.org:/tor into maint-0.2.7

Merge branch 'maint-0.2.6' of git-rw.torproject.org:/tor into maint-0.2.6

Merge branch 'maint-0.2.5' of git-rw.torproject.org:/tor into maint-0.2.5

Merge branch 'maint-0.2.9'

Merge branch 'bug21278_extra_029' into maint-0.2.9