Add a dire warning about not reformatting the whole codebase yet.

clang-format: remove StatementMacros usage

This change lets us use clang-format-6.0, and is okay since we now
require semicolons after HT_PROTOTYPE/GENERATE.

Use semicolons after HT_PROTOTYPE and HT_GENERATE.

ht.h: Require a semicolon after HT_PROTOTYPE and HT_GENERATE[2]

Try to shorten an #error in address.c

Break CONNECTION_TESTCAE_ARG across multiple lines

Extract verbatim table in uname.c

checkSpace: permit wide lines for LCOV_EXCL

We're telling clang-format that a line with LCOV_EXCL must not be
split -- that's fine, but we shouldn't complain when it indents it.

Fix a couple more long warning lines

These are not a problem with 2-space indentation, but cocci will
start getting confused when clang-format wraps them with 4-space
indentation.

circuitbuild: expect bug message that clang-format will generate.

clang-format wants to put no space here, so we need to tell the test
to expect a lack of a space.

checkSpace: be more careful about bad function headers.

Previously we would forbid macro indentations like this:

FOO({
  int x;
  })

But clang-format sometimes generates those.

Wrap columnar tables in "clang-format off/on"

These tables have aligned comments, so we don't want clang-format to
mess with them.

Use parentheses to avoid mis-indentations of stringified macro args

clang-format sometimes thinks that "#name" should be written as
"#     name" if it appears at the start of a line.  Using () appears
to suppress this, while confusing Coccinelle.

maps: do not include _ as part of expected prefix in macros.

Doing this makes our macro usage a little clear IMO, and also avoids
having to use an unadorned "new" within a macro.  (Clang-format
seems to think that "new" means we're doing C++, and so it generates
some output that checkSpace.pl doesn't care for.)

Remove senseless CHECK_PRINTF()s from util_bug.c

These belong in util_bug.h (and they already are there).

Their presence made clang-format misindent these functions in a way
that checkSpace.pl dislikes.

log_test_helpers: remove semicolons from end of macros

We want our code to require semicolons after use of these macros,
so that our code formatters and/or analysis tools don't get
confused.

Use smaller CPP error messages

Clang-format wants to split these messages across multiple lines,
which confuses the heck out of coccinelle.

onion_queue.c: use TAILQ_HEAD less confusingly.

When we use macro inline, clang-format likes to break it in the
middle, which makes checkSpace get confused.

checkSpace: don't treat an unindented label as starting a function.

prob_distr: use "clang-format off" to avoid wide lines for URLs

Use a compile-time assert in control_events.h

(The original idiom here led clang-format to generating a too-wide line.)

checkSpace.pl: Allow space between iteration macros and ().

Clang-format wants to put these in, and they do make sense for
consistency.

Also allow more types.

checkSpace.pl: Use a data structure for a list of non-function names

checkspace: allow spaces in cpp directives.

checkSpace.pl: allow {{, ){, and ({.

checkSpace: remove the nosplabel test.

Temporary clang-format configuration and script.

The format is the same as in my previous efforts here.

The script is a little tricky, since it invokes both clang-format
and codetool, and it makes sure that files do not have a changed
mtime unless there is actually some change in the file.

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.