Merge branch 'ticket5526'

Tweak/cleanup 5526 fix.

Print accounting information in heartbeat messages.

Implements ticket 5526.

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

Document the correct loglevel for the heartbeat message

Add a changes file for 9108.

Rename circuit_get_global_list to remove trailing _

Merge remote-tracking branch 'majek/bug9108'

Conflicts:
src/or/circuitlist.h

Add changes file for #4773.

Fix a wide line in test_geoip_with_pt

Merge branch 'bug5040_4773_rebase_3'

Test ExtORPort cookie initialization when file writing is broken.

Test some fail conditions of ext_orport.c USERADDR/TRANSPORT parsing.

Fix some #5040 bugs found by Nick's tests.

- Set conn->address when we receive a USERADDR command.
- Set conn->state to a sane value when we transition from Extended
  ORPort to ORPort.

Write some free_all functions to free the auth. cookies.

We started allocating space for them on the heap in the previous
commit.

Conflicts:
src/or/ext_orport.h

Implement and use a generic auth. cookie initialization function.

Use the generic function for both the ControlPort cookie and the
ExtORPort cookie.

Also, place the global cookie variables in the heap so that we can
pass them around more easily as pointers.

Also also, fix the unit tests that broke by this change.

Conflicts:
src/or/config.h
src/or/ext_orport.c

Warn if the Extended ORPort listens on a public IP address.

Better documentation for ext_or_auth_correct_client_hash.

White-box tests for the succeeding case of ext_or_port handshake.

(Okay, white-box plus mocking enough other functions so they don't
crash.)

Test for initializing ext_or_auth_cookie file

Make 0x01==SAFECOOKIE a macro, not a magic number

Unit test for the ext_orport safe_cookie handshake

Expose/mock some functions to make ext_orport.c testing possible

More unit tests for handle_client_auth_nonce

Incidentally, this business here where I make crypto_rand mockable:
this is exactly the kind of thing that would make me never want to
include test-support stuff in production builds.

Unit test for basic ext_or_cookie authentication backend

Split the cryptographic part of handle_client_auth_nonce into new fn

Fix some ext_orport.c DOCDOCs.

Tests for connection_write_ext_or_command.

Unit tests for ext_or_id_map.

Unit tests for fetch_ext_or_cmd

Split out buffers and socks tests into separate modules.

No other changes were made here.  Keeping everything in
src/test/test.c was a legacy of back when we had all our unit tests in
one big file.

Doing this now because I'm adding an ext_or_command test.

Add a clientmap_entry_free().

Remove a nedless strdup/free pair.

Fix hash functions for transport_name in client entry

Use memdup_nulterm and check for NULs in handle_cmd_transport

Fix a variety of issues in 4773

memwipe some stack-allocated stuff
Add DOCDOC comments for state machines
Use memdup_nulterm as appropriate
Check for NULs in useraddr
Add a macro so that <= AUTH_MAX has a meaning.

Break up <??> differently, and explain why

Add guards to ext_orport.h, rename get_file to get_file_name

Use only uintptr_t for the value of transport_count

If a single client connects with multiple transports, note all transports.

Fix a number of issues with the #5040 code.

- Don't leak if a transport proxy sends us a TRANSPORT command more
  than once.

- Don't use smartlist_string_isin() in geoip_get_transport_history().
  (pointed out by Nick)

- Use the 'join' argument of smartlist_join_strings() instead of
  trying to write the separator on our own.
  (pointed out by Nick)

- Document 'ext_or_transport' a bit better.
  (pointed out by Nick)

- Be a bit more consistent with the types of the values of 'transport_counts'.
  (pointed out by Nick)

Trivially change unittests to make them more readable.

Pluggable transport names are now in alphabetical order like they
appear in the bridge-ip-transports string.

Make check-spaces happy.

Add a changes file for #5040.

Fix some unittests that broke when we added bridge-ip-transports.

Add a unittest for the bridge-ip-transports extra-info line.

Correction in the 0.2.4.16-rc changelog

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

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

Merge remote-tracking branch 'asn/bug9363_take2'

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 August 2013 GeoIP database.

Add 2, update 1, and remove 3 manual A1 substitutions.

new location for tarballs; and you should edit Makefile too

forward-port the 0.2.4.16-rc changelog

Merge branch 'maint-0.2.4'

Fix an uninitialized-read when parsing v3 introduction requests.

Fortunately, later checks mean that uninitialized data can't get sent
to the network by this bug.  Unfortunately, reading uninitialized heap
*can* (in some cases, with some allocators) cause a crash if you get
unlucky and go off the end of a page.

Found by asn.  Bugfix on 0.2.4.1-alpha.

Update coverage instructions

Also test that server transports get written to Tor's state.

test_pt_configure_proxy(): verify the internals of the managed proxy.

forward-port 0.2.4.15-rc changelog

forward-port the reformat of changelog stanza

Use size_t for length in replaycache_add_test_and_elapsed()

For consistency with the rest of buffer lengths in Tor.

See #8960

Fix tor_get_lines_from_handle() for make check-spaces

Unit test for smartlist_ints_eq

(This was the only wholly untested function in containers.c)

Try to make the win32 ersatz_socketpair pass.

Fixed "unused parameter cloexec" warnings on windows

Unit test for tor_{ersatz_,}socketpair.

This catches tor_accept as well.

Split the socketpair replacement code into its own function for testing

Simple unit test for tor_open_socket_with_extensions

Actually use the cloexec argument in the !defined(SOCK_CLOEXEC) case

Prepare patch for ticket 5129 for merging.

- Preserve old eventdns code.
- Add function to close sockets cross-platform, without accounting.
- Add changes/ file.

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

Merge remote-tracking branch 'arma/bug9354' into maint-0.2.4

Fix compilation on Windows

(Bugfix on tests for #9288 fix; bug not in any released Tor)

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

Fix invalid-read when a managed proxy configuration fails.

Merge branch 'bug9288_rebased'

Conflicts:
src/test/test_pt.c

Fix mixed declaration/statement warning

Update pt/configure_proxy until it stops segfaulting

Modifications to transports.c for the unit tests to work.

Both 'managed_proxy_list' and 'unconfigured_proxies_n' are global
src/or/transports.c variables that are not initialized properly when
unit tests are run.

Write unit tests for configure_proxy().

Prepare some mock functions to test #9288.

Fix invalid-read when a managed proxy configuration fails.

NumDirectoryGuards now tracks NumEntryGuards by default

Now a user who changes only NumEntryGuards will get the behavior she
expects. Fixes bug 9354; bugfix on 0.2.4.8-alpha.

Allow {,k,kilo,m,mega,g,giga,t,tera}bit{,s} in torrc

Patch from CharlieB for ticket #9214

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

Merge branch 'bug9337' into maint-0.2.4

Avoid assertion failure on unexepcted address family in DNS reply.

Fixes bug 9337; bugfix on 0.2.4.7-alpha.

Fix bug9309, and n_noncanonical count/continue code

When we moved channel_matches_target_addr_for_extend() into a separate
function, its sense was inverted from what one might expect, and we
didn't have a ! in one place where we should have.

Found by skruffy.

fix typo

Fix bug9309, and n_noncanonical count/continue code

When we moved channel_matches_target_addr_for_extend() into a separate
function, its sense was inverted from what one might expect, and we
didn't have a ! in one place where we should have.

Found by skruffy.

Whoops; check in test_circuitmux.c

Merge branch 'bug9296'

Regression test for destroy cell queue

This exercises the code that crashed and caused #9296.

Initialize destroy_cell_queue.

Likely fix for the crash bug of #9296, which was introduced through a
combination of #7912 and #8586.  Bugfix not in any released Tor.

fix typo

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

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

Fix for bug 9295: Don't crash when running --hash-password.

Fix on 0.2.4.15-rc; I'm doing this branch against 0.2.3, since we're
considering #9122 for an 0.2.3 backport, and if we do, we should merge
this too.