Give access to type size info without explicit use of dict_attr_sizes[]

The change provides min_size() and max_size() macros and a Boolean
function, is_fixed_size(). All take an fr_type_t value and do the
subscripting of dict_attr_sizes for the user. References to
dict_attr_sizes[] other than definition and initialization have
been changed to use these, to exercise them and I believe make
the intent of the rewritten code clearer.

Add fr_pair_list_empty() to test for empty lists of fr_pair_ts (#3846)

In anticipation of migration to dlists

don't install local HTML files

just do "adoc -> man" into the local "man" directory

where they can be commited to git, like the "conf -> adoc"
conversions, and where the "install" process no longer needs
asciidoctor or pandoc

printf \t is more portable

add support for casts on existence checks on cond_eval()

the parser doesn't support it yet, and there are no test cases.
but it should eventually work.

clear out vpt for the NULL case

quiet here, too

quiet clang scan

suppress 'make install' doc if there's no asciidoctor or pandoc

more gratuitous renaming to keep people happy

and comment the code to explain why it's not *quite* the same
as the grandparent commit

Might not have any capture data

Don't free runtime-compiled regular expressions if they've been assigned to the request

Fix typo in build framework which stopped regex named capture groups running with libpcre2

Different way of doing sanity checks for regexes

install the docs on `make install` target

normalize paths before sort (i.e. unique), not after

Strip unused args from cond_do_regex

precompile regexes

so that we don't expand and compile the RHS of TMPL_TYPE_REGEX_XLAT
for every attribute we're trying to match

fix installation of "man" files.

make "install.man" also do "install.doc.man"

remove the "order dependency" on the "man" directory, and add in
a manual "mkdir".

The top-level Makefile has a wildcard rule for the man files
which just happens to match the directories, because GNU Make is
too stupid to know what's a directory and what's a file

Update Palo Alto Dictonary (#3842)

From https://docs.paloaltonetworks.com/resources/radius-dictionary.html

remove unused function

allow (&LDAP-Group == &Filter-Id)

with the appropriate casts!

ensure that the RHS is cast to the correct data type

add new attr to attr comparison functions

which removes all of the old code.

also update the "switch" handler to create conditions which
match what the cond_eval() code is expecting

forbid list / regex for switch / case

we can _probably_ allow regexes for case, but that requires
additional changes to switch.c.  The condition created there
must satisfy the requirements of the run-time evaluator

hoist more sanity checks to cond_promote_values()

and ensure that for regexes, LHS is always cast to string.

This ensures less work for the run-time evaluator.

clear box _before_ exiting

add and/or to cond_debug()

remove check_list from paircmp() functions

now that prefix / suffix are gone, it's no longer needed

remove Prefix and Suffix comparisons

nothing in the default config uses them any more.

They are historical crap and should have been removed a while ago.

define and use paircmp_virtual()

which only takes one VP, and only does comparisons to virtual
attributes.

This is the first step in getting rid of the old paircmp() API

fr_ext_debug() Fix invalid reference (#3840)

Add Key Identifier extensions to server.cnf

The Intel Wireless Daemon (iwd, alternative to wpa_supplicant) heavily
relies on Linux kernel interfaces for all its crypto, but the Linux
kernel doesn't seem to accept certificates without these extensions and
hence TLS will fail without them.

Because we're not allocating radiuses

added cond_realize_tmpl()

which does a bit more than the previous code in order to
realize a tmpl

commented out for now, but all of the tests pass

add assertion and ensure data has the correct type.

because it has always been parsed as a real data type, we
no longer need / print a cast.

we no longer allow unresolved tmpls in the cond_eval()

move to switch statements / arrays

so that types don't need to be ordered

allow bare words for xlat's

allow string/octets cast for regexes

and if there's no cast, add a cast to string.  This change
removes one more run-time check.

all tmpls have now been resolved

hoist "move attr to LHS" from normalise to promote function

because promotion is called from the unlang compiler, and
normalise isn't.

We don't really need to normalise conditions with dynamically
defined attributes.  It's useful and nice, but not critical.

ensure that the LHS of regexes are resolved

by casting them to strings, so it's explicit what we are comparing
against.

allow RHS cast

because most of the time we can just promote things as necessary

treat TMPL_TYPE_EXEC as FR_TYPE_STRING, too

resolve ALL tmpls before passing them to cond_eval()

which means that cond_eval() just has to "realize" the tmpls
before evaluating.  Which makes its life a lot easier.

We also add assertions to cond_eval() in order to force that
all of the callers have realized the tmpls.

It makes no sense to call cond_eval() on things which are
tmpl_is_unresolved().  *Someone* should have determined what that
thing is.

xlats are strings for the purpose of conditional comparisons

don't parse double/single quoted strings as booleans

we should instead use 0/1 or bare true/false

ensure that the "case" tmpl is always resolved

expose fr_cond_promote_types() and call it from unlang pass2

so that we can do all of the normalizations / checks necessary,
and not just whatever hacks have gone into the compiler

remove unused variables

remove old functions and simplify

we now have cond_cast_tmpl(), which casts a tmpl_t, and resolves
it if necessary.  It also checks for IP addr -> prefix casting,
as that depends on the values, and can't be done in fr_type_promote()

We now also have cond_promote_types(), which checks LHS / RHS
for casts, ensures that they're compatible, and calls cond_cast_tmpl()
in order to cast the LHS / RHS tmpl to the appropriate type.

As a result, some unit tests have changed.  Generally more things
are now allowed, as the compiler is smarter.

The one (minor) regression is that we don't do compile-time checks
for things like (&Attr-uint8 > 256), which should arguably be a
parse error.  Instead, the RHS is a 16-bit value, so we promote
the LHS to a 16-bit value.  It ends up being a run-time check
which is always false.

update cond_promote_types() to resolve unresolved tmpls

which changes the test outputs a bit, but that seems OK

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)