Move libcurl init to global_lib framework

Having pre-generated certificates saves very little time and breaks CI regularly

Rename some more global_lib functions

Table formatting in libldap

Move global_lib_init and instantiation calls inside the module code

Minor fixes

v4: Framework for configuring, initialising and freeing libraries (#4541)

* Add endforeach macro to close code blocks opened by foreach macros

* Define framework for initialising and freeing libraries

Reading library configuration from a subsection of a global {} config
section.

* Initialise global list of libraries

* Register handlers for "lib" symbols to auto initialise and free libraries

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

Check fr_sbuff_in_bstrncpy() return in xlat_func_rpad() (CID #15039490) (#4515)

Cast to void where we know a call will work or just want side effects (#4537)

This deals with the following CIDs: 1469133, 1503911, 1503964,
1504282 (#1 of 5)

Deal with unchecked return value Coverity reports (#4532)

This deals with the following CIDs: 1503943, 1503949,
1455233, 1503981

Remove needless checks from xlat_config_escape(). (#4536)

The VLA escaped[] is declared to be large enough to hold the text
from the value box even if every single character needs the
largest possible encoding (MIME, which takes three characters to
encode one). The bound checks will therefore never cause an error
return.

Add some more text about identity privacy

Use eapol_test-build.sh to build eapol_test on deb CI

Update test certificates

Always build jlibtool with debugging symbols

Add $(E) suffix for executable targets and $(L) suffix for dynamic libraries

Add rule for .js files and fix Linux specific extension substitution

Use BUILD and TARGET not HOST and TARGET

fix final issues

Remove module bootstrapping from conf parsing (#4538)

set and print quotes for string values.

tests for unary not

append argument as group, not as raw node

validate data type where possible.

This doesn't catch all of the cases, but it catches enough to be
useful.

Fix typo (#4531)

just pass all of the tmpl_rules to the xlat tokenize functions

so that they all take the same thing, and so that we have better
control over non-attribute parsing

check fields only when they're valid

lots of fixes.

Set flags better.  Do more resolution of binary operations.
We still need to add xlat_expr_resolve_unary()

update tests to show that most pure things can now be purified.

comments

allow for instantiated xlats to be put into other ones

tmpl_tokenize() will call xlat_instantiate_ephemeral() on things
it sees.  However, if we then hoist that xlat out of the tmpl
and into the main xlat tree, we can't re-instantiate the node.

Rather than adding flags to change the rest of the code, we just
catch that here.  This should likely be cleaned up later

allow for pruning of xlat walk

print data types, too

set more flags correctly

print out indexes, which can help at times.

check for xlat / exec tmpls

which should really be handled, but let's debug other code first.

more debugging

clean up flag handling

things are pure unless marked impure.

check input allow_unresolved once per loop, and not in some of the
individual "case" statements

CI FreeBSD: Bump vmactions/freebsd-vm and prune before pushback (#4533)

Pushback of the working directory over rsync sometimes fails with
"ssh_dispatch_run_fatal: Connection to A.B.C.D: message authentication code
incorrect" irrespective of the MAC used by the SSH connection.

Cleaning the unnecessary parts of the working directory should reduce the
chances of such failures until the cause is determined.

We keep any changes to the eapol_test build since this is cached between runs.

Raise limit for number of Redis xlat components (#4534)

Correct arguments in the enscripten target

Init proto

Typo

Fixes for tmpl_resolve

add void to shut up compiler

add custom resolve callback handler

add more debug output for xlat_purify

which can help track down issues with the flags

purify arguments, even if the function can't be purified

and hoist can_purify up to the parent.

We should really have a merge flags just for XLAT_FUNC, which
would cut down on some of this repetition.

move more logic into tmpl_resolve()

rlm_expr: Fix typo with double '&' (#4525)

head can be NULL

update comments

just use talloc_const_free()

just use talloc_const_free

Placate -Wincompatible-pointer-types-discards-qualifiers (#4523)

re-enable purify.txt tests

With some hacks to do xlat thread instantiation only when running
the xlat_purify tests.  If "-p" is passed in other circumstances,
you will get weird and esoteric errors.  Oh well.

That will be fixed once we get rid of conditions, and replace them
with the new xlat expressions.

try to resolve tmpls, and give better errors

resolve function arguments before the function itself

otherwise the function may steal the arguments into its internal
structures, where they cannot be resolved.

lists are resolved as attributes

initialize more fields

set more flags properly, and try to resolve things.

There's (as yet) no "resolve" callback for the xlat functions,
so we can't parse things like

&Service-Type == Framed-User

The LHS will be an attribute, and the RHS will be TMPL_UNRESOLVED.

The interpreter currently does this resolution in all kinds of
complex pass2_fixup functions.  We will need to move those fixups
to common code, and call the custom resolvers from xlat_resolve()

let's assert at the right point.

move prototype of xlat_purify()

heads start out as pure

initialize more fields

only set destructors if they're going to do work.

This papers over certain issues with pure functions, and purifying
them.  But arguably pure functions should use no instance data,
and should take no destructors.  If they do, they're not pure.

print flags, and don't assume XLAT_TMPLs are all attributes

let's set can_purify here, too.

Note that we don't deal well with an N-ary operation which
starts off with pure children, and then has impure children
later appended.  That needs a bit more thinking

Fix permissions for mods_config/sql extras in rpms

Don't assert on resync Close #4522

rlm_couchbase: Fix several issues (#4517)

Check return values in rlm_date.c (CID #1503947, #1503971) (#4520)

Catch the non-OSX cmd->install_path == NULL case (CID #1504302) (#4521)

add xlat_purify() function

for now, it calls unlang_interpret_synchronous().  We'll fix
that later.

shut up clang scan

on second thought, this isn't necessary

double-check and fix flags.

We now have a "can_purify" flag.  This is set to "true" for
pure functions which have pure arguments.  And for alternations
which have pure arguments.

%% is set to pure.  XLAT_BOX and XLAT_LITERAL are pure, but
they are not "can_purify".

The result is that we tell if there are portions of the tree
which need additional work for purification.

this loop isn't necessary.  It's done by xlat_copy.

update APIs to allow for "evaluate only one node"

instead of looping through everything

Add util-linux as dependency for freeradius-config rpm

Required to run the certificate generation post install script

Always put something in temporary buffer, if only "" (CID #1503992) (#4519)

From the point where src->dst_type is determined to have fixed
size, fr_value_box_from_substr() works with a local copy... but
if fr_sbuff_remaining() returned zero, nothing was written in the
local copy. With this change, in that case, the local copy is made
an empty string, NUL-terminated.

Add a second run of installing rpm build dependencies

It has been observed that not all dependencies get installed first time
on rocky 8

Revert "clean up internal requests if they have no parent"

This reverts commit 9f8b9bb3ad94687b378f5539a1010310a089bc80.

the interpreter should still clean up some requests which have
no parent, as there are very much cases where those are leaked.
however, until we track all of those issues down, let's not
break CI

clean up internal requests if they have no parent

Specifically for trigger_exec(), which can be called with
a real interpreter, but no parent for the "request" which is
running the trigger.

Revert "put yielded increment / decrement into the callbacks"

This reverts commit a3aee1615ab0986cebe596c536e840e667de2943.

intp->func.resume() is called when a request _starts_ running
for the first time, too.  Which means that we can't decrement
the "yielded" count, because it's already at zero

add "help" target for laziness

used

remove old code and just use unlang_module_yield_to_section()

there are a few other places which use unlang_interpret_synchronous()
which could likely also be converted.

check for a new event list only once.

put yielded increment / decrement into the callbacks

which makes the main interpret loop a little clearer

add dict in preparation for xlat_purify

Revert "close on error. Fixes #4518"

This reverts commit 4040e326d7bc637f55cdb029ea3c3127de3d2a96.

this is done by cf_stack_cleanup(), and there are many other paths
in cf_file_include() which do "return -1" without closing the
file.

radsniff: Add '-Z' option allowing to dump the packets (#4504)

* log: Fix  fr_pair_list_log() to accept level identation parameter

* log: Fix fr_log_init_file() 'fd' set

* log: Fix fr_log_close() 'dst' check

* radsniff: Add '-Z' option allowing to dump the packets