forward-port the 0.2.1.31 release notes

add a release blurb

bump to 0.2.2.34

fold in changes entries

Merge branch 'maint-0.2.2' into release-0.2.2

Add option to give guard flag to relays without the CVE-2011-2768 fix

This way, all of the DA operators can upgrade immediately, without nuking
every client's set of entry guards as soon as a majority of them upgrade.

Until enough guards have upgraded, a majority of dirauths should set this
config option so that there are still enough guards in the network. After
a few days pass, all dirauths should use the default.

Don't give the Guard flag to relays without the CVE-2011-2768 fix

Make tor_version_same_series non-static

Merge branch 'maint-0.2.1_secfix' into maint-0.2.2_secfix

Conflicts:
src/or/connection_or.c

Reject create cells on outgoing OR connections from bridges

Mark which OR connections are outgoing

Don't use any OR connection which sent us a CREATE_FAST cell for an EXTEND

Fix suggested by Nick Mathewson.

Don't send a certificate chain on outgoing TLS connections from non-relays

Merge branch 'maint-0.2.2' into release-0.2.2

Merge branch 'maint-0.2.1' into maint-0.2.2

Remove the -F option from tor-resolve.

It used to mean "Force": it would tell tor-resolve to ask tor to
resolve an address even if it ended with .onion.  But when
AutomapHostsOnResolve was added, automatically refusing to resolve
.onion hosts stopped making sense.  So in 0.2.1.16-rc (commit
298dc95dfd8), we made tor-resolve happy to resolve anything.

The -F option stayed in, though, even though it didn't do anything.
Oddly, it never got documented.

Found while fixing GCC 4.6 "set, unused variable" warnings.

manually backport a5232e0c4c

Merge branch 'maint-0.2.2' into release-0.2.2

Merge branch 'maint-0.2.1' into maint-0.2.2

always more commits waiting!

Merge branch 'maint-0.2.2' into release-0.2.2

closer to an 0.2.2.34 changelog

stop asserting at boot

The patch for 3228 made us try to run init_keys() before we had loaded
our state file, resulting in an assert inside init_keys. We had moved
it too early in the function.

Now it's later in the function, but still above the accounting calls.

Reinit keys at the start of options_act().

Previously we did this nearer to the end (in the old_options &&
transition_affects_workers() block).  But other stuff cares about
keys being consistent with options... particularly anything which
tries to access a key, which can die in assert_identity_keys_ok().

Fixes bug 3228; bugfix on 0.2.2.18-alpha.

Conflicts:

src/or/config.c

Don't crash a bridge authority on SIGHUP if it's not in the consensus

Fixes bug 2572.

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.

Conflicts:

src/or/config.c

Merge remote-tracking branch 'public/cov_run224_022' into maint-0.2.2

Don't crash when accountingmax is set in non-server Tors

We use a hash of the identity key to seed a prng to tell when an
accounting period should end.  But thanks to the bug998 changes,
clients no longer have server-identity keys to use as a long-term seed
in accounting calculations.  In any case, their identity keys (as used
in TLS) were never never fixed.  So we can just set the wakeup time
from a random seed instead there.  Still open is whether everybody
should be random.

This patch fixes bug 2235, which was introduced in 0.2.2.18-alpha.

Diagnosed with help from boboper on irc.

Properly refcount client_identity_key

In a2bb0bf we started using a separate client identity key. When we are
in "public server mode" (that means not a bridge) we will use the same
key. Reusing the key without doing the proper refcounting leads to a
segfault on cleanup during shutdown. Fix that.

Also introduce an assert that triggers if our refcount falls below 0.
That should never happen.

Add some asserts to get_{tlsclient|server}_identity_key

We now require that:
  - Only actual servers should ever call get_server_identity_key
  - If you're being a client or bridge, the client and server keys should
    differ.
  - If you're being a public relay, the client and server keys
    should be the same.

Rename get_client_identity_key to get_tlsclient_identity_key

Maintain separate server and client identity keys when appropriate.

Fixes a bug described in ticket #988.

Conflicts:

src/or/main.c
src/or/router.c

Make crypto_free_pk_env tolerate NULL arg in 0.2.1.  Error-proofing against bug 988 backport

Maintain separate server and client TLS contexts.

Fixes bug #988.

Conflicts:

src/or/main.c
src/or/router.c

Refactor tor_tls_context_new:

* Make tor_tls_context_new internal to tortls.c, and return the new
  tor_tls_context_t from it.

* Add a public tor_tls_context_init wrapper function to replace it.

Conflicts:

src/or/main.c
src/or/router.c

Add public_server_mode function.

Merge branch 'maint-0.2.2' into release-0.2.2

Merge branch 'maint-0.2.1' into maint-0.2.2

Fix zlib macro brokenness on osx with zlib 1.2.4 and higher.

From the code:
   zlib 1.2.4 and 1.2.5 do some "clever" things with macros.  Instead of
   saying "(defined(FOO) ? FOO : 0)" they like to say "FOO-0", on the theory
   that nobody will care if the compile outputs a no-such-identifier warning.

   Sorry, but we like -Werror over here, so I guess we need to define these.
   I hope that zlib 1.2.6 doesn't break these too.

Possible fix for bug 1526.

Merge branch 'maint-0.2.2' into release-0.2.2

unmangle the fingerprint example in the man page

Remove a confusing dollar sign from the example fingerprint in the
man page, and also make example fingerprint a valid one.

Merge branch 'maint-0.2.2' into release-0.2.2

Merge remote-tracking branch 'rransom-tor/bug4251-022' into maint-0.2.2

Fix crash when changing node restrictions with DNS lookup in progress

Fixes bug 4259, bugfix on 0.2.2.25-alpha.  Bugfix by "Tey'".

Original message by submitter:

  Changing nodes restrictions using a controller while Tor is doing
  DNS resolution could makes Tor crashes (on WinXP at least). The
  problem can be repeated by trying to reach a non-existent domain
  using Tor:

    curl --socks4a 127.0.0.1:9050 inexistantdomain.ext

  .. and changing the ExitNodes parameter through the control port
  before Tor returns a DNS resolution error (of course, the following
  command won't work directly if the control port is password
  protected):

    echo SETCONF ExitNodes=TinyTurtle | nc -v 127.0.0.1 9051

  Using a non-existent domain is needed to repeat the issue so that
  Tor takes a few seconds for resolving the domain (which allows us to
  change the configuration). Tor will crash while processing the
  configuration change.

  The bug is located in the addressmap_clear_excluded_trackexithosts
  method which iterates over the entries of the addresses map in order
  to check whether the changes made to the configuration will impact
  those entries. When a DNS resolving is in progress, the new_adress
  field of the associated entry will be set to NULL. The method
  doesn't expect this field to be NULL, hence the crash.

Free rend_data and intro_key when extra intro circs become general-purpose

Merge branch 'maint-0.2.1' into maint-0.2.2

Update to the October 2011 GeoIP database.

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.

Note ticket and source version for feature3951 in changes file

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

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

Make internal error check for unrecognized digest algorithm more robust

Fixes Coverity CID 479.

Check return of init_keys() ip_address_changed: fix Coverity CID 484

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.

Looks like Windows version 6.2 will be Windows 8

Thanks to funkstar for the report

bridges should use create_fast cells for their own circuits

fixes bug 4124, as noticed in bug 4115

bug 4115: make bridges use begindir for their dir fetches

removes another avenue for enumerating bridges.

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

changes file for bug 4059

Fix a potentially useless integer overflow check.

GCC 4.2 and maybe other compilers optimize away unsigned integer
overflow checks of the form (foo + bar < foo), for all bar.

Fix one such check in `src/common/OpenBSD_malloc_Linux.c'.

Merge branch 'maint-0.2.1' into maint-0.2.2

Update to the September 2011 GeoIP database.

prepare a release blurb

bump to 0.2.2.33

fold in one more

Merge branch 'maint-0.2.2' into release-0.2.2

Merge branch 'maint-0.2.1' into maint-0.2.2

Conflicts:
src/or/main.c
src/or/router.c

Generate our ssl session certs with a plausible lifetime

Nobody but Tor uses certs on the wire with 2 hour lifetimes,
and it makes us stand out. Resolves ticket 4014.

fold in recent changelog entries

Merge branch 'maint-0.2.2' into release-0.2.2

Update to the September 2011 GeoIP database.

Demote 'INTRODUCE2 cell is too {old,new}' message to info level

Demote HS 'replay detected' log message for DH public keys to info level

Describe rend_service_descriptor_t more completely

Describe rend_intro_point_t more completely

Fix log message typo.

typos on line 1 of your changelog make you look dumb

fold in latest changes files

Merge branch 'maint-0.2.2' into release-0.2.2

Merge remote-tracking branch 'public/gcc-295-fix' into maint-0.2.2

Merge remote-tracking branch 'public/enhance_replay_detection' into maint-0.2.2

Check for replays in PK-encrypted part of intro cell, not just in the g^x value

Add changes file for bsd4 fixes

Merge remote-tracking branch 'public/bug2649_squashed' into maint-0.2.2

Look for correct "ar" to cross-compile

For some reason, autoconf doesn't by default have an "AC_PROG_AR" for
this -- possibly it's assumed that any "ar" you have will work
everyplace.

Fixes bug 3909; found by sid77.

This fixes a build issue first present in fdbdb4dc1500, but the bug
(of not using a correct ar) has been in every Tor version ever: it
just didn't matter until then.

fold in changes files so far

Merge branch 'maint-0.2.2' into release-0.2.2

Fix assertion in addressmap_clear_excluded_trackexithosts

Fixes bug 3923; bugfix on 0.2.2.25-alpha; bugfix from 'laruldan' on trac.

Correct man page: multiple control auth styles can be set at once

Check for lround with autoconf; fall back to rint.

Fix a compilation issue on older FreeBSDs

Use %f with printf-style formatting, not %lf

For printf, %f and %lf are synonymous, since floats are promoted to
doubles when passed as varargs.  It's only for scanf that we need to
say "%lf" for doubles and "%f" for floats.

Apparenly, some older compilers think it's naughty to say %lf and like
to spew warnings about it.

Found by grarpamp.

Switch a SMARTLIST_FOREACH in circuitbuild.c to BEGIN/END

It had some cpp stuff inside, and older GCCs don't like preprocessor
directives inside macro arguments.

Found by grarpamp.

update punctuation now that we've seen the source text

expand and relocate the dedication

our windows ce support isn't actually support