Finish documenting request_state_replace

Use the new state replace function for restoring state for children

add "limit" configuration parsing and document it

don't bother setting "single connection" in listener

it's already set by the encoder

only set the single connection flag on the first packet

RFC 8907 Section 4.3

move memory allocation to outside of the mutex lock

tacacs: Silence flexible array extension warnings

Flag enums need to be marked up with flag_enum

Else the compiler gets annoyed when you try and assign flags or'd together

Formatting

tacacs: Listener is used

Can't embed conditions in macro args

Remove unused table lookups

< 30 instances of "???" vs > 200 instances of "<INVALID>"

tacacs: We already have a hex printing function, and it doesn't use depricated sprintf functions

don't mangle request->session_state_ctx manually

now that it's an actual list, and in the pair_root, we need to
remove it from the pair_root if it's freed, and add it to the
pair_root when it's allocated or restored.

and a check of

git grep -- 'request->session_state_ctx = '

should return only one assignment in request_state_replace().
Nothing else should muck with the state

compile "auth-type"

session_state_ctx should always exist

and even if it doesn't, talloc_free() is NULL-safe

Abandon redis tests when the cluster fails to start

Don't trash the data set in &control.Tmp-String-0

be less restrictive in the decoder which helps with CI

get the synthesized packet code from the decrypted packet #4882

not from the encrypted packet.

don't close the socket after one reply packet on single connect

the "single connect" flag refers to a session, not to a particular
packet exchange.

document local clients

cleanups and more checks on corner cases

the body_xor() function just does xor, and relies on the caller
to check / set / clear the header flags

wrap macros in more brackets and update flags for the compiler

Parameterise choice of raddb for debugging

Add launcher for EAP test debugging

dictionary.aruba: Update dicto from ClearPass 6.11 (#4876)

v4: Fix up module boostrap sequence (#4883)

* Bootstrap all modules once module config parsing is complete

This allows submodules access to the fully parsed parent config data
when bootstrapping.

* Use bootstrapping for initialising SQL drivers

Moving these set up steps to bootstrap means the driver is fully set up
by the time the sql module is being instantiated and so it is safe to
start connections.

enfore connected sockets for TCP, and check for hash table

before dereferencing it

print the correct amount of data remaining

Typos

clean up error messages, and print actual value

"secretlen" can't be zero, either

relax checks.  it seems clients don't pay attention to this.

use correct names

use and look up local client in virtual server

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

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

Ensure each LDAP test that updates the directory uses a different user

Avoids clashes when tests are run in parallel

rpm:  Split smtp and imap into separate packages

This means we don't have to pull in curl or our wrapper library unless rest/curl/imap are used

rpm: Need to use %define outside of the macro files

rpm: Typo

rpm: Simplify specifying optional build components and libraries

rpm: Remove lua config files if we're not building lua

rpm: Use the RPM macros for calling utilities

rpm: Package rlm_lua separately

rpm: More reordering

rpm: EAP-PWD is not experimental, and is always supported by the versions of OpenSSL we require

rpm: Remove dup EAP-PWD entry

rpm: Only install the mruby example scripts with mruby

rpm: TNC got removed ages ago

rpm: List each .so individually as this appears to be the only way to fix the duplicates

We were seeing the SQL and LDAP .so files being included in both the main package and SQL/LDAP packages, which was adding incorrect dependencies for MySQL/openldap to the main FreeRADIUS RPM

rpm: Use defattr to specify default dir permissions so we don't need to manually set permissions on each %dir

The vast majority of config files have the same permissions (640) so just set that as the default

rpm: Add support back for symas flavoured openldap

rpm:  Just use the standard with/without macros

free the filter on error path.  Fixes #4881

remove unnecessary restriction

Index the headers starting with 1, just like all the data. (#4880)

Allow redis cluster time to propagate changes after failover

allow a debugger via FR_DEBUGGER="lldb --" or similar

Annotate remaining tainted data issues (CID #1519002, #1503896) (#4879)

Sequence ldap tests that update the directory to avoid conflicts

Use different keys in each cache test

Allows for parallel runs of cache tests that use redis without the tests
interfering with each other.

SQL driver needs to be instantiated before starting connections

CI: Correct REST port env variables

Very short cache ttl can cause issues with CI

When running under memory sanitizers, tests can be slow enough for the
cache entry to expire

Ensure cache resources are freed before returning

Tidy and fix up redis clust_node_fail test

Resequence tests so redis ones run sequentially

Redis tests all call scripts to reset the same cluster - so cannot run
in parallel

Tidy and correct redis_ippool test

CI: Tests need a redis-cluster, created with cluster-create script

The script creates nodes on the local host and is not designed to
operate on a remote host

CI: Check that redis nodes are up before creating the cluster

The create-cluster script starts redis-server with the --daemonize
option and on slow CI hosts, this may not be fully ready if `start` is
immediately followed by `create`

CI: Fix up redis create-cluster script for version on Ubuntu 20.04

Allow for \1f style escape strings inside single quotes

Fixup ldap tests - missing dn doesn't actually return an LDAP error

Replace != with !( == ) in ldap module tests

CI: Correct test ldap data

- Enums now need to be quoted strings
- Test looks for &control.Reply-Message to be set

post-auth is now send

uc is a GNU Make Standard Library function, not builtin

Don't call xlat_process_return() with node->call.func == NULL

just return

let's not go to "next" on pushed child or pushed unlang

not all failed expansions will produce fr_strerror()

so print them as warnings

this should now pass

It should have been commited with "go to "next" xlat node on RESUME"

let's not use vpt after it's been freed

and clean up some of the other code to avoid leaks on error

fix "success" policy to use new edit sections

we don't need to do "if" checks for parent, we can just use
the "=" operator, and let that fail-safe.

We also revert the "subrequest" test to the previous version.

The next step is to figure out why changing the xlat expansions
broke the "success" policy.  But we can check that tomorrow,
once the CI tests pass again.

go to "next" xlat node on RESUME

previously, if an intermediate xlat node yielded, it would resume
and _not_ go to process the next sibling.  So make xlat_frame_eval_resume()
call xlat_frame_eval() when it's done, to resume.

Then fix up the xlat indentation, through the simple expedient
of saving the indent level, indenting, and restoring the indent
level on success/failure.  This is easier than trying to figure out
the proper indent/exdent macros for every possible code path.

Then also fix up the keywords.subrequest test, which returns an
extra Packet-Type attribute from inside of the subrequest.

don't lose casts when they're explicitly given

(foo)%{xlat..}

was being set to just

%{xlat...}

and that's wrong.  :(

expanding the expression may fail

send reply if the reply is set

which can be set in a policy or in a module

Add -p to mkdir coverity_tool/config. (#4877)

Handle unchecked returns in src/protocols/dhcpv4/encode.c (CIDs below) (#4874)

CID 1503896: change fr_dbuff_in() to FR_DBUFF_IN_RETURN()
CID 1504021: annnotate; a preceding FR_DBUFF_REMAINING_RETURN() assures
             that there's enough space for the PEN and data-len1.

Start using modeling for Coverity (#4853)

This involves
* making the coverity workflow build the model
* adding model source file(s)