Merge remote-tracking branch 'public/bug8059' into maint-0.2.4

Remove an unused local variable.

Merge branch 'bug8206_squashed' into maint-0.2.4

Check return values from fcntl and setsockopt

(Based on a patch from flupzor; bug #8206)

Check for 0.2.4.8, not 0.2.4.7, to determine EXTEND2 support

Fixes bug 8464; bugfix on b2863739 in 0.2.4.8-alpha

Merge remote-tracking branch 'public/bug8180_023_v2' into maint-0.2.4

Upgrade the warn for EntryNodes without UseEntryGuards to an error

fixes bug 8180

Merge remote-tracking branch 'public/unused_stuff' into maint-0.2.4

Fix some basic socket issues with tor-fw-helper-natpmp on windows

This isn't going to be the last of these issues, but we might as
well take the fixes as we find them.

Patch from Gisle Vanem, fixes bug 7280.

Remove incentives.txt from tor git; putting it into torspec.

Fix brackets in tor-fw-helper help output.

Change nesting from [(]) to [()]. Formerly it made it look to me at
first glance that "internal port" was optional.

      [Trivial change; fixes #7767  --nickm]

Improve asciidoc-helper's suggestion of what to install

Fixes bug #7766.

Patch by David Fifield, revised by Peter Palfrader.

Be explicit that we want not only nacl but nacl-with-a-fast-curve25519

Resolves the user experience part of #8014.

Make circuit_purpose_to_string handle CIRCUIT_PURPOSE_PATH_BIAS_TESTING

Look at the right variable when warning about signed size_t.

In 81d69f4c2d8a451 (0.2.21-alpha) we added a compile-time check for
a (totally broken) signed size_t.  In 0e597471af (not yet released)
I switched to a better configure-time check, which stored its output
in a different variable.  I didn't change the code which looked at
the output, however.

This bug is not in any released version of Tor, and would not affect
anybody with a working Tor.

Merge remote-tracking branch 'origin/maint-0.2.3' into maint-0.2.4

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

Fix a framing bug when reading versions from a versions cell.

Our ++ should have been += 2.  This means that we'd accept version
numbers even when they started at an odd position.

This bug should be harmless in practice for so long as every version
number we allow begins with a 0 byte, but if we ever have a version
number starting with 1, 2, 3, or 4, there will be trouble here.

Fix for bug 8059, reported pseudonymously. Bugfix on 0.2.0.10-alpha
-- specifically, commit 6fcda529, where during development I
increased the width of a version to 16 bits without changing the
loop step.

start part-way through the ssl cert lifetime

also, snap the start time and end time to a day boundary, since most
certs in the wild seem to do this.

randomize SSLKeyLifetime by default

resolves ticket 8443.

parameterize SSLKeyLifetime

no actual changes in behavior yet

Merge remote-tracking branch 'arma/bug6783_big_hammer' into maint-0.2.4

Fix typos in a few log messages

set DisableV2DirectoryInfo_ off by default

since it's only enableable by authorities, nobody else would be able
to start their tor

Tweak bug6783 patch.

Add a DisableV2DirectoryInfo_ option to 404 all v2 ns requests

I have no idea whether b0rken clients will DoS the network if the v2
authorities all turn this on or not.  It's experimental. See #6783 for
a description of how to test it more or less safely, and please be
careful!

Merge branch 'bug8447_squashed' into maint-0.2.4

Fix 8447: use %u to format circid_t.

Now that circid_t is 4 bytes long, the default integer promotions will
leave it alone when sizeof(int) == 4, which will leave us formatting an
unsigned as an int.  That's technically undefined behavior.

Fixes bug 8447 on bfffc1f0fc7616a25c32da2eb759dade4651659e.  Bug not
in any released Tor.

Update to the March 2013 GeoIP database.

Correctly set entry->is_dir_cache when adding an entry guard for the first time

                    (Second part of a bug8367 fix. -NM)

Fix bootstrapping with bridges by making is_dir_cache is set on them.

This fixes bug 8367, introduced in d7089ff228227259137b5a8b. Not in
any released Tor.

update the in-progress bandwidth requirements for authorities

Fix a comment in tor_main.c

We don't use subversion or src/*/makefile.am any longer

Remove some functions which were unused except for their tests

Remove some totally unused functions

Remove a bunch of unused macro definitions

fix a wide line

Merge branch 'bug8065_v2'

Downgrade an assert to LD_BUG

This should prevent crashes on further recurrence of 8065, and help
diagnose such if they occur

Fix a may-be-used-uninitialized warning.

Merge branch 'bug2286_unit_test_squashed'

Test alternate values of maxunmeasuredbw too

Update unmeasured bandwidth clipping unit test per NickM's bugfix for the Unmeasured=1 issue

Refactor storing of measured_bw versus Unmeasured=1.

This patch moves the measured_bw field and the has_measured_bw field
into vote_routerstatus_t, since only votes have 'Measured=XX' set on
their weight line.

I also added a new bw_is_unmeasured flag to routerstatus_t to
represent the Unmeasured=1 flag on a w line.  Previously, I was using
has_measured_bw for this, which was quite incorrect: has_measured_bw
means that the measured_bw field is set, and it's probably a mistake
to have it serve double duty as meaning that 'baandwidth' represents a
measured value.

While making this change,I also found a harmless but stupid bug in
dirserv_read_measured_bandwidths: It assumes that it's getting a
smartlist of routerstatus_t, when really it's getting a smartlist of
vote_routerstatus_t.  C's struct layout rules mean that we could never
actually get an error because of that, but it's still quite incorrect.
I fixed that, and in the process needed to add two more sorting and
searching helpers.

Finally, I made the Unmeasured=1 flag get parsed.  We don't use it for
anything yet, but someday we might.

This isn't complete yet -- the new 2286 unit test doesn't build.

Add unit test for unmeasured bandwidth clipping in consensus

Refactor v3_networkstatus test to allow reuse of test for measuredbw

Note some annoyinc copy-and-paste code

Tweak consensus method 17 based on arma's comments

Instead of capping whenever a router has fewer than 3 measurements,
we cap whenever a router has fewer than 3 measurements *AND* there
are at least 3 authorities publishing measured bandwidths.

We also generate bandwidth lines with a new "Unmeasured=1" flag,
meaning that we didn't have enough observations for a node to use
measured bandwidth values in the authority's input, whether we capped
it or not.

touchups on 78582760

Document (on man page) which DNS record types Tor knows.

Fixes bug 7876

Revise some comments in router.c

Avoid the phrase "legally valid" (as opposed to valid but not legal)?
And document what functions really do.

Fix for bug 6935.

Fix a bug that roger found in the wide_circ_id code

Merge remote-tracking branch 'public/bug8209'

Merge remote-tracking branch 'public/bug8207'

Merge remote-tracking branch 'public/wide_circ_ids'

Conflicts:
src/or/channel.h
src/or/connection_or.c
src/or/cpuworker.c

make check-spaces

Merge remote-tracking branch 'public/feature4994-rebased'

Fix unreachable use-before-assign in test_util_join_win_cmdline

Apparently there is a compiler that believes this is something to
warn about.

Merge remote-tracking branch 'public/integers_donna'

Authorities were adding downtime for every relay every restart

Stop marking every relay as having been down for one hour every
time we restart a directory authority. These artificial downtimes
were messing with our Stable and Guard flag calculations.

Fixes bug 8218 (introduced by the fix for 1035). Bugfix on 0.2.2.23-alpha.

Use torint.h in curve25519-donna*.c

This will get it building on systems that lack stdint.h

Merge remote-tracking branch 'arma/bug1992'

Restore the entry/dirguard distinction.

We shouldn't be calling choose_random_entry() for directory
conncetions; that's what choose_random_dirguard() is for.

tor.1: Specify what Auto means for RefuseUnknownExits in more detail

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

oops; add a missing semicolon

(Cherry-picked from fc35ee4910326dc1ae718482b30e57666a71df85)

get rid of the new caching notion in resolve_my_address()

and replace it with the good old-fashioned two functions approach

simplify timing checks

now that both timers are on the same schedule, there's no point
tracking separate timers.

Check for IP address change every minute, not 15 minutes

Relays used to check every 10 to 60 seconds, as an accidental side effect
of calling directory_fetches_from_authorities() when considering doing
a directory fetch. The fix for bug 1992 removes that side effect. At the
same time, bridge relays never had the side effect, leading to confused
bridge operators who tried crazy tricks to get their bridges to notice
IP address changes (see ticket 1913).

The new behavior is to reinstate an every-60-seconds check for both
public relays and bridge relays, now that the side effect is gone.

Stop trying to resolve our hostname so often

For example, we were doing a resolve every time we think about doing a
directory fetch. Now we reuse the cached answer in some cases.

Fixes bugs 1992 (bugfix on 0.2.0.20-rc) and 2410 (bugfix on
0.1.2.2-alpha).

Don't tor_close_socket(-1) in tor-resolve.

Bugfix on 96b1bd4fb8e64.  Not in any released Tor.

Teach resolve_my_address() to return a cached answer

I didn't make any of the callers use this feature yet.

Fix two more coverity-spotted leaks in master.

One is a probably-impossible leak if we fail to sign a consensus;
another occurs when we can't look up the user we're trying to chown
our sockets to.

Avoid null-pointer deref in pathbias_is_new_circ_attempt

Coverity is worried about this (CID 980653).  It hasn't happened in
testing, but we might as well make sure it can't happen.

Fix a nigh-impossible overflow in cpuworker.c

When we compute the estimated microseconds we need to handle our
pending onionskins, we could (in principle) overflow a uint32_t if
we ever had 4 million pending onionskins before we had any data
about how onionskins take.  Nevertheless, let's compute it properly.

Fixes bug 8210; bugfix on 0.2.4.10. Found by coverity; this is CID
980651.

Fix a null-deref-on-fail in unit tests

If geoip_format_bridge_stats() returned NULL when it should have
returned a string, we would have tried to deref NULL, and died.  Not
a big deal in the unit tests, but still worth fixing.

Found by coverity; This is CID 743384.

Make _SC_OPEN_MAX actually get used when closing fds before exec.

Fixes bug 8209; bugfix on 0.2.3.1-alpha.

oops; add a missing semicolon

Merge remote-tracking branch 'public/bug7816_023'

Fix another memory leak

This one occurs when changing configuration options. Found by
coverity.

Merge remote-tracking branch 'public/bug8208'

Check whether ei is non-NULL before altering it.

This fixes a crash bug if we fail to generate an extrainfo
descriptor.

Fixes bug 8208; bugfix on 0.2.3.16-alpha.

Suppress a coverity false positive in connection_edge_package_raw_inbuf

Coverity is worried that we're checking entry_conn in some cases,
but not in the case where we set entry_conn->pending_optimistic_data.

This commit should calm it down (CID 718623).

Make hidden service authorization work again.

The refactoring in commit 471ab340325 wasn't complete enough: we
were checking the auth_len variable, but never actually setting it,
so it would never seem that authentication had been provided.

This commit also removes a bunch of unused variables from
rend_service_introduce, whose unusedness we hadn't noticed because
we were wiping them at the end of the function.

Fix for bug 8207; bugfix on 0.2.4.1-alpha.

Fix even more dead code and resource leaks in the unit tests

Found by coverity

Fix a very short-lived socket leak in tor-resolve

This shouldn't actually matter, since tor-resolve will return soon
after this function exits, but it's nice to be warning-free

Found by coverity, fixes CID 718633

Resolve memory leaks in the unit tests and benchmarks (found by coverity)

These shouldn't really matter, but it's nice to be leak-free.

Fix a copy-and-paste issue found by coverity

Fixes CID 980650; bugfix on 0.2.4.10-alpha.

Speed up the curve25519 unit test by doing fewer iterations

This test was accounting for about 2/3 of our unit tests' runtime,
even on systems with a fast curve25519.  No test should take so long.

Merge remote-tracking branch 'public/bug8158'

Avoid generating extra spaces when explaining how we guessed our address

Refactor resolve_my_address() so logs are more accurate / helpful

It returns the method by which we decided our public IP address
(explicitly configured, resolved from explicit hostname, guessed from
interfaces, learned by gethostname).

Now we can provide more helpful log messages when a relay guesses its IP
address incorrectly (e.g. due to unexpected lines in /etc/hosts). Resolves
ticket 2267.

While we're at it, stop sending a stray "(null)" in some cases for the
server status "EXTERNAL_ADDRESS" controller event. Resolves bug 8200.

log the hostname that resolve_my_address() used to guess our IP

Merge remote-tracking branch 'public/bug7801_v2'

Replace magic constants for wide_circ_ids with inline function calls

Wrap more macro definitions in (parentheses)

To avoid surprises, good coding practice suggests parenthesizing every
macro definition -- or at the very least, all those involving an
expression.

Improve comment at Andrea's request

Merge remote-tracking branch 'public/signof_enum'

changes file for bug7801

Unit test for tor_weak_random_range