Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.2

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.

Bump to 0.3.3.8-dev.

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge remote-tracking branch 'github/bug26269_031' into maint-0.3.2

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.2

Merge branch 'bug26269_029' into maint-0.2.9

Bump to 0.3.3.8

Clear all control.c flags on control_free_all()

Fixes bug 25512.

(Cherry-picked from 3519d0c8086ede

Merge remote-tracking branch 'catalyst-github/bug26455_033' into maint-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.2

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge remote-tracking branch 'mikeperry/bug25705_v3_033' into maint-0.3.3

Update geoip and geoip6 to the July 3 2018 database.

Merge branch 'maint-0.2.9' into maint-0.3.2

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'bug26535_032' into maint-0.3.2

Merge branch 'bug26535_029' into maint-0.2.9

hs_ntor_ref.py: pass only strings to subprocess.Popen

Recent Python3 versions seem to require this on Windows.

Fixes bug 26535; bug copied from ntor_ref.py on 0.3.1.1-alpha.

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.

Merge remote-tracking branch 'dgoulet/bug26523_033_01' into maint-0.3.3

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.1

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)

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.1

Fix a memory error in test_shared_random

Bug not in any released Tor.

Fix some memory errors in the recent coverity fixes.

Found by asan on travis :/

Merge branch 'maint-0.3.2' into maint-0.3.3

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

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge remote-tracking branch 'ahf-github/maint-0.3.0' into maint-0.3.1

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

Changes file for 26467

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.3.1' into maint-0.3.2

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

control: Make HSPOST properly parse HSADDRESS= param

For HSv3, the HSADDRESS= wasn't properly parsed for the HSPOST command. It now
correctly use it and furthermore sends back a "200 OK" in case the command is
successful for a v3 descriptor.

Fixes #26523

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

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.3.1' into maint-0.3.2

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 memory leak in helper_add_hsdir_to_networkstatus().

This patch fixes a memory leak in helper_add_hsdir_to_networkstatus()
where the rs object might not get properly freed.

See: Coverity CID 1437427.

Fix memory leak in new_establish_intro_cell().

This patch fixes a memory leak in new_establish_intro_cell() that could
happen if a test assertion fails and the *cell_out value isn't properly
free'd.

See: Coverity CID 1437445

Fix memory leak in test_channelpadding_consensus().

The relay variable is always allocated, but might not be freed before we
return from this function.

See: Coverity CID 1437431

Fix memory link in test_link_specifier().

This patch fixes a memory leak in test_link_specifier() where ls might
not get freed in case one of the test macros fails.

See: Coverity CID 1437434.

Fix memory leak in decode_link_specifiers().

This patch fixes a memory leak in decode_link_specifiers() where the
hs_spec variable might leak if the default label is taken in the
switch/case expression.

See: Coverity CID 1437437.

Fix memory leak in client_likes_consensus().

This patches fixes a memory leak in client_likes_consensus() where if
consensus_cache_entry_get_voter_id_digests() would fail we would return
without having free'd the voters list.

See: Coverity CID 1437447

Fix potential memory leak in hs_helper_build_hs_desc_impl().

This patch fixes a memory leak in hs_helper_build_hs_desc_impl() where
if a test assertion would fail we would leak the storage that `desc`
points to.

See: Coverity CID 1437448

Fix memory leak in pick_hsdir_v3().

This patch fixes a memory leak in pick_hsdir_v3() where we might return
early, but forgot to free the responsible_hsdirs variable. We solve this
by not allocating storage for responsible_hsdirs until it's actually
needed.

See: Coverity CID 1437449

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

Fix potential memory leak in hs_helper_build_intro_point().

This patch fixes a potential memory leak in
hs_helper_build_intro_point() where a `goto done` is called before the
`intro_point` variable have been assigned to the value of the `ip`
variable.

See: Coverity CID 1437460
See: Coverity CID 1437456

Use correct CARGO_HOME in test_rust.sh

Out-of-tree builds could fail to run the rust tests if built in
offline mode.  cargo expects CARGO_HOME to point to the .cargo
directory, not the directory containing .cargo.

Fixes bug 26455; bug not in any released tor.

Prefer recent Python 3 in autoconf.

Closes ticket 26372.

Fix changes file for memory leak.

fix memory leak in protover.c

Fix a memory leak where directory authorities would leak a chunk of
memory for every router descriptor every time they considered voting.

This bug was taking down directory authorities in the live network due
to out-of-memory issues.

Fixes bug 26435; bugfix on 0.3.3.6.

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'maint-0.3.2' into maint-0.3.3

Add .editorconfig to follow coding standards style

Fix a bug in my fix for #26258

The fix here is use a different bourne shell subsitution for
CARGO_ONLINE, so that an empty string counts as set.

Merge remote-tracking branch 'public/bug26258_033' into maint-0.3.3

squash! Make sure that the test_rust.sh script fails when a test fails

Also make sure that we're actually running the test from within the right
cwd, like we do when we're building.  This seems necessary to avoid
an error when running offline.

Amusingly, it appears that we had this bug before: we just weren't
noticing it, because of bug 26258.

test_entrynodes: fix a GCC warning

Some versions of GCC complain that the bfn_mock_node_get_by_id
function might return NULL, but we're assuming that it won't.
(We're assuming it won't return NULL because we know in the tests
that we're passing it valid IDs.)

To make GCC happy, tt_assert() that each node_t is set before using
it.

Fixes a second case of bug26269; bugfix on 0.3.0.1-alpha.

Fix a GCC "potential null dereference" warning.

Fixes bug 26269; bugfix on c30be5a82d47328 in 0.2.8.2-alpha

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.3.2' into maint-0.3.3

Fix an extra space in a changes file

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'bug26158_031' into maint-0.3.1

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'bug25686_diagnostic_032' into maint-0.3.2

bump to 0.3.3.7-dev

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'dannenberg_ipv6_029' into maint-0.2.9

Add a changelog entry.

Add IPv6 orport address for dannenberg.

Bump to 0.3.3.7

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.3.2' into maint-0.3.3

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

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.1

Several attempts to diagnose ticket 25686

There are a few reasons that relays might be uploading desciptors
without saying X-Desc-Gen-Reason:
  1. They are running an old version of our software, before 0.3.2.stable.
  2. They are not running our software, but they are claiming they
     are.
  3. They are uploading through a proxy that strips X-Desc-Gen-Reason.
  4. They somehow had a bug in their software.

According to the 25686 data, 1 is the most common reason.  This
ticket is an attempt to diagnose case 4, or prove that case 4
doesn't actually happen.

Update geoip and geoip6 to the June 7 2018 database.

Merge branch 'maint-0.3.2' into maint-0.3.3