Fix most instances of SC2086 warning

Fix shellcheck SC2006 warnings in 'coverage' script

Merge branch 'maint-0.3.5'

Merge branch 'maint-0.3.4' into maint-0.3.5

Appveyor: add comments to the config file

Documentation for 28399, 28454, and previous fixes.

Appveyor: manually add zstd flags to configure

The mingw zstd from MSYS2 doesn't come with a pkg-config file.

Fixes 28454 on Tor 0.3.4.1-alpha.

Appveyor: remove incorrect OpenSSL path

Fixes 28399 on 0.3.4.1-alpha.

Appveyor: Improve diagnostics

Stop installing MSYS2 packages.
We're compiling for mingw, so we only need mingw packages.

Run pacman in verbose mode.

Explicitly install pkg-config and xz, to future-proof our builds.

Diagnostics for 28399.

Merge branch 'maint-0.3.5'

Merge branch 'ticket27750_034_01_squashed' into maint-0.3.5

conn: Close the read side of a closing connection when write limit is reached

In conn_close_if_marked(), we can decide to keep a connection open that still
has data to flush on the wire if it is being rate limited on the write side.

However, in this process, we were also looking at the read() side which can
still have token in its bucket and thus not stop the reading. This lead to a
BUG() introduced in 0.3.4.1-alpha that was expecting the read side to be
closed due to the rate limit but which only applies on the write side.

This commit removes any bandwidth check on the read side and simply stop the
read side on the connection regardless of the bucket state. If we keep the
connection open to flush it out before close, we should not read anything.

Fixes #27750

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

Merge branch 'maint-0.3.5'

Merge remote-tracking branch 'teor/bug28441-035' into maint-0.3.5

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.5'

Merge branch 'maint-0.3.4' into maint-0.3.5

Work around check-spaces.

Changes file for bug 28419

libtorrunner: fix memory leak in child() error path

This avoids leaking memory in case libtorrunner's child() function fails.

Merge branch 'maint-0.3.5'

Fix Doxyfile for 0.3.5 source tree moves

Make Doxygen work again after the 0.3.5 source tree moves.  Fixes bug
28435; bugfix on 0.3.5.1-alpha.

Merge branch 'maint-0.3.5'

log: stop talking about the Named flag in log messages

Clients have ignored the Named flag since 0.3.2.

Fixes bug 28441; bugfix on 0.3.2.1-alpha.

comment: circuit_list_path_impl() does not check Named any more

comment: Fix a typo in nodes_in_same_family()

doc: Remove an outdated sentence from KeepalivePeriod in the man page

Merge branch 'maint-0.3.5'

Merge branch 'bug28183_029' into maint-0.3.5

Merge branch 'tor-github/pr/501'

Merge branch 'maint-0.3.5'

Merge branch 'tor-github/pr/487' into maint-0.3.5

Comment for rend_cache_failure in feature/rend/rendcache.c: "usuable" should be "usable"

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.5'

Merge branch 'maint-0.3.4' into maint-0.3.5

Merge branch 'maint-0.2.9' into maint-0.3.3

Merge branch 'bug28413_029' into maint-0.2.9

Fix a compiler warning in aes.c.

Apparently some freebsd compilers can't tell that 'c' will never
be used uninitialized.

Fixes bug 28413; bugfix on 0.2.9.3-alpha when we added support for
longer AES keys to this function.

seccomp2: Add "shutdown" to the list of permitted system calls.

We don't use this syscall, but openssl apparently does.

(This syscall puts a socket into a half-closed state. Don't worry:
It doesn't shut down the system or anything.)

Fixes bug 28183; bugfix on 0.2.5.1-alpha where the sandbox was
introduced.

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.5'

Merge branch 'maint-0.3.4' into maint-0.3.5

Merge branch 'maint-0.2.9' into maint-0.3.3

Fix a bug in usage of SSL_set1_groups_list()

Apparently, even though the manpage says it returns an int, it
can return a long instead and cause a warning.

Bug not in any released Tor.  Part of #28399

Merge branch 'maint-0.3.5'

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.4' into maint-0.3.5

Merge branch 'maint-0.2.9' into maint-0.3.3

Update geoip and geoip6 to the November 6 2018 database.

Annotate subsystem list with their levels.

Merge branch 'subsystems'

Log before performing a subsystem operation

Add list of levels in subsystem_list.c

Rename subsystem callback functions to make them consistent

fixup! Make initialization for the "err" library into a subsystem.

Check for failure to install backtrace handler.

Document that subsystem callbacks are optional.

Explain that configuration should happen elsewhere, but not init.

Merge branch 'maint-0.3.5'

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.4' into maint-0.3.5

Merge branch 'maint-0.2.9' into maint-0.3.3

Merge branch 'bug28245_029' into maint-0.2.9

Always declare groups when building with openssl 1.1.1 APIs

Failing to do on clients was causing TLS 1.3 negotiation to fail.

Fixes bug 28245; bugfix on 0.2.9.15, when we added TLS 1.3 support.

Merge branch 'maint-0.3.5'

Copy changelog to maint-0.3.5 so master will get it.

Merge branch 'maint-0.3.5'

"ours" to avoid version bump.

Bump version to 0.3.5.4-alpha-dev.

Merge branch 'maint-0.3.5'

Make the hibernate.c changes for systemd compile.

Merge branch 'maint-0.3.5'

"ours" to avoid version bump.

Bump to 0.3.5.4-alpha

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

Merge branch 'maint-0.3.5'

Merge remote-tracking branch 'teor/ticket28318-035' into maint-0.3.5

simplify now that it uses tor's copyright and license

Change version on master to 0.4.0.0-alpha-dev.

Merge branch 'ticket27225_squashed'

Memoize summarize_protover_flags()

Our tests showed that this function is responsible for a huge number
of our malloc/free() calls.  It's a prime candidate for being
memoized.

Closes ticket 27225.

Switch ctassert.h to 3bsd (with permission)

At intro points, don't close circuits on NACKs

Allow lib/cc to include its own files.

Change copyright statement (with permission)

Remove a tab.

Add parentheses to the ctassert macro expansions

Add a user of CTASSERT().

New macro CTASSERT(condition) to assert condition at compile-time.

To get it, use: #include "lib/cc/ctassert.h"

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

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

Merge branch 'maint-0.3.5'

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

Merge branch 'maint-0.3.5'

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

Merge branch 'maint-0.3.5'

Add a changes file for bug 28303

Include sys/time.h in timers.c and time_fmt.c

This fixes compilation on OpenBSD.

Signed-off-by: Kris Katterjohn <katterjohn@gmail.com>

refuse to start with relative paths + RunAsDaemon

Resume refusing to start with relative file paths and RunAsDaemon
set (regression from the fix for bug 22731).

Fixes bug 28298; bugfix on 0.3.3.1-alpha.

Merge branch 'bug28348_036'

Merge branch 'maint-0.3.5'

Merge branch 'bug28348_035' into bug28348_036

Merge branch 'bug28348_034' into bug28348_035