Python v4 nesting

Whitespace

Rename fr_dict_attr_enum functions to fr_dict_enum

Unused

Check if we have  a trigger args list

Add fr_rb_node_t fields to structures that are inserted into rbtrees

This is being done ahead of removing malloc calls in the rbtree.

Use names field initialisers

Move shellcheck RC from root to workflow config; disable JSCPD (#3904)

Fuzzer artifact fetcher (#3893)

* Fuzzer: Make sure that the fuzzer is built before attempting to run it

* Fuzzer: Script to fetch fuzzing artifacts from GH Actions workflows

Requires a Personal Access Token with "repo:public_repo" scope to be set in
GITHUB_TOKEN environment variable to download artifacts.

* Shut up linter

POSIX-compliant methods of getting around this are fugly.

Revise dbuff_tests to avoid provoking jscpd (#3902)

Only run the free walker if we have a free callback

The rest of the nodes are parented off the tree

More rbmonkey fixes

Delete data as the _last_ step when removing or inserting a node

LLVM snapshot repository is broken; switch to LLVM 10 for now

Add Arista Captive-Portal and fix WebAuth start/complete values (#3900)

* Fix WebAuth start/complete values

* Add Arista-Captive-Portal

* Remove Arista- prefix

Typo

look at listener only if it exists

by pushing checks for size to code where it exists

Fix condition

Don't crash if env_pairs are NULL

Add Hitron dictionary

sqlcounter: fixed configuration for PostgreSQL #3867

ABSTIME is deprecated since PostgreSQL 7.0 and removed in PostgreSQL 12.
GREATER doesn't seem to be documented, but GREATEST is available
since PostgreSQL 9.

Add inverse type mappings for bool

More helpful error messages

rlm_python: The detach() function in example.py expects parameter

Add extra tests to rlm_attr_filter to exercise the different operators

radclient: Fix 'invalid attribute' test

It is necessary to match the output files built with/without -NDEBUG

Changed prematurely

thresh should be an fr_rb_test_node_t

Grr stupid generics

Avoid type conflicts in fr_dbuff_advance

radclient: Add tests validating invalid attributes

test: Add 'IGNORE_ERROR' option for radclient tests

Avoid backing up markers and make naming and usage consistent.

Modernize and clean up dhcpv4 encoding code to make better use of dbuffs and markers

Don't allow the length argument of fr_dbuff_advance to be a signed integer

Remove FR_DBUFF_RESERVE*() now that nothing uses it.

This allows a simpler, more direct expression of FR_DBUFF_MAX*()
(for now, at least; things will get interesting again when we
let FR_DBUFF_MAX*() take advantage of extensibility)

Fixup rbmonkey so it does not duplicate private structs

- Use a test structure with room for an rb node instead of just an integer.
- Minor style corrections.

ensure that we do adoc -> man conversions, too

because $(wildcard ...) will only pick up files which exist
in the directory.  the auto-generated "man" files might not,
so we need to add a dependency so that they are created, too

Correct path for man pages made from adoc files

We should always have list head pointer values now Closes #3889

Can't write to a const

CI: Allow one-off fuzzing runtime to be specified in the branch name

CI: Set explicit retention days for uploaded assets; name fixes

Default retention period is 90 days which is excessive for large
artifacts, especially now that this can be controlled.

Do not retain the packages built for post-install testing. We limit to
the minimum of 1 day. These assets are large and we don't want people to
use them.

Make attr_filter test pass

Remove pair from temporary list before adding to correct list

unify checks for buffer_len==1 and do_not_respond

always use protocol size, even for do_not_respond

we rely on proto_foo encode() to look at request->reply->code,
and then do "the right thing" to signal it's own write() layer
that we're not replying.

tell the IO handler that we're done

DHCPv4: Still need to copy pairs to a local list so we can search for options

add flag / assertion for request is finished

so that we don't erroneously refuse to clean it up

Use a local var for easier debugging

tests for the attr_filter module

print out when we're cleaning up the packet

typo

man files are built on "make install", not "make"

Fix issue with TLV presence checks for attribute resolved during pass2

Prefix rbtree types with fr_

Expose rbnode_t in public headers

Avoid zeroing out memory and then filling it with data

Simplify uses of rbtrees in radclient/packet list tracking code

Deal with error paths in callers of dict_attr_alloc_null

Update license

CI: Fix fuzzer asset naming

Scheduled fuzzing improvements

Change scheduled fuzzing to a matrix workflow so that we fuzz protocols in separate
jobs: More CPUs; less context switching.

If HEAD doesn't build then walk up the tree until we find something that does,
then fuzz that.

Set the runtime based on the calculated time remaining in the job since we
don't know ahead of time how much time we will spend walking the tree to find a
working build.

Name assests after "<commit>:<protocol>"

Copy the job output text into the asset that is uploaded when there are fuzzing
failures.

Rename parameters of fr_dbuff_in_bytes() and FR_DBUFF_IN_BYTES_RETURN()

Though they could be either dbufs or markers, the parameter names
suggested that only dbuffs would do. They now follow the convention
of "_dbuff_or_marker" to indicate their possible use. There was
already an fr_dbuff_in() call in dbuff_tests using a marker, but
none for fr_dbuff_in_bytes(), dbuff or marker; the latter is now
present, and both include tests to explicitly test cases in which
fr_dbuff_used(dbuff) and fr_dbuff_used(marker) differ.

Rename dpair_legacy.c to pair_legacy.c

Rename dpair.c to pair.c

try to shut up the linter.

docs, and provisions for lldb

wrapper script so we don't need to remember anything

clean up tags on fail, too

GCC doesn't deal with multiple pragmas in the same line correctly

Missing inline

Fix various build issues

Can't warn about unused results when there's no result

Format build deps correctly

Mark up return values of pair allocation functions as needing to be assigned

Note used

Rework DIAG_ON and DIAG_OFF macros to behave consistently.

Add DIAG_OFF_OPTIONAL and DIAG_ON_OPTIONAL which ignore unknown pragmas

CI: Activate tmate on failure when pushing to the ci-debug branch

Use new function for clearing error stack

CI: Rework package installation for on-push CI and scheduled fuzzing

Install packages from debian/control file.

Install extra CI-specific packages from new scripts/ci/extra
packages.control file

Defer fixture package installation until after build.

Disable rebuild of initramfs to optimise runtime.

Remove util/pair.c and util/pair_legacy.c

Remove USE_DOUBLE_LIST from build options

Set correct function type for eval function in protocol test points.

Replace references to cursor.h with dcursor.h

and remove un-used references to cursor.h

Replace util/pair_legacy.h with util/dpair_legacy.h

Replace util/pair.h with util/dpair.h

Add build dependency for deb packages on Perl

Run fuzzer.c through the formatter

ws

Add function for explicitly clearing outstanding messages

typo

we have one byte of option length, too

use p / end, and data / data_len

mixing and matching is a bad idea

ensure variables are initialized

clear errors after every run.

Fiz various issues with fuzzer.c

- Initialise the atexit handlers
- Initialise error pools before starting to fuzz
- Use fr_perror() so errors get printed correctly
- Use macros when calling failure functions

Revert "just call printf, not push"

This reverts commit 02209ba71c112c9f72dc0e6b9304dd2bdfb222e9.

just call printf, not push

include stdbool