Call cpu_init if we change to being a relay

The issue is that we use the cpuworker system with relays only, so if we
start up as a client and transition to being a relay later, we'll be
sad.

This fixes bug 14901; not in any released version of Tor.

Don't leak a cond var when starting threads in a pool

Merge remote-tracking branch 'sebastian/bug14875'

Avoid undefined behaviour in ifreq_to_smartlist

This could trigger where _SIZEOF_ADDR_IFREQ() might not return a
multiple of sizeof(void *). Fixes bug 14875; not in any released version
of Tor.

Actually get all interface addresses

If we guessed a buffer size too small, we never increased the buffer and
tried again

Also simplify the interface of ifreq_to_smartlist a little

Merge branch 'bug14478_squashed'

Put error messages to stderr

Thanks weasel :)

Also remove stray debug output

Change exit code of zero_length_keys script

Weasel notes that exit codes above 128 are for when a script dies from a
signal.

Avoid undefined behaviour of test

Fixes part of bug #14478, patch idea suggested by an anonymous
contributor. Thanks!

Check that mktemp result is a directory

Fixes part of bug #14478, patch idea suggested by an anonymous
contributor. Thanks!

Clean up after zero length key test

Merge remote-tracking branch 'public/bug14759'

fix a compilation warning

Merge remote-tracking branch 'public/remove_freelist'

Merge branch 'bug12844'

Conflicts:
src/or/circuituse.c
src/test/include.am
src/test/test_entrynodes.c

Additional paranoia: do not even build tor2web stuff if not using.

(That is, don't build it unless we're building for tor2web, or we
are building for tests.)

Fix crash on glibc __libc_message()

__libc_message() tries to open /dev/tty with O_RDWR, but the sandbox
catches that and calls it a crash.  Instead, I'm making the sandbox
setenv LIBC_FATAL_STDERR_, so that glibc uses stderr instead.

Fix for 14759, bugfix on 0.2.5.1-alpha

Remove mempools and buf freelists

They have been off-by-default since 0.2.5 and nobody has complained. :)

Also remove the buf_shrink() function, which hasn't done anything
since we first stopped using contiguous memory to store buffers.

Closes ticket 14848.

Merge remote-tracking branch 'public/feature_13555'

Merge remote-tracking branch 'sebastian/bug14819'

Merge remote-tracking branch 'sysrqb/bug14802_025'

Merge remote-tracking branch 'teor/avoid-NULL-smartlist-foreach'

Add some comments to or_circuit_t.workqueue_entry

These were suggested by dgoulet, thanks!

Avoid use-after-free of circ belonging to cancelled job

This fixes a bug where we decide to free the circuit because it isn't on
any workqueue anymore, and then the job finishes and the circuit gets
freed again.

Fixes bug #14815, not in any released version of Tor.

Re-remove the --disable-threads configure option

Reserve enough space for rend_service_port_config_t

In #14803, Damian noticed that his Tor sometimes segfaults. Roger noted
that his valgrind gave an invalid write of size one here. Whenever we
use FLEXIBLE_ARRAY_MEMBER, we have to make sure to actually malloc a
thing that's large enough.

Fixes bug #14803, not in any released version of Tor.

Return 0 when detecting the amount of memory fails

Fixes bug 14802;  bugfix on 0.2.5.4-alpha.

Avoid calling SMARTLIST_FOREACH on a NULL smartlist in tests

Check if each smartlist is NULL before calling SMARTLIST_FOREACH on it.

Bug discovered by the clang static analyzer.
Apple clang 600.0.56 (LLVM 3.5svn) on x86_64-apple-darwin14.1.0.

Control: getinfo entry-guards report down-since

If the guard unreachable_since variable was set, the status "up" was
reported which is wrong. This adds the "down" status followed by the
unreachable_since time value.

Fixes #14184

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Remove useless export in zero length key test

Fixes part of bug #14478, patch idea suggested by an anonymous
contributor. Thanks!

Don't use invalid exit values in zero length key test

Shell exit values must fall into the range of [0-255], so let's honour
this. In practice, the "exit -1" from the old code set an exit value of
255 on most systems, so let's pick that.

Fixes part of bug #14478, patch idea suggested by an anonymous
contributor. Thanks!

Merge remote-tracking branch 'sebastian/bug13993'

Fix check-spaces complaints

Avoid logging startup messages twice

Add unit test for #13290

Fix scheduler compilation on targets where char is unsigned.

Per discussion with nickm, the `dir` argument should be a int rather
than a signed char.

Fixes bug #14764.

changes file for 13796

Merge remote-tracking branch 'public/bug13796'

Run the format_changelog script for initial cleanup

Start working on an 0.2.3 changelog

This is just sorting the entries and lightly editing a couple of
problems I found.

Fix a work-counting bug introduced by the workqueue merge

David Goulet finds that when he runs a busy relay for a while with the
latest version of the git code, the number of onionskins handled
slowly dwindles to zero, with total_pending_tasks wedged at its
maximum value.

I conjecture this is because the total_pending_tasks variable isn't
decremented when we successfully cancel a job.  Fixed that.

Fixes bug 14741; bugfix not on any released version of tor.

use ARRAY_LENGTH macro in domain_to_string

Merge remote-tracking branch 'yawning/bug14740'

Add a string representation for LD_SCHED, and a extra sanity check.

This both fixes the problem, and ensures that forgetting to update
domain_list in the future will trigger the bug codepath instead of
a NULL pointer deref.

improve changes files more

14554 doesn't need a changes file: the bug wasn't in a release

Bump the minimum relay version to 0.2.4.18-rc

Closes #13555

Use getsockname, not getsockopt, on TPROXY sockets

fix a unit tests memory leak (my fault)

Work around test_status.c weirdness

Ordinarily, get_options() can never return NULL, but with
test_status.c mocking, it can.  So test for that case.

The best fix here would be to pass the options value to a
bridge_server_mode() function.

Merge branch 'bug6852'

Conflicts:
src/or/status.c

Choose a more deliberate cutoff for clients in heartbeat

Merge remote-tracking branch 'public/bug9635_warnings_025'

Conflicts:
src/test/test.c

tweak based on comments from dgoulet

Merge branch 'feature13865_test'

Test for 13865.

Add more parenthesis to the definition of ARRAY_LENGTH

Fix an unused-variable warning.

Merge branch 'feature13864_squashed'

Updating manpage for 13865.

Changes file for 13865.

Allow reading torrc from stdin.

Merge remote-tracking branch 'public/bug13319'

Merge remote-tracking branch 'sysrqb/bug14216_bad_since'

Merge remote-tracking branch 'dgoulet/bug14202_026_v1'

Merge remote-tracking branch 'public/14188_part1'

Updating OpenBSD section of doc/TUNING.

Only retry connecting to configured bridges

After connectivity problems, only try connecting to bridges which
are currently configured; don't mark bridges which we previously
used but are no longer configured.  Fixes 14216.  Reported by
and fix provided by arma.

Merge remote-tracking branch 'dgoulet/bug14554_026_v1'

Fix: check r < 0 before checking errno

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Fix possible infinite loop on pipe/sock_drain()

If the returned value of read/recv is 0 (meaning EOF), we'll end up in an
infinite loop (active wait) until something is written on the pipe which is
not really what we want here especially because those functions are called
from the main thread.

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Fix some unused-argument warnings

Fix an uninitialized-variable warning.

Fix: test -ENOENT after config_parse_unix_port()

Check for -ENOENT instead of ENOENT after the HS port is parsed.

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Merge remote-tracking branch 'public/prop227_v2'

Conflicts:
src/test/test_dir.c

Correctly handle OutboundBindAddress again.

ca5ba2956bcd4b5ee1e526ccf5914f52fe6e6d51 broke this; bug not in any
released Tor.

Also fix a typo.

Fixes 14541 and 14527.  Reported by qbi.

Merge branch 'bug14451_026_v1'

changes file for 14451

Use HT_BUCKET_NUM_ in more places

(patch from sysrqb)

Merge remote-tracking branch 'public/ticket11737'

Document unix: addresses

Restrict unix: addresses to control and socks for now

Remove SocksSocket; it's now spelled differently thanks to 14451

Also, revise bug12585 changes file to mention new syntax

When there are no package lines, make consensus/packages say "".

Also, give a better error message when there is no consensus.

Correctly reject packages lines with empty entries

Define 'digesttype' correctly

Remove obsolete workaround in dirserv_thinks_router_is_hs_dir()

Fixes #14202

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Make hidden service use the config unix prefix

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Support unix: prefix in port configuration

It's now possible to use SocksPort or any other kind of port that can use a
Unix socket like so:

  SocksPort unix:/foo/bar/unix.sock

Fixes #14451

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Merge branch 'bug11485_026_v2_squashed'

fixup! Refactor the use of ifdef HAVE_SYS_UN_H

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Changes file for ticket #11485

Actually make connections to HSes on AF_UNIX sockets

Support connection_exit_connect() to AF_UNIX sockets

Handle config options for AF_UNIX hidden services rendservice.c

changes file for 8405

Bug #8405: Report SOCKS username/password in CIRC status events

Introduces two new circuit status name-value parameters: SOCKS_USERNAME
and SOCKS_PASSWORD. Values are enclosing in quotes and unusual characters
are escaped.

Example:

    650 CIRC 5 EXTENDED [...] SOCKS_USERNAME="my_username" SOCKS_PASSWORD="my_password"

Add GETINFO consensus/{valid-{after,until},fresh-until}

more typo fixes from mcs and gk

Tweak tor-resolve docs and logs

Resolves 14325