Fix names of functions that convert strings to addrs

Now let's have "lookup" indicate that there can be a hostname
resolution, and "parse" indicate that there wasn't.  Previously, we
had one "lookup" function that did resolution; four "parse" functions,
half of which did resolution; and a "from_str()" function that didn't
do resolution.  That's confusing and error-prone!

The code changes in this commit are exactly the result of this perl
script, run under "perl -p -i.bak" :

  s/tor_addr_port_parse/tor_addr_port_lookup/g;
  s/parse_addr_port(?=[^_])/addr_port_lookup/g;
  s/tor_addr_from_str/tor_addr_parse/g;

This patch leaves aton and pton alone: their naming convention and
behavior is is determined by the sockets API.

More renaming may be needed.

Fix a bunch of whitespace errors

Merge branch 'prop176-v2'

Add some points to make it easy to turn off v3 support

Changes file for prop176 branch

Quiet two notices, and spelling mistake cleanup

Fix a few 64bit compiler warnings

Add more log statements for protocol/internal failures

Remove auth_challenge field from or_handshake_state_t

We didn't need to record this value; it was already recorded
implicitly while computing cell digests for later examination in the
authenticate cells.

spec conformance: allow only one cert of each type

Give tor_cert_get_id_digests() fail-fast behavior

Right now we can take the digests only of an RSA key, and only expect to
take the digests of an RSA key.  The old tor_cert_get_id_digests() would
return a good set of digests for an RSA key, and an all-zero one for a
non-RSA key.  This behavior is too error-prone: it carries the risk that
we will someday check two non-RSA keys for equality and conclude that
they must be equal because they both have the same (zero) "digest".

Instead, let's have tor_cert_get_id_digests() return NULL for keys we
can't handle, and make its callers explicitly test for NULL.

Fix some more issues wrt tor_cert_new found by asn

Make more safe_str usage happen for new logs in command.c

Set up network parameters on non-authenticated incoming connections

Also add some info log messages for the steps of the v3 handshake.

Now my test network bootstraps!

Make sure we stop putting cells into our hash at the right time.

Bugfixes for authenticate handling and generation

Fix log message about what cells we are sending

more verbose log for recording an odd cell

Actually accept cells in SERVER_RENEGOTIATING

Generate certificates that enable v3 handshake

Allow "finished flushing" during v3 handshake

Hook up all of the prop176 code; allow v3 negotiations to actually work

Remove a no-longer-relevant comment

Make tor_tls_cert_is_valid check key lengths

New functions to record digests of cells during v3 handshake

Also, free all of the new fields in or_handshake_state_t

Implement cert/auth cell reading

Basic function to write authenticate cells

Also, tweak the cert cell code to send auth certs

Function to return peer cert as tor_tls_cert

Add AUTH keys as specified in proposal 176

Our keys and x.509 certs are proliferating here.  Previously we had:
   An ID cert (using the main ID key), self-signed
   A link cert (using a shorter-term link key), signed by the ID key

Once proposal 176 and 179 are done, we will also have:
   Optionally, a presentation cert (using the link key),
       signed by whomever.
   An authentication cert (using a shorter-term ID key), signed by
       the ID key.

These new keys are managed as part of the tls context infrastructure,
since you want to rotate them under exactly the same circumstances,
and since they need X509 certificates.

Functions to get a public RSA key from a cert

Function to detect certificate types that signal v3 certificates

Function to get digests of the certs and their keys

More functions to manipulate certs received in cells

Function to extract the TLSSECRETS field for v3 handshakes

Functions to send cert and auth_challenge cells.

Cell types and states for new OR handshake

Also, define all commands > 128 as variable-length when using
v3 or later link protocol.  Running into a var cell with an
unrecognized type is no longer a bug.

Add a sha256 hmac function, with tests

Turn X509 certificates into a first-class type and add some functions

New function to get all digests of a public key

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

Merge remote-tracking branch 'sebastian/osxcompile'

Consider hibernation before dropping privs

Without this patch, Tor wasn't sure whether it would be hibernating or
not, so it postponed opening listeners until after the privs had been
dropped. This doesn't work so well for low ports. Bug was introduced in
the fix for bug 2003. Fixes bug 4217, reported by Zax and katmagic.
Thanks!

Fix a compile warning on OS X 10.6 and up

Add a missing comma in tor_check_port_forwarding

My fault; fix for bug 4213.

Update documentation comment for rend_client_reextend_intro_circuit

One of its callers assumes a non-zero result indicates a permanent failure
(i.e. the current attempt to connect to this HS either has failed or is
 doomed).  The other caller only requires that this function's result
never equal -2.

Bug reported by Sebastian Hahn.

Don't launch a useless circuit in rend_client_reextend_intro_circuit

Fixes bug 4212.  Bug reported by katmagic and found by Sebastian.

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

This merge is here to take a commit (feature 3951) that we already
have in master, so use "merge -s ours"

Note ticket and source version for feature3951 in changes file

Merge remote-tracking branch 'karsten/feature3951' into maint-0.2.2

Merge remote-tracking branch 'public/bug2003_nm'

reinstate a notice for the non-loopback socksport case

Thanks to prop171, it's no longer a crazy thing to do, but you should
make sure that you really meant it!

Merge remote-tracking branch 'rransom-tor/bug4018'

Merge remote-tracking branch 'public/bug2430'

Merge remote-tracking branch 'asn2/bug3656'

Conflicts:
src/common/util.c
src/common/util.h
src/or/config.h
src/or/main.c
src/test/test_util.c

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

Avoid running DNS self-tests if we're operating as a bridge

Revive our beautiful unit tests.

They broke when the PT_PROTO_INFANT proxy state was added.

Make it compile on Windows™.

Support multiple transports in a single transport line.

Support multiple comma-separated transpotrs in a single
{Client,Server}TransportPlugin line.

Turn on directory request statistics by default.

Change the default values for collecting directory request statistics and
inlcuding them in extra-info descriptors to 1.

Don't break if we are configured to collect directory request or entry
statistics and don't have a GeoIP database. Instead, print out a notice
and skip initializing the affected statistics code.

This is the cherry-picked 499661524b0a572303087af721325608dd91f7ce.

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

Fix compilation of 3335 and 3825 fixes

In master, they ran into problems with the edge_conn/entry_conn split.

Merge remote-tracking branch 'rransom-tor/bug3335-v2'

Conflicts:
src/or/connection_edge.c
src/or/rendclient.c

Add changes file for bug 4094

Rephrase the log messages emitted if the TestSocks check is positive

Previously Tor would always claim to have been given a hostname
by the client, while actually only verifying that the client
is using SOCKS4A or SOCKS5 with hostnames. Both protocol versions
allow IP addresses, too, in which case the log messages were wrong.

Fixes #4094.

Remove an HS's last_hid_serv_requests entries when a conn. attempt ends

Record the HS's address in last_hid_serv_request keys

Fix comment typo

Detect and remove unreachable intro points

Clear the timed_out flag when an HS connection attempt ends

Record intro point timeouts in rend_intro_point_t

Refetch an HS's desc if we don't have a usable one

Previously, we wouldn't refetch an HS's descriptor unless we didn't
have one at all.  That was equivalent to refetching iff we didn't have
a usable one, but the next commit will make us keep some non-usable HS
descriptors around in our cache.

Code bugfix on the release that introduced the v2 HS directory system,
because rend_client_refetch_v2_renddesc's documentation comment should
have described what it actually did, not what its behaviour happened
to be equivalent to; no behaviour change in this commit.

Looks like Windows version 6.2 will be Windows 8

Thanks to funkstar for the report

bump to 0.2.3.5-alpha

give 0.2.3.5-alpha a blurb

fix default for TokenBucketRefillInterval in man page

fold in recent changes entries

refill our token buckets 10 times/sec, not 100

refilling often is good, but refilling often has unclear side effects
on a) cpu load, and b) making sure every cell, ever, is sent out one at
a time

Merge branch 'maint-0.2.2'

bridges should use create_fast cells for their own circuits

fixes bug 4124, as noticed in bug 4115

Merge branch 'maint-0.2.2'

start preparing the 0.2.3.5-alpha changelog

bug 4115: make bridges use begindir for their dir fetches

removes another avenue for enumerating bridges.

Make sure the microdesc cache is loaded before setting a v3 md consensus

Otherwise, we can wind up munging our reference counts if we set it in
the middle of loading the nodes.  This happens because
nodelist_set_consensus() and microdesc_reload_cache() are both in the
business of adjusting microdescriptors' references.

Add some debugging code to microdesc.[ch]

Fix a crash bug in tor_assert(md->held_by_node)

The fix is to turn held_by_node into a reference count.

Fixes bug 4118; bugfix on 0.2.3.1-alpha.

if we have enough usable guards, just pick one

we don't need to check whether we don't have enough guards right after
concluding that we do have enough.

slight efficiency fix suggested by an anonymous fellow on irc.

Merge branch 'maint-0.2.2'

trivial whitespace changes, take two

Trivial whitespace fixes

Add a changes file for Tom Lowenthal's patch for #4063

Ticket #4063 - change circuit build timeout log entries from NOTICE to INFO

Improve wording in some comments and log messages.

Merge branch 'feature3630-rebased'

Fix issues in 3630 patch noted by Karsten

Changes file for feature3630

Make bufferevents work with TokenBucketRefillInterval

Refactor connection_bucket_refill(_helper) to avoid roundoff

We were doing "divide bandwidth by 1000, then multiply by msec", but
that would lose accuracy: instead of getting your full bandwidth,
you'd lose up to 999 bytes per sec.  (Not a big deal, but every byte
helps.)

Instead, do the multiply first, then the division.  This can easily
overflow a 32-bit value, so make sure to do it as a 64-bit operation.

Make documentation for TokenBucketRefillInterval match its behavior

Move around check for TokenBucketRefillInterval; disallow changes to it