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

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

doc: Use `` for commands and "" for quotes in ReleasingTor.md

doc: Spell make test-network-all correctly in ReleasingTor.md

Closes ticket 28821.

HSv3: Correctly memwipe client auth keystream.

Wipe the whole thing, not just the size of the pointer.

Add a couple more checks to test_parsecommon.c

These checks should make coverity stop giving us a "dereference
before null check" warning here.

Merge branch 'ticket28177'

Merge branch 'maint-0.3.5'

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 'maint-0.3.5'

Merge branch 'ticket28115_035' into maint-0.3.5

Re-alphabetize the list of tests in tests.[ch]

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

Merge branch '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 'maint-0.3.5'

Merge branch 'nss-travis' into maint-0.3.5

Merge branch '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.5'

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 'tor-github/pr/421'

clean up a tor2web comment

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

Merge branch 'maint-0.3.5'

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

Merge remote-tracking branch 'onionk/rust-protocommas1'

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 remote-tracking branch 'dgoulet/ticket27471_035_02'

test: Add test for closing intro circuits when storing a new descriptor

This is client side and related to 27471 for previous commit that fixes this
issue.

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

hs-v3: Close client intro circuits if the descriptor is replaced

When storing a descriptor in the client cache, if we are about to replace an
existing descriptor, make sure to close every introduction circuits of the old
descriptor so we don't have leftovers lying around.

Ticket 27471 describes a situation where tor is sending an INTRODUCE1 cell on
an introduction circuit for which it doesn't have a matching intro point
object (taken from the descriptor).

The main theory is that, after a new descriptor showed up, the introduction
points changed which led to selecting an introduction circuit not used by the
service anymore thus for which we are unable to find the corresponding
introduction point within the descriptor we just fetched.

Closes #27471.

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

Merge branch 'ticket27549_035_01_squashed'

hs-v3: Consolidate descriptor cookie computation code

Both client and service had their own code for this. Consolidate into one
place so we avoid duplication.

Closes #27549

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

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.4' into maint-0.3.5

Merge branch '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.

evloop: fix docs for threadpool_register_reply_event

Commit 6a5f62f68f2c73dbbbbddb4fa3759586f4c2b0dc ultimately didn't
include the base argument, and the callback is named cb.

evloop: fix docs

alert_sockets_t was moved in 544ab27a949406628809869111b7288017a5bcb1.

Bump to 0.3.6.0-alpha-dev.

Merge branch 'maint-0.3.5'

"ours" to avoid version bump.

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

Add changes file

One testcase for annotation handling in tokenize_string()

Test AT_END checking in tokenize_string()

Add testcase for too many elements in tokenize_string() input

Add testcase for too few elements in tokenize_string() input

Unit-test multiple line parsing with tokenize_string()

Unit-test some error conditions in get_next_token()

Test object parsing in get_next_token()

Fix memory management in test_parsecommon_get_next_token_parse_keys

Test RSA private key parsing with get_next_token()

Test-case for public key parsing using get_next_token()

Test argument concatenation in get_next_token()

First testcase for get_next_token()

Add new source file to test target

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