Clarify some documentation and comments wrt resetting OR token buckets

Remove workaround code for bug539

We fixed bug 539 (where directories would say "503" but send data
anyway) back in 0.2.0.16-alpha/0.1.2.19.  Because most directory
versions were affected, we added workaround to make sure that we
examined the contents of 503-replies to make sure there wasn't any
data for them to find.  But now that such routers are nonexistent,
we can remove this code.  (Even if somebody fired up an 0.1.2.19
directory cache today, it would still be fine to ignore data in its
erroneous 503 replies.)

Fix some 'impossible' overflow bugs in byte counting

The first was genuinely impossible, I think: it could only happen
when the amount we read differed from the amount we wanted to read
by more than INT_MAX.

The second is just very unlikely: it would give incorrect results to
the controller if you somehow wrote or read more than 4GB on one
edge conn in one second.  That one is a bugfix on 0.1.2.8-beta.

Look at the right errno when sending reason for connect() failure

In afe414 (tor-0.1.0.1-rc~173), when we moved to
connection_edge_end_errno(), we used it in handling errors from
connection_connect().  That's not so good, since by the time
connection_connect() returns, the socket is no longer set, and we're
supposed to be looking at the socket_errno return value from
connection_connect() instead.  So do what we should've done, and
look at the socket_errno value that we get from connection_connect().

Triage the XXX022 and XXX021 comments remaining in the code

Remove some, postpone others, leave some alone.  Now the only
remaining XXX022s are ones that seem important to fix or investigate.

Comment out ancient asserts for bug 930; resolve an xxx021

Fixup tor_addr_to_sockaddr return convention

Clean up a comment-conversation about bad libevent version/method combos

Remove the "fuzzy time" code

It was the start of a neat idea, but it only got used in 3 places,
none of which really needed it.

Fix signed/unsigned compare warning

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

Add 2696 changes file

Remove superfluous -g -O2 compiler argument

Autoconf adds -g -O2 by default, so adding it ourselves is not required.
It also caused a warning with clang for every source file, so remove it
here. Fixes last issue of ticket 2696.

Futz with the clang patch a bit and tidy some geoip.c stuff

Document a few more members.

Resolve the one DOCDOC in the 0.2.2 code atm

Remove a useless doxygen comment

Move the decl for tor_gettimofday_cache_clear to the right header

add changes file for the doxygen issues

Clean up whitespace

Doxygen documentation for about 100 things that didn't have any

About 860 doxygen-less things remain in 0.2.2

Fix up all doxygen warnings other than "foo is not documented"

Turn on the "source browser" feature for our generated HTML.

Run "doxygen -u" to update doxygen config file.

Merge branch 'bug2756_relay' into maint-0.2.2

Fix comment for connection_edge_consider_sending_sendme

Merge branch 'bug2757' into maint-0.2.2

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

Fix a compile warning when using clang

Issue noticed by Steven Murdoch; fixes bug 2689. The cast didn't do
anything, and we don't need to look at the return value of the function
here.

Fix a log message typo

Log the source of a rejected POSTed v3 vote

Related to ticket 2683.

Check newconn, not conn, before marking newconn for close

This fixes bug 2757.  It's a bugfix on 85da676108f0d.

Consider sending stream-level SENDME cells on partial flushes.

Right now, we only consider sending stream-level SENDME cells when we
have completely flushed a connection_edge's outbuf, or when it sends
us a DATA cell.  Neither of these is ideal for throughput.

This patch changes the behavior so we now call
connection_edge_consider_sending_sendme when we flush _some_ data from
an edge outbuf.

Fix for bug 2756; bugfix on svn r152.

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

Merge remote branch 'arma/bug2510' into maint-0.2.2

Merge branch 'bug2511' into bug2510

Merge remote branch 'arma/bug2716' into maint-0.2.2

fix two issues pointed out by nickm

we're not reachable if we don't have a routerinfo yet

fix one more typo

Tweak bug2716 patch a little

Name the magic value "10" rather than re-deriving it.

Comment more.

Use the pattern that works for periodic timers, not the pattern that
doesn't work. ;)

Merge branch 'hsdir_assignment' into maint-0.2.2

Clarify threat description and avoid negative uptimes.

improve accuracy for when a relay went unreachable

Merge commit 'sebastian/bug2663' into maint-0.2.2

Merge branch 'maint-0.2.1' into maint-0.2.2

put the 0.2.1.30 changes into releasenotes

Use observed instead of declared uptime for HSDir

It is important to verify the uptime claim of a relay instead of just
trusting it, otherwise it becomes too easy to blackhole a specific
hidden service. rephist already has data available that we can use here.

Bugfix on 0.2.0.10-alpha.

Add a step to the release process: tell trac about the version.

Fix compilation under LLVM/clang with --enable-gcc-warnings

- When compiling using clang (2.9 or lower) do not enable
  -Wnormalized=id or -Woverride-init when --enable-gcc-warnings
  or --enable-gcc-warnings-advisory is set as these options
  are unsupported.

Fix two compile warnings when using clang

Issue found by Steven Murdoch

oops. I got the bug number wrong.

openSUSE initscript create /var/run/tor fixes Ticket #2573

(changes file added by nickm)

Merge branch 'maint-0.2.1' into maint-0.2.2

move to march 2011 geoip file

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

steps roger takes when making a new release

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

update spec locations

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

Avoid crash in any_pending_bridge_descriptor_fetches

This is based on shitlei's fix for bug2629, with extra parens removed.
Fixes bug 2629, bugfix on 0.2.1.2-alpha.

make nickm's proposed convention from 2003 be gospel

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

Fix log message when we have too few dirauths

The calculation of when to send the logmessage was correct, but we
didn't give the correct number of relays required: We want more than
half of all authorities we know about. Fixes bug 2663.

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

Merge branch 'real_ipv6_fix' into maint-0.2.1

Oops, here's the *REAL* fix for the ipv6 issue

We need to _REJECT_ descriptors with accept6/reject6 lines.  If we
let them onto the network , other un-upgraded tors will crash.

Merge branch 'careful_with_all_descs' into maint-0.2.1

changes file for careful_with_all_descs

Revert "Disallow reject6 and accept6 lines in descriptors"

This reverts commit b3918b3bbbfa9097246d63746c8b540eff2ec8e8.

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

Conflicts:
src/or/policies.c

exit_policy_is_general_exit is IPv4 only; it should admit it.

Changes file for ipv6 parsing issue

Merge branch 'maint-0.2.1' into maint-0.2.2

Disallow reject6 and accept6 lines in descriptors

This fixes a remotely triggerable assert on directory authorities, who
don't handle descriptors with ipv6 contents well yet. We will want to
revert this once we're ready to handle ipv6.

Issue raised by lorth on #tor, who wasn't able to use Tor anymore.
Analyzed with help from Christian Fromme. Fix suggested by arma. Bugfix
on 0.2.1.3-alpha.

Fix setting target port in get_interface_address6

We want to use the discard port correctly, so a htons() was missing.
Also we need to set it correctly depending on address family.

Review provided by danieldg

Fix connect() failures in get_interface_address6()

The third argument for connect should be dependent on the address
family. Issue spotted by piebeer who also wrote the patch.

Do not serve encrypt-only descriptors with the "all" request. Reported by piebeer

Merge remote branch 'rransom/typo-fix-2011-03-03-01' into maint-0.2.2

Fix typo in man page

Avoid spurious bwhist parsing failures

This should fix a bug that special ran into, where if your state file
didn't record period maxima, it would never decide that it had
successfully parsed itself unless you got lucky with your
uninitialized-variable values.

This patch also tries to improve error messags in the case where a
maximum value legitimately doesn't parse.

Tweak the bug2250 fix slightly

Rename the _UsingTestingTorNetwork hidden option to
_UsingTestNetworkDefaults (thanks Nick for the suggestion) and added a
changes file.

clarify an assert

also log about running changes, even on a bridge authority.

Fix GETINFO config-text for private networks

In private networks, the defaults for some options are changed. This
means that in options_validate(), where we're testing that the defaults
are what we think they are, we fail. Use a workaround by setting a
hidden configuration option _UsingTestingTorNetwork when we have altered
the configuration this way, so that options_validate() can do the right
thing.

Fixes bug 2250, bugfix on 0.2.1.2-alpha (the version introducing private
network options).

Merge branch 'bug1863_bwhist' into maint-0.2.2

Fix two potential bugs in the bug1863 code.

Merge remote branch 'arma/bug2403' into maint-0.2.2

Add some dollar signs in an attempt to appease older asciidocs

We should really require a modern asiidoc: backporting stuff to 8.2 is
a timesink.

prevent same entry and exit for insane edge case

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

Conflicts:
configure.in

Remove doc/spec/Makefile.in from list of generated files

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

(This is an "ours" merge of the code for bug1859 backported to 0.2.1.)

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

changes file for removing torspec from the tarball

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

Use "ours" strategy" to avoid taking bug2402 fix.

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

changes file for removing torspec from the tarball

Merge branch 'maint-0.2.1' into maint-0.2.2

prefer https urls

Don't let bad DNS make exit policy and declared exit policy get out of sync

Patch from "postman" on trac. Fixes bg 2366. Bug on 0.1.2.5-alpha.