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

Merge branch 'maint-0.3.4' into maint-0.3.5

Merge branch 'maint-0.3.3' into maint-0.3.4

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'

Fix a bug where we would crash on --version.

Bug not in any released Tor.

test-stem would have caught this.

changes file for splitting routerparse.c (#27924)

Remove routerparse include from files that dont use it

Remove versions.h include from routerparse.h

Remove unused headers from routerparse.c

Extract addr-policy parsing code.

Move summarize_protover_flags to versions.c

extract networkstatus parsing to its own file.

Pull detached-signatures code into dirauth.

Split microdescriptor parser into its own file.

Remove addr_policy_assert_ok() as unused

Remove dump_distinct_digest_count()

It was disabled-by-default for ages, and it no longer compiles. I
think it's safe to call it obsolete.

Remove router_get_dir_hash as unused.

Move v2 hs parsing into feature/rend

Split the authority-cert and signature/hash code from routerparse

Extract the version-managing code from routerparse.c

Leave the versions.h include in routerparse.h for now; I'll remove
it later.

Extract logic for dumping unparseable junk from routerparse.c