Merge remote-tracking branch 'asn/bug5646'

Extract data from DESTROY cell _after_ protocol violation checks.

Ignore unknown lines from managed proxies.

Another dirport/orport correction; this one from "cypherpunks"

Document that the ORPort flags apply to DirPort too. (thanks, Roger.)

5597: document new ORPort options

Clarify SessionGroup documentation. Bug 5437.

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

Make the succeeding parse_http_time tests more obviously right

(When the correct answer is given in terms of seconds since the
epoch, it's hard to be sure that it really is the right answer
just by reading the code.)

Merge branch 'bug5346_squashed' into maint-0.2.2

Fix month check in parse_http_time, add test

Remove more dubiosity in struct tm handling. related to bug5346

changes file for branch bug5346

Reject an additional type of bad date in parse_http_time

Fix parse_http_time and add tests

* It seems parse_http_time wasn't parsing correctly any date with commas (RFCs
  1123 and 850). Fix that.
* It seems parse_http_time was reporting the wrong month (they start at 0, not
  1). Fix that.
* Add some tests for parse_http_time, covering all three formats.

Merge remote-tracking branch 'public/bug5139'

Add changes file for bug 4108

In connection_ap_handshake_process_socks(), mark the socks request as finished if a reply is send after a parse error

Silences the log message:
[warn] {BUG} _connection_mark_unattached_ap(): Bug: stream (marked at connection_edge.c:2224) sending two socks replies?
after the client triggered the "Tor is not an HTTP Proxy" response.

No additional socks reply was sent, though.

Merge remote-tracking branch 'public/bug3964'

Merge branch 'bug5380'

Remove over-two-months-old entry guards even while running.

Previously, we only did this check at startup, which could lead to
us holding a guard indefinitely, and give weird results.  Fixes bug
5380; bugfix on 0.2.1.14-rc.

(Patch by Roger; changes file and commit message by Nick)

Correct documentation for remove_obsolete_entry_guards.

Merge remote-tracking branch 'public/bug2297'

Merge remote-tracking branch 'public/bug2822'

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

Correct the bulletproofing of routerlist_insert()

The original code updated some variables, but forgot to remove a
replaced old-routerdesc from rl->old_routers.

Related to bug 1776.

Merge remote-tracking branch 'public/bug3296'

Merge branch 'bug5095_squashed'

When ReloadTorrcOnSIGHUP=1, do non-reload activities anyway

Previously, we skipped everything that got invoked from
options_init_from_torrc.  But some of the stuff in
options_act_reversible and options_act is actually important, like
reopening the logs.

Now, a SIGHUP always makes the effects of an options_set() happen,
even though the options haven't changed.

Fix for bug 5095; bugfix on 0.2.1.9-alpha, which introduced
__ReloadTorrcOnSIGHUP.

Fix some remaining nmake/msvc build issues

Merge branch 'win32_winnt'

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

Clarify MaxCircuitDirtiness behavior with hidden services. Bug 5259.

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

Merge branch 'bug5796_022_squashed' into maint-0.2.2

Fix a crash bug on SETCIRCUITPURPOSE.

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

Merge remote-tracking branch 'karsten/geoip-may2012' into maint-0.2.2

Merge remote-tracking branch 'linus/task-5891'

Clarify changes file regarding which bugs get fixed.

Rename changes file.

Assert that rep_hist_format_desc_stats() returns !NULL.

The guard against this is the test for
start_of_served_descs_stats_interval != 0 done earlier.

Fix desc stats on bridge authorities that didn't serve anything.

Bump _WIN32_WINNT to 0x0501 throughout the code

This tells the windows headers to give us definitions that didn't
exist before XP -- like the ones that we need for IPv6 support.

See bug #5861.  We didn't run into this issue with mingw, since
mingw doesn't respect _WIN32_WINNT as well as it should for some of
its definitions.

Remove all instances of WIN32_WINNT (without leading _)

We started adding it in 59e2c77824840f back in 2004, 8 years and 3
days ago.  It's time to deprogram ourselves from this cargo cult.

Add a changes file for the misc bug5859 build issues

MSVC build issue: it can't tell that tor_assert(0) aborts.

MSVC build issue: make 'const' in declaration match 'const' in definition

MSVC warns if you declare a function as having a "int foo" argument
and then implement it with a "const int foo" argument, even though
the latter "const" is not a part of the function's interface.

MSVC build issue: we use INLINE as the one that will magically work

MSVC build issue: add magic to make openssl headers in aes.c work

Be a good git person: store nmakefiles in correct text fmt

Remove the unused torrc.bridge.in. Bug 5622.

Add some MSVC stuff to gitignore

We do not need to define _WIN32 by hand; MSVC does that for us

Bug 5858; fix on 0.2.3.12-alpha

Fix win32 compilation of 31eb73f88e8

Do not publish the "git-XXX" tag in server descriptors

Instead, allow packagers to put a 'TOR_BUILD_TAG' field in the
server descriptor to indicate a platform-specific value, if they
need to.  (According to weasel, this was his use for the git- tag
previously.)

This is part of 2988

Cut down on the OS information we give.

For uname-based detection, we now give only the OS name (e.g.,
"Darwin", "Linux".)  For Windows, we give only the Operating System
name as inferred from dw(Major|Minor)version, (e.g., "Windows XP",
"Windows 7"), and whether the VER_NT_SERVER flag is set.

For ticket 2988.

Fix a couple of wide lines

When no usable exit satisfies a predicted port, stop predicting it.

Fix for bug 3296.

Merge remote-tracking branch 'asn/bug4865_take2'

Merge remote-tracking branch 'public/bug5091'

Merge branch 'maint-0.2.2'

fix over-wide line from f661747370

Fix O(n^2) performance when parsing a big pile of extrainfos

We were doing an O(n) strlen in router_get_extrainfo_hash() for
every one we tried to parse.  Instead, have
router_get_extrainfo_hash() take the length of the extrainfo as an
argument, so that when it's called from
extrainfo_parse_from_string(), it doesn't do a strlen() over the
whole pile of extrainfos.

Merge remote-tracking branch 'public/bug4591'

Merge branch 'bug5786'

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

Conflicts:
src/common/util.c
src/test/test_util.c

Merge branch 'bug5786_range_022' into maint-0.2.2

Merge remote-tracking branch 'asn/bug5540'

Merge remote-tracking branch 'linus/bug5146'

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

Add changes/bug5760

Add missing CRLFs to AUTHCHALLENGE failure replies
Fix #5760

Fix a segfault in pt/protocol test

Now that the pt code logs mp->argv[0] all over the place, we need to
be sure to set up mp->argv in our tests.

Bugfix on e603692adcd, not in any released version.

Detect out-of-bounds bwweightscale values early in the voting process

If the authorities agreed on a sufficiently bad bwweightscale value
(<=0 or == INT32_MAX), the bandwidth algorithm could make the voters
assert while computing the consensus.

Fix for bug5786; bugfix on 0.2.2.17-alpha

Check more thoroughly for dups when parsing networkstatus parameters

See changes file for details.

Partial fix for bug 5786; fix on 0.2.2.2-alpha.

Handle out-of-range values in tor_parse_* integer functions

The underlying strtoX functions handle overflow by saturating and
setting errno to ERANGE.  If the min/max arguments to the
tor_parse_* functions are equal to the minimum/maximum of the
underlying type, then with the old approach, we wouldn't treat a
too-large value as genuinely broken.

Found this while looking at bug 5786; bugfix on 19da1f36 (in Tor
0.0.9), which introduced these functions.

Apply a patch from Gisle Vanem to make tor-gencert build under MSVC

(Note: It makes sense to use tor-gencert on Windows for testing
purposes only.  If you are a directory authority operator, and you
are contemplating running tor-gencert on a Windows box in an actual
production environment, you are probably making a mistake.)

Merge branch 'bug5645_take2'

Make a cast less const-violating; make a field size explicit.

Reorder rend_mid_rendezvous() to do protocol violation checks on top.

Merge branch 'bug5070_take2'

Using %d to printf an enum may not be by-the-standard okay.

Fix an overwide line

Fix comments: There is no such thing as a NUL pointer

Changes file for bug 5070

Document some transports.c behaviors and assumptions

Update to the May 2012 GeoIP database.

fix quad typo in comments

i assume if nickm maintained "libeven" this would never have been
introduced. :)

Fix bug 5762: detect missing accept4 that gives ENOSYS

We had been checking for EINVAL, but that means that SOCK_* isn't
supported, not that the syscall itself is missing.

Bugfix on 0.2.3.1-alpha, which started to use accept4.

Ignore [::] when building descriptors.

This is how IPv6 says "0.0.0.0" and something we will have to
translate into a globally reachable address before putting it in a
descriptor.

The fix is a short term solution until a real one is implemented.

Closes #5146.

Make transports.c logs a bit more helpful.

Add a missing ntohl to tell_controller_about_resolve_result

Fix for bug 5723; bugfix on 0.2.3.1-alpha (commit 22f723e4)

bump to 0.2.3.15-alpha-dev

bump to 0.2.3.15-alpha

fold in new changes entries

Remove __ from HAVE_EXTERN_ENVIRON_DECLARED__

I think that the trailing __ got added in false analogy to
HAVE_MACRO__func__, HAVE_MACRO__FUNC__, and HAVE_MACRO__FUNCTION__.
But those macros actually indicate the presence of __func__,
__FUNC__, and __FUNCTION__ respectively.  The __ at the end of
HAVE_EXTERN_ENVIRON_DECLARED would only be appropriate if the
environ were declared__, whatever that means.

(As a side-note, HAVE_MACRO__func__ and so on should probably be
renamed HAVE_MACRO___func__ and so on.  But that can wait.)

This is an identifier renaming only.

Fix headers in test for whether environ is declared in stdlib/unistd

We'd had our configure.in test include unistd.h unconditionally,
which would fail on Windows/mingw, even though environ _was_
declared there.  Fix for 5704; bugfix on 0.2.3.13-alpha.

Thanks to Erinn for finding this and rransom for figuring out the
problem.

Only disable cert chaining on the first TLS handshake

If the client uses a v2 cipherlist on the renegotiation handshake,
it looks as if they could fail to get a good cert chain from the
server, since they server would re-disable certificate chaining.

This patch makes it so the code that make the server side of the
first v2 handshake special can get called only once.

Fix for 4591; bugfix on 0.2.0.20-rc.

Bridges should never set the send_unencrypted flag on any of their descs

Fix for bug 5139.