Merge branch 'bug28202_033' into bug28202_035

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.4' into maint-0.3.5

"ours" to avoid taking a second instance of 27797.

node: Make node_supports_v3_rendezvous_point() also check for the key

It is not enough to look at protover for v3 rendezvous support but also we
need to make sure that the curve25519 onion key is present or in other words
that the descriptor has been fetched and does contain it.

Fixes #27797.

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

Merge branch 'ticket28115_035' into maint-0.3.5

rust/tor_log: fix C_RUST_COUPLED documentation

This file was moved in 97b15a1d7c51764888d2172711e3f3a71fb01916
and moved again in e7f5f48d68553206b95cbb4f610702c887500124.

Merge branch 'nss-travis' into maint-0.3.5

Merge branch 'maint-0.3.4' into maint-0.3.5

"ours" version to avoid removing rust use declaration.

Merge branch 'maint-0.3.3' into maint-0.3.4

Remove a now-unused rust "use".

Merge branch 'maint-0.3.4' into maint-0.3.5

Merge branch 'maint-0.3.3' into maint-0.3.4

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

Merge remote-tracking branch 'onionk/rust-protocommas1' into maint-0.3.5

Rename a function; it is used to convert a value _From_ le.

We can't use htons()/ntohs() -- those are no-ops on exactly the
wrong platforms.

Fix a misspelled macro test that was breaking big-endian OPE

Fixes bug 28115; bugfix on 0.3.5.1-alpha.

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.4' into maint-0.3.5

Merge branch 'bug27800-034' into bug27800-035

Merge branch 'bug27800-033' into tor-034

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.

Bump to 0.3.5.3-alpha-dev

expand the blurb

Reflow the changelog

merge in some fixes i found in a sandbox

0.3.5.3-alpha changelog touch-ups

Fold in the entry for 28089

Merge branch 'maint-0.3.4'

conn: Stop writing when our write bandwidth limist is exhausted

Commit 488e2b00bf881b97bcc8e4bbe304845ff1d79a03 introduced an issue, most
likely introduced by a bad copy paste, that made us stop reading on the
connection if our write bandwidth limit was reached.

The problem is that because "read_blocked_on_bw" was never set, the connection
was never reenabled for reading.

This is most likely the cause of #27813 where bytes were accumulating in the
kernel TCP bufers because tor was not doing reads. Only relays with
RelayBandwidthRate would suffer from this but affecting all relays connecting
to them. And using that tor option is recommended and best practice so many
many relays have it enabled.

Fixes #28089.

Light changelog edits

Bump to 0.3.5.3-alpha

Run format-changelog

Fold changes file into changelog

Merge remote-tracking branch 'tor-github/pr/406'

Merge remote-tracking branch 'tor-github/pr/408'

Argh. The unset value for OwningControllerFD is NOT -1.

Merge branch 'bug27849_redux'

Merge remote-tracking branch 'tor-github/pr/404'

Add a tor_free() in tor_gencert to fix a coverity warning

Fix typo in comment for hs_cell_parse_introduce2()

mainloop: Set client role if ControlPort is set

It turns out that if _only_ the ControlPort is set and nothing else, tor would
simply not bootstrap and thus not start properly. Commit 67a41b63063370c2
removed that requirement for tor to be considered a "client".

Unfortunately, this made the mainloop enable basically nothing if only the
ControlPort is set in the torrc.

This commit now makes it that we also consider the ControlPort when deciding
if we are a Client or not. It does not revert 67a41b63063370c2 meaning
options_any_client_port_set() stays the same, not looking at the control port.

Fixes #27849.

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

Merge branch 'maint-0.3.4'

test: Update approx_time before the test starts

This way we have the same time source when the IP is created and tested later.

Fixes #27810

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

Merge remote-tracking branch 'public/bug27990'

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.3.3' into maint-0.3.4

Fix another tor_assert(0) case

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.2.9' into maint-0.3.3

Fix make check-spaces.

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.3.3' into maint-0.3.4

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.4'

Merge branch 'maint-0.3.3' into maint-0.3.4

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

Write a comment to explain may_need_rebind condition

Refrain from listener rebinding when address families differ

Remove a double-newline and improve a comment

Merge branch 'bug27772_squashed'

Fix various GCC LTO warnings in the unit tests.

In tor_tls_get_my_certs(), set cert ptrs even on failure

Nothing should ever look at them on failure, but in some cases,
the unit tests don't check for failure, and then GCC-LTO freaks out.

Fixes part of 27772.

Initialize some locals in socks5 parsing code.

These confused GCC LTO, which thought they might be used
uninitialized.  I'm pretty sure that as long as 'res' indicates
success, they will always be set to something, but let's unconfuse
the compiler in any case.

Use the correct function signatures in test_relaycell.c

This is now officially an antipattern: please let's never copy a
function declaration in two places again.  That's what headers are
for.

Use assertions so GCC LTO doesn't worry about TLS channel conversion

Part of #27772

Merge remote-tracking branch 'tor-github/pr/402'

report origin of mystery core file

Report what program produced the mysterious core file that we
occasionally see on Travis CI during make distcheck.  Closes ticket
28024.

Merge remote-tracking branch 'tor-github/pr/380'

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.2.9' into maint-0.3.3

Merge branch 'maint-0.3.4'

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

Merge branch 'bug27738-034-merged' into bug27738-master-merged

Merge branch 'bug27738-033' into bug27738-034

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

Merge branch 'maint-0.3.4'

Merge branch 'appveyor_pacman_fixes_034' into maint-0.3.4

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.2.9' into maint-0.3.3

Update geoip and geoip6 to the October 9 2018 database.

Add sys/time.h include back to procmon.c

fixup! Changes file for apveyor changes

Changes file for apveyor changes

Appveyor: Don't try to install the mingw openssl package

This package conflicts with a preinstalled mingw openssl.

Fixes bug 27943. I think this was an appveyor change, not a Tor bug.

Install only one of the two mingw architectures in appveyor

Closes ticket 27765

This commit also changes the appveyor file to a unified line ending
style, since the old one was pretty confused.  Use "-b" to see the
real differences.

Merge remote-tracking branch 'tor-github/pr/384'

Merge remote-tracking branch 'tor-github/pr/385'

Merge remote-tracking branch 'public/ticket27856'

Add changes file

Add newline between hs_client_get_random_intro_from_edge() and hs_client_receive_introduce_ack()

Make sure we're removing conn from old_conns *once*

Refrain from using SMARTLIST_DEL_CURRENT in retry_all_listeners

Merge remote-tracking branch 'public/bug27893'

Merge remote-tracking branch 'public/bug27728'