Don't warn when compiling with --disable-threads

STMT_VOID semantics suggested by nick, thanks!

Robert says that this bug was not in fact one of frosty's

Merge branch 'bug4424' into maint-0.2.2

Remove an extraneous "if" in the 4424 fix

Don't leak an extend_info_t in rend_client_any_intro_points_usable

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

Add a changes file for 4410

Fix remotely triggerable assert during ip decryption

Fixes bug 4410.

Merge branch 'bug4383_nm' into maint-0.2.2

Fix a memleak when fetching descriptors for bridges in ExcludeNodes.

Merge remote-tracking branch 'erinn/win-bundle-fix' into maint-0.2.2

remove absolute path from contrib/package_nsis-mingw.sh in order to make it easier to automatically build tor expert bundle

Add a changes file for the 4340 fix

Disable stats requiring geoip info if we have none

In other parts of the code we will otherwise attempt to collect these
statistics, and that will lead to crashes.

Discard all cells on a marked connection

Fix for bug 4299

Fix typo, spotted by tmpname0901. Thanks!

bump maint-0.2.2 to 0.2.2.34-dev

Merge branch 'maint-0.2.1' into maint-0.2.2

bump maint to 0.2.1.31

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.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.1' into maint-0.2.2

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.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.

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 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.

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.

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.

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.

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.

Mention that HiddenServiceDir must exist

lround() missing in MSVC

lround() is missing in MS Visual-C's <math.h>. Not available anywhere.
Here is an easy patch.

Don't write ControlPorts to disk till after we setuid and make datadir

Fix for bug 3747; bugfix on 0.2.2.26-beta.

Remove extra quotation marks around the result of esc_for_log

Ignore deprecation warnings on OS X

Starting with Lion, Apple decided to deprecate the system openssl. We
can start requiring users to install their own openssl once OS X doesn't
ship with it anymore.

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

Update to the August 2011 GeoIP database.

Check return value in fmt_addr

Previously, if tor_addr_to_str() returned NULL, we would reuse the
last value returned by fmt_addr().  (This could happen if we were
erroneously asked to format an AF_UNSPEC address.)  Now instead we
return "???".