codetool: post-processor for clang-format

This code transformer makes a couple of changes that we want for our
source code, and can be expanded to handle more.

Merge branch 'bug33103'

Merge remote-tracking branch 'public/practracker_regen_overbroad_2'

test: Missing hs_init() in some hs_control tests

Without the hs_init(), the caches are not initialized and the previous commit
was causing an assert due to the missing caches.

But regardless of that, those tests were not initializing but were calling
hs_free_all().

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

hs-v3: Remove descriptor when we remove client authorization

When the ONION_CLIENT_AUTH_REMOVE command is given to tor, now also remove the
descriptor associated with the client authorization credentials.

Fixes #33148

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

Fix a memory leak in handling GETINFO.

Fixes bug 33103; bugfix on 0.4.3.1-alpha.

Merge branch 'pr1701-fix-typo'

configure: Fix and improve summary pretty printing

Fixes #32230

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

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

Merge branch 'tor-github/pr/1704'

Merge branch 'tor-github/pr/1700'

address: Fix comments in address.h

And improve inline function spacing, and function declaration
spacing.

Comment-only change.

go in to fix one typo, end up fixing more

practracker: use shutil.move so --regen will work on windows

On windows you can't os.rename() a file if the target filename
already exists.

practracker: integration tests for --regen and --regen-overbroad

practracker: add a --regen-overbroad option to remove overbroad exceptions.

Closes ticket 32372.

hs-v3: Purge ephemeral client auth on NEWNYM

Fixes #33139.

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

Remove unused import UserDict

Fixes: https://trac.torproject.org/projects/tor/ticket/32791#comment:3
Unblocks: #1686

Merge branch 'tor-github/pr/1687'

Merge branch 'tor-github/pr/1692'

Merge branch 'tor-github/pr/1697'

Fix a changelog typo

Merge branch 'maint-0.4.2'

Using "ours" strategy to avoid version bump.

Merge branch 'maint-0.4.1' into maint-0.4.2

Using "ours" strategy to avoid version bump

bump to 0.4.2.6-dev

Bump to 0.4.1.8-dev

hs-v2: Turn logs into protocol warning

All of those can be triggered remotely so change them to protocol warnings.

Fixes #32706

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

copy 0.4.1.8 and 0.4.2.6 entries to master ChangeLog, ReleaseNotes.

Merge branch 'bug32673_035'

Merge branch 'bug33093_logging_035'

Do not set "once" when calling tor_bug_occurred_ from BUG().

The "once" flag makes tor_bug_occurred_() say that future instances
of the warning will be suppressed -- but that's not something that
BUG() does.  Fixes bug 33095; bugfix on 0.4.1.1-alpha.

Merge branch 'maint-0.4.2'

Use "-s ours" to avoid version bump.

Bump version to 0.4.2.6

Merge branch 'maint-0.4.1' into maint-0.4.2

"Ours" merge to avoid version bump.

bump to 0.4.1.8

Merge branch 'maint-0.4.2'

Merge branch 'maint-0.4.1' into maint-0.4.2

Merge branch 'maint-0.4.0' into maint-0.4.1

Merge branch 'maint-0.3.5' into maint-0.4.0

Merge remote-tracking branch 'tor-github/pr/1634' into maint-0.4.1

Merge remote-tracking branch 'tor-github/pr/1614' into maint-0.3.5

buf_read_from_tls: Return ERROR_MISC, not WANTWRITE, on BUG().

Fixes bug 32673; bugfix on 0.3.0.4-alpha.  We introduced these
checks in ee5471f9aab5526 to help diagnose 21369, but we used "-1"
when "TOR_TLS_ERROR_MISC" would have been correct.  Found by opara.

I don't think that this is actually getting triggered in the wild,
but if it were, it could cause nasty behavior: spurious
WANTREAD/WANTWRITE returns have a way of turning into CPU-eating
busy-loops.

Change BUG() messages in buf_flush_to_tls() to IF_BUG_ONCE()

We introduced these BUG() checks in b0ddaac07428a06 to prevent a
recurrence of bug 23690.  But there's a report of the BUG() message
getting triggered and filling up the disk.  Let's change it to
IF_BUG_ONCE().

Fixes bug 33093; bugfix on 0.3.2.2-alpha.

changes: file for 33091

mainloop: Simplify ip_address_changed()

Simplify ip_address_changed() by removing redundant checks
(all exits are relays, and need to rebuild their descriptor when their
IP addresses change).

Rewrite the function comment, and rename the argument, so the function
is easier to understand and use.

Closes 33091.

connection: Improve function comments

Improve the function comments on connection_finished_connecting(),
to describe what the function actually does.

Part of 33091.

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

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

Travis: Stop allowing stem test failures

Stop allowing failures on the Travis CI stem tests job. It looks like
all the stem hangs we were seeing are now fixed, but let's make sure we
see them if they happen again.

Closes ticket 33075.

Use print() function in both Python 2 and Python 3

Merge branch 'tor-github/pr/1683'

changes file for ticket 33039

tests for ticket 33039

fix leak in GETCONF

Fix a memory leak introduced by refactoring of control reply
formatting code.  Fixes bug 33039; bugfix on 0.4.3.1-alpha.

Git hooks: remove support for 0.2.9 directory layout.

We no longer will be pushing or committing things in branches that
use src/common or src/or.

Resolves ticket 32957.

update doc/HACKING/ReleasingTor.md

Bump to 0.4.3.1-alpha-dev

Merge branch 'tor-github/pr/1677'

Lower log level of standard error messages from PT's.

This patch lowers the log level of error messages from PT processes from
warning to debug.

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

pick a release date

Add some changelog edits from seborn

draft a changelog blurb for 0.4.3.1-alpha

Add changes file

Run doxygen makefile target on Travis

dir_connection_t: Explain dirconn_direct better

Direct connections can use a DirPort or ORPort.
Indirect connections must use a multi-hop Tor circuit.

Comment-only changes.

Bump to 0.4.3.1-alpha

changelog: small edits

Run format_changelog.py

Initial incorporation of 0.4.3.x changelogs.

Remove changes files that already appeared in 0.4.2.5

Merge branch 'tor-github/pr/1675'

Merge branch 'tor-github/pr/1668'

doc: Minor man page fixes

Fix a client port list that's missing HTTPTunnelPort.

Wrap a long line.

doc: Improve port flags in man page

Explain that:
* most flags default to off
* flags that default to on are specified

Document PreferIPv6Automap in its "No" form, because it's on by
default.

Cleanup after 32637.

config: Make clients tell dual-stack exits they prefer IPv6

Make Tor clients tell dual-stack exits that they prefer IPv6
connections.

This change is equivalent to setting the PreferIPv6 flag on
SOCKSPorts (and most other listener ports). Tor Browser has
been setting this flag for some time, and we want to remove a
client distinguisher at exits.

Also update the man page, putting all the IP flags in their
non-default forms.

Closes ticket 32637.

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

test/nodelist: Remove ROUTERSTATUS_FORMAT_NO_CONSENSUS_METHOD

When we removed obsolete consensus methods in 32695, we also
removed ROUTERSTATUS_FORMAT_NO_CONSENSUS_METHOD, so we need to
remove it from the routerstatus format tests.

Part of 20218.

Merge branch 'ticket20218_rebased_squashed' into ticket20218_merged

* ticket 32695 removed networkstatus_consensus_has_ipv6(),
  keep that change in master.
* ticket 20218 modifies the function name and comment for
  routerstatus_has_visibly_changed(), keep that change
  in ticket20218_rebased_squashed.

Rename test to match new identifier.

split a wide line

Rename routerstatus_has_changed to routerstatus_has_visibly_changed

This is an automated commit, generated by this command:

./scripts/maint/rename_c_identifier.py \
        routerstatus_has_changed routerstatus_has_visibly_changed

It was generated with --no-verify, since it introduces a wide line.
I'll fix it in a subsequent commit.

Update routerstatus_has_changed tests.

Now they also check whether output of routerstatus_format_entry()
has changed.

Add unit test for routerstatus_has_changed()

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

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

add_c_file: Fix "control line not preceded by a blank line"

Fix dirauth and relay module include.am add_c_file.py
"control line not preceded by a blank line" errors.

Also remove a duplicate ADD_C_FILE: SOURCES in the relay module.

Obviously correct fixes to already-reviewed code.

add_c_file: Replace asserts with exceptions

Closes 32962.

add_c_file: Improve script documentation

Part of 32962.

add_c_file: Improve tor source directory checks

Check that the script isn't in a tor build directory, by looking
for a src/include.am file.

Part of 32962.

Fix a small lint-changes warning.

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

Initialize publish/subscribe code when running as an NT service.

Fixes bug 32778; bugfix on 0.4.1.1-alpha.

Revert "Use tor_api.h entry points in ntmain.c."

This reverts commit dca2629480f143ed8deb71b73b15c7c5ff890571.

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

This reverts commit 5c240db0bf7751d74ba438a1ca4ef0d051a53df7.

Merge branch 'ticket32806'

Remove some dead checks

The only code that could set these options to be negative was in the
unit tests.

fix a stray asterisk in a comment

Merge branch 'ticket32487_squashed_and_merged'

Merge branch 'ticket32695_squashed'

Remove functions that checked for pre-ipv6 consensus.

We no longer need or need to test:
  * node_awaiting_ipv6()
  * networkstatus_consensus_has_ipv6().