Improve log messages for diagnosing 13762

Merge remote-tracking branch 'yawning/cid1270894'

Actually free measured_guardfraction.

CID 1270894

whitespace fixes

Fix an implicit-narrowing warning

Merge remote-tracking branch 'yawning/bug14918'

Combine test_helpers.[ch] and testhelper.[ch]

Merge branch 'bug9321_rerebase'

Conflicts:
src/or/dirvote.h
src/test/include.am
src/test/test_entrynodes.c

Don't exit if we can't find the GuardfractionFile when booting.

...just disable the feature in that case.

Final guardfraction preparations for upstream merge.

- Write a changes file.
- Change some logs to lesser severities.

Add unittests for the guardfraction feature.

Unittest prep: Move get_yesterday_date_str() to helper file.

We want to use this function in our guardfraction unittests, so make a
test_helpers module and move it there.

Calculate relay weighted bandwidth using guardfraction bandwidths.

Now use the guardfraction function to calculate individual relay
bandwidth weights.

Calculate total bandwidth weights using the guardfraction bandwidths.

Now use the function introduced in the previous commit, to calculate the
total bandwidth weights when making the consensus.

Calculate the guardfraction bandwidth of a guard.

Parse GuardFraction info from consensuses and votes.

Also introduce the UseGuardFraction torrc option which decides whether
clients should use guardfraction information found in the consensus.

Write GuardFraction information to consensus.

If we've seen enough votes with guardfraction information, write down
the GuardFraction string in the consensus.

Write guardfraction information to votes.

If a dirauth has guardfraction information about a guard, write it down
when serializing the routerstatus.

Parse Guardfraction file and apply results to routerstatuses.

Parse the file just before voting and apply its information to the
provided vote_routerstatus_t. This follows the same logic as when
dirauths parse bwauth files.

Fix bootstrap directory information logging.

`dir_info_status` is used from main.c:directory_info_has_arrived() to
provide useful (INFO/NOTICE) level logging to users, and should always
be updated regardless of the rate limiting.

Add description to the precious Python variable.

Fix compute_frac_paths_available, when ExitNodes is not set.

Merge branch 'bug14918'

Merge remote-tracking branch 'public/feature_13822'

Fix some issues with reporting exit-free networks

Fixes bug 14918.

Reflow 0.2.6.3 changelog.

Fold new entries into 0.2.6.3 changelog

Fix warnings from lintChanges.py

Check thread count for negative; realloc->reallocarray

CID 1268069

Have tinytest tolerate systems where char has > 8 bytes

CID 1064418

Use --list-fingerprint instead of process control in zero_length_keys.sh

Using kill and wait in this way may have been making windows
builders unhappy.

Wrap two long lines

trivial whitespace fix

Maybe this will make coverity not care that this check is dead. CID 1268063

No, coverity, this is not a memory leak.  1268065

Fix a few coverity "Use after NULL check" warnings

Also remove the unit test mocks that allowed get_options() to be
NULL; that's an invariant violation for get_options().

Fix deadcode warning in get_interface_addresses_raw().

CID  1268070

Fix a trivial double-close in address.c. CID 1268071

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.

Search-and-replace to regain coding style in wake of 13822.

(all-caps should be reserved for global or module-global constants and

Split ROUTER_REQUIRED_MIN_BANDWIDTH into RELAY_ and BRIDGE_ variants

Also raise those minima from 20 to 75 and 50 respectively.

Closes ticket 13822.

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.