Revert 4438ef32's changes to test_address.c

Apparently some versions of the mac sdk care about the ordering of
net/if.h wrt other headers.

Fixes bug 25319; bug not in any released tor.

Remove a bunch of other redundant #includes

Folks have found two in the past week or so; we may as well fix the
others.

Found with:

\#!/usr/bin/python3
import re

def findMulti(fname):
    includes = set()
    with open(fname) as f:
        for line in f:
            m = re.match(r'^\s*#\s*include\s+["<](\S+)[>"]', line)
            if m:
                inc = m.group(1)
                if inc in includes:
                    print("{}: {}".format(fname, inc))
                includes.add(m.group(1))

import sys
for fname in sys.argv[1:]:
    findMulti(fname)

Merge remote-tracking branch 'fristonio/ticket-25261'

Use autoconf to check for optional zstd functionality.

Fixes a bug in our zstd-static code.  Bug not in any released
version of Tor.

Merge branch 'maint-0.3.3'

Merge branch 'maint-0.3.2' into maint-0.3.3

"ours" merge to avoid taking 24902 a second time and having to reconcile
all the merge conflicts.

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 'dgoulet/ticket24902_029_05' into maint-0.2.9

Whoops. 256 was not big enough.

Allow checkpointing of non-sha1 digests.

This is necessary because apparently v3 rendezvous cpath hops use
sha3, which I had forgotten.

Bugfix on master; bug not in any released Tor.

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.3.3'

Remove changes file for 24898-029 backports in maint-0.3.2

These patches were already merged, and so don't need a changes file
in these branches.

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch '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

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

stop calling channel_mark_client in response to a create_fast

since all it does is produce false positives

this commit should get merged into 0.2.9 and 0.3.0 *and* 0.3.1, even
though the code in the previous commit is already present in 0.3.1. sorry
for the mess.

[Cherry-picked]

stop calling channel_mark_client in response to a create_fast

since all it does is produce false positives

this commit should get merged into 0.2.9 and 0.3.0 *and* 0.3.1, even
though the code in the previous commit is already present in 0.3.1. sorry
for the mess.

backport to make channel_is_client() accurate

This commit takes a piece of commit af8cadf3a9 and a piece of commit
46fe353f25, with the goal of making channel_is_client() be based on what
sort of connection handshake the other side used, rather than seeing
whether the other side ever sent a create_fast cell to us.

Remove the return value of node_get_prim_orport() and node_get_prim_dirport()

Compilation workaround for windows, which lacks O_SYNC

Bug not in any released Tor.

tor_zstd_format_version shouldn't be built when !HAVE_ZSTD

Fixes bug 25276; bugfix not in any released Tor.

Merge branch 'maint-0.3.3'

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

Merge remote-tracking branch 'ffmancera-1/bug18918'

Merge branch 'bug23909'

Merge branch 'bug24914'

Fix a stack-protector warning: don't use a variable-length buffer

Instead, define a maximum size, and enforce it with an assertion.

Merge branch 'onion_ntor_malloc_less'

Documentation fixes suggested by catalyst.

Merge remote-tracking branch 'isis/bug25185'

Merge branch 'maint-0.3.3'

Merge remote-tracking branch 'isis/bug25171' into maint-0.3.3

Merge branch '25162_zstd_static'

This stats_n_seconds_working variable needs to be static now.

(When a variable isn't going to be declared extern in the header, we
require that it be static.)

Merge remote-tracking branch 'valentecaio/t-25081'

Merge remote-tracking branch 'valentecaio/t-24714'

tests: Remove duplicate included header file in src/test/test.c.

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

Merge branch 'bug18105'

Merge branch 'bug24484_squashed'

Remove workaround code for systems where free(NULL) is busted.

Add an autoconf test to make sure we won't regret it.

Closes ticket 24484.

Bump version in master to 0.3.4.0-alpha-dev

Merge branch 'protover_rust_compat_v3'

rust protover: match the C implementation on empty-str cases

Empty versions lists are permitted; empty keywords are not.

Forbid u32::MAX as a protover range element in rust

Part of the 25249 fix to make rust match the C.

Rust protover compat: forbid more than MAX_VERSIONS_TO_EXPAND in a range

Also correct MAX_VERSIONS_TO_EXPAND to match the C.

NOTE that this patch leads to incorrect behavior: the C code allows
huge ranges; it just doesn't allow votes on them (currently).  For
full compatibility, we'll need to make the rust code store ranges as
ranges natively, possibly using something like the range_map crate.

Still, this patch is smaller than a "proper" fix.

Fixes TROVE-2018-003.

ticket 25261: Removed multiple includes of transports.h in connection.c

Merge remote-tracking branch 'mikeperry/bug24769'

Merge branch 'tests_rust'

Merge branch 'maint-0.3.2'

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.2.9' into maint-0.3.1

fix make check-changes

Merge remote-tracking branch 'dgoulet/ticket24902_029_05'

Make check-changes happy

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

man: Document default values if not in the consensus for DoS mitigation

Fixes #25236

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

Merge remote-tracking branch 'dgoulet/bug25223_029_01' into ticket24902_029_05

dos: Add extra safety asserts in cc_stats_refill_bucket()

Never allow the function to set a bucket value above the allowed circuit
burst.

Closes #25202

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

dos: Don't set consensus param if we aren't a public relay

We had this safeguard around dos_init() but not when the consensus changes
which can modify consensus parameters and possibly enable the DoS mitigation
even if tor wasn't a public relay.

Fixes #25223

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

Remove 25170 changes file from master: it was already merged in 0.3.3.2-alpha

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.3.2'

Merge branch 'maint-0.2.9' into maint-0.3.1

dirserv: Improve returned message when relay is rejected

Explicitly inform the operator of the rejected relay to set a valid email
address in the ContactInfo field and contact bad-relays@ mailing list.

Fixes #25170

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

Merge remote-tracking branch 'dgoulet/ticket24902_029_05'

Have tor_addr hashes return a randomized hash for AF_UNSPEC.

We don't expect this to come up very much, but we may as well make
sure that the value isn't predictable (as we do for the other
addresses) in case the issue ever comes up.

Spotted by teor.

Fix a typo in an address_set.c comment.

Merge branch 'maint-0.3.2'

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'bug23318-redux_029' into maint-0.2.9

Tweaks into router_should_be_dirserver() log msg.

Fixed log message that has been changed in commit 5ea993fa5a.

Signed-off-by: Fernando Fernandez Mancera <ffmancera@riseup.net>

Merge branch 'maint-0.3.2'

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 'public/bug24198_029' into maint-0.2.9

Merge branch 'maint-0.3.2'

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'ticket24315_029' into maint-0.2.9

Merge branch 'maint-0.3.2'

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 'public/bug21074_029' into maint-0.2.9

Merge remote-tracking branch 'isis/bug25127_redux'

fix compilation.

chnages file for 25120

Merge branch 'bug25120'

Lower log-level in different error conditions in entropy selection.

This patch lowers the log-level from warning to info in the cases where
we are going to attempt another method as entropy source to hopefully
make the user feel less concerned.

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

Merge branch 'maint-0.3.1' into maint-0.3.2

Merge branch 'maint-0.3.2'

Merge branch 'maint-0.2.9' into maint-0.3.1

Merge branch 'bug24978_029_enable' into maint-0.2.9

Bump version to 0.3.3.2-alpha-dev

Increment the release date in the changelog.

   "We fray into the future, rarely wrought
     save in the tapestries of afterthought"
       -- Richard Wilbur, _Years End_

rust: Remove now unused byte_slice_is_c_like() utility.

rust: Remove empty_static_cstr() in favour of new cstr!() macro.

rust: Use tor_util::strings utils for protover_compute_for_old_tor.