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.

cross-check name / number for attributes

we previously looked up (and checked) duplicate attributes by name.

We now check if attributes of the same number are compatible.
If not, error out.

As a result, a number of dictionaries have to be updated in order
to have compatible types.  Mostly by commenting out one of the
incompatible types.

fix typo

Try and track down issue with redis cache driver

Don't add conf pairs to sections twice. (#3955)

control sockets don't have dictionaries

add NDEBUG to shut up compiler

Quiet GCC

use new names

rework arp to use process_foo methods

it still uses the entry_point_set() method until such time as
all of the other protocols are cleaned up

add provisions to load process_foo OR proto_foo_bar

add bootstrap and instantiate methods

lowercase field names

it's 2021.  "foo.bar" is much easier to read than "Foo.Bar".
Random Shouting Is Bad.

rename and use new structures

start of new process state machines

update to new standards

we don't always need a "set priority" function

re-run EAP tests after re-building server / util libraries

regenerate from the source

more cleanups

remove "load dictionary" code from bootstrap phase.
Replace it with an assert that the dictionary exists.

move the "compile" function back to the instantiate phase.
it was erroneously moved to the bootstrap phase a few commits ago.

if there is a compile function, assert that the dictionary it uses
is the same as the dictionary used for the namespace.

load the compiler function by namespace, not by server second
name.

Fix documentation flag and re-enable -Wall for GCC

Reorder warning args

Fix other warnings

Disable -Wpoison-system-directories

Fix pedantic warning

Fix -Weverything check

add const

remove unused assignment

call cf_section_has_parent

add cf_section_has_parent()

hard-code some dictionary rules

which makes the rest of the code much simpler, during the cleanup
process.

the "control" server has no dictionary.

eap-aka-prime, eap-aka, and eap-sim all have the same root
dictionary.

Restrict deletion during iteration to in-order. (#3952)

This requires different iterator structures for inorder traversal,
but it does enforce the constraint at compile time.

rbtree_iter_done() is now a macro using _Generic(), so at least that
doesn't have to change.

Relax asserts if we're in a policy section

all foreign attributes should be qualified by full path

add namespace_on_read() callback

which loads the dictionaries VERY early in the process

Be more explicit about failing when the dictionary isn't set

push the SUB section rules, not the PARENT section rules

This bug has been the bane of my existence for quite a while now.

move namespace_config around, and rename things

remove FR_TYPE_ON_READ

minor rearrangement

update comments

add back code

Init ctrl

Add TLS engine configuration code

Indent

remove unused assignment

Add rbtree_iter_delete() and corresponding test. (#3949)

rbtree_iter_delete() deletes the current node in an iterator
and remembers the next one; rbtree_iter_next_inorder() notices
when that's happened and uses the saved node, otherwise it
behaves as usual.

Correct typo (#3951)

Double semi-colon

Fix cf_remove to auto covert input args

remove unused assignments

remove unused variable

remove FR_TYPE_ON_READ and rename function

rely on rule->on_read, instead of type flag

move on_read callback to separate struct element

in preparation for getting rid of FR_TYPE_ON_READ

add section / pair / data in alloc functions, not elsewhere

on review, most of the calls to cf_section_add() were wrong.
the child section was already added to the parent section by
_cf_section_alloc().  There were many, many, calls to
cf_item_add() or cf_section_add() which resulted in the
section being added twice.

However, cf_pair_alloc() and cf_data_alloc() did not add the item
to the parent.

In the interest of less code and fewer errors, we've removed
cf_section_add() and cf_pair_add().  The calls to cf_item_add()
are now done only from the alloc functions.

We still have many calls to cf_data_add(), because that function
is in fact allocating *and* adding the CONF_DATA items.

remove crap code

it makes no sense to look up the parents name in the parent
CONF_PARSER?

Also add the new section to the parent AFTER the callback is done

Add pre-order and post-order rbtree iterators. (#3945)

The test we have is based on static test data with the values
appropriate for pre and post iterators determined by hand. We
can generate additional test cases as desired.

Fix rlm_couchbase to match new dlists of map_t (#3947)

plus correct definition of list parameter in mod_json_object_to_map()

Extend timeout on SMTP delivery tests to allow for slower exim performance (#3948)

v4: Change lists of CONF_ITEM to be dlists (#3946)

* Replace next and child pointers in CONF_ITEM with dlist entries

* Initialise dlists

* Use fr_dlist_empty() to identify empty children lists

* Replace uses of ->next pointers with dlist functions

* Don't add a section to it's parent twice

It's already added by _cf_section_alloc()

* _cf_item_add() only adds single items to the parent's children list

* Replace uses of cursor with dlist functions

* Remove cursor from CONF_ITEM structure

* Correct __cplusplus wrapper

Have the config file parser check if the files exist

Print extra operand information when debugging

Revert "disable a number of raw tests until we fix the underlying issue"

This reverts commit 1a0623b0cff8691551dcb701d22d2896c207e3a1.

We must parse the namespace before allowing the rest of the virtual server to be parsed.

disable a number of raw tests until we fix the underlying issue

So that we don't block others who wish to do work.

It seems that commit a6055f25e52 changed various assumptions
throughout the code, which has these side effects.  We need to
track down the root cause, and fix the assumptions.  Then
re-enable the tests

set dictionary namespace in bootstrap

so that when we compile X that calls Y, both X and Y have
their default dictionaries set

print out virtual server name, not dictionary name

no need for duplicate checks

Revert "Fix sign comparison"

This reverts commit 72fd3fbc45a2600a89be9a8eb9b7c472d6f27ea5.

Fix sign comparison

add cast

Free the child's session-state ctx if the state_child_entry_t is freed

remove extraneous copy

v3 puts the certificates into FR_EX_TLS_INDEX_CERTS.  v4 doesn't.

so there's no need to create the certs in the wrong context,
then copy them to the correct context, then delete the certs.
We can just create them in the correct context.

Move cert_vps variable into the condition in order to highlight
the fact that it's a temporary variable, used only to limit the
list of certificates being printed.

This change doesn't affect the current issues of EAP-TLS not
caching certs in session state, or it leaking the certs.  But
this change is not wrong.

use the ctx passed in, not hard-coded request_ctx

use parent sequence number in child name

so that we don't have multiple children labelled with the same
name

vp_date is fr_unix_time_t, not fr_time_t

unfortunately the C compiler checks on underlying data types
(uint64_t), and doesn't give warnings about typedefs.

We might want to have automated tests which check the code
for these kind of errors.

vp_date is unix time, not fr_time_t

clean up old code

after some inspection, the code was left over from other things.
Before we go crazy adding new features, let's clean things up
first.

added enum / name for process_foo

so we can have processing modules for each protocol

Move fr_pair_list_sort() to use fr_dlist_sort() (#3943)

Convert lists of map_t to doubly linked lists (#3913)

* Define header of list of map_t

* Add dlist pointers to map_t structure and add macros to simplify calls

fr_map_list_init() - consistent way to initialise a dlist of map_t
fr_map_list_head() - return head of list cast to map_t * - simplifies
  code when accessing elements of the head item in the list.

* Set child of map_t to be a list header and initialise list on allocation.

* Alter structures that contain lists of maps to use fr_map_list_t

* Operate directly on list head in gext

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

* Pass fr_map_list_t * between functions

* Use dlist functions for traversing lists rather than ->next or cursors

* Ensure all new fr_map_list_t are correctly initialised

* New version of map_sort() for fr_map_list_t

* Use dcursors in place of cursors for tracking in fr_map_list_t

* Amend calls to map_afrom_cs to pass fr_map_list_t *

* Use fr_dlist_insert_tail() to add a map_t to fr_map_list_t

* Ensure fr_map_list_t is initialised when allocating vp_list_mod_t

* Correct calls to functions which want a fr_map_list_t *

* Handle references to the first (only?) map in a structure.

* Cannot just point two map items at the start of a list now

* Correct way to remove an item from the start of fr_map_list_t

Make dbuff usage more idiomatic (#3940)

Only set TEST_SERVER variables for feature tests (#3938)

Prevents builds without the feature test servers from attempting to test
against servers that don't exist.

rbtree: Add inorder iterator

rbtree: Parent should be NIL not NULL

Seems like we have a mixture of comparisons.  Some areas compared parent to NIL and some to NULL.

move detail to the new location

for now, proto_detail_process is in src/listen/detail,
as a reminder that it will go away as soon as possible.

move arp to new location