Merge branch 'maint-0.3.3' into maint-0.3.4

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

Merge remote-tracking branch 'catalyst-github/bug25895_034-squashed' into maint-0.3.4

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.1' into maint-0.3.2

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)

Restor EOL@EOF in routerlist.c

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 'maint-0.3.3' into maint-0.3.4

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 branch 'maint-0.3.3' into maint-0.3.4

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.3' into maint-0.3.4

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.3' into maint-0.3.4

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

Fix Rust cross compilation.

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.3' into maint-0.3.4

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

Bump version to 0.3.4.3-alpha-dev

Bump to 0.3.4.3-alpha.

Add some casts to get test_dir.c compiling on windows.

Fixed bug 26479.  Bug introduced by 0a6f4627a4292e4; bug not in any
released version.

Fix test_geoip failures on windows by writing file in binary mode.

Fixes bug 26480; bug appeared when we re-enabled the geoip tests on
windows.  Bug originally introduced by our fix to 25787; bug not in
any released Tor.

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

Prefer recent Python 3 in autoconf.

Closes ticket 26372.

Hang on; this branch is supposed to be 0.3.4.2-alpha-dev.

Merge branch 'bug25787_squashed' into maint-0.3.4

Revise geoip tests to not require paths of actual geoip config

When I wrote the first one of these, it needed the path of the geoip
file.  But that doesn't translate well in at least two cases:

   - Mingw, where the compile-time path is /c/foo/bar and the
     run-time path is c:\foo\bar.

   - Various CI weirdnesses, where we cross-compile a test binary,
     then copy it into limbo and expect it to work.

Together, these problems precluded these tests running on windows.

So, instead let's just generate some minimal files ourselves, and
test against them.

Fixes bug 25787

Merge branch 'maint-0.3.3' into maint-0.3.4

Fix changes file for memory leak.

Merge branch 'additional_rust_test_fixes' into maint-0.3.4

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge remote-tracking branch 'catalyst-github/allow_fail_rust' into maint-0.3.4

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.

add build.rs to EXTRA_DIST

Allow spaces in CCLD in linker.sh

Hello71 spotted this first.

eol@eof is still the law in 0.3.4 :(

Add a leak suppression for backtrace_alloc

This appears to be an internal rust thing: I don't know why it's
leaking.  We should investigate further.

Merge branch 'maint-0.3.4' into additional_rust_test_fixes

Use different RUST_LINKER_OPTIONS for sanitizers

These appear to work for me.

Fix clang warnings introduced by 9f2b887c5d251b93ee9def708

Fix memory leak in CryptoDigest type

If you're owning a C pointer, you need to implement Drop.

Disable doctests in src/rust/crypto module.

These all need C linking to work, and so far, rustdoc does not seem
to respect cargo setting about build scripts or RUSTOPTIONS.

Fix bugs in rust digest tests

Merge branch 'rust_build_script_v3' into additional_rust_test_fixes

rust: Remove --all-features flag from `cargo test` call in test_rust.sh.

We'd like to feature gate code that calls C from Rust, as a workaround
to several linker issues when running `cargo test` (#25386), and we
can't feature gate anything out of test code if `cargo test` is called
with `--all-features`.

 * FIXES #26400: https://bugs.torproject.org/26400

Allow Rust to fail for now

Multiple issues are preventing Travis CI from passing with Rust right
now.

eol@eof in test-dir.c

Merge branch 'asn_bug24977_final_squashed' into maint-0.3.4

Add changes file for #24977.

Recreate nodelist before use if it's outdated.

We currently only do the check when we are about to use the HSDir indices.

Recreate voting schedule before use if it's outdated.

rust: Remove redundant "testing" feature from tor_log crate.

It was synonymous with the builtin "test" feature.

 * FIXES #26399: https://bugs.torproject.org/26399

Merge branch 'rl1987_bug26196_rebased_034' into maint-0.3.4

Add changes file

Enable DEBUG_SMARTLIST for entire codebase when expensive hardening is on

Enable DEBUG_SMARTLIST in unit tests

Refrain from accessing empty smartlist in test_bridges_clear_bridge_list

Just check that smartlist length is zero instead

Merge remote-tracking branch 'catalyst-github/bug26415_034' into maint-0.3.4

Fix linker errors when building Tor.

This patch fixes linking errors when compiling Tor with the dirauth
module disabled.

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

Fix compilation of Rust crypto doctests

The doctests for src/rust/crypto don't compile for multiple reasons,
including some missing exports and incorrect identifier paths.  Fixes
bug 26415; bugfix on 0.3.4.1-alpha.

Use a rust build script to set linker options correctly for tests.

We need this trick because some of our Rust tests depend on our C
code, which in turn depend on other native libraries, which thereby
pulls a whole mess of our build system into "cargo test".

To solve this, we add a build script (build.rs) to set most of the
options that we want based on the contents of config.rust.  Some
options can't be set, and need to go to the linker directly: we use
a linker replacement (link_rust.sh) for these.  Both config.rust and
link_rust.sh are generated by autoconf for us.

This patch on its own should enough to make the crypto test build,
but not necessarily enough to make it pass.

Remove duplicate MODULE_DIRAUTH_SOURCES from libtor_testing.a

This was already added to LIBTOR_A_SOURCES; it doesn't need to get
added again.

Fixes bug 26402. Bugfix on 0.3.4.1-alpha.

Revert "Remove duplicate MODULE_DIRAUTH_SOURCES from libtor_testing.a"

This reverts commit 70d91bd059869a0ecf24ceb66942ada4f66f03c9.

Remove duplicate MODULE_DIRAUTH_SOURCES from libtor_testing.a

This was already added to LIBTOR_A_SOURCES; it doesn't need to get
added again.

Fixes bug 26402. Bugfix on 0.3.4.1-alpha.

Merge branch 'rust_rand_tests' into maint-0.3.4

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

Merge branch 'maint-0.3.3' into maint-0.3.4