Merge remote-tracking branch 'tor-github/pr/1256' into maint-0.3.5

Merge branch 'maint-0.2.9' into maint-0.3.5

`git merge --strategy ours` to take the 0.3.5 changes for 31001,
and ignore the conflicting 0.2.9 changes.

Merge remote-tracking branch 'tor-github/pr/1178' into maint-0.2.9

Merge remote-tracking branch 'tor-github/pr/1376' into maint-0.3.5

Merge remote-tracking branch 'origin/maint-0.3.5' into maint-0.3.5

Travis: Disable the Rust macOS build

This build was added in 0.3.5.

We need to mitigate slow scheduling of Travis macOS jobs.

Closes ticket 32177.

Merge branch 'ticket32177_029' into ticket32177_035

Stem from maint-0.3.5, disabled macOS chutney from ticket32177_029.

Travis: Disable all but one macOS build

We need to mitigate slow scheduling of Travis macOS jobs.

Closes ticket 32177.

Appveyor: Use Windows Server 2019

Instead of Windows Server 2016.

Closes ticket 32086.

Merge branch 'maint-0.2.9' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1370' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1369' into maint-0.2.9

Merge branch 'maint-0.2.9' into maint-0.3.5

Update geoip and geoip6 to the October 1 2019 database.

Travis: Keep the macOS Rust job, but don't wait for it to finish

Since Rust on macOS is slow, don't wait for the macOS Rust job to finish.
Instead, split rust into slow rust (macOS) and fast rust (Linux). And
allow the build to finish before slow rust finishes.

Also make sure that we have:
* a Rust build on each platform,
* a Rust build with each compiler, and
* a check on all our Rust builds.

Finally, sort builds: allow fail last, macOS first, slowest first.

Closes 31859 for 0.3.5.

Travis: Split jobs, add essential jobs, remove redundant jobs

Part of 31859 for 0.3.5.

Merge branch 'bug31859_029' into bug31859_035

While merging:
* leave out some redundant jobs and build matrix entries

Travis: Add a macOS chutney job, but don't wait for it to finish

Since Travis macOS has IPv6 support (and Travis Linux does not), chutney
will now run its IPv6 networks as part of Travis CI.

But since chutney is slow, don't wait for the macOS chutney to finish.
(Travis have fixed the duplicate notification bug in fast_finish. So we
can use fast_finish and allow_failure to finish early. Unfortunately,
allow_failure also means we ignore failures in macOS chutney.)

Also make sure that we have:
* a compile on each platform, with each compiler,
* a check on each platform, and
* a check on each compiler.

Finally, sort builds: allow fail last, macOS first, slowest first.

Closes ticket 30860.
Closes ticket 31859 for 0.2.9.

Travis: Remove a redundant clang Linux job

Part of 31859.

Appveyor: Avoid spurious errors in Appveyor CI builds

When Appveyor fails before the install step, some of the finish step's
functions were not defined.

Fixes bug 31884; bugfix on 0.3.4.2-alpha.

Travis: Unroll the build matrix into matrix: include:

The jobs should be the same, but Travis may display them differently.

Part of 31859.

Merge remote-tracking branch 'tor-github/pr/1300' into maint-0.3.5

Merge branch 'maint-0.2.9' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1216' into maint-0.2.9

Merge remote-tracking branch 'tor-github/pr/1307' into maint-0.3.5

Merge branch 'maint-0.2.9' into maint-0.3.5

"ours" merge to avoid picking up 0.2.9 version of ticket 31673.

pf: when extracting an IPv6 address, make sure we got an IPv6 address

Our code assumes that when we're configured to get IPv6 addresses
out of a TRANS_PF transparent proxy connection, we actually will.
But we didn't check that, and so FreeBSD started warning us about a
potential NULL pointer dereference.

Fixes part of bug 31687; bugfix on 0.2.3.4-alpha when this code was
added.

fp.c: Suppress float-conversion warnings on FreeBSD.

We used to do this on Windows only, but it appears to affect
multiple platforms when building with certain versions of GCC, and a
common pattern for defining the floating-point classifier functions.

Fixes part of 31687. I'm calling this a bugfux on 31687, when we
started suppressing these warnings on Windows.

build: The <sys/sysctl.h> is now deprecated on Linux

Closes #31673

build: The <sys/sysctl.h> is now deprecated on Linux

Closes #31673

entrynodes: Make routine descriptor expiry notice logs less alarming

When tor is missing descriptors for some primary entry guards, make the
log message less alarming. It's normal for descriptors to expire, as long
as tor fetches new ones soon after.

Fixes bug 31657; bugfix on 0.3.3.1-alpha.

Merge remote-tracking branch 'tor-github/pr/1271' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1181' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1117' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1114' into maint-0.3.5

test: Change "make test-stem" so it only runs the stem tests that use tor

This change makes test-stem faster and more reliable.

Use "make test-stem-full" to run all of stem's tests.

Closes ticket 31554.

changes: file for 23507, v3 onion service fix

hs: v3 single onion services fall back to 3-hop intro for unreachable nodes

Previously, v3 single onion services failed when all intro nodes were
unreachable via a 1-hop path. Now, we select intros that are only available
via a 3-hop path, and use a 3-hop path to connect to them.

Fixes bug 23507; bugfix on 0.3.2.1-alpha.

changes: file for 23818, v3 onion service fix

hs: Always use a 3-hop path when a v3 single onion intro fails

Previously, we always used a 1-hop path, no matter how many times a v3
single onion intro failed.

Fixes bug 23818; bugfix on 0.3.2.1-alpha.

hs: Always use a 3-hop path when a v3 single onion rend fails

Previously, we used a 1-hop path when a single onion rend failed
immediately, and a 3-hop path when it failed after trying to build
a circuit.

Fixes bug 23818; bugfix on 0.3.2.1-alpha.

Merge branch 'bug23818_029' into bug23818_035

changes: file for 23818, v2 onion service fix

rendservice: Always use a 3-hop path when a v2 single onion intro fails

Previously, we always used a 1-hop path, no matter how many times a v2
single onion intro failed.

Fixes bug 23818; bugfix on 0.2.9.3-alpha.

rendservice: Always use a 3-hop path when a v2 single onion rend fails

Previously, we used a 1-hop path when a single onion rend failed
immediately, and a 3-hop path when it failed after trying to build
a circuit.

Fixes bug 23818; bugfix on 0.2.9.3-alpha.

Fix gcc build exclude on travis

The main build doesn't use HARDENING_OPTIONS, so the exclude shouldn't,
either. Reduces the number of builders by one.

Merge branch 'maint-0.2.9' into maint-0.3.5

dirauth: Change dizum IP address

New IP address from 194.109.206.212 to 45.66.33.45.

Signed request from Alex de Joode, operator of dizum:
https://trac.torproject.org/projects/tor/ticket/31406

Published descriptor by dizum on August 12th, 2019:

--
r dizum fqbq1v2DCDxTj0QDi7+gd1h911U GZmZtCLaPDQNxkhIFj8UcgTRAuA 2019-08-12 15:28:40 45.66.33.45 443 80
s Authority Fast Running Stable V2Dir Valid
v Tor 0.4.0.5
pr Cons=1-2 Desc=1-2 DirCache=1-2 HSDir=1-2 HSIntro=3-4 HSRend=1-2 Link=1-5 LinkAuth=1,3 Microdesc=1-2 Relay=1-2 Padding=1
w Bandwidth=20 Unmeasured=1
p reject 1-65535
--

Finally, confirmed by DNS:

  $ dig +short tor.dizum.com
  45.66.33.45

Closes #31406

Signed-off-by: David Goulet <dgoulet@torproject.org>

Merge remote-tracking branch 'tor-github/pr/1077' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1066' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1065' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1056' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1039' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1034' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1020' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/987' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/952' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/951' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/937' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/926' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/911' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/848' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/741' into maint-0.3.5

Merge branch 'maint-0.2.9' into maint-0.3.5

Delete src/or/buffers.c which was fixup'ed in 0.2.9 but moved in 0.3.5.
The 0.2.9 fix is not required for the 0.3.5 patch.
Resolves merge conflict caused by 30041.

Merge remote-tracking branch 'tor-github/pr/1078' into maint-0.2.9

Merge remote-tracking branch 'tor-github/pr/1052' into maint-0.2.9

Merge remote-tracking branch 'tor-github/pr/1229' into maint-0.2.9

Modify "Protect buffers against INT_MAX datalen overflows." for 0.2.9

Merge remote-tracking branch 'tor-github/pr/1080' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1054' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/920' into maint-0.3.5

Fix #28525 changes file that is breaking CI.

Fix #28525 changes file that is breaking CI.

Merge remote-tracking branch 'tor-github/pr/763' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/762' into maint-0.2.9

Merge remote-tracking branch 'tor-github/pr/958' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/957' into maint-0.2.9

Merge branch 'bug31343_31374_029' into bug31343_31374_035

Merge remote-tracking branch 'tor-github/pr/1222' into combined31343_31374_035

Merge remote-tracking branch 'tor-github/pr/1210' into combined31343_31374_035

Merge remote-tracking branch 'tor-github/pr/1221' into combined31343_31374_029

Merge remote-tracking branch 'tor-github/pr/1209' into combined31343_31374_029

Merge branch 'ticket31374_029' into ticket31374_035

Fix a warning about casting the results of GetProcAddress.

Fixes bug 31374; bugfix on 0.2.9.1-alpha.

Merge branch 'ticket31343_029' into ticket31343_035

Fix another time_t/long warning for 31343.

Merge branch 'ticket31343_029' into ticket31343_035

Restore proper behavior of netinfo skew check

My previous fix removed a comparison, which would have caused us to
warn about every skew instead of skews of over an hour.

Hardcode the image into tor-exit-notice.html

Merge remote-tracking branch 'tor-github/pr/1044' into maint-0.3.5

Merge changed chutney and stem script lines from 29280 with changed
stem script lines from 30591.

Merge branch 'maint-0.2.9' into maint-0.3.5

Merge rust additions in 0.3.5 with sudo deletions in 0.2.9.

Merge remote-tracking branch 'tor-github/pr/1156' into maint-0.3.5

Merge remote-tracking branch 'tor-github/pr/1155' into maint-0.2.9

Merge remote-tracking branch 'tor-github/pr/991' into maint-0.2.9

Merge branch 'ticket31343_029' into ticket31343_035

Avoid using labs() on time_t in channeltls.c

On some windows builds, time_t is 64 bits but long is not.  This is
causing appveyor builds to fail.

Also, one of our uses of labs() on time_t was logically incorrect:
it was telling us to accept NETINFO cells up to three minutes
_before_ the message they were responding to, which doesn't make
sense.

This patch adds a time_abs() function that we should eventually move
to intmath.h or something.  For now, though, it will make merges
easier to have it file-local in channeltls.c.

Fixes bug 31343; bugfix on 0.2.4.4-alpha.

Set 'routerlist' global to NULL before freeing it.

There is other code that uses this value, and some of it is
apparently reachable from inside router_dir_info_changed(), which
routerlist_free() apparently calls.  (ouch!)  This is a minimal fix
to try to resolve the issue without causing other problems.

Fixes bug 31003. I'm calling this a bugfix on 0.1.2.2-alpha, where
the call to router_dir_info_changed() was added to routerlist_free().

Changes file for bug 31001

Prevent UB on signed overflow.

Overflowing a signed integer in C is an undefined behaviour.
It is possible to trigger this undefined behaviour in tor_asprintf on
Windows or systems lacking vasprintf.

On these systems, eiter _vscprintf or vsnprintf is called to retrieve
the required amount of bytes to hold the string. These functions can
return INT_MAX. The easiest way to recreate this is the use of a
specially crafted configuration file, e.g. containing the line:

FirewallPorts AAAAA<in total 2147483610 As>

This line triggers the needed tor_asprintf call which eventually
leads to an INT_MAX return value from _vscprintf or vsnprintf.

The needed byte for \0 is added to the result, triggering the
overflow and therefore the undefined behaviour.

Casting the value to size_t before addition fixes the behaviour.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

Merge branch 'maint-0.2.9' into maint-0.3.5