Merge branch 'maint-0.2.9' into maint-0.3.1

Merge remote-tracking branch 'ahf-github/maint-0.2.9' into maint-0.2.9

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'feature26372_029' into maint-0.2.9

Fix memory leak in test_sr_setup_commits().

This patch fixes a memory leak in test_sr_setup_commits() where the
place_holder is allocated, but never freed again.

See: Coverity CID 1437440.

Fix memory leak in disk_state_parse_commits().

This patch fixes a memory leak in disk_state_parse_commits() where if
commit is NULL, we continue the internal loop, but without ever freeing
the args variable.

See: Coverity CID 1437441.

Fix memory leak in frac_nodes_with_descriptors().

This patch fixes a memory leak in frac_nodes_with_descriptors() where
we might return without free'ing the bandwidths variable.

See: Coverity CID 1437451.

Fix potential memory leak in test_hs_auth_cookies().

This patch fixes a potential memory leak in test_hs_auth_cookies() if a
test-case fails and we goto the done label where no memory clean up is
done.

See: Coverity CID 1437453

Prefer recent Python 3 in autoconf.

Closes ticket 26372.

Merge branch 'maint-0.2.9' into maint-0.3.1

Add .editorconfig to follow coding standards style

Fix an extra space in a changes file

Merge branch 'bug26158_031' into maint-0.3.1

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'dannenberg_ipv6_029' into maint-0.2.9

Add a changelog entry.

Add IPv6 orport address for dannenberg.

Merge remote-tracking branch 'teor/bug26272-031' into maint-0.3.1

Merge branch 'maint-0.2.9' into maint-0.3.1

Update geoip and geoip6 to the June 7 2018 database.

Merge branch 'maint-0.2.9' into maint-0.3.1

Avoid out-of-bounds smartlist access in protover_compute_vote()
and contract_protocol_list()

Silence unused-const-variable warnings in zstd.h on some gcc versions

Fixes bug 26272; bugfix on 0.3.1.1-alpha.

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'bug26116_029' into maint-0.2.9

Add a unit test for PEM-encrypted documents.

Add a missing "return -1" when checking for Ed25519 ID loops

Fixes bug 26158; bugfix on 0.3.0.1-alpha.

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'bug26072_029' into maint-0.2.9

Return -1 from our PEM password callback

Apparently, contrary to its documentation, this is how OpenSSL now
wants us to report an error.

Fixes bug 26116; bugfix on 0.2.5.16.

Merge branch 'maint-0.2.9' into maint-0.3.1

Update geoip and geoip6 to the May 1 2018 database.

Add a missing return after marking a stream for bad connected cell

Fixes bug 26072; bugfix on 0.2.4.7-alpha.

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

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge remote-tracking branch 'juga/ticket26007_029_02' into maint-0.2.9

hs-v3: Add an extra white-space when parsing descriptor

The specification describes the signature token to be right after a newline
(\n) then the token "signature" and then a white-space followed by the encoded
signature.

This commit makes sure that when we parse the signature from the descriptor,
we are always looking for that extra white-space at the end of the token.

It will allow us also to support future fields that might start with
"signature".

Fixes #26069

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

Merge branch 'maint-0.2.9' into maint-0.3.1

Having a ControlPort open doesn't mean we are a client

The any_client_port_set() returns true if the ControlPort is set which is
wrong because we can have that port open but still not behave as a tor client
(like many relays for instance).

Fixes #26062

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

Test read bandwidth measurements with empty file

Stop logging stack contents when reading a zero-length bandwidth file

When directory authorities read a zero-byte bandwidth file, they log
a warning with the contents of an uninitialised buffer. Log a warning
about the empty file instead.

Fixes bug 26007; bugfix on 0.2.2.1-alpha.

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge remote-tracking branch 'catalyst-github/bug25936-031' into maint-0.3.1

Merge remote-tracking branch 'catalyst-github/bug25936-029' into maint-0.2.9

Merge branch 'maint-0.2.9' into maint-0.3.1

Stop logging stack contents when reading a zero-length bandwidth file

When directory authorities read a zero-byte bandwidth file, they log
a warning with the contents of an uninitialised buffer. Log a warning
about the empty file instead.

Fixes bug 26007; bugfix on 0.2.2.1-alpha.

Merge branch 'bug25936-029' into bug25936-031

Show test-suite.log for distcheck on Travis CI

When Travis CI runs make distcheck, test-suite.log doesn't exist in
the expected place.  Add a new make target to show this file and use
it when DISTCHECK=yes in .travis.yml.  Fixes bug 25814; bug not in any
released Tor.

Fix a copy-paste error in the fix for #23693.

Found by coverity; CID 25912; bug not in any released Tor.

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'travis_distcheck_029' into maint-0.2.9

Merge remote-tracking branch 'public/bug23693_031_redux' into maint-0.3.1

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'bug24969_029_v2' into maint-0.2.9

Permit the nanosleep system call in the seccomp2 callbox

Fixes bug 24969; bugfix on 0.2.5.1-alpha when the sandbox was introduced.

Allow cpuworkers to exist without onion keys

Now that we allow cpuworkers for dirport-only hosts (to fix 23693),
we need to allow dup_onion_keys() to succeed for them.

The change to construct_ntor_key_map() is for correctness,
but is not strictly necessary.

Add distcheck support to travis configuration.

Implements 25814.

Merge branch 'maint-0.2.9' into maint-0.3.1

Add support for the coveralls.io coverage tool in travis config

Closes ticket 25818.

Merge branch 'maint-0.2.9' into maint-0.3.1

Fix an LCOV exclusion pattern in address.c

Fix another crash-on-no-threadpool bug.

This one happens if for some reason you start with DirPort enabled
but server mode turned off entirely.

Fixes a case of bug 23693; bugfix on 0.3.1.1-alpha.

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge remote-tracking branch 'ahf-github/bugs/24854_029_2' into maint-0.2.9

Lift the list of default directory servers into their own file.

This patch lifts the list of default directory authorities from config.c
into their own auth_dirs.inc file, which is then included in config.c
using the C preprocessor.

Patch by beastr0.

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

Switch Travis to stable rust

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'maint-0.2.5' into maint-0.2.9

Update geoip and geoip6 to the April 3 2018 database.

Merge branch 'maint-0.2.9' into maint-0.3.1

Fix CID 1430932

Coverity found a null pointer reference in nodelist_add_microdesc().
This is almost certainly impossible assuming that the routerstatus_t
returned by router_get_consensus_status_by_descriptor_digest() always
corresponds to an entry in the nodelist.  Fixes bug 25629.

Merge branch 'maint-0.2.9' into maint-0.3.1

Remove sb_poll check: all poll() calls are ok.

Add the poll() syscall as permitted by the sandbox

Apparently, sometimes getpwnam will call this.

Fixes bug 25513.

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'maint-0.2.5' into maint-0.2.9

Merge remote-tracking branch 'catalyst-github/bug25388-025' into maint-0.2.5

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'maint-0.2.5' into maint-0.2.9

Update geoip and geoip6 to the March 8 2018 database.

version bump to 0.3.1.10-dev

Merge branch 'maint-0.2.9' into maint-0.3.1

"ours" merge to avoid version bump.

Bump version to 0.2.9.15-dev

Update to 0.3.1.10

Merge branch 'maint-0.2.9' into maint-0.3.1

"ours" merge to avoid version bump.

version bump to 0.2.9.15

Merge branch 'maint-0.2.9' into maint-0.3.1

Protover tests: disable some obsoleted tests

These were meant to demonstrate old behavior, or old rust behavior.

One of them _should_ work in Rust, but won't because of
implementation details.  We'll fix that up later.

Spec conformance on protover: always reject ranges where lo>hi

Forbid UINT32_MAX as a protocol version

The C code and the rust code had different separate integer overflow
bugs here.  That suggests that we're better off just forbidding this
pathological case.

Also, add tests for expected behavior on receiving a bad protocol
list in a consensus.

Fixes another part of 25249.

Forbid "-0" as a protocol version.

Fixes part of 24249; bugfix on 0.2.9.4-alpha.

Add more of Teor's protover tests.

These are as Teor wrote them; I've disabled the ones that don't pass
yet, with XXXX comments.

Add some protover vote round-trip tests from Teor.

I've refactored these to be a separate function, to avoid tricky
merge conflicts.

Some of these are disabled with "XXXX" comments; they should get
fixed moving forward.

Add another NULL-pointer fix for protover.c.

This one can only be exploited if you can generate a correctly
signed consensus, so it's not as bad as 25074.

Fixes bug 25251; also tracked as TROVE-2018-004.

Correctly handle NULL returns from parse_protocol_list when voting.

In some cases we had checked for it, but in others we had not.  One
of these cases could have been used to remotely cause
denial-of-service against directory authorities while they attempted
to vote.

Fixes TROVE-2018-001.

Document how to allow partial Travis failures

Add some commented-out allow_failures clauses to make it easier to
temporarily allow less-critical sub-builds to fail while still
reporting success.

Update the .gitmodules to refer to project-level tor-rust-dependencies

Closes most of #25323.

Merge branch 'maint-0.2.9' into maint-0.3.1

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

Merge branch 'maint-0.2.9' into maint-0.3.1

"ours" merge to avoid conflicts with the cherry-picked fix for 24898.