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.

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.

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>

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

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

Try to work around changes in openssl 1.1.0

Prefer not to use a couple of deprecated functions; include more
headers in tortls.c

This is part of  ticket 14188.

Bump a client authorization message from debug to info.

A user wants this for 14015, and it seems fairly reasonable.

New GETINFO consensus/packages to expose package information from consensus

Fixes on prop227 comments, based on comments by mcs on #10395

Merge branch 'doc13702_squashed'

Adding section on OpenBSD to doc/TUNING

Groundwork for AF_UNIX hidden services in rendservice.c

Bail early in cpuworker_onion_handshake_replyfn if the circuit is marked

changes file for bug14350

configure.ac: fix disabling systemd notification support

If --disable-systemd is given, $enable_systemd is set to "no", not "false".
As a result, if libsystemd is found, we still turn on systemd support even
if we explicitly disable it with --disable-system.

Try to fix some more memory leaks in the unit tests

Fix some memory leaks in new address tests

whitespace fixes

Split the slow unit tests into their own binary

This can run in parallel with the faster ones and the other tests.

Merge branch 'if_addr_refactoring_squashed'

Conflicts:
src/test/include.am
src/test/test.c

Refactor code that looks up addresses from interfaces

Now the code has separate implementation and examination functions,
uses smartlists sanely, and has relatively decent test coverage.

changes file for 9969

Tweak the 9969 fix a little

If we have busy nodes and excluded nodes, then don't retry with the
excluded ones enabled.  Instead, wait for the busy ones to be nonbusy.

Merge remote-tracking branch 'public/ticket9969'

Conflicts:
src/or/directory.c
src/or/routerlist.c
src/or/routerlist.h
src/test/include.am
src/test/test.c

fix warnings from lintChanges

Merge remote-tracking branch 'public/ticket13762'

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

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

Merge remote-tracking branch 'karsten/geoip6-jan2015' into maint-0.2.4

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

fix some warnings in compat_threads.c