Merge branch 'pr1599_squashed'

practracker: Wrap long lines in the test script

practracker: Test practracker exit status

Part of 32705.

practracker: Fail tests if any process exits unexpectedly

Part of 32705.

practracker: Capture stderr output in test_practracker.sh

And canonicalise file paths, so that output is consistent, even in
out-of-tree builds.

Capturing stderr output produces better diagnostics.

Diagnostics for 32705.

scripts/git: Improve branch list comments

Hopefully, these explanations make it easier to get the next branch
transition right.

Part of 32772.

scripts/git: Fix git-merge-forward after removing 0.2.9

We need a test branch for the earliest maint branch (maint-0.3.5),
but we can't merge forward into it, because there is no previous
active maint branch.

Also, edit a git-push-all comment that still said "029".

Bugfix on 32772; bug not in any released version of tor.

Merge branch 'tor-github/pr/1633'

Merge branch 'ticket32892_043_01_squashed'

chan: Remove dead var cell handler from channel_t

The variable lenght cells are handled directly by
channel_tls_handle_var_cell() from an OR connection reading its inbuf. The
channel var cell handler (agnostic) was never used.

Closes #32892

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

Merge remote-tracking branch 'tor-github/pr/1636'

Merge branch 'ticket32883'

Merge branch 'ticket32825_squashed'

changes file for 32825

Rename UTIL_TEST_NO_WIN to reflect intent

Now that these tests are skipped on windows _and_ android, we should
name the macro that skips them after the functionality that the
tests depend on.

Closes last part of 32825.

Merge remote-tracking branch 'tor-github/pr/1659'

Merge branch 'tor-github/pr/1656'

Merge branch 'maint-0.4.2'

practracker: print a notice to stderr when disabled

When TOR_DISABLE_PRACTRACKER is set, print a message to
stderr when skipping practracker checks.

Part of 32705.

Deprecate the ClientAutoIPv6ORPort option

test_practracker.sh: never disable practracker

When practracker is disabled, its output will be empty.  We don't
want that happening during our tests.

Fixes bug 32705; bugfix on 0.4.2.1-alpha, when test_practracker.sh
was introduced.

Merge branch 'pr_1643_squashed'

manpage: group some SafeSocks-related options

Move some SafeSocks-related options near each other.  Part of ticket
32846.

manpge: reorder some padding-related options

Move ReducedCircuitPadding and ReducedConnectionPadding immediately
after the corresponding non-Reduced options.  Part of ticket 32846.

manpage: GuardfractionFile is authority-only

Only directory authorities use the GuardfractionFile option, so move
it to that section.  Part of ticket 32846.

manpage: alphabetize client options

Alphabetize client options in tor.1.txt.  Closes ticket32846.

hs-v2: Always check rend_cache validity before using it

When looking up an entry in the rend_cache, stop asserting that it exists but
rather confirm it exists and if not, return that no entry was found.

The reason for that is because the hs_circ_cleanup_on_free() function (which
can end up looking at the rend_cache) can be called from the
circuit_free_all() function that is called _after_ the rend cache is cleaned
up in tor_free_all().

We could fix the free all ordering but then it will just hide a future bug.
Instead, handle a missing rend_cache as a valid use case as in while we are in
the cleanup process.

As Tor becomes more modular, it is getting more and more difficult to ensure
subsystem callstack ordering thus this fix aims at making the HSv2 subsystem
more robust at being called while tor is pretty much in any kind of state.

Fixes #32847.

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

Merge branch 'disable_ns_macro'

Merge branch 'ticket22029_attempt_squashed'

Add changes file for Bug #22029

Update tor.1 man page for Bug #22029

Add dirserv_load_fingerprint_file() test

Allow ed25519 keys to be banned in approved-routers

Changes file for ticket 32887 (NS*() removal)

Fix wide lines from NS() removal fallout.

Remove NS*() macros from test.h.

We never used them very much, and although they had potential to
clarify some of our tests, they also made some of the logic harder
for people to follow.  Clang-format can't make head or tail of them,
so the time has come to say goodbye to them.

Remove all usage of the NS*() macros in test*.c

This is an automatically generated commit, made with the following
kludgey perl script. It results in a number of wide lines, which
I'll clean up in a subsequent commit.

#/usr/bin/perl -w -i
$mod = "NS_MODULE";
$submod = "NS_SUBMODULE";
$last_was_empty = 0;

while (<>) {

    s/\bASPECT\(\s*(\w+)\s*,\s*(\w+)\s*\)/$1_$2/;

    if (/# *define +NS_MODULE +(\w+)/) {
        $mod = $1;
        next;
    } elsif (/# *define +NS_SUBMODULE +(\w+)/) {
        $submod = $1;
        next;
    }

    next if (/#undef NS_(SUB)?MODULE/);

    s/NS\(\s*test_main\s*\)/test_${mod}_${submod}/;
    s/NS\(\s*(\w+)\s*\)/${mod}_${submod}_$1/g;
    s/NS_FULL\(\\s*(\w+)\s*,\s*(\w+),\s*(\w+)\s*\)/$1_$2_$3/;
    s/^(\s*)NS_MOCK\(\s*(\w+)\s*\)/$1MOCK($2,\n$1     ${mod}_${submod}_$2)/;
    s/NS_UNMOCK\(\s*(\w+)\s*\)/UNMOCK($1)/;
    s/TEST_CASE\(\s*(\w+)\s*\)/{ "$1", test_${mod}_$1, TT_FORK, NULL, NULL }/;
    s/TEST_CASE_ASPECT\(\s*(\w+)\s*,\s*(\w+)\s*\)/{ "$1_$2", test_${mod}_$1_$2, TT_FORK, NULL, NULL }/;
    s/NS_DECL\(\s*([^,]+)\s*,\s*([^,]+)\s*,\s*(\(.*)\);/static $1 ${mod}_${submod}_$2$3;\nATTR_UNUSED static int ${mod}_${submod}_$2_called = 0;/;
    s/\bCALLED\(\s*(\w+)\s*\)/${mod}_${submod}_$1_called/;

    if (/^$/) {
        print if (! $last_was_empty);
        $last_was_empty = 1;
    } else {
        $last_was_empty = 0;
        print;
    }

    if (eof) {
        $mod = "NS_MODULE";
        $submod = "NS_SUBMODULE";
        $last_was_empty = 0;
    }
}

# Please enter the commit message for your changes. Lines starting
# with '#' will be kept; you may remove them yourself if you want to.
# An empty message aborts the commit.
#
# Date:      Thu Jan 9 10:26:10 2020 -0500
#
# On branch disable_ns_macro
# Changes to be committed:
# modified:   src/test/test_accounting.c
# modified:   src/test/test_compat_libevent.c
# modified:   src/test/test_dir.c
# modified:   src/test/test_dir_handle_get.c
# modified:   src/test/test_dns.c
# modified:   src/test/test_options.c
# modified:   src/test/test_procmon.c
# modified:   src/test/test_rendcache.c
# modified:   src/test/test_router.c
# modified:   src/test/test_routerset.c
# modified:   src/test/test_status.c
# modified:   src/test/test_tortls.c
# modified:   src/test/test_tortls_openssl.c
# modified:   src/test/test_util_format.c
# modified:   src/test/test_util_process.c
#
# Untracked files:
# experiments/
# locate_options.sh
# un_ns.pl
#

# Please enter the commit message for your changes. Lines starting
# with '#' will be kept; you may remove them yourself if you want to.
# An empty message aborts the commit.
#
# Date:      Thu Jan 9 10:26:10 2020 -0500
#
# On branch disable_ns_macro
# Changes to be committed:
# modified:   src/test/test_accounting.c
# modified:   src/test/test_compat_libevent.c
# modified:   src/test/test_dir.c
# modified:   src/test/test_dir_handle_get.c
# modified:   src/test/test_dns.c
# modified:   src/test/test_options.c
# modified:   src/test/test_procmon.c
# modified:   src/test/test_rendcache.c
# modified:   src/test/test_router.c
# modified:   src/test/test_routerset.c
# modified:   src/test/test_status.c
# modified:   src/test/test_tortls.c
# modified:   src/test/test_tortls_openssl.c
# modified:   src/test/test_util_format.c
# modified:   src/test/test_util_process.c
#
# Untracked files:
# experiments/
# locate_options.sh
# un_ns.pl
#

Replace various NS_MODULE/SUBMODULE vals with shorter names.

This will help avoid wide lines in our output.

Put all NS_DECL invocations on single lines.

This is going to make my script happier; these lines will soon
disappear.

Always define NS_MODULE and NS_SUBMODULE when NS is used.

When these macros aren't defined, the expansions of the NS macros
can get particularly ugly.

Merge branch 'pre_formatter_cleanups_squashed'

changes file for 32764.

process.h: include stdbool.h

We use bool, so we should include stdbool.

conscache.h: declare config_line_t structure.

This frees us from a dependency on include order.

Turn authdir_mode_v3() into a non-inline function

This change means that authmode.h no longer needs to see
or_options_t, and frees us from an ordering dependency.

Include ht.h in all headers that use HT_ENTRY()

Without this change, compilation success depends on include order in
several tricky ways.

consdiffmgr.h: use struct declarations for several types

This prevents a dependency on include order.

hs_circuitmap.h: use a struct declaration

This fixes a case where our compilation would depend on include order.

routerparse.h: include parsecommon.h when exposing token table.

Without this include, our compilation depends more on include order.

process.h: add a struct declaration for buf_t.

This lets us avoid a dependency on include order.

timers.h: add declaration for timeval.

This frees us from some dependencies on include order.

lib/conf: include torint.h in files that use it.

This frees us from some dependencies on include order.

control_hs.h: use a couple of struct delcarations

Doing this frees us from some assumptions about include order.

connection.h: new includes and struct declarations

Using these frees us from several dependencies on include order.

siphash.h: include stdint.

Doing this gives us a valid uint64_t type, freeing us from
dependencies on include order.

hs_circuit: use struct declaration.

This frees us from a dependency on include order.

or_options_st.h: Add an include and a struct declaration.

This frees us from some dependencies on include order.

Include x509.h in tortls_internal.h.

This gives us the definition of tor_x509_cert_impl_t, and makes us
less dependent on include order.

Include sys/types.h in fdio.h, for "off_t".

Otherwise our compilation depends on include order.

Use raw_assert in ht.h

Also, include torerr.h from ht.h if we are using raw_assert.

Otherwise, our includes need to be ordered so that ht.h comes after
util_log.h.

Remove extra ; from tt_assert() macro definition.

We were actually omitting the semicolon in a few places, leading to
confusing indentation and some cocci failures.

Use new ENABLE/DISABLE_GCC_WARNING

This is an automated commit, generated by:

perl -i -pe 'next if /define/; s/((?:ENABLE|DISABLE)_GCC_WARNING)\(([A-Za-z0-9_\-]+)\)/$1(\"-W$2\")/' src/*/*/*.[ch] src/*/*.[ch]

Replace DISABLE/ENABLE_GCC_WARNING with something that will confuse clang-format less

core/include.am: spelling fix

It's 2020. Update the copyright dates with "make update-copyright"

Run "make autostyle"; add an endif comment

router: Improve port search function comments

speaking of utf-8, remove strange char from comment

Merge branch 'ticket32845_squashed'

changes: file for 32845

string: Check UTF-8 string pointer and length

If they are inconsistent, output a raw bug log.

Part of 32845.

Merge remote-tracking branch 'tor-github/pr/1640'

test_helpers: add a missing free for CID 1457527.

Add new state to connection_proxy_state_to_string()

Also, add a CTASSERT() to make sure that we don't let this array get
out-of-sync again.

Found by coverity; this is CID 1457528.

string: Add extra UTF-8 test cases

These test cases are validated differently by some
programming languages, because those languages have
incorrect UTF-8 implementations.

We want to make sure that tor validates them correctly.

Closes ticket 32845.

Space the a-d unsigned ints in tor_inet_aton()

Merge branch 'haxxpop/tcp_proxy_squashed' into tcp_proxy_squshed_and_merged

circuit: Add change file

test: HTTP CONNECT protocol

test: HAPRoxy protocol

test: Implement haproxy

circuit: Implement haproxy

Merge remote-tracking branch 'tor-github/pr/1615'

Add documentation about building a TAGS file.

Patch from opara.  Closes ticket 32779.

Merge remote-tracking branch 'tor-github/pr/1620'

Merge branch 'maint-0.4.2'

Merge remote-tracking branch 'tor-github/pr/1612' into maint-0.4.2

When initializing pthreads, always set the main thread.

Fixes bug 32884.  This is a bugfix on 0.3.3.1-alpha, when we started
allowing restart-in-process with tor_api.h.

Make ntmain.h functions stubs when NT_SERVICE not enabled.

This lets us simplify main.c a little, and avoid a practracker
exception.

Followup from #32883.

Restore feature where nt-services detect non-"run_tor" modes.

Followup for #32883.

Use tor_api.h entry points in ntmain.c.

When we updated main.c to use the same entry points as tor_api.h, we
didn't update ntmain.c.  This led to bugs as the two got out of
sync.

There are two calls that we drop in this patch: first, I drop the
call to set_main_thread(), since that's redundant with the init
function in compat_winthreads.c (but see #32884).  Second, I drop
the check for CMD_RUN_TOR: I'll add that back with a subsequent
commit.

Closes ticket 32883.

Merge branch 'maint-0.4.2'

Merge branch 'maint-0.4.1' into maint-0.4.2

update changes file to pass "make check-changes"

Merge branch 'maint-0.4.1' into maint-0.4.2

Fix sandbox crash during reload of logging configuration

Allow calls to dup() which was introduced in commit a22fbab986.

From a security perspective, I don't think this should impact the
security of the sandbox significantly. As far as I can tell, there
is nothing an adversary can do with a duplicated FD that can't be
done with the original.

Add changes file

Remove 0.2.9 from git scripts

Fix spacing in tor_sscanf() call in tor_inet_aton()

Fix spacing in tor_inet_aton() function header