fix issues noted by traumschule

plus fix a bonus typo

Merge branch 'maint-0.3.3' into release-0.3.3

Bump to 0.3.3.11

ChangeLog and ReleaseNotes for 0.3.3.11

Merge branch 'maint-0.2.9' into maint-0.3.3

Merge branch 'maint-0.3.3' into release-0.3.3

Update geoip and geoip6 to the January 3 2019 database.

Merge branch 'maint-0.2.9' into maint-0.3.3

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'bug21394_029_redux' into maint-0.2.9

Merge branch 'maint-0.3.3' into release-0.3.3

Merge remote-tracking branch 'dgoulet/ticket28619_033_01' into maint-0.3.3

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'ticket28973_033_squashed' into maint-0.3.3

Detect openssl bug 7712 and work around it.

In theory it would be better to detect this bug in advance, but this
approach is much simpler, and therefore safer to backport.

This closes tor issue 28973.

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

Note ticket number in changes file for 24803

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

Fallbacks: Update the hard-coded fallback list in December 2018

Merge Phoul's two lists into teor's list.

Replace the 150 fallbacks originally introduced in Tor 0.3.3.1-alpha in
January 2018 (of which ~115 were still functional), with a list of
157 fallbacks (92 new, 65 existing, 85 removed) generated in
December 2018.

Closes ticket 24803.

Fallbacks: Update the hard-coded fallback list in December 2018

Replace the 150 fallbacks originally introduced in Tor 0.3.3.1-alpha in
January 2018 (of which ~115 were still functional), with a list of
148 fallbacks (89 new, 59 existing, 91 removed) generated in
December 2018.

Closes ticket 24803.

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

Update geoip and geoip6 to the December 5 2018 database.

Merge branch 'maint-0.3.3' into release-0.3.3

hs-v3: Do not close RP circuits when deleting an ephemeral service

Bug reported on tor-dev@ and here is the detail explanation of the issue:
https://lists.torproject.org/pipermail/tor-dev/2018-November/013558.html

Fixes bug #28619

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

Test: Fix memory leaks and missing unmocks in entry guard tests

test_entry_guard_outdated_dirserver_exclusion leaks memory, and is
missing some unmocks.

Fixes 28554; bugfix on 0.3.0.1-alpha.

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'bug26913_033' into maint-0.3.3

Merge remote-tracking branch 'public/bug24104_029_squashed' into maint-0.2.9

Resolved conflicts with the 26269 fix in 015fcd0e1191aa6f.

Merge branch 'maint-0.3.3' into release-0.3.3

Merge remote-tracking branch 'dgoulet/bug27550_033_01' into maint-0.3.3

Merge branch 'maint-0.3.3' into release-0.3.3

Work around check-spaces.

Changes file for bug 28419

libtorrunner: fix memory leak in child() error path

This avoids leaking memory in case libtorrunner's child() function fails.

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

Merge branch 'bug28413_029' into maint-0.2.9

Fix a compiler warning in aes.c.

Apparently some freebsd compilers can't tell that 'c' will never
be used uninitialized.

Fixes bug 28413; bugfix on 0.2.9.3-alpha when we added support for
longer AES keys to this function.

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

Fix a bug in usage of SSL_set1_groups_list()

Apparently, even though the manpage says it returns an int, it
can return a long instead and cause a warning.

Bug not in any released Tor.  Part of #28399

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

Update geoip and geoip6 to the November 6 2018 database.

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

Merge branch 'bug28245_029' into maint-0.2.9

Always declare groups when building with openssl 1.1.1 APIs

Failing to do on clients was causing TLS 1.3 negotiation to fail.

Fixes bug 28245; bugfix on 0.2.9.15, when we added TLS 1.3 support.

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'bug26470_032' into maint-0.3.3

Merge branch 'maint-0.3.3' into release-0.3.3

Merge remote-tracking branch 'dgoulet/ticket27410_032_01' into maint-0.3.3

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

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

Merge branch 'maint-0.3.3' into release-0.3.3

Merge remote-tracking branch 'tor-github/pr/346' into maint-0.3.3

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'bug28202_029' into bug28202_033

Fix two other cases of (buf + N > end) pattern

Related to fix for 28202.

Fix possible UB in an end-of-string check in get_next_token().

Remember, you can't check to see if there are N bytes left in a
buffer by doing (buf + N < end), since the buf + N computation might
take you off the end of the buffer and result in undefined behavior.

Fixes 28202; bugfix on 0.2.0.3-alpha.

Merge branch 'maint-0.3.3' into release-0.3.3

Remove a now-unused rust "use".

Merge branch 'maint-0.3.3' into release-0.3.3

Merge remote-tracking branch 'onionk/rust-allsupportednull1' into maint-0.3.3

Treat backtrace test failures as expected on most BSD-derived systems

Treat backtrace test failures as expected on NetBSD, OpenBSD, and
macOS/Darwin, until we solve bug 17808.

(FreeBSD failures have been treated as expected since 18204 in 0.2.8.)

Fixes bug 27948; bugfix on 0.2.5.2-alpha.

Merge branch 'maint-0.3.3' into release-0.3.3

Log more info for duplicate ed25519 IDs

Occasionally, key pinning doesn't catch a relay that shares an ed25519
ID with another relay.  Log the identity fingerprints and the shared
ed25519 ID when this happens, instead of making a BUG() warning.

Fixes bug 27800; bugfix on 0.3.2.1-alpha.

Merge branch 'maint-0.3.3' into release-0.3.3

Fix another tor_assert(0) case

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

Fix make check-spaces.

Merge branch 'maint-0.3.3' into release-0.3.3

Adjust tor_assert_unreached()

This version of the macro makes it obvious to the compiler that
"tor_assert_unreached()" will always abort().

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

Merge branch 'bug27709_029' into maint-0.2.9

Explain a bit more about branch prediction in the unit-test case

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

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

Merge remote-tracking branch 'teor/bug27738-032' into bug27738-033

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.3

Update geoip and geoip6 to the October 9 2018 database.

hs-v3: Silence some logging for client authorization

If a tor client gets a descriptor that it can't decrypt, chances are that the
onion requires client authorization.

If a tor client is configured with client authorization for an onion but
decryption fails, it means that the configured keys aren't working anymore.

In both cases, we'll log notice the former and log warn the latter and the
rest of the decryption errors are now at info level.

Two logs statement have been removed because it was redundant and printing the
fetched descriptor in the logs when 80% of it is encrypted wat not helping.

Fixes #27550

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

Merge branch 'maint-0.3.3' of https://git.torproject.org/tor into maint-0.3.3

Merge branch 'maint-0.3.3' into release-0.3.3

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.2

rust/protover: return C-allocated string in protover_all_supported()

The result of CString::into_raw() is not safe to free
with free() except under finicky and fragile circumstances
that we definitely don't meet right now.

This was missed in be583a34a3815c2c10e86094ab0610e4b7f9c869.

rust/protover: fix null deref in protover_all_supported()

Fortunately with the current callers it couldn't happen in practice.

Fix on d1820c1516a31a149fc51a9e5126bf899e4c4e08.

Merge branch 'maint-0.3.3' of https://git.torproject.org/tor into maint-0.3.3

hs-v3: Close all SOCKS request on descriptor failure

Client side, when a descriptor is finally fetched and stored in the cache, we
then go over all pending SOCKS request for that descriptor. If it turns out
that the intro points are unusable, we close the first SOCKS request but not
the others for the same .onion.

This commit makes it that we'll close all SOCKS requests so we don't let
hanging the other ones.

It also fixes another bug which is having a SOCKS connection in RENDDESC_WAIT
state but with a descriptor in the cache. At some point, tor will expire the
intro failure cache which will make that descriptor usable again. When
retrying all SOCKS connection (retry_all_socks_conn_waiting_for_desc()), we
won't end up in the code path where we have already the descriptor for a
pending request causing a BUG().

Bottom line is that we should never have pending requests (waiting for a
descriptor) with that descriptor in the cache (even if unusable).

Fixees #27410.

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