use fr_type_promote() in condition tokenizer

and update one test:  we can cast a "string" to "ipaddr"
for comparisons

add fr_type_promote()

which returns what data type should be used when we have two
different data types in a comparison.

Note that the output data type _may_ be different from either
of the input data types!

Note also that there may be failures at run-time, if the values
don't fit.

hoist checks to common code

don't parse "Foo-Bar" as an attribute for tmpl_is_unresolved()

v4 requires "&" prefixes.  So "&Foo-Bar" is tmpl_is_attr_unresolved()

clean up checks for regex

it's never allowed on the RHS.  and casts aren't allowed

fix name

have tmpl_cast_set() call fr_type_cast()

and update tests to match.

add fr_type_cast()

which checks if you can cast type src->dst

move 'size' to be with unsigned ints

Fix rlm_smtp

Typo

More fixups for the raw api in dhcpclient

Move remaining list heads out of fr_radius_packet_t into the request_t

Use pair list abstraction macros in more places

Request can be const when passed to log_rdebug_enabled

Quiet clang scan

use FR_TYPE_VALUE

More dhcpclient fixes

typo

remove PACKAGE_* definitions from config.h.in

likely added by running "autoconf" locally, instead of doing
"make reconfig" at the top level.

We don't use PACKAGE_* definitions for anything, and they
should only be defined at the top level. i.e. the package is
"freeradius-server", and not "whatever module this happens to be"

remove c->cast from everything

we never allowed casts for existence checks

remove c->cast from switch statements

remove c->cast from pass2_fixup_cond_map()

remove all references to c->cast from conditional evaluations

which means that we only rely on the LHS tmpl casting

ensure that c->data.map->lhs->cast matches c->cast

move more normalisation to cond_normalise()

update error messages

check tmpl types for casting

there's no reason to cast a regex to anything.  casts only apply
to data types (data, attr, exec, xlat)

and update tests to match

Separate requests from packets in dhcpclient

request|reply_pairs already used

Remove dependency on packet->vps for radsnmp

Fixup radsniff to use packet lists distinct from packets

Fix packet verify to take a separate list pointer

...and some more

Make _fr_dbuff_init() and FR_DBUFF_TMP() set the buff field (#3837)

...and some more

Clean up uses of request->state

Pass list headers (separate from packets) into radius encoding/decoding functions

This is with a view of removing packet->vps, and moving it to request_t, after which we can maybe merge all the list heads under a single root.

Better names for some tmpl functions

Fix fr_packet_log to take a separate list

add HW-Ext-Specific RADIUS attribute for bounce port using CoA (#3833)

catch more edge conditions in cond_eval()

and fixup parenting after normalization

Add code that honors the "max" field in fr_sbuff_uctx_file_t (#3786)

Note: buffering in the FILE may cause it to read more bytes
than the sbuff will be allowed to access.

Simplify internal_encode(). (#3830)

With a child dbuff to encode data into,
1. We needn't repeat ourselves with FR_DBUFF_RESERVE().
2. The length of the data when it's encoded is just the
   used portion of the child dbuff.

We also switched to fr_dbuff_in_memcpy() to get the data length
into the dbuff; it gets rid of the clutter of FR_DBUFF_TMP().

Propagate shifted from parent to child (#3828)

Add fr_value_box_from_network_dbuff() (#3738)

With this change, fr_value_box_from_network() is made a wrapper
around fr_value_box_from_network_dbuff()

make cond_eval() iterative

and set c->parent correctly in cond_normalise()

Add FR_DBUFF_COPY() analogous to FR_SBUFF_COPY() (#3834)

"next" is no longer valid

add more optimizations for

FOO || true

in addition to pre-existing

true || FOO

fix up talloc parentage

children are talloc parented from the parent.
next siblings are talloc parented from the older sibling.

The result is that we can free c->next (if necessary), and
ensure that all of the child *and* next siblings are free'd

add parent to fr_cond_t

remove CONF_ITEM from fr_cond_t.

it's not needed by anyone

pass CONF_SECTION instead of getting it from fr_cond_t

Fix legacy tmpl function names

TMPL_TYPE_UNRESOLVED should never occur

simplify cond_eval_tmpl()

remove unused parameter

rename COND_TYPE_EXISTS to COND_TYPE_TMPL

which is rather a lot more correct

more cleanups

clean up optimizations for COND_TYPE_CHILD

parent all of the conditions from the ctx

instead of from the parent fr_cond_t.  This change means that
we no longer need to call talloc_steal(), which is slow.

add AND / OR to pass2_cond_callback()

don't re-use variable name

make && and || into conditional types

so that the fr_cond_t has less magic in it

make format string match parameters

Happy new year

Add the ability to disallow filters in the tmpl_tokenizer

Fix debug output for network/worker instantiation

Remove bad asserts

Minor tweaks to eapol_test-build.sh; Disable shellcheck SC2223 (#3826)

Fixes for issues SC2086 and SC2236 from Super Linter output...

In scripts/ci/eapol_test-build.sh line 41:
: ${BUILD_CONF_DIR:="$(dirname $0)/eapol_test"}
  ^-- SC2223: This default assignment may cause DoS due to globbing. Quote it.
                               ^-- SC2086: Double quote to prevent globbing and word splitting.

In scripts/ci/eapol_test-build.sh line 51:
    if [ ! -z "${WHICH_EAPOL_TEST}" ]; then
         ^-- SC2236: Use -n instead of ! -z.

Disable SC2223: "This default assignment may cause DoS due to globbing. Quote
it" since this is easy to trigger and rarely a concern.

rlm_redis_ippool_tool: Link provided OpenSSL (#3827)

FreeBSD build fixes and basic CI build with eapol_test checks (#3825)

* FreeBSD build fixes

* Fix eapol_test build options

Enable CONFIG_EAP_SIM on FreeBSD.
"n" means yes!

* Remove bashisms from eapol_test-build

* CI: FreeBSD build

Pass &vp to radius_pairmove() rather than vp in add_vp_tuple() (#3821)

This doesn't include code testing rlm_mruby, but it is consistent
with the prototype of radius_pairmove() and with other uses of
the function (e.g. in mod_vptuple() in rlm_python/rlm_python.c).

autoconf fixes for --with/--without options (#3820)

* configure.ac fixes for --with/--without of core modules

Rewrite the module name for the --with/--without test so that it matches the
mangling performed by autoconf.

Declare AC_ARG_WITH for the module name to avoid spurious warnings about
invalid --with/--without options.

* configure.ac fixes for --with/--without of rlms

Declare AC_ARG_WITH for the module name to avoid spurious warnings about
invalid --with/--without options.

CI package tests: Workaround to permit builds with Fedora Rawhide (#3818)

clean up and fix build rules

remove double //

because GNU make is too stupid to know what's a filename and
what's a string

PROGRAM_INSTALL is jlibtool, so we need that, too

add missing ${Q}

update and fix rules for installing "man" pages

don't use $(R)/, it already ends in /

update to be correct

this is done automatically by the framework

ensure that R has a trailing /

remove unused argument.  Fixes #3817

minor simplification

Don't use fr_sbuff_in_bstrcpy_buffer log messages may not be talloced

rename strerror to error in log_request_error

Pass --embed only if python3-config supports it

Pass embed to python3 configure otherwise we don't get the '-l' arg for python back

Allow insertion of new dlist items at a specific position

rlm_mschap: Fix wrong cast (#3815)

let's actually include the relevant make file

and don't print out the commands we're using

move "man" pages to antora

and add rules to build / install the "man" pages

fix references now that "unlang" has moved

ability to pass args to antora

describe the outline

add unlang/ to xref prefixes

now that we've moved the pages