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

...and the mk file

Nightly, continuous fuzzing using LLVM snapshot

Parallel fuzzing of all protocols restarting every 5 minutes or on failure.

Fuzzer: Honour -D dictdir when using jobs=N option

Fuzzer: Correctly remove -D dictdir argument

Typo

Move type functions into a separate source file

Fix type ordering, add missing types to FR_TYPE_FIXED_SIZE

Correctly remove pairs from "from" list in raidus_pairmove()

and remove a stray debugging printf

move "sort $INCLUDE dir/" code to using heap

in preparation for moving rbtree nodes in-line in functions

typo

rename function

if we're comparing inodes, call it inode_cmp

remove empty array at the end of a structure

it's a GCC extension, and very useful.  But clang complains.

add libfreeradius-PROTOCOL.la as dependency of test.unit.PROTOCOL

because GNU make is too stupid to figure it out

allocate child_struct for all key fields

even if it doesn't actually use child structs

Call trigger_args_afrom_server() with a fr_pair_list to populate

rather than allocating a new list in the function

Correct order of parameters in eap_packet_to_vp()

fr_pair_list_clear() is not needed to clear and initialise a list.

fr_pair_list_free() leaves the header in the correct state

Call map_list_mod_to_vps() with a fr_pair_list_t to populate

rather than allocating a new list in the function.

Move cursor tests to correct framework

Remove un-used cursor

Replace cursor functions with pair_list functions in rlm_couchbase.c

Correct docuemntation

Correct method to add pairs to existing list in rlm_mruby.c

Pass pointer to request_pairs to lua_pushlightuserdata()

s/fr_cursor/fr_dcursor/ in rlm_lua/lua.c