Move around some LCOV_EXCLs in src/common

Apparently, my compiler now generates coverage markers for
label-only lines, so we need to exclude those too if they are meant
to be unreachable.

Improve cov-diff script to account for new files.

Add unit test for buf_get_line().

Note an unreachable (?) section in buffers.c

Let's get 100% coverage for proto_http while we're at it.

There was some coverage before, but it was mostly accidental.

Test more error cases of our socks code.

Coverage is now respectable. :)

Add unit tests for SOCKS functions that parse server responses

We use these when we're acting as a SOCKS client, but we'd never
actually written tests for them :/

socks5 unit tests: add check for ipv6 address type.

Socks tests for bad socks5 username/passwd auth.

Mark some tests in parse_socks.c as unreachable (BUG, LCOV)

These tests aren't reachable, given their actual arguments.  I'm
going to mark them as BUG(), and as unreachable with LCOV.

testing: test many possible truncated SOCKS commands

Many of the 'truncated command' paths in fetch_from_buf_socks() were
not reached by the tests.  This new unit test tries to check them
exhaustively.

Move ext_or_command tests to test_proto_misc.c

No other code changes.

Add unit tests for var_cell and control0 proto functions.

Eat redundant semicolons

fix the changes file

hs-v3: Don't non fatal assert if we can't get the intro extend info

Fixes #23159.

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

Merge branch 'ticket23602_029'

Merge branch 'typecheck4'

changes file for my confparse typechecking fun

Type bug in shared_random_state -- make sure Version is int.

The confparse field has type UINT, which corresponds to an int
type.  We had uint32_t.

This shouldn't cause trouble in practice, since int happens to
4-bytes wide on every platform where an authority is running.  It's
still wrong, though.

Make the TransProxyType field non-const

The correct type for a STRING confparse value is char *, not const
char *.

Correct two state-file variable types.

These should have been int, but we had listed them as unsigned.
That's an easy mistake to make, since "int" corresponds with either
INT or UINT in the configuration file.

This bug cannot have actually caused a problem in practice, since we
check those fields' values on load, and ensure that they are in
range 0..INT32_MAX.

Add test to make sure all confparse variables are well-typed

New approach, suggested by Taylor: During testing builds, we
initialize a union member of an appropriate pointer type with the
address of the member field we're trying to test, so we can make
sure that the compiler doesn't warn.

My earlier approach invoked undefined behavior.

add changes file for 23645

Merge remote-tracking branch 'asn/ticket23645_032_01'

hs: Improve logging

Also demote a log message that can occur under natural causes
(if the circuit subsystem is missing descriptors/consensus etc.).
The HS subsystem will naturally retry to connect to intro points,
so no need to make that log user-facing.

circuit: Log n_circ_id and global identifier of HS circuits

So we can track them more easily in the logs and match any open/close/free
with those identifiers.

Part of #23645

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

circuit: Log circuit identifiers when cannibalizing

This removes the "nickname" of the cannibalized circuit last hop as it is
useless. It now logs the n_circ_id and global identifier so we can match it
with other logging statement.

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

circuit: Keep a copy of n_circ_id for logging

Prior to the log statement, the circuit n_circ_id value is zeroed so keep a
copy so we can log it at the end.

Part of #23645

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

circuit: Closing reason are signed values

Part of #23645

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

Merge branch 'bug22109'

Make testing rend1_payload variables static

This fixes a clang warning.

Merge branch 'bug23539_032_01_squashed'

Whitespace fix.

sched: make interval a plain int; initialize with macro

sched: Make KISTSchedRunInterval non negative

Fixes #23539.

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

fix changes file

Merge remote-tracking branch 'asn/ticket23420_032_01'

prop224: Add test for parsing RENDEZVOUS1 cell.

Just making sure that added padding does not screw up cell parsing.

fix typo

Mention even more about changes files in doc/HACKING

Fix check-spaces and a weird copyright.

Use setup_full_capture_of_logs for 21800

setup_capture_of_logs() doesn't actually suppress warnings :p

Remove unused chosen_scheduler_type variable.

Merge remote-tracking branch 'pastly/bug23552_032_03'

Merge branch 'maint-0.3.1'

Merge branch 'maint-0.3.0' into maint-0.3.1

hs: Avoid possible double circuit close on error

Fixes #23610

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

Merge branch 'bug22805_v2_squashed'

Stop using CREATE_FAST as a signifier of clienthood.

Relays send it, and we may as well let them.  Part of our fix for
22805.

changes file for bug22805

Use channel_is_client() accessor in channelpadding.c.

Also, allow channel_is_client() to take a const channel.

Stop clearing the is_client flag on channel directly

Remove or_circuit_t.is_first_hop; use channel_is_client() instead

The is_first_hop field should have been called used_create_fast,
but everywhere that we wanted to check it, we should have been
checking channel_is_client() instead.

sched: move code to respect comments

The diff is confusing, but were two static scheduler functions that
needed moving to static comment block.

No code change. Thanks dgoulet for original commit

sched: only log when scheduler type changes

Closes 23552. Thanks dgoulet for original impl

test_util: capture the log in the correct place.

Merge branch 'bug21800'

check-changes fix

Merge remote-tracking branch 'pastly/bug23581_032_03'

Whoops; only run lintchanges when usepython

Update CodingStandards.md for changes-file stuff

'check-changes' is now part of "make check", and must pass.

Whoops -- put a changes file in the wrong place :(

Fix warning in bug23610

Merge branch 'ticket23564_squashed'

Make check-spaces part of the standard "make check" process

To do this, we had to make sure it passes when the changes directory
is empty.  I also tried to improve the quality of the output, and
fix some false-positive cases.  Let's see how this goes!

Closes ticket 23564.

Capture "Result does not fit in tor_timegm()" warnings in util tests

Fixes bug 21800; bugfix on 0.2.9.x

Merge remote-tracking branch 'dgoulet/ticket23303_032_01'

Merge remote-tracking branch 'catalyst-oniongit/bug23611'

Merge branch 'maint-0.3.1'

hs: Avoid possible double circuit close on error

Fixes #23610

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

Remove wrong config parameter mention from manpage

Don't refer to the obsolete (and misspelled)
UseEntryGuardsAsDirectoryGuards config parameter in the manpage.

hs: Improve comments for when we change timestamps

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

sched: Hard exit if we can't select a scheduler

Fixes #23581

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

hs: Avoid possible double circuit close on error

Fixes #23610

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

Merge remote-tracking branch 'catalyst-oniongit/bug23607'

lintchanges fix

Merge branch 'maint-0.3.1'

Bug 23608: Mock time for all channelpadding tests.

Avoid assertion failure for state file clock skew

The clock_skew_warning() refactoring allowed calls from
or_state_load() to control_event_bootstrap_problem() to occur prior
bootstrap phase 0, causing an assertion failure.  Initialize the
bootstrap status prior to calling clock_skew_warning() from
or_state_load().

Use correct sign for state file clock skew

or_state_load() was using an incorrect sign convention when calling
clock_skew_warning() to warn about state file clock skew.  This caused
the wording of the warning to be incorrect about the direction of the
skew.

When detecting OpenSSL, look for 1.0.1, and in homebrew location

Previously we would detect the system openssl on OSX, and then fail
to use it, since we required Open 1.0.1 or later.  That's silly!

Instead of looking for RAND_add(), look for TLSv1_1_method(): it was
introduced in 1.0.1, and is also present in LibreSSL.

Also, add the hombebrew path to our search path here.

Fixes bug 23602; bugfix on 0.2.7.2-alpha.

Use correct name for authtype 3 in a comment.

hs: Log the intro point when we clean it up

When we remove an intro point from the service list, log info about it and
some useful data.

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

circuit: Log n_circ_id and global identifier

When we can, log the n circuit id and global identifier for origin circuit.

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

Merge branch 'bug23080'

Unit test to make sure that our platform string is parseable

Closes 22109.

Merge branch 'maint-0.3.1'

changes file for bug23568

in zstd compression, fix 32 bit build

format '%lu' expects argument of type 'long unsigned int', but argument ... has type 'size_t'

Closes ticket 23568.

Fix a comment that misunderstood is_canonical

is_canonical doesn't mean "am I connected to the one true address of
this relay"; it means "does this relay tell me that the address I'm
connected to belong to it."  The point is to prevent TCP-based MITM,
not to prevent the relay from multi-homing.

Related to 22890.

Warn on ExtORPort USERADDR that is missing or 0.

Diagnoses violations on #23080.

Merge remote-tracking branch 'dgoulet/ticket23305_032_01'

Merge branch 'maint-0.2.9' into maint-0.3.0

Merge branch 'maint-0.2.8' into maint-0.2.9

Merge branch 'maint-0.3.0' into maint-0.3.1

Merge branch 'maint-0.3.1'

Remove longclaw's IPv6 address, as it will soon change

Authority IPv6 addresses were originally added in 0.2.8.1-alpha.

This leaves 3/8 directory authorities with IPv6 addresses, but there
are also 52 fallback directory mirrors with IPv6 addresses.

Resolves 19760.

sched: reorder code to fit comment bodies; comment typos

Closes 23560

fix typos/etc found while starting to look at scheduler