Merge remote-tracking branch 'rransom/bug2089' into maint-0.2.2

Merge maint-0.2.1 with -s ours.

Make the DH parameter we use for TLS match the one from Apache's mod_ssl

Our regular DH parameters that we use for circuit and rendezvous
crypto are unchanged.  This is yet another small step on the path of
protocol fingerprinting resistance.

(Backport from 0.2.2's 5ed73e3807d90dd0a3)

Document what descriptors HSAuthoritativeDir serves

Fixes bug 2089.

Update documentation for PublishServerDescriptor (0.2.2.x)

Merge branch 'bug2408-v2-021' into bug2408-v2-022 (keep-ours)

Merge branch 'bug2408-v2-021-common' into bug2408-v2-022

Update documentation for PublishServerDescriptor

Ignore and warn about "PublishServerDescriptor hidserv"

Fixes #2408.

Merge remote branch 'origin/maint-0.2.1' into maint-0.2.2

Locking failures on windows are indicated by EACCES

Patch our implementation of tor_lockfile_lock() to handle this case
correctly. Also add a note that blocking behaviour differs from windows
to *nix. Fixes bug 2504, issue pointed out by mobmix.

move the clause above the "if bw is too low" check

dtrt when only relaybandwidthburst is set

fixes bug 2470

Merge branch 'maint-0.2.1' into maint-0.2.2

Update to the February 1 2011 Maxmind GeoLite Country database.

typo in manpage fix

Merge branch 'bug2279' into maint-0.2.2

Some cleanups to bug2279 messages/docs from rransom

Merge branch 'bug2203_rebased' into maint-0.2.2

changes file for ASLR/DEP build on windows

Enable ASLR and permanent DEP for Windows executables

Fix for #2358

Merge remote branch 'sebastian/bug2450' into maint-0.2.2

Merge remote branch 'public/bug2378' into maint-0.2.2

Remove country codes from EntryNodes manpage entry

add documentation for ClientRejectInternalAddresses

Merge branch 'bug2181' into maint-0.2.2

Explain soft shutdown mode a little better in the accountingmax documentation

Fix typo in dir-spec.txt [found by sebastian]

Add an option to disable the block-private-addresses feature

Suggested by rransom.  Probably necessary for testing network mode.

Merge remote branch 'origin/maint-0.2.1' into maint-0.2.2

Merge-forward our exit-notice backport (empty merge)

fix the links in the exit-list notice we give out to users.

Backport current tor-exit-notice to 0.2.1

Merge branch 'bug2409' into maint-0.2.2

Merge branch 'bug2321' into maint-0.2.2

Avoid sketchy integer cast in cbt code

When calling circuit_build_times_shuffle_and_store_array, we were
passing a uint32_t as an int.  arma is pretty sure that this can't
actually cause a bug, because of checks elsewhere in the code, but
it's best not to pass a uint32_t as an int anyway.

Found by doorss; fix on 0.2.2.4-alpha.

Add client code to detect attempts to connect to 127.0.0.1 etc

We detect and reject said attempts if there is no chosen exit node or
circuit: connecting to a private addr via a randomly chosen exit node
will usually fail (if all exits reject private addresses), is always
ill-defined (you're not asking for any particular host or service),
and usually an error (you've configured all requests to go over Tor
when you really wanted to configure all _remote_ requests to go over
Tor).

This can also help detect forwarding loop requests.

Found as part of bug2279.

Fix double-mark bug when failing to init transparent connection

Fixes part of bug 2279.  Bugfix on 0.1.2.1-alpha.

Log more about soft-hibernation

Correctly detect BIO_new failures

This bug was noticed by cypherpunks; fixes bug 2378.

Bugfix on svn commit r110.

Always treat failure to allocate an RSA key as an unrecoverable allocation error

Handle failing cases of DH allocation

Remove an unused function in crypto.c

Comment remaining CBT functions.

Left circuit_build_times_get_bw_scale() uncommented because it is in the wrong
place due to an improper bug2317 fix. It needs to be moved and renamed, as it
is not a cbt parameter.

Describe consensus method 11 in dir-spec.txt

Add changelog entry for bug2203.

Fix authority side of 2203.

Do not add Exit bandwidth to E if BadExit is set.

Fix client side of 2203: Do not count BadExits as Exits.

Add changes file for bug2004

Fix bug #2004 by demoting a log message.

To quote arma: "So instead of stopping your CBT from screaming, you're just
going to throw it in the closet and hope you can't hear it?"

Yep. The log message can happen because at 95% point on the curve, we can be
way beyond the max timeout we've seen, if the curve has few points and is
shallow.

Also applied Nick's rule of thumb for rewriting some other notice log messages
to read like how you would explain them to a raving lunatic on #tor who was
shouting at you demanding what they meant. Hopefully the changes live up to
that standard.

Fix bug in verifying directory signatures with short digests

If we got a signed digest that was shorter than the required digest
length, but longer than 20 bytes, we would accept it as long
enough.... and then immediately fail when we want to check it.

Fixes bug 2409; bug in 0.2.2.20-alpha; found by piebeer.

Tell which geoip file we're parsing

Fix assert for relay/bridge state change

When we added support for separate client tls certs on bridges in
a2bb0bfdd5 we forgot to correctly initialize this when changing
from relay to bridge or vice versa while Tor is running. Fix that
by always initializing keys when the state changes.

Fixes bug 2433.

Make the DH parameter we use for TLS match the one from Apache's mod_ssl

Our regular DH parameters that we use for circuit and rendezvous
crypto are unchanged.  This is yet another small step on the path of
protocol fingerprinting resistance.

Merge remote branch 'origin/maint-0.2.1' into maint-0.2.2

Merge remote branch 'rransom/policy_summarize-assert' into maint-0.2.1

Fix bounds-checking in policy_summarize

Found by piebeer.

Merge remote branch 'origin/maint-0.2.1' into maint-0.2.2

Oops; actually add the code to the last patch. :/

Fix two more SIZE_T_CEILING issues

This patch imposes (very long) limits on the length of a line in a
directory document, and on the length of a certificate.  I don't
think it should actually be possible to overrun these remotely,
since we already impose a maximum size on any directory object we're
downloading, but a little defensive programming never hurt anybody.

Roger emailed me that doorss reported these on IRC, but nobody seems
to have put them on the bugtracker.

Merge branch 'maint-0.2.1' into maint-0.2.2

be the winner, rewrite history

Merge commit 'sebastian/bug2317' into maint-0.2.2

Merge branch 'maint-0.2.1' into maint-0.2.2

0.2.1.29 changelog and blurb

Merge remote branch 'origin/maint-0.2.1' into maint-0.2.2

Fix a couple of non-cleared key issues in hidden services

we need to do more hunting, but this fixes the ones mentioned in 2385.

Zero out some more key data before freeing it

Found by cypherpunks; fixes bug 2384.

Update the spec with the new bounds

Tighten accepted circwindow parameters

Based on discussion in bug 2317, these values seem to be sane.

Provide constant limits for all consensus params

This addresses Nick's concern about doing non-constant bounds checking
inside networkstatus_get_param().

Fix a typo spotted by Roger

Sanity-check consensus param values

We need to make sure that the worst thing that a weird consensus param
can do to us is to break our Tor (and only if the other Tors are
reliably broken in the same way) so that the majority of directory
authorities can't pull any attacks that are worse than the DoS that
they can trigger by simply shutting down.

One of these worse things was the cbtnummodes parameter, which could
lead to heap corruption on some systems if the value was sufficiently
large.

This commit fixes this particular issue and also introduces sanity
checking for all consensus parameters.

Make get_net_param_from_list() static

This prepares for making the accessor method for consensus parameters
safer in the next commit.

Merge remote branch 'origin/maint-0.2.1' into maint-0.2.2

Conflicts:
src/or/routerparse.c
src/or/test.c

Merge branch 'bug2352_obsize' into maint-0.2.1

Merge remote branch 'origin/maint-0.2.1' into maint-0.2.2

Add missing check for hostname answer_len in dnsserv size

This is checked elsewhere too, but let's be RFC-conformant.

Merge remote branch 'origin/maint-0.2.1' into maint-0.2.2

Merge branch 'bug2332_part2' into maint-0.2.1

Merge remote branch 'origin/maint-0.2.1' into maint-0.2.2

Merge branch 'bug2324_uncompress' into maint-0.2.1

clean up message; explain a magic number in a comment

Merge remote branch 'origin/maint-0.2.1' into maint-0.2.2

Conflicts:
src/or/config.c
src/or/networkstatus.c
src/or/rendcommon.c
src/or/routerparse.c
src/or/test.c

make the description of tolen_asserts more dire

We have a CVE # for this bug.

Fix a heap overflow found by debuger, and make it harder to make that mistake again

Our public key functions assumed that they were always writing into a
large enough buffer.  In one case, they weren't.

(Incorporates fixes from sebastian)

Always nul-terminate the result passed to evdns_server_add_ptr_reply

In dnsserv_resolved(), we carefully made a nul-terminated copy of the
answer in a PTR RESOLVED cell... then never used that nul-terminated
copy.  Ouch.

Surprisingly this one isn't as huge a security problem as it could be.
The only place where the input to dnsserv_resolved wasn't necessarily
nul-terminated was when it was called indirectly from relay.c with the
contents of a relay cell's payload.  If the end of the payload was
filled with junk, eventdns.c would take the strdup() of the name [This
part is bad; we might crash there if the cell is in a bad part of the
stack or the heap] and get a name of at least length
495[*]. eventdns.c then rejects any name of length over 255, so the
bogus data would be neither transmitted nor altered.

  [*] If the name was less than 495 bytes long, the client wouldn't
     actually be reading off the end of the cell.

Nonetheless this is a reasonably annoying bug.  Better fix it.

Found while looking at bug 2332, reported by doorss.  Bugfix on
0.2.0.1-alpha.

catch another overlong malloc possibility. found by cypherpunks

Fix another instance of "128" in buffers.c.  More bug2330.

Merge remote branch 'origin/maint-0.2.1' into maint-0.2.2

Make our replacement INT32_MAX always signed

The C standard says that INT32_MAX is supposed to be a signed
integer.  On platforms that have it, we get the correct
platform-defined value.  Our own replacement, however, was
unsigned.  That's going to cause a bug somewhere eventually.

Merge remote branch 'public/bug2363' into maint-0.2.2

Merge remote branch 'sebastian/bug2337' into maint-0.2.2

Merge branch 'bug2331' into maint-0.2.2

add a missing "not" in bug2331 changelog

Merge branch 'bug2346' into maint-0.2.2

Explain bug2346 fix better based on suggestions from arma

Fix a autoconf warning

Pull up more data when parsing socks messages

Previously, we only looked at up to 128 bytes.  This is a bad idea
since socks messages can be at least 256+x bytes long.  Now we look at
up to 512 bytes; this should be enough for 0.2.2.x to handle all valid
SOCKS messages.  For 0.2.3.x, we can think about handling trickier
cases.

Fixes 2330.  Bugfix on 0.2.0.16-alpha.

Wait 60 minutes before retrying failed state save; bug2346