Enable sanitizers on GCC

CI package tests: Run bleeding-edge distro builds for RPM and DEB on push (#3982)

GH now lets you build the matrix from job output so we can excute different
plans on push vs on schedule.

This lets us perform package testing on each push without consuming lots of
workers.

Typically all DEB or RPM distros either succeed together or fail together, so
on push we just run a job for each of DEB and RPM using a bleeding-edge distro.

Remove redundant declaration

Move commonly called time functions into time.h

Minor CI improvements (#3980)

* Use pragmas not hacks to disable build warnings for homebrew libs

Current libjson-c no longer triggering errors.

talloc.h errors silenced by b65415f985bdb4a823afd5d7e1acac4de4314f0b

* CI package tests: Use Azure-hosted mirrors for better reliability

New version of parallel keyword

Removes request signalling inversion

Makes parallel children "first class" citizens

Remove unecessary check

Unused

Change process and signal functions to pass the current frame

For signal functions this is important because all frames are signalled.  For process functions it removes significant amounts of boilerplate.

Ignore documentation issues in talloc headers

Don't include talloc.h directly

Typo

Fix pre func arg order

use process module, not dynamic client module

move DHCPv6 to using new process / listen API

Cache the child request name

typo

request not set yet

Improve debug output consistency for parallel

More request context debugging

Fix macro order

Log in the request context if possible

Set udp_result_t treq to NULL when the treq has been released

Add additional sanity checks on freeing udp_result

Add destructor to _udp_result_free so we can easily set break points

Remove duplicate comparison macros

Requests must not be allocated in passed in ctx

Use the comparison macro

Use merge not rebase

Print out configured MRD and MRC

Use a macro for checking heap insertion

Not the first time the wrong assert has been used for this

assert that the various heap IDs are marked as "free"

Lets see if the github actions bot will sign commits

Have strftime extend the output buffer

Ensure all fields in dl_symbol_free_t are initialised

Correct value box foreach to use dlist functions (#3973)

Don't free pairs which were already freed with u->packet

CI packages: Remove scomp bug workaround; fix test.eap dependencies within post-install tests (#3970)

* CI RPMs: Remove workaround for scomp filter bug with faccessat2

No longer required.

* post-install tests: Don't depend on server build artifacts

Within the post-install test mock build environment we are not building the
server, so don't depend on it's build assets.

test (#3971)

* Use rebase instead of merge when attempting to reconcile dev's branch and main repo

* Init the extra vp list

* Attempt to fix push to local repo

Add new models in checkrad script for Juniper MX and Ericsson SSR (#3968)

Cherry pick from #3234

Co-authored-by: Jorge Pereira <jpereiran@gmail.com>

Add name back in?

Everything happens on upstream branch

Don't waste so much of GitHub's CPU time

Remove waffle

WS

Don't attempt to merge the main repo into itself

Update the developer's branch too

Just use a static value to differentiate tests running on different branches

Note additional permissions

Use simpler test names

Add documentation for auto-merge

Auto-merge devs' local dev branches into same upstream branch (#3966)

master and v3.0.x branches only at present with ff-only merges.

Make fr_dbuff_{ptr, ptr_const, buff, start, current, end} work with casts (#3967)

Not having some of the alternatives of the above-mentioned macros fully
parenthesized leads to the counterintuitive result that, for example,

(char const *) fr_dbuff_current(dbuff_or_marker)

fails to compile. This change corrects that.

Check unlang action not the rcode

remove dependencies on libfoo.a

it doesn't work, and needs to be fixed later

Set return code explicitly on yield

move TACACS+ to new process / listen method

some servers don't have this (yet)

it helps to initialize fields

allow new method in get_process_by_name()

which allows 'call' to work

move dhcpv4 to using new process / listen API

clean up and remove references to old proto_vmps_process

remove old maintainer who is no longer active

track may be NULL

move to new naming scheme

hoist dynamic client processing to virtual servers

so we don't have the same code everywhere else

Typo

Cleanup rbtree functions

Bump return and element count to uint64_t

Switch rbtree_flatten to using iterators

These are really asserts

Yes if len == 0 then SIGN_BIT_HIGH produces an invalid shift, but len is never 0, so...

Alter lists of vp_list_mod_t to be dlists (#3950)

v4: Amend lists of PAIR_LIST to be doubly linked lists (#3954)

* Create new list of PAIR_LIST header and initialisation function

* Amend pairlist_free() to operate on new list header

* Amend PAIR_LIST structure to work with dlist

fr_rb_node_t is now in the header.

* Amend pairlist_read to populate a dlist of PAIR_LIST

* pairlist_cmp() now works on list heads rather than individual entries

* Amend getusersfile() to put dlist headers into the rb tree

* Amend file_common to search the new rb tree of dlist heads

* Update rlm_attr_filter_t to use dlist header

* Update attr_filter_getfile() to operate on dlist of PAIR_LIST

* Update attr_filter_common() to operate on dlist of PAIR_LIST

v4: Convert lists of fr_value_box_t to doubly linked lists (#3964)

* Typo in dlist.h

* Make fr_value_box_list_t a dlist header

* Add initialisation function for list of fr_value_box_t

* Amend other structs containing lists of value boxes to use fr_value_box_list_t

* Replace fr_cursor use with fr_dcursors

* Amend exiting fr_value_box_list helper functions to use dlist properties

* Initialise lists of fr_value_box_t

* Use fr_dlist_empty() to test for empty lists of fr_value_box_t

* Alter fr_value_box_clear_value() to correctly clear child lists.

* Amend fr_vasprintf() to correctly take fr_value_box_list_t for %pM parameter

and differentiate between logging a single fr_value_box_t and a
fr_value_box_list_t.

* Use fr_dlist_talloc_free() to free fr_value_box_list_t

* Use dlist functions to walk fr_value_box_list_t

* Amend unlang_xlat_push() to take fr_value_box_list_t

* Amend definition of xlat_func_t and xlat_funct_resume_t ...

... to take fr_value_box_list_t

* Amend xlat_func_base64_encode() to take fr_value_box_list_t

* Amend xlat_func_base64_decode() to take fr_value_box_list_t

* Amend xlat_func_bin() to take fr_value_box_list_t

* Amend xlat_func_concat() to take fr_value_box_list_t

* Amend xlat_func_hex() to take fr_value_box_list_t

* Amend xlat_hmac() to take fr_value_box_list_t

* Amend xlat_func_hmac_* to use fr_value_box_list_t

* Amend xlat_func_length() to take fr_value_box_list_t

* Amend xlat_func_md4() to take fr_value_box_list_t

* Amend xlat_func_md5() to take fr_value_box_list_t

* Amend xlat_func_pairs() to take fr_value_box_list_t

* Amend xlat_func_rand() to take fr_value_box_list_t

* Amend xlat_func_randstr() to take fr_value_box_list_t

* Amend xlat_func_regex() to take fr_value_box_list_t

* Amend xlat_func_sha1() to take fr_value_box_list_t

* Amend xlat_func_##_md to take fr_value_pair_list_t

* Amend xlat_func_string() to take fr_value_box_list_t

* Amend xlat_func_strlen() to take fr_value_box_list_t

* Amend xlat_func_sub_regex() to take fr_value_box_list_t

* Amend xlat_func_sub() to take fr_value_box_list_t

* Amend case change xlat_funcs to take fr_value_box_list_t

* Amend xlat_func_urlquote() to take fr_value_box_list_t

* Amend xlat_func_urlunquote() to take fr_value_box_list_t

* Amend remaining builtin xlat funcs to take fr_value_box_list_t

* Amend always_xlat() to take fr_value_box_list_t

* Amend xlat_func_chap_password() to take fr_value_box_list_t

* Amend cipher_rsa_encrypt_xlat() to take fr_value_box_list_t

* Amend cipher_rsa_sign_xlat() to take fr_value_box_list_t

* Amend cipher_rsa_decrypt_xlat() to take fr_value_box_list_t

* Amend cipher_rsa_verify_xlat() to take fr_value_box_list_t

* Amend xlat_delay() and xlat_delay_resume() to take fr_value_box_list_t

* Amend dhcp4_decode_xlat and dhcpv4_encode_xlat to take fr_value_box_list_t

* Amend xlat_icmp_resume() and xlat_icmp() to take fr_value_box_list_t

* Amend rest_xlat() and rest_xlat_resume() to take fr_value_box_list_t

* Amend redis_remap_xlat() to take fr_value_box_list_t

* Amend redis_node_xlat() to take fr_value_box_list_t

* Amend json_quote_xlat() to take fr_value_box_list_t

* Amend json_encode_xlat() to take fr_value_box_list_t

* Correctly initialise dlist entry in fr_value_box_copy_meta()

* Amend fr_value_box_list_concat() to take fr_value_box_list_t

* Amend fr_value_box_list_aprint() to take fr_value_box_list_t

* Amend fr_value_box_list_acopy() to take fr_value_box_list_t

* Amend fr_value_box_list_tainted() to take fr_value_box_list_t

* Remove unused function fr_value_box_list_get()

* Amend fr_value_box_list_flatten_argv() to take fr_value_box_list_t

* Amend map_proc() to take fr_value_box_list_t

* Amend definition of map_proc_func_t to take fr_value_box_list_t

* Amend rlm_client map_proc_client() to take fr_value_box_list_t

* Amend rlm_csv mod_map_proc() to take fr_value_box_list_t

* Amend rlm_json mod_map_proc() to take fr_value_box_list_t

* Amend rlm_ldap mod_map_proc() to take fr_value_box_list_t

* Amend map_to_list_mod() to take fr_value_box_list_t

* Maintain list pointers when performing cast in place on fr_value_box_t

* Amend unlang_module_yield_to_*() to take fr_value_box_list_t

* Amend rlm_sql mod_map_proc() to take fr_value_box_list_t

* Initialise list entry pointers correctly

* Amend fr_jpath_evaluate_leaf to take fr_value_box_list_t

* Amend unlang_tmpl_push() to take fr_value_box_list_t

* Amend unlang_tmpl_resume() to add fr_value_box_t to new list header

* Amend unlang_tmpl_exec_wait_final() to populate box and add to list header

* Alter fr_exec_nowait() to take fr_value_box_list_t

* Amend fr_exec_wait_start() to take fr_value_box_list_t

* Amend unit_test_module mod_map_proc() to take fr_value_box_list_t

* Handle fr_value_box_list_t in list_mod_create(), map_proc_apply() ...

... and unlang_map_state_init()

* Amend xlat_frame_eval_resume() to take fr_value_box_list_t

* Amend xlat_frame_eval_repeat() to handle fr_value_box_list_t

* Amend xlat_sync_eval() to hand fr_value_box_list_t

* Amend _json_map_proc_get_value() to take handle fr_value_box_list_t

* Pass parent to fr_value_box_cast_to_strvalue() to extract children as string

* Amend mod_exec_nowait_resume(), mod_exec_wait_resume() and mod_exec_dispatch()

... to handle fr_value_box_list_t

* Comment out code which allows for a fr_value_box_t in a fr_tmpl_t ...

... to be the head of a singly linked list.

Now that lists of fr_value_box_t are doubly linked lists, this will
no-longer work.

Code left in place commented out as a reminder as to where this
behaviour was used.

Quiet GCC

Change scope of filename_tree to shutup clang analyzer Closes #3962

Use appropriate exit functions

Use correct _Noreturn specifier

Extend timeout for exim delivery - allowing for slow ci hosts (#3963)

Typo in dlist.h (#3961)

Lock critical trees in developer builds to expose iterator conflicts

Expose module prefix table

Accumulate the encoding byte in a variable and write it once (#3953)

This lets us stick with the dbuff API

Formatting fixes

rbtree pointer should never be NULL

Make rbtree_walk() users use iterators instead. (#3959)

That makes rbtree_walk() unused, so it goes away, too.
In turn, since rbtree_walk() is the only caller of the
walk_*_order() functions, they go away as well, and it
means the various "compare" functions (that do what's
usually called "visiting" nodes in the context of tree
traversal) will typically be inlined into the one place
they're used, in the iterator loop, and control flow
that's implied by what the "compare" functions return
can just be explicitly done in said loop.

Correct parenthesization for rbtree_iter_done() macro (#3958)

There's also now a test in rbtree_tests.c confirming that
rbtree_iter_done(), which one must invoke when exiting
an iterator early,  does unlock the tree if it has a lock.

Typo

Initialise all fields

Rbtree iterator fixes

Whitespace

Whitespace

Minor fixes

Correct EAP-AKA dictionary

don't retun void

shut up clang scan

rearrange for unused variable

don't use the internal namespace for these dictionary tests

fix ethernet dictionary

add assertions to shut up scan

for some incredibly frustrating reason, scan and -W errors are
substantially different on CI and on local builds, even for
the same rev of the same compiler.