conn: Fix memleaks in retry_all_listeners

Fixes #27670

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

Report UNIX connection addresses that we opened correctly.

This is an aside on ticket27670.

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.3.3' into maint-0.3.4

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

fixup! changes file

Merge remote-tracking branch 'dgoulet/ticket27545_035_01'

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

Merge branch 'pr278_squashed'

Actually, just disable test_rebind.sh on Appveyor

Disable test_rebind.sh on Windows

Update/fix CI build

Update integration test to Python 3

Pick random ports in test_rebind.py

Always include socket rebinding code

Tweak test_rebind.py for future-proofness

Integration test for socket rebinding

squash! Integration test for socket rebinding

Avoid mentioning ticket number in comments

Comments/explanation for #17873

Minor code cleanups

Add changes file

Log a notice *after* creating connection

Log a notice when changing to/from wildcard IP address

Always close old listeners in retry_all_listeners

Refrain from compiling socket rebinding code on system that don't need it

Try rebinding new listener after closing old one if first bind failed with EADDRINUSE

Merge branch 'maint-0.3.4'

Merge remote-tracking branch 'teor/bug27464-034' into maint-0.3.4

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

rust/protover: fix check for overlapping ranges

Closes ticket 27649. Bugfix on e6625113c98c281b0a649598d7daa347c28915e9.

rust/protover: remove version zero from tests

This isn't legal according to dir-spec.txt.

We can write separate tests for it if the spec
is changed to make it legal.

test/protover: remove version zero from tests

This isn't legal according to dir-spec.txt.

We can write separate tests for it if the spec
is changed to make it legal.

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

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.4'

Bug 25505: Check circuitmux queues before padding.

Merge remote-tracking branch 'mikeperry/bug25505'

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

Merge remote-tracking branch 'onionk/doublevote1'

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.2.9' into maint-0.3.2

Merge remote-tracking branch 'teor/ticket27252-034' into maint-0.3.4

Merge remote-tracking branch 'teor/ticket27252-033' into maint-0.3.3

Merge remote-tracking branch 'teor/ticket27252-032' into maint-0.3.2

Merge remote-tracking branch 'teor/ticket27252-029' into maint-0.2.9

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

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

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.2.9' into maint-0.3.2

hs-v3: Shuffle the list of authorized clients

This commit makes it that the authorized clients in the descriptor are in
random order instead of ordered by how they were read on disk.

Fixes #27545

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

Initialize 't' in ge25519_scalarmult_base_niels()

OSS-Fuzz's version of memorysanitizer can't tell that this value is
not going to be used unsafely.

Update geoip and geoip6 to the September 6 2018 database.

Defer reporting directory bootstrap progress

Existing cached directory information can cause misleadingly high
bootstrap percentages.  To improve user experience, defer reporting of
directory information progress until at least one connection has
succeeded to a relay or bridge.

Closes ticket 27169.

Bugfix: sizeof(socklen) doesn't make sense when calling connect()

Bugfix on 2f657a1416f2f81dd1be900269c4ae9bdb29f52d; bug not in
any Tor release.

Track bootstrap phase independently of progress

Track bootstrap phase (enumerated by bootstrap_status_t) independently
from the bootstrap progress (which can represent intermediate
progress).  This allows control_event_bootstrap_problem() to avoid
doing a linear search through the bootstrap progress space to find the
current bootstrap phase.

Refactor control_event_bootstrap_core() more

Eliminate a few conditional expressions in
control_event_bootstrap_core() by overwriting the status parameter.

Refactor control_event_bootstrap() somewhat

Move the mostly-invariant part of control_event_boostrap() into a
helper control_event_bootstrap_core().  The helper doesn't modify any
state beyond doing logging and control port notifications.

Deindent much of control_event_bootstrap

Make control_event_bootstrap() return void

Simplify control_event_bootstrap() by making it return void again.  It
is currently a fairly complicated function, and it's made more
complicated by returning an int to signal whether it logged at NOTICE
or INFO.

The callers conditionally log messages at level NOTICE based on this
return value.  Change the callers to unconditionally log their verbose
human-readable messages at level INFO to keep NOTICE logs less
cluttered.

This partially reverts the changes of #14950.

test: Fix coverity CID 1439129

One HSv3 unit test used "tor_memeq()" without checking the return value. This
commit changes that to use "tt_mem_op()" to actually make the test validate
something :).

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

test: Fix coverity CID 1439130

Trivial fix of removing an uneeded NULL check in an HS v3 unit test.

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

test: Fix coverity CID 1439131

Simple uninitialized object that we could free in an HS v3 unit test.

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

Copy changelogs and releasenotes to master

Address coverity warnings (CID 1439133/1439132).

>>>>    CID 1439133:  Null pointer dereferences  (REVERSE_INULL)
>>>>    Null-checking "fields" suggests that it may be null, but it
>>>> has already been dereferenced on all paths leading to the check.

>>>>    CID 1439132:  Null pointer dereferences  (REVERSE_INULL)
>>>>    Null-checking "fields" suggests that it may be null, but it
>>>> has already been dereferenced on all paths leading to the check.

Merge branch 'maint-0.3.4'

"ours" to avoid version bump.

Bump to 0.3.4.8-dev

Merge branch 'maint-0.3.3' into maint-0.3.4

"ours" to avoid version bump.

Bump to 0.3.3.10-dev.

Merge branch 'maint-0.3.2' into maint-0.3.3

"ours" to avoid version bump.

Merge branch 'maint-0.2.9' into maint-0.3.2

"ours" to avoid version bump.

Bump to 0.3.2.12-dev

Bump to 0.2.9.17-dev

Call event_set_mem_functions during initialization

Refactor initialization in curve25519_basepoint_spot_check

This is an attempt to work around what I think may be a bug in
OSS-Fuzz, which thinks that uninitialized data might be passed to
the curve25519 functions.

make ipv6-only config complaint clearer

(a relay operator hit this on #tor-relays and couldn't make sense
of it. i couldn't either until i went to go read the code.)

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.3.3' into maint-0.3.4

"ours" to avoid bump.

Merge branch 'maint-0.3.2' into maint-0.3.3

"ours" to avoid bump

Merge branch 'maint-0.2.9' into maint-0.3.2

"ours" to avoid bump

Bump to 0.3.3.10

Bump to 0.3.2.12

Bump to 0.2.9.17

Merge remote-tracking branch 'dgoulet/ticket20700_035_03'

HSv3: Don't assert when reading bad client-side privkeys.

HSv3: Add subcredential in client auth KDF on the client-side.

HSv3: Add subcredential in client auth KDF on the service-side.

Also update some client auth test vectors that broke...

hs-v3: Make hs_desc_build_fake_authorized_client() return an object

Return a newly allocated fake client authorization object instead of taking
the object as a parameter.

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

bug: Use PATH_SEPARATOR instead of slash

In function get_fname_suffix, previously it uses /, but in fact it
should use PATH_SEPARATOR.

hs-v3: Improve v3 client authorization logging

Part of #20700.

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

hs-v3: Log client auth load activities client side

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

hs-v3: Log client auth load activities service side

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

hs-v3: Rename client_pk to client_auth_pk

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

hs-v3: Rename client_sk to client_auth_sk

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

test: HS v3 client auth is config equal function

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

hs-v3: Republish descriptors if client auth changes

When reloading tor, check if our the configured client authorization have
changed from what we previously had. If so, republish the updated descriptor.

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

hs-v3: Make all descriptor content free functions public

Series of functions that we now need in hs_service.c.

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

hs-v3: Re-enable the decoding in the encoding function

Previously, the validation by decoding a created descriptor was disabled
because the interface had to be entirely changed and not implemented at the
time.

This commit re-enabled it because it is now implemented.

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