Tweak #6752 patch based on comments by nickm.

Lower dir fetch retry schedules in testing networks.

Also lower maximum interval without directory requests, and raise
maximum download tries.

Implements #6752.

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

Fix unit tests to pass after fix for #8879

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

Fix socks5 handshake for username/password auth

The fix for bug 8117 exposed this bug, and it turns out real-world
applications like Pidgin do care. Bugfix on 0.2.3.2-alpha; fixes bug 8879.

Changes file for bug8791

Check the return value of memchr().

For quality-of-implementation in munge_extrainfo_into_routerinfo().
See #8791

Add a changes file for bug 8790

Remove strcpy from unit tests.

See #8790.

Merge branch 'maint-0.2.4'

Merge bug5595-v2-squashed into maint-0.2.4

Add some unit tests for fp_pair_map_t to test/containers.c based on the strmap tests

Rephrase comment in trusted_dirs_load_certs_from_string() to reflect 5595 fix

Make warning in authority_cert_dl_failed() LD_BUG per NickM code review

Use tor_asprintf() and clean up string handling in authority_certs_fetch_missing()

Clean up ugly constants in connection_dir_download_cert_failed(), and fix a broken one

Avoid duplicate downloads by (fp,sk) and by fp for authority certs when bootstrapping

When downloading certificates, distinguish requesting by identity digest from requesting by ID digest, signing key pair; fixes bug 5595

Implement fp_pair_map_t

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

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

Merge branch 'bug8844_v2' into maint-0.2.3

Don't run off the end of the array-of-freelists

This is a fix for bug 8844, where eugenis correctly notes that there's
a sentinel value at the end of the list-of-freelists that's never
actually checked.  It's a bug since the first version of the chunked
buffer code back in 0.2.0.16-alpha.

This would probably be a crash bug if it ever happens, but nobody's
ever reported something like this, so I'm unsure whether it can occur.
It would require write_to_buf, write_to_buf_zlib, read_to_buf, or
read_to_buf_tls to get an input size of more than 32K.  Still, it's a
good idea to fix this kind of thing!

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

Fix bug 8845: check the right length of memory in aes unit tests

This couldn't actually be a buffer overrun unless AES somehow turned
into memcpy, but still it's good to fix it.

Merge branch 'maint-0.2.4' into master

Use a clearer idiom for node identity in router_counts_toward_thresholds

Fix 8833: crash bug from using NULL node->ri in dirserv.c

It appears that moria1 crashed because of one instance of this (the
one in router_counts_toward_thresholds).  The other instance I fixed
won't actually have broken anything, but I think it's more clear this
way.

Fixes bug 8833; bugfix on 0.2.4.12-alpha.

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

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

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

Merge branch 'bug8235-fix-rebased' into maint-0.2.4

Increase the pathbias state file miscounting version check.

We now know the bug is present in 0.2.4.12-alpha too. It should be fixed in
0.2.4.13-alpha, though.

Changes file for 8235.

Bug 8235: Fix scaling adjustments.

We need to subtract both the current built circuits *and* the attempted
circuits from the attempt count during scaling, since *both* have already been
counted there.

Change dflt branch to inspect in findMergedChanges.pl

Remove changes files which were merged in release-0.2.4

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

Add a boolean to flag-thresholds for "we have enough measured bandwidth"

Implements #8711.

Turn on ntor by default client-side

Implements #8561.

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

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

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

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

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

Conflicts:
src/common/crypto.c

Remove a double-newline

Merge branch 'less_charbuf_rebased' into maint-0.2.4

Conflicts:
src/or/dirserv.c
src/or/dirserv.h
src/test/test_dir.c

Add unit test for encoding ntor key in routerinfo

Remove the unused pk3 variable from test_dir_formats

Re-enable test for parsing and generating descriptor with exit policy

Looks like I turned this off in 6ac42f5e back in 2003 and never got
around to making it work again.  There has been no small amount of
code drift.

Test networkstatus_getinfo_helper_single

Add a quick-and-dirty-test for generate_v2_networkstatus.

It sure is a good thing we can run each test in its own process, or
else the amount of setup I needed to do to make this thing work
would have broken all the other tests.

Test mocking would have made this easier to write too.

Remove the now-unused router_get_networkstatus_v3_hash

Test improvement: include microdesc lines in our synthetic microdesc consensuses.

Fix a couple of documentation issues.

Remove some now-needless length defines

Remove RS_ENTRY_LEN

Nothing uses it any longer now that we use smartlists of strings for
stuff that manipulates iles of formatted routerstatuses.

Initial changelog for less_charbuf

Refactor dirobj signature generation

Now we can compute the hash and signature of a dirobj before
concatenating the smartlist, and we don't need to play silly games
with sigbuf and realloc any more.

Use chunks, not buffers, for router descriptors

Refactor v2 networkstatus generation to avoid buffer-style

Refactor routerstatus_format_entry to avoid character-buffers

Fix some wide lines

forward-port 0.2.4.1[12] changelogs

Don't leak a waiting-for-certs consensus when accepting it.

I believe this was introduced in 6bc071f765d2829249db52, which makes
this a fix on 0.2.0.10-alpha.  But my code archeology has not extended
to actually testing that theory.

Fix memory leak when sending configuration-changed event

Fix for bug #8718; bugfix on 0.2.3.3-alpha.

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

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

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

Merge branch 'bug8037_squashed' into maint-0.2.4

Reject most directory documents with an internal NUL.

(Specifically, we reject all the ones that aren't NUL-terminated,
since a NUL-terminated thing can't have a NUL in the middle.)

Another fix for #8037.

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

Conflicts:
src/or/dirserv.c

Merge branch 'bug8683_rebased' into maint-0.2.4

Remove a now-empty if body; invert the sense of its condition

Remvoe total_bandwidth and total_exit_bandwidth as unused.

Rename all fields which measure bw in kb to end with _kb

Fix some KB/B confusion in flag threshold minima.

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

Manpage: refer to ExcludeExitNodes, not the nonexistent ExcludeEntryNodes

Spotted on tor-talk by "hamahangi".

fix trivial typo

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

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

Avoid busy-looping on WANTREAD within connection_handle_write

Fix for bug 5650.  Also, if we get a WANTREAD while reading while
writing, make sure we're reading.

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

Merge branch 'bug8587_v3' into maint-0.2.4

Better test program for 128-bit math support

Clang 3.2 does constant-folding and variable substitution to determine
that the program is equivalent to "return 1".  Splitting the 128-bit
math into a new function seems sufficient to fix this.

Merge branch 'maint-0.2.4'

socks5 will ask for username/password if we offer it

Commit a264c4fe made the socks5 server prefer auth 2 if it's offered,
but it didn't update the unit test to expect it.

Use credible_bandwidth uniformly in setting/using fast_bandwidth

We were using credible_bandwidth to build the fast_bandwidth
threshold, but comparing it to bandwidth_for_router.

Use correct units for dirserv_get_{credible_bandwidth,bandwidth_for_router}

We were mixing bandwidth file entries (which are in kilobytes) with
router_get_advertised_bw() entries, which were in bytes.

Also, use router_get_advertised_bandwidth_capped() for credible_bandwidth.

socks5 will ask for username/password if we offer it

Commit a264c4fe made the socks5 server prefer auth 2 if it's offered,
but it didn't update the unit test to expect it.

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

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

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

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

Conflicts:
src/or/status.c

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

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

Conflicts:
doc/tor.1.txt
src/or/config.c
src/or/connection.c

fix text in function comment

(now that we moved to libevent)

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