With COCCI defined, avoid PERIODIC_EVENT.

Coccinelle doesn't understand the particular pattern of token
pasting we have going on here.

COCCI: disable DOWLOAD_SCHEDULE macro

tor-coccinelle.h: add MOCK_DECL

tor-coccinelle.h: handle SLIST and LIST.

With coccinelle, avoid includes in the middle of an expression.

Add a "COCCI" macro that we tell coccinelle is always defined.

This will let us give specific in-file equivalents to given macros
or preprocessor directives, to make things parse.

tor-coccinelle.h: handle NS and NS_DECL.

tor-coccinelle: handle SIMPLEQ and TAILQ better.

tor-coccinelle.h: Add {EN,DIS}ABLE_GCC_WARNING.

tor-coccinelle: expect parentheses after HT_INITIALIZER

Use #warning instead of #warn: one is C and one isn't.

try_parse.sh: add a verbose mode and a meaningful exit code.

document test-operator-cleanup

Document tor-coccinelle.h

Changes file for coccinelle scripts

Run test_operator_cleanup on our unit tests

Coccinelle doesn't understand it when we use "==" and "!=" and so on as
arguments to macros.  To solve this, we prefer OP_EQ, OP_NE, and so
on.

This commit is automatically generated by running
./scripts/coccinelle/test_operator_cleanup over all of the source
code in src.

Add some more of our trickier macros to tor-coccinelle.h

Note that this header file behaves a bit strangely.  It is used by
coccinelle just for the purpose of knowing how to parse
difficult-to-parse stuff.  It doesn't need to produce good C -- just
grammatical C.

Add a script to tell whether a file can be perfectly parsed by spatch

spatch can let us know whether a file has parsed "perfectly" or
not.  The more perfect it parses, the likelier any semantic patches
are to apply.  I've used this script to identify problem areas in
our code.

Add a script to run spatch with appropriate arguments

It's a bit tricky to remember the right incantation to get the
proper include paths and incantations for coccinelle, but without
it, coccinelle is less effective at parsing our C.

Merge branch 'tor-github/pr/1296'

Document inconsistent usage of config_var_is_listable()

See also ticket 31654.

Extract common list of flags to use for obsolete variables.

Clarify documentation on config_var_is_listable()

Here we make it clear we're only looking at listable variable names,
not at whether the variables themselves are gettable.

Also, remove an extraneous h.

(This commit is not a fixup, because of rebase conflicts.)

Add test_cmdline.sh to distribution.

integration test for --list-torrc-options

(This option tests our existing behavior, not necessarily the most
sensible behavior.)

Test: Make sure NOLIST options are not listed.

Changes file for 31625 (config flag refactor)

Revise documentation on CFLG_* flags

Remove all CVFLAG_* usage.

Remove all VTFLAG_* usage.

confparse, conftypes: Replace flags with their new names.

The old names remain as #defines that cause variables to get one or
more flags.

Now every flag-testing function in confparse.c tests exactly one flag.

Replace low-level {var_type,struct_var}_is_*() with flag inspection

Since the flags are now stored with compatible numbering, we can
just OR them together and see whether the flag we want is in the
result.

(Net code removal!)

Re-number VTFLAG_* values so they don't conflict with CVFLAG_*

Move VTFLAG_* declarations to conftypes.h

typed_var: Make flags into an unsigned OR of bits.

Using a bitfield here will enable us to unify the var_type_def_t flags
with the config_var_t flags.

(This commit does not yet do that unification, and does not yet
rename or refactor any flags.  It only changes booleans into bits.)

struct_var: refactor struct_var_is*() functions to delegate

Previously they checked the individual flags inside var_type_def_t;
now they call the appropriate var_type_is_*() functions.

(These functions will be removed entirely by the end of this branch.)

config: make config_var_is_dumpable static.

config: rename "contained" to "derived", and explain it better.

Don't use "derived" directly, but check its implications for listing
and copying.

config: Introduce the concept of an "ungettable" variable.

We had though to make all obsolete and invisible variables
ungettable, so that GETCONF would reject them.  But it turns out
that this isn't the current behavior of GETCONF with those
variables.  So for now, I'm leaving the current behavior unchanged.
(See ticket 31647 for a proposal to change the behavior.)

Document config_var_is_dumpable and config_var_is_settable.

Merge branch 'tor-github/pr/1311'

Merge branch 'tor-github/pr/1307'

Merge branch 'ticket31578' into ticket31578_merged

main: add some newlines to the subsystem list, for readability

main: remove level number comments from the subsystem list (master)

Part of 31615.

Merge branch 'bug31615_041' into bug31615_master

Merged modified lines from bug31615_041, and unmodified lines
from master.

main: remove level number comments from the subsystem list (0.4.1)

Part of 31615.

Merge branch 'bug31615_040' into bug31615_041

Merged modified lines from bug31615_040, and unmodified lines
from maint-0.4.1.

main: remove level number comments from the subsystem list (0.4.0)

These levels get out of date really easily: we'll implement a level
dump command in tor in 31614.

They also cause conflicts and inconsistencies when merging forward
level changes.

Part of 31615.

build: The <sys/sysctl.h> is now deprecated on Linux

Closes #31673

Merge branch 'tor-github/pr/1264'

Merge branch 'ticket30924_042_04_squashed_merged'

Merge branch 'ticket30924_042_04_squashed' into ticket30924_042_04_squashed_merged

hs-v3: Rename validation function in hs_intropoint.c

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

hs-v3: Move DoS parameter check against 0

Move it outside of the validation function since 0 is a valid value but
disables defenses.

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

practracker: Make you happy funny script

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

Merge branch 'tor-github/pr/1298'

Merge branch 'tor-github/pr/1287'

subsys: Make the subsystem init order match the module dependencies

Fix levels for subsystems that depend on log/err
* winprocess (security) doesn't use err:
  * call windows process security APIs as early as possible
  * init err after winprocess
  * move wallclock so it's still after err
* network and time depend on log:
  * make sure that network and time can use logging.
  * init network and time after log

Add comments explaining the module init order.

Fixes bug 31615; bugfix on 0.4.0.1-alpha.

changes file for ticket31477

Add new practracker test files to Makefile.am

New practracker exceptions for dependency violations in headers

I've done this manually, since I don't want to override the existing
exceptions in this branch.

Fix a bug in practracker's handling of .may_include in headers

I was expecting our filter code to work in a way it didn't.  I
thought that saying that DependencyViolation applied to "*" would
hit all of the files -- but actually, "*" wasn't implemented.  I had
to say "*.c" and "*.h"

Remove unused struct_var_* functions.

These turned out to be unnecessary, so let's not keep them around
and let them start getting complicated.

Closes ticket 31630.

config: Invert sense of _is_invisible, and rename to is_listable()

config: replace config_var_is_cumulative with is_replaced_on_set()

This change replaces the higher-level property check to match our
intended flag factoring.

Merge branch 'tor-github/pr/1295'

Merge branch 'tor-github/pr/1294'

Merge branch 'tor-github/pr/1293'

Merge branch 'tor-github/pr/1291'

Merge branch 'tor-github/pr/1292'

Merge branch 'tor-github/pr/1261'

Add tests for IPv6 exit policies on microdescriptors

Check IPv6 exit policies on microdescriptors in node_exit_policy_rejects_all()

config: note that some arguments are required.

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

Merge branch 'maint-0.4.1'

Merge branch 'maint-0.4.0' into maint-0.4.1

Merge branch 'maint-0.3.5' into maint-0.4.0

Merge remote-tracking branch 'tor-github/pr/1271' into maint-0.3.5

Merge branch 'maint-0.4.1'

Resolved modified/deleted conflict on changes/bug30649 by deleting
the file.

tor-github/pr/1282 modifies ChangeLog and ReleaseNotes instead of
changes/bug30649. We cherry-picked it to master, as well as merging
to release-0.4.1.

Merge branch 'maint-0.4.0' into maint-0.4.1

Merge commit '73890a86ef' into maint-0.4.1

tor-github/pr/1283, with the following changes:
* cherry-pick the merge commit in 1283 on top of...
* tor-github/pr/1174, but with the last commit re-worded to remove
    the fixup, because fixups break our push rules.

This is an "ours" merge, except for the bugfix version change in
changes/bug30649.

Add #30649 to ReleaseNotes.

Fix #30649 changelog entry.

Merge branch 'bug30649_040' into bug30649_maint_041

Fix bugfix version in Bug 30649: Changes file.

Merge commit '763fd0ad66' into maint-0.4.0

tor-github/pr/1174, but with the last commit re-worded to remove
the fixup, because fixups break our push rules.

Fix bugfix version in Bug 30649: Changes file.

Makefile: include checkShellScripts.sh in EXTRA_DIST

Part of 30967.

changes: file for 30967

shellcheck: Add shellcheck to the pre-commit hook

* Move the shellcheck script from the Makefile to its own script file
* Reformat the shellcheck script so it's easier to read and modify
* Call the shellcheck script from the pre-commit hook

Fixes bug 30967; not in any released version of Tor.

Makefile: Explicitly list the subdirectories that contain scripts

Part of 30967.

Remove _ex suffix from typed_var_*_ex() functions.

Now that the variants of these functions that took config_line_t are
gone, there is no longer any reason for the remaining variants to
have "ex" at the end of their names.

This commit was made by running this perl script over all the files
in src/:

#!/usr/bin/perl -w -i -p

s{typed_var_(assign|free|encode|copy|eq|ok|kvassign|kvencode|mark_fragile)_ex}
 {typed_var_$1}g;

typedvar: remove now-unused functions taking config_type_t.

These functions are no longer used.

Part of 31629.

Document fields of struct_magic_decl_t.

Document warn_deprecated_option().

Document return value of config_mgr_add_format().

Document validate_fn_t as it stands.

Also document that it will be changed in a later branch.

config_assign: Document CAL_* options.

These were sort of described in config_assign() documentation, but
not so well.