Explain bug2346 fix better based on suggestions from arma

Wait 60 minutes before retrying failed state save; bug2346

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

Update to the January 1 2011 Maxmind GeoLite Country database.

Describe tor-resolve defaults. Bug 2364.

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

Remove a loud info log message

Correctly detect and exclude addresses outside of our virtual address range

Found by cypherpunks; fixes more of 2328.  Bug was introduced in 3623a122;
first appeared in 0.2.0.5-alpha.

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

Merge branch 'bug2328_021' into maint-0.2.1

Notice a little faster if we're running out of virtual addresses

We were not decrementing "available" every time we did
++next_virtual_addr in addressmap_get_virtual_address: we left out the
--available when we skipped .00 and .255 addresses.

This didn't actually cause a bug in most cases, since the failure mode
was to keep looping around the virtual addresses until we found one,
or until available hit zero.  It could have given you an infinite loop
rather than a useful message, however, if you said "VirtualAddrNetwork
127.0.0.255/32" or something broken like that.

Spotted by cypherpunks

Handle a NULL return from addressmap_get_virtual_address

Fix for bug 2328; bugfix on 0.1.2.1-alpha; bug found by doorss.

Fix a double-counting bug in addrmap_get_virtual_address

We were decrementing "available" twice for each in-use address we ran
across.  This would make us declare that we ran out of virtual
addresses when the address space was only half full.

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

Note that Tor requires Automake 1.7. Fixes bug 2305

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

Avoid assertion on read_file_to_str() with size==SIZE_T_CEILING-1

Spotted by doors, fixes bug 2326.

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

Check size against SIZE_T_CEILING in realloc too.

Fixes bug 2324.

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

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

Never include pthread.h when building for Windows.

On Windows, we never use pthreads, since it doesn't usually exist,
and when it does it tends to be a little weirdly-behaved.  But some
mingw installations have a pthreads installed, so autoconf detects
pthread.h and tells us about it.  This would make us include
pthread.h, which could make for trouble when the iffy pthread.h
tried to include config.h.

This patch changes compat.h so that we never include pthread.h on
Windows.  Fixes bug 2313; bugfix on 0.1.0.1-rc.

Fix a function formatting warning in rephist.c

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

Conflicts:
src/or/routerparse.c

fix whitespace issues

Bump copyright statements to 2011 (0.2.2)

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

Conflicts:
src/common/test.h
src/or/test.c

Bump copyright statements to 2011

Fix compile wanrings revealed by gcc 4.5 on mingw

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

Merge remote branch 'rransom/bug2190_the_hard_way' into maint-0.2.2

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

Merge remote branch 'public/bug2190_021' into maint-0.2.1

Merge branch 'maint-0.2.1' into maint-0.2.2

put 0.2.1.28 release notes in place too

Merge branch 'maint-0.2.1' into maint-0.2.2

Merge commit 'nickm/fix_security_bug_022' into maint-0.2.2

Merge commit 'nickm/fix_security_bug_021' into maint-0.2.1

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

Change gabelmoo's IP address and ports.

Merge remote branch fix_security_bug_021 into fix_security_bug_022

Conflicts:
src/common/memarea.c
src/or/or.h
src/or/rendclient.c

Add a changelog entry

Make payloads into uint8_t.

This will avoid some signed/unsigned assignment-related bugs.

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

Conflicts:
src/config/geoip

Have all of our allocation functions and a few others check for underflow

It's all too easy in C to convert an unsigned value to a signed one,
which will (on all modern computers) give you a huge signed value.  If
you have a size_t value of size greater than SSIZE_T_MAX, that is way
likelier to be an underflow than it is to be an actual request for
more than 2gb of memory in one go.  (There's nothing in Tor that
should be trying to allocate >2gb chunks.)

Base SIZE_T_CEILING on SSIZE_T_MAX.

Only add each log message to pending_cb_messages once.

Don't call flush_pending_log_callbacks while logging LD_NOCB messages.

Found by boboper.

Update to the December 1 2010 Maxmind GeoLite Country database.

Merge branch 'bug2081_followup_022' into maint-0.2.2

Reject relay versions older than 0.2.0.26-rc

This was the first version to cache the correct directory information.

Fixes bug 2156.

revise bug2081_followup changelog

Fix a bug in calculating wakeup time on 64-bit machines.

If you had TIME_MAX > INT_MAX, and your "time_to_exhaust_bw =
accountingmax/expected_bandwidth_usage * 60" calculation managed to
overflow INT_MAX, then your time_to_consider value could underflow and
wind up being rediculously low or high.  "Low" was no problem;
negative values got caught by the (time_to_consider <= 0) check.
"High", however, would get you a wakeup time somewhere in the distant
future.

The fix is to check for time_to_exhaust_bw overflowing INT_MAX, not
TIME_MAX: We don't allow any accounting interval longer than a month,
so if time_to_exhaust_bw is significantly larger than 31*24*60*60, we
can just clip it.

This is a bugfix on 0.0.9pre6, when accounting was first introduced.
It fixes bug 2146, unless there are other causes there too.  The fix
is from boboper.  (I tweaked it slightly by removing an assignment
that boboper marked as dead, and lowering a variable that no longer
needed to be function-scoped.)

Add a missing ! to directory_fetches_from_authorities

The old logic would have us fetch from authorities if we were refusing
unknown exits and our exit policy was reject*.  Instead, we want to
fetch from authorities if we're refusing unknown exits and our exit
policy is _NOT_ reject*.

Fixed by boboper.  Fixes more of 2097.  Bugfix on 0.2.2.16-alpha.

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.

Fix a harmless off-by-one error in counting controller argument lengths

Bugfix on 0.1.1.1-alpha; found by boboper.

Specified grammars for orconn-status and entry-guards for Tor versions 0.1.2.2-alpha through 0.2.2.1-alpha with feature VERBOSE_NAMES turned off.

Correct information about support for guards being called helper nodes.

The spec stated that support for the helper-nodes command would be removed
in 0.1.3.x, however support for this command is still in Tor. Updated the spec
to reflect this and added a node that the command is deprecated.

Correct grammars to reflect that VERBOSE_NAMES is part of the protocol.

Several updates to grammars for events and GETINFO results.  All relate
to the fact that LongName has replaced ServerID since 0.2.2.1-alpha. See
documentation of VERBOSE_NAMES for more information. The following
grammars were changed:
  * orconn-status GETINFO result
  * entry-guards GETINFO result
  * Path general token
  * OR Connection status changed event
  * New descriptors available event
In all cases a note was added about when the old grammar applies.

Several changes to the way tokens describing servers are documented.

(1) Made the wording of the comments consistant with token names.
Digest/Fingerprint and Name/Nickname were being used interchangeably.
Better to just use Fingerprint and Nickname becuase they are the names
of the tokens.

(2) Places the tokens currently in use before the tokens used in older
versions.  ServerSpec should be documented before ServerID.

(3) Added a note to the comments about ServerID that cross reference
the VERBOSE_FEATURE, allowing users to see when and why ServerID was
replaced with LongName.

Clarify description of FEATURES in control-spec.

(1) On by default is a bad way to describe features. Rather, they
are always on and should be viewed as a part of the control
protocol. Updated the wording in USEFEATURE to reflect this.

(2) Made descriptions of Tor versions consistant across all
features. There is the version in which a feature was introduced and
the version in which it became part of the protocol.

(3) Reworded the description of the VERBOSE_NAMES feature. The
previous wording describes the way things used to be first. Better to
lead with the current state of things and then describe how it differs
from old versions.

Merge branch 'maint-0.2.1' into maint-0.2.2

Conflicts:

doc/Makefile.am

add 0.2.1.27 blurb and changelog to release notes

fix changes file for 2210

Revise comment on 2210 a little; clean up n_streams/num_streams confusion

Also add a changes file

Add wrappers function for libc random()

On windows, it's called something different.

Fix whitespace in patch for 2210 and backport to 0.2.2

Improve fairness when activating streams in circuit_resume_edge_reading_helper

 The reason the "streams problem" occurs is due to the complicated
interaction between Tor's congestion control and libevent. At some point
during the experiment, the circuit window is exhausted, which blocks all
edge streams. When a circuit level sendme is received at Exit, it
resumes edge reading by looping over linked list of edge streams, and
calling connection_start_reading() to inform libevent to resume reading.
When the streams are activated again, Tor gets the chance to service the
first three streams activated before the circuit window is exhausted
again, which causes all streams to be blocked again. As an experiment,
we reversed the order in which the streams are activated, and indeed the
first three streams, rather than the last three, got service, while the
others starved.

 Our solution is to change the order in which streams are activated. We
choose a random edge connection from the linked list, and then we
activate streams starting from that chosen stream. When we reach the end
of the list, then we continue from the head of the list until our chosen
stream (treating the linked list as a circular linked list). It would
probably be better to actually remember which streams have received
service recently, but this way is simple and effective.

Merge branch 'exitstats' into maint-0.2.2

comment karsten's bug2196 patch a little

Merge remote branch 'rransom/fix2195-fix' into maint-0.2.2

Avoid dereferencing NULL if a bridge fails to build an ei descriptor.

Reported by an anonymous commenter on Trac.

Report only the top 10 ports in exit-port stats.

Fix compilation with mingw and OpenSSL 0.9.8m+

Use S_CASE for ehostunreach, not E_CASE. Partial backport of 69deb22f. Fixes 0.2.1 compilation on windows

stop shipping doc/img and doc/website in the tarball

pick a more accurate release date

Disable DirPort when BridgeRelay is set

Have authorities reject routers running verions susceptible to bug 1038.

bump to 0.2.2.19-alpha

prepare for our emergency openssl fix release

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

Merge branch 'fix2204' into maint-0.2.1

Do not set the hostname TLS extension server-side; only client-side

This may fix bug 2204, and resolve the incompatibility with openssl
0.9.8p/1.0.0b.

Better fix for 2190: defer libevent->controller messages instead of dropping

Do not send Libevent log messages to a controller (0.2.1 backport)

Doing so could make Libevent call Libevent from inside a Libevent
logging call, which is a recipe for reentrant confusion and
hard-to-debug crashes.  This would especially hurt if Libevent
debug-level logging is enabled AND the user has a controller
watching for low-severity log messages.

Fix bug 2190; fix on 0.1.0.2-rc.

Do not send Libevent log messages to a controller.

Doing so could make Libevent call Libevent from inside a Libevent
logging call, which is a recipe for reentrant confusion and
hard-to-debug crashes.  This would especially hurt if Libevent
debug-level logging is enabled AND the user has a controller
watching for low-severity log messages.

Fix bug 2190; fix on 0.1.0.2-rc.

Fix a unit test broken by fix for 2195

Merge branch 'fix2183', remote branch 'rransom/fix2195-v2' into maint-0.2.2

Add changes file for bug2195.

Fix logic error in router_dump_router_to_string.

Spotted by Nick Mathewson.

Add comments to some of the bug2183 fix code

Generate a router descriptor even if generating an extra-info descriptor fails.

Fixes bug #2195.

Do not emit an extra-info-digest descriptor line if the digest is zero.

Tweak the bugfix for 2183 a bit more.

new development version

final touchups

bump to 0.2.2.18-alpha

warn more about AllowSingleHopExits

merge in more changelog entries