Merge branch 'bug27658_029' into maint-0.2.9

Check waitpid return value and exit status in tinytest.c

It's possible for a unit test to report success via its pipe, but to
fail as it tries to clean up and exit.  Notably, this happens on a
leak sanitizer failure.

Fixes bug 27658; bugfix on 0.2.2.4-alpha when tinytest was
introduced.

Merge remote-tracking branch 'teor/ticket27252-029' into maint-0.2.9

Update geoip and geoip6 to the September 6 2018 database.

Bump to 0.2.9.17-dev

Bump to 0.2.9.17

Merge branch 'ticket27344_029' into maint-0.2.9

Tell openssl to build its TLS contexts with security level 1

Fixes bug 27344, where we'd break compatibility with old tors by
rejecting RSA1024 and DH1024.

Merge remote-tracking branch 'teor/bug27461-029' into maint-0.2.9

Merge remote-tracking branch 'teor/bug27463-029' into maint-0.2.9

Windows: Stop calling SetProcessDEPPolicy() on 64-bit Windows

It is not supported, and always fails. Some compilers warn about the
function pointer cast on 64-bit Windows.

Fixes bug 27461; bugfix on 0.2.2.23-alpha.

hs: Silence a spurious warning in rend_client_send_introduction()

gcc 8 warns that extend_info_t.nickname might be truncated by strncpy().

But it doesn't know that nickname can either contain a hex id, or a
nicknames. hex ids are only used for general and HSDir circuits.

Fixes bug 27463; bugfix on 0.1.1.2-alpha.

Windows: Silence a spurious warning in the GetAdaptersAddresses cast

GetProcAddress() returns FARPROC, which is (long long int(*)()) on
64-bit Windows:
https://msdn.microsoft.com/en-us/library/windows/desktop/ms683212(v=vs.85).aspx

But GetAdaptersAddresses() is (long unsigned int(*)()), on both 32-bit
and 64-bit Windows:
https://docs.microsoft.com/en-us/windows/desktop/api/iphlpapi/nf-iphlpapi-getadaptersaddresses

So gcc 8 issues a spurious "incompatible function pointer" warning
about the cast to GetAdaptersAddresses_fn_t.

Silence this warning by casting to a void function pointer, before
the cast to GetAdaptersAddresses_fn_t.

This issue is already fixed by 26481 in 0.3.5 and later, by removing
the lookup and cast.

Fixes bug 27465; bugfix on 0.2.3.11-alpha.

Merge remote-tracking branch 'teor/bug27418-029' into maint-0.2.9

Travis: don't call echo with a --flag as the first argument

When we use echo in Travis, don't pass a --flag as the first argument.

Fixes bug 27418; bugfix on 0.3.4.7-rc.

Travis: when showing a log fails, keep trying to show other logs

When a Travis build fails, and showing a log fails, keep trying to
show the other logs.

Fixes bug 27453; bugfix on 0.3.4.7-rc.

Test: avoid spurious failures in make test-network-all

Before running make test-network-all, delete old logs and test result
files, to avoid spurious failures.

Fixes bug 27295; bugfix on 0.2.7.3-rc.

Test: consistently use $(TEST_NETWORK_ALL_LOG_DIR) in Makefile.am

Part of 27295.

Silence a compilation warning on MSVC 2017 and clang-cl

test.c no longer uses lround(), so we don't need to declare it,
and we can use math.h for fabs().

Fixes bug 27185; bugfix on 0.2.2.2-alpha.

Travis: Skip gcc on Linux with default settings

It's redundant, because all the non-default builds use gcc on Linux.

Part of 27252.

Travis: make the exclude descriptions shorter

Part of 27252.

Travis: Skip a duplicate hardening-off build in Tor 0.2.9

Part of 27252.

Travis: skip gcc on OSX, because the default compiler is clang

Part of #27252.

Mark cert_matches_key as not-intrusive; fix stretch compilation.

Merge remote-tracking branch 'teor/travis-osx-029' into maint-0.2.9

Rewrite test_tortls_cert_matches_key()

Unlike the old test, this test no will no longer mess around with
the forbidden internals of any openssl data structures.

Additionally, it verifies several other behaviors of
tor_tls_cert_matches_key() that we had wanted to verify, such as
the possibility of the certificate's key not matching.

Fixes bug 27226; bugfix on 0.2.5.1-alpha.

Use our x509 wrapper code in tor_tls_cert_matches_key()

This allows us to mock our own tor_tls_get_peer_certificate()
function in order to test ..cert_matches_key(), which will in turn
allow us to simplify test_tortls_cert_matches_key() considerably.

Prep work for the fix for 27226.

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

Travis: add a missing macOS asciidoc env var

Travis: add a missing macOS asciidoc dependency

Travis: put distcheck first for readability

Changes file for Travis: enable macOS builds

Travis: fix a typo

Travis: list installed package versions before building

Part of 24629.

Travis: run an asciidoc build

Implements 27087.

Travis: Use ccache

Part of ticket 26952.

Travis: make macOS builds work for Tor 0.2.9

Tor 0.2.9 needs extra help to find OpenSSL on macOS.

Part of 24629.

Travis: create configure flags once, then echo the flags

Creating the configure flags once avoids inconsistent flags
between configure and distcheck configure.

Echoing the flags helps developers work out what configure is
doing.

(Backported to 0.2.9 and later as a precaution.)

Fixes 27088 on 0.3.4.1-alpha, adds logging in previous releases.

Travis: enable macOS builds

Also:
* explain why we don't install zlib

Part of 24629.

Travis: Rewrite .travis.yml

Build on all compilers:
* default options + hardening

Build on gcc:
* coverage (+ no hardening)
* distcheck
* no hardening

Add some extra logging:
* tail config.log on failure
  (config.log is too long for travis to render)

Put the config in a more logical order
* Sort config items in chronological order
* Put related items together

Part of 24629.

Update geoip and geoip6 to the August 7 2018 database.

Add more optional packages to Travis

Apparently we weren't building with either libcap or libseccomp on
Travis.  Install libcap-dev and libseccomp-dev in .travis.yml.  Closes
ticket 26560.

Don't search for -lpthread on Windows

If we're building for Windows, we want to use windows threads no
matter what, and we don't want to link a pthread library even if it
is present.  Fixes bug 27081; bugfix on 1790dc67607799a in 0.1.0.1-rc.

Changes file for 25440

Fix crash when calling openat with sandbox enabled #25440

The seccomp rule for the openat syscall checks for the AT_FDCWD
constant. Because this constant is usually a negative value, a
cast to unsigned int is necessary to make sure it does not get
converted to uint64_t used by seccomp.

More info on:
https://github.com/seccomp/libseccomp/issues/69#issuecomment-273805980

Merge branch 'fix_nonstandard_malloc_029' into maint-0.2.9

Stop logging link auth warnings on Single Onion Services and Tor2web

Instead, log a protocol warning when single onion services or
Tor2web clients fail to authenticate direct connections to relays.

Fixes bug 26924; bugfix on 0.2.9.1-alpha.

Changes file for bug 26787

Try putting ulimit -c 0 in test_bt.sh to see if it fixes bug 26787

Merge branch 'bug26485_029_squashed' into maint-0.2.9

Add a test for format_recommended_version_list.

Warn when an authority has voted for a version with a space in it.

Another way to try to prevent a recurrence of 26485.

Warn the directory authority operator if their versions list is bogus

Prevents bug 26485; bugfix on 0.1.1.6-alpha.

SKIP test_keygen.sh on Windows until the underlying issue is resolved

Skip an unreliable key generation test on Windows, until the underlying
issue in bug 26076 is resolved.

Fixes bug 26830; bugfix on 0.2.7.3-rc.

put new bridge auth Serge into place

The "Bifroest" bridge authority has been retired; the new bridge authority
is "Serge", and it is operated by George from the TorBSD project.

Closes ticket 26771.

When setting a nonstandard malloc, disable the system malloc.

Closes ticket 20424.

Merge branch 'bug26269_029' into maint-0.2.9

Update geoip and geoip6 to the July 3 2018 database.

Merge branch 'bug26535_029' into maint-0.2.9

ntor_ref.py: pass only strings to subprocess.Popen

Recent Python3 versions seem to require this on Windows.

Fixes bug 26535; bug introduced in f4be34f70d6f277a0f3f73e, which
was apparently intended itself as a Python3 workaround.

Finally remove our EOL@EOF check.

This check was added by mistake long ago.  It will be nice to see
these EOLs disappear from our code over time.

(backport from master)

Fix a memory error in test_shared_random

Bug not in any released Tor.

Changes file for 26467

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

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.

Add .editorconfig to follow coding standards style

Fix a GCC "potential null dereference" warning.

Fixes bug 26269; bugfix on c30be5a82d47328 in 0.2.8.2-alpha

Merge branch 'dannenberg_ipv6_029' into maint-0.2.9

Add a changelog entry.

Add IPv6 orport address for dannenberg.

Update geoip and geoip6 to the June 7 2018 database.

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

Merge branch 'bug26116_029' into maint-0.2.9

Add a unit test for PEM-encrypted documents.

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.

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 'juga/ticket26007_029_02' into maint-0.2.9

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 remote-tracking branch 'catalyst-github/bug25936-029' into maint-0.2.9

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.

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.

Merge branch 'travis_distcheck_029' into maint-0.2.9

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.

Add distcheck support to travis configuration.

Implements 25814.

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

Closes ticket 25818.

Fix an LCOV exclusion pattern in address.c

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

Merge branch 'maint-0.2.5' into maint-0.2.9

Update geoip and geoip6 to the April 3 2018 database.