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>

test: HS v3 descriptor decoding with client authorization

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

hs-v3: Decrypt the descriptor with client private key

Parse the client authorization section from the descriptor, use the client
private key to decrypt the auth clients, and then use the descriptor cookie to
decrypt the descriptor.

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

hs-v3: Refactor the descriptor decryption/decoding

This commit refactors the existing decryption code to make it compatible with
a new logic for when the client authorization is enabled.

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

hs-v3: Refactor secret data building logic

Because this secret data building logic is not only used by the descriptor
encoding process but also by the descriptor decoding, refactor the function to
take both steps into account.

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

test: HS v3 client authorization loading secret key

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

hs-v3: Load client authorization secret key from file

The new ClientOnionAuthDir option is introduced which is where tor looks to
find the HS v3 client authorization files containing the client private key
material.

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

test: HS v3 descriptor encoding with client authorization

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

hs-v3: Encrypt the descriptor using a cookie

Previously, we encrypted the descriptor without the descriptor cookie. This
commit, when the client auth is enabled, the descriptor cookie is always used.

I also removed the code that is used to generate fake auth clients because it
will not be used anymore.

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

test: Build an HSv3 descriptor with authorized client

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

test: HS v3 building a descriptor with client auth

This commit tests that the descriptor building result, when the client
authorization is enabled, includes everything that is needed.

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

hs-v3: Generate all descriptor related keys

We need to generate all the related keys when building the descriptor, so that
we can encrypt the descriptor.

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

test: HS v3 loading client auth keys service side

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

hs-v3: Load all client auth keys to the service

This commit loads all client public keys from every file in
`authorized_clients/` directory.

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

Merge remote-tracking branch 'teor/bug27521'

Merge branch 'maint-0.3.4'

"ours" to avoid version bump

Bump to 0.3.4.8

in master, remove changes files for already-merged changes

Copy updated 0.3.4.6-rc changelog to master

Forward-ports fix for 27488.

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.2.9' into maint-0.3.2

Merge branch 'ticket27344_029' into maint-0.2.9

Tell openssl to build its TLS contexts with security level 1

Fixes bug 27344, where we'd break compatibility with old tors by
rejecting RSA1024 and DH1024.

Merge branch 'maint-0.3.4'

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

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.2

Merge branch 'maint-0.3.3' into maint-0.3.4

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

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

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.2.9' into maint-0.3.2

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

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.3.3' into maint-0.3.4

Merge branch 'maint-0.2.9' into maint-0.3.2

Appveyor: Disable gcc hardening in Windows 64-bit builds

As of August 29, 2018, Appveyor images come with gcc 8.2.0 by
default. 64-bit Windows executables compiled with gcc 8.2.0 and
tor's --enable-gcc-hardening crash.

Fixes bug 27460; bugfix on 0.3.4.1-alpha.

Merge branch 'bug27461-029' into bug27461-032

Fix a minor merge conflict due to an #endif comment.

Windows: Stop calling SetProcessDEPPolicy() on 64-bit Windows

It is not supported, and always fails. Some compilers warn about the
function pointer cast on 64-bit Windows.

Fixes bug 27461; bugfix on 0.2.2.23-alpha.

hs: Silence a spurious warning in rend_client_send_introduction()

gcc 8 warns that extend_info_t.nickname might be truncated by strncpy().

But it doesn't know that nickname can either contain a hex id, or a
nicknames. hex ids are only used for general and HSDir circuits.

Fixes bug 27463; bugfix on 0.1.1.2-alpha.

Comment: Fix typos in get_interface_addresses_win32()

Closes 27521.

Windows: Silence a spurious warning in the GetAdaptersAddresses cast

GetProcAddress() returns FARPROC, which is (long long int(*)()) on
64-bit Windows:
https://msdn.microsoft.com/en-us/library/windows/desktop/ms683212(v=vs.85).aspx

But GetAdaptersAddresses() is (long unsigned int(*)()), on both 32-bit
and 64-bit Windows:
https://docs.microsoft.com/en-us/windows/desktop/api/iphlpapi/nf-iphlpapi-getadaptersaddresses

So gcc 8 issues a spurious "incompatible function pointer" warning
about the cast to GetAdaptersAddresses_fn_t.

Silence this warning by casting to a void function pointer, before
the cast to GetAdaptersAddresses_fn_t.

This issue is already fixed by 26481 in 0.3.5 and later, by removing
the lookup and cast.

Fixes bug 27465; bugfix on 0.2.3.11-alpha.

Revert "Avoid double-close on TCP sockets under NSS."

This reverts commit b5fddbd24144a94580e50886cd07a82968a1b86c.

The commit here was supposed to be a solution for #27451 (fd
management with NSS), but instead it caused an assertion failure.

Fixes bug 27500; but not in any released Tor.

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

Changes file for 25132

Merge remote-tracking branch 'ageis/control-getinfo-uptime'

Merge remote-tracking branch 'teor/ticket27467'

Remove GetAdaptersAddresses_fn_t

The code that used it was removed as part of the 26481 refactor.

Closes ticket 27467.

Backport to older NSS, which does not have SEC_DerSignDataWithAlgorithmID

Fix checkspaces

Suppress strict-prototypes warnings in one more batch of NSS headers

Add a last-ditch memwipe() implementation for nss+old glibc

On new glibc versions, there's an explicit_bzero().  With openssl,
there's openssl_memwipe().

When no other approach works, use memwipe() and a memory barrier.

Merge branch 'maint-0.3.4'

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.2.9' into maint-0.3.2

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

Merge branch 'maint-0.3.4'

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

Merge branch 'maint-0.3.2' into maint-0.3.3

Merge branch 'maint-0.2.9' into maint-0.3.2

Merge branch 'maint-0.3.4'

Merge branch 'maint-0.3.3' into maint-0.3.4

Fix a type, and hopefully the win64 builds.

Add note about use of tor_memcmp()

Fix a reverse-inull warning from coverity in new code.

Try to fix new coverity warnings in unit tests.

Travis: don't call echo with a --flag as the first argument

When we use echo in Travis, don't pass a --flag as the first argument.

Fixes bug 27418; bugfix on 0.3.4.7-rc.

Appveyor: tail config.log and cat test-suite.log after failed builds

Also zip the full logs, and upload them as build artifacts.

Implements ticket 27430.

Travis: when showing a log fails, keep trying to show other logs

When a Travis build fails, and showing a log fails, keep trying to
show the other logs.

Fixes bug 27453; bugfix on 0.3.4.7-rc.

Appveyor: always log the compiler path and version

Implements 27449.

Debug one last reference-counting issue that only appeared on openssl master

Fix an easy refcounting bug in a unit test