final step of edit fixups and tests

typos, and work around merge issue with hex <=

update test until we find time to track down other issues

warnings, so they don't eat the real error

more "rewrite to state functions"

rename for clarity

avoid using lhs.vp where possible

in preparation for making LHS into a cursor

more cleanups to unify and simplify code

allow multiple selectors for leaf attributes

notes so that we don't forget

remove unnecessary code

more cleanups

move expand RHS list to its own function.
Remove in_parent_list as redundant

more cleanups to use only one cursor for the LHS

split up functions some more

more cleanups to edit state functions

more rearrange to state functions

move to function-based approach

in preparation for further breaking down the code into smaller
functions, with less if / then / else logic.  And then to move to
using dcursors

%{...} can be T_BARE_WORD

No need to copy back files from FreeBSD VM

Check return of fr_pair_list_copy() (CID #1514672 and 1514673)

minor rearrangement

we should check for bad LHS before parsing the RHS

we should use test_fail, not fail for most situations

try to get tmpl_dcursor insert / remove working

commented out because it doesn't work, and there's not yet time
to track it down

print program to run after printing logs

typo

notes on how to quickly find failing tests

fix fr_pair_iter_next_dcursor_value() and friends

more cleanups

perl -p -i -e 's/&request\./&/' src/tests/keywords/*

except for "comments", because &Reply-Message is likely parsed
as &Reply - &Message.  We'll have to track that down again

simplification

perl -p -i -e 's/&request.Tmp/&Tmp/' src/tests/keywords/*

update doxygen

Bump FreeBSD vm version

Don't use uninitialized variable

Annotate false positive uninit_use (CID #1503898) (#4719)

find is uninitialized and only has a small portion of find.addr.inet
set in fr_redis_cluster_pool_by_node_addr(), but then all of find.addr
is assigned to spare->pending_addr, hence coverity complains. It turna
out, though, that cluster_node_connect() only uses the part that did
get set.

Typo (#4714)

copy/paste strikes again

Deal with complaints about len parameter of fr_udp_checksum (CIDs below) (#4712)

1504068, 1503957, 150468: use the value l4_len directly so it's not
                          fed through two byte swaps and hence tainted.

Annotate xlat uses of known good values from dlsts (CIDs follow) (#4708)

By the time these functions are called, the dlists are known
to have the needed list items.

CIDs: 1506634, 1506635, 1506636, 1506637, 1506638, 1506639,
      1506640, 1506641, 1506642, 1506643

Cast remaining non-checked fr_pair_list_copy() calls to void (CIDs below) (#4682)

CIDs: #1504051, #1507255

Consistently use bracket, not brace, for annotations (CID #1448182) (#4704)

Typo

one more sbuff_tests.c annotation (CID #1504019) (#4699)

Missed onefr_sbuff_out_bstrncpy_until() that coverity doesn't
realize will always put *something* in the sbuff

Annotate false positive tainted_data (CID #1503893) (#4713)

In fr_dhcv4_raw_packet_recv() (is there a reason for that
spelling?), coverity claims the downcast of packet->data
in the fr_dhcpv4_packet_get_option() call taints the contents
of packet->data, but it's cast to, and the called function
takes, a const-qualified pointer, so in what sense can it be
tainted?

Deal with coverity tainted data defect in mod_track_create() (CID #14… (#4718)

* Deal with coverity tainted data defect in mod_track_create() (CID #1469134)

Added a test to make sure option + option_len doesn't extend off
the end of the packet. Since in other cases, coverity doesn't
recognize the range check that it asks for, we annotate it, too.

* Brackets

Co-authored-by: Arran Cudbard-Bell <a.cudbardb@freeradius.org>

move first use after NULL check (CID #1503920) (#4715)

Check for NULL return from xlat_register() (CID #1507059) (#4721)

Barring issues with args, xlat_func_args() will dereference the
xlat_t * handed it, and xlat_register()'s callers do check for
error return, so it makes sense to check what xlat_register()
returns and return error in that case.

Annotate false positive uninit (CID #1503938, #1504037) (#4716)

Both instances appear in the same printf() call.

1503938: flags is large enough that fr_dict_attr_flags_print()
         will not run out of space, the only way it can return
         an error.
1504037: NULL is passed as the ancestor, and coverity doesn't
         complain about evaluating da->type, so it must not
         think da is NULL. Given that, the fr_dict_attr_oid_print()
         call here also can only fail by running out of room.
         oid_str is 512 bytes, but here the length depends on
         da->depth. It appears that 512 bytes suffices in practice,
         but we'll check the return value and exit after logging
         an error if it proves not to.

Annotate false positive tainted_data (CID #1243443) (#4717)

Coverity doesn't realize that eap_validation(), which is called
before eap_identity(), range checks the length.

Wordsmithing

sbuff-ng

Ensure out is set on all return paths

Use the FR_SBUFF_SET_RETURN macro

Use the FR_SBUFF_RETURN_ERROR macro

Switch the majority of xlat functions and condition functions to returning fr_sbuff_error

Have base* functions return fr_slen_t

Move regex_flag_parse to fr_slen_t

Use fr_sbuff_error in more places

Fix unsafe dereferences of fr_sbuff_current

Add significantly simpler error handling logic

Rework filter parsing

Now prouces fr_slen_t offsets which are increased by one...

Rework is to prep for more complex filters in future...

Fix output format so jump to line works with vscode

Add safe macros for switching over sbuffs

Use accessors for more tmpl fields

Update example site with correct location of session-id for eap_aka_sim

Scheduled fuzzing: Update src/tests/fuzzer-corpus/dhcpv4.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/radius.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/util.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/tftp.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/vmps.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/tacacs.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/dns.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/dhcpv6.tar

Annotate false positive tainted_data (CID #1451665) (#4709)

This one is interesting. fr_radius_packet_log_hex(), called
if NDEBUG is defined and the debug level is high enough, in
turn calls fr_nbo_to_uint16() and fr_box_octets(). The results
are used in log messages, Typically when bytes are swapped, the
entity to which the result is stored is considered tainted, but
here coverity thinks the data being read is tainted. That
propagates back to place where the defect is claimed.

More convert from 'update {....}' to 'edit' against src/tests/keywords (#4710)

Annotate false positive tainted_data (CID #1503921) (#4711)

By construction, 0 <= offset <= 0x3fff, so it is in range as
a subscript for fr_dns_marker.

typo

Better way to check test server is defined

only create VPs for := and =

reference the correct tmpl

more "convert to update"

list may be empty

convert whitespace to tabs

add help target

more "convert to update"

test index assignments

[1] is not the same as NUM_UNSPEC

forbid [*] and [#] on the LHS of edit assignments

and update the scripts in all.mk to handle [*] in output

add CONF_ITEM* to unlang_t

so that we can track where each instruction was created from.
and instructions generated from CONF_PAIRs can also get queried
for their filename and line number.

ensure that the group always has a CONF_SECTION pointer

Initialize rather than memcpy() (CID #1508486) (#4707)

Add Active Direcotory test server variable

Define tests for Active Directory LDAP server

Run tests on Active Directory LDAP server

Add config for testing Active Directory LDAP server

Add samba to test build

Deal with (possibly false positive) untrusted values. (CIDs below) (#4701)

1445221: length is checked for consistency with packet length;
         annotated.
1448175: pulled the checksum validation into the block declaring
         udp_len, to make it clear that it *is* range checked.
         Annotated anyway, because coverity hasn't noticed other
         such clear range checks.
1448175: len is range checked just before the call to memcpy(),
         which is what coverity says should be done. Annotated.
1503937: the check of p effectively sanity checks count; annotated.
1503954: before the fr_pair_tlvs_from_network() call, option_len
         is indeed checked; annotated.
1503968: length is checked; perhaps coverity doesn't recognize
         that (option + 4 + length) > end is equivalent to
         length > end - (option + 4). Annotated.

Co-authored-by: Alan DeKok <aland@freeradius.org>

Correct print_hex_data() parameters in fr_vmps_print_hex() (#4700)

For the loop printing the trailing portions of the packet to be
correct, length must include the four-byte id and two-byte length
as well as the following data. id and length are explicitly printed,
so print_hex_data() would presumably show what follows, but that
would start at attr + 6 and only be length - 6 bytes.

Add test server variable for persistent search tests

Define tests for persistent search LDAP server

Run tests for persistent search LDAP server

Add config for testing persistent search LDAP server

Add 389 Directory Server to test build

Add ldap_sync tests to test target

Add test server variables for RFC4533 ldap sync tests