Rename tor_escape_str_for_socks_arg() to something more generic.

Since we are going to be using that function to also escape parameters
passed to transport proxies using environment variables.

Pass characters to be escaped to tor_escape_str_for_socks_arg().

This is in preparation for using tor_escape_str_for_socks_arg() to
escape server-side pluggable transport parameters.

Write unit tests for the ServerTransportOptions parsing function.

Write function that parses ServerTransportOptions torrc lines.

And use it to validate them.

Lightly refactor and test format_hex_number_sigsafe

Better tests for upper bounds, and for failing cases.

Also, change the function's interface to take a buffer length rather
than a maximum length, and then NUL-terminate: functions that don't
NUL-terminate are trouble waiting to happen.

HEX_ERRNO_SIZE is no longer the correct upper limit for format_hex_number_sigsafe

Expose format_hex_number_..., and rename it to ..._sigsafe().

There are some other places in the code that will want a signal-safe
way to format numbers, so it shouldn't be static to util.c.

Don't build format_helper_exit_status on win32

The only thing that used format_helper_exit_status on win32 was the
unit tests. This caused an error when we tried to leave a static
format_helper_exit_status lying around in a production object file.

The easiest solution is to admit that this way of dealing with process
exit status is Unix-only.

Merge remote-tracking branch 'linus/unused_param'

Merge remote-tracking branch 'linus/binshify'

Merge remote-tracking branch 'public/fancy_test_tricks'

Conflicts:
src/common/include.am

Conflict was from adding testsupport.h near where sandbox.h had
already been added.

Traditional /bin/sh is unhappy about {a,b} globbing.

Avoid compiler warning 'unused param'.

Fixes #9261.

Fix two pre-coffee typos

And a changes file for #9258

Reject relative control socket paths and emit a warning.

Previously we would accept relative paths, but only if they contained a
slash somewhere (not at the end).

Otherwise we would silently not work.  Closes: #9258.  Bugfix on
0.2.3.16-alpha.

Document get_parent_directory more

cosmetic cleanups

put sandbox.h in the tarball, so the tarball builds

Merge remote-tracking branch 'public/gsoc-ctoader-cap-phase1-squashed'

Remove a bogus semicolon spotted by Gisle Vanem

Add a basic seccomp2 syscall filter on Linux

It's controlled by the new Sandbox argument.  Right now, it's rather
coarse-grained, it's Linux-only, and it may break some features.

Add some basic unit tests for the circuit map data structure.

These show off the new mocking code by mocking the circuitmux code
so that we can test the circuit map code in isolation.

Add rudimentary test mocking support.

This is not the most beautiful possible implementation (it requires
decorating mockable functions with ugly macros), but it actually
works, and is portable across multiple compilers and architectures.

Add a script to compare directories full of gcov output

We can't just use "diff", since we don't care about changes in line
numbers, or changes in the exact number of times a line was called.
We just care about changes that make lines covered or non-coverd.  So
pre-process the files before calling diff.

Coverage support: build with --enable-coverage to have tests run with gcov

If you pass the --enable-coverage flag on the command line, we build
our testing binaries with appropriate options eo enable coverage
testing.  We also build a "tor-cov" binary that has coverage enabled,
for integration tests.

On recent OSX versions, test coverage only works with clang, not gcc.
So we warn about that.

Also add a contrib/coverage script to actually run gcov with the
appropriate options to generate useful .gcov files.  (Thanks to
automake, the .o files will not have the names that gcov expects to
find.)

Also, remove generated gcda and gcno files on clean.

Completely refactor how FILENAME_PRIVATE works

We previously used FILENAME_PRIVATE identifiers mostly for
identifiers exposed only to the unit tests... but also for
identifiers exposed to the benchmarker, and sometimes for
identifiers exposed to a similar module, and occasionally for no
really good reason at all.

Now, we use FILENAME_PRIVATE identifiers for identifiers shared by
Tor and the unit tests.  They should be defined static when we
aren't building the unit test, and globally visible otherwise. (The
STATIC macro will keep us honest here.)

For identifiers used only by the unit tests and never by Tor at all,
on the other hand, we wrap them in #ifdef TOR_UNIT_TESTS.

This is not the motivating use case for the split test/non-test
build system; it's just a test example to see how it works, and to
take a chance to clean up the code a little.

Start work on fancy compiler tricks to expose extra stuff to our tests

This is mainly a matter of automake trickery: we build each static
library in two versions now: one with the TOR_UNIT_TESTS macro
defined, and one without.  When TOR_UNIT_TESTS is defined, we can
enable mocking and expose more functions. When it's not defined, we
can lock the binary down more.

The alternatives would be to have alternate build modes: a "testing
configuration" for building the libraries with test support, and a
"production configuration" for building them without.  I don't favor
that approach, since I think it would mean more people runnning
binaries build for testing, or more people not running unit tests.

Merge remote-tracking branch 'origin/maint-0.2.4'

Merge branch 'bug9200' into maint-0.2.4

Add a comment and a check for why flag indices will be <= 63

Merge remote-tracking branch 'karsten/geoip-manual-update-jul2013'

Merge remote-tracking branch 'origin/maint-0.2.4'

Merge remote-tracking branch 'origin/maint-0.2.3' into maint-0.2.4

Merge remote-tracking branch 'origin/maint-0.2.2' into maint-0.2.3

Update to the July 2013 GeoIP database.

Add 2 new manual A1 substitutions.

FIx undefined behavior in dirvote.c

 Fix a bug in the voting algorithm that could yield incorrect results
 when a non-naming authority declared too many flags. Fixes bug 9200;
 bugfix on 0.2.0.3-alpha.

Found by coverity scan.

Merge remote-tracking branch 'linus/bug8530'

Merge remote-tracking branch 'origin/maint-0.2.4'

Merge branch 'bug9147' into maint-0.2.4

Give a warning when bufferevents are enabled.

Ticket 9147.

Merge remote-tracking branch 'origin/maint-0.2.4'

Fix bug 9122: don't allow newdefaultoptions to be NULL

(This caused a crash that was reported as bug 9122, but the underlying
behavior has been wrong for a while.)

Fix on 0.2.3.9-alpha.

Fix #5584 - raise awareness of safer logging - warn about potentially unsafe config options

Forward-port the 0.2.4.14-alpha changelog

Merge remote-tracking branch 'origin/maint-0.2.4'

Merge remote-tracking branch 'origin/maint-0.2.3' into maint-0.2.4

Check more return values in the unit tests

Fix some problems with the bug9002 fix.

Fixes bug 9090; bug not in any released Tor.

Attempt to un-confuse coverity about (false) double-free in bench_onion_ntor.

Fix a couple of resource leaks in test_config.c

Spotted by Coverity Scan. Not in any released Tor.

Merge remote-tracking branch 'origin/maint-0.2.4'

Tweak bug9063_redux patch: {n_p}_chan_cells, not {n,p}_conn_cells

Merge remote-tracking branch 'origin/maint-0.2.3' into maint-0.2.4

Conflicts:
src/or/config.c
src/or/relay.c

Merge branch 'bug9063_redux_023_squashed' into maint-0.2.3

Implement a real OOM-killer for too-long circuit queues.

This implements "algorithm 1" from my discussion of bug #9072: on OOM,
find the circuits with the longest queues, and kill them.  It's also a
fix for #9063 -- without the side-effects of bug #9072.

The memory bounds aren't perfect here, and you need to be sure to
allow some slack for the rest of Tor's usage.

This isn't a perfect fix; the rest of the solutions I describe on
codeable.

Invoke binaries in $PATH rather than by absolute path.

That security measure costs more than it gives.
So, keep your PATH sane and we can run this script on more than
one system.

Merge remote-tracking branch 'public/bug9082'

Fix bug 9082: avoid leak when freeing destroy cell queues

In my #7912 fix, there wasn't any code to remove entries from the
(channel, circuit ID)->circuit map corresponding to queued but un-sent
DESTROYs.

Spotted by skruffy. Fixes bug 9082; bug not in any released Tor.

Limit hidden service descriptors to at most 10 guard nodes.

Fixes bug 9002; bugfix on 0.1.1.11-alpha (which introduced guard
nodes), or on 0.0.6pre1 (which introduced hidden services).

forward-port 0.2.4.13-alpha changelog

Merge branch 'bug9072-024' into bug9072-025

Merge branch 'bug9072-023' into bug9072-024

Disable middle relay queue overfill detection code due to possible guard discovery attack

Instead of testing for __GNUC__, use CHECK_SCANF, like CHECK_PRINTF.

This lets us have the possibility of fine-tuning the check in the tor_sscanf test cases at a later date.

Use CHUTNEY_PATH to find Chutney.

Merge branch 'circuit_queue_cap-0.2.5-squashed'

Conflicts:
src/or/relay.c

Merge remote-tracking branch 'origin/maint-0.2.4'

Using "ours" strategy to avoid taking commits

Merge branch 'circuit_queue_cap-0.2.4-squashed' into maint-0.2.4

Merge remote-tracking branch 'origin/maint-0.2.3' into maint-0.2.4

Using "ours" strategy to avoid taking changes.

Merge branch 'circuit_queue_cap-0.2.3-squashed' into maint-0.2.3

Fix signed/unsigned comparison warning

Increase the limit so leaky pipe might work

Increase the limit so leaky pipe might work

Increase the limit so leaky pipe might work

Don't queue more cells as a middle relay than the spec allows to be in flight

Don't queue more cells as a middle relay than the spec allows to be in flight

Don't queue more cells as a middle relay than the spec allows to be in flight

Merge remote-tracking branch 'origin/maint-0.2.4'

Merge branch 'bug9047' into maint-0.2.4

Define SEEK_SET for platforms that lack it.

Rename networkstatus_dl_interval() -> networkstatus_dl_check_interval()

Merge remote-tracking branch 'linus/bug8532'

If we write the annotation but not the microdescriptor, rewind.

This fixes bug 9047 (and some parts of 9031, 8922, 8883 that weren't
fixed in 8822).  Bugfix on 0.2.2.6-alpha.

Add make target test-network running traffic tests in a Chutney network.

This implements ticket #8530.

Merge remote-tracking branch 'public/bug7912_squashed'

Fix compile warnings wrt printf formating of int64_t

Add a changes file for bug 7912

I'm calling it a bugfix on 0.2.0.1-alpha, since that's where cell
queues were first introduced.

Add destroy balance tracking and logging to circuitmux

Implementation of a fix for bug 7912

I added the code to pass a destroy cell to a queueing function rather
than writing it immediately, and the code to remember that we
shouldn't reuse the circuit id until the destroy is actually sent, and
the code to release the circuit id once the destroy has been sent...
and then I finished by hooking destroy_cell_queue into the rest of
Tor.

Code to track on a circuit whether it has a "pending" delete cell

This will be used in a fix for bug7912.

Merge remote-tracking branch 'origin/maint-0.2.4'

Merge remote-tracking branch 'public/bug8822' into maint-0.2.4

Merge remote-tracking branch 'origin/maint-0.2.4'

Fix the biggest errors in doc/HACKING

We can wait for 0.2.5 for a full rewrite.  #8964

Merge remote-tracking branch 'origin/maint-0.2.4'

Remove various outdated documents.

doc/TODO and doc/spec/README were placeholders to tell people where to
look for the real TODO and README stuff -- we replaced them years ago,
though.

authority-policy, v3-authority-howto, and torel-design.txt belong in
torspec.  I'm putting them in attic there since I think they may be in
large part obsolete, but someone can rescue them if they're not.

translations.txt is outdated, and refers to lots of programs other
than Tor.  We have much better translation resources on the website
now.

tor-win32-mingw-creation.txt is pending review of a revised version
for 0.2.5 (see ticket #4520), but there's no reason to ship this one
while we're waiting for an accurate version.

the tor-rpm-creation.txt isn't obsolete AFAIK, but it belongs in
doc/contrib if anywhere.

Resolves bug #8965.

Merge remote-tracking branch 'origin/maint-0.2.4'

Merge remote-tracking branch 'andrea/bug8639_v3' into maint-0.2.4