Unused macros

Fix uninitialised fields

Make NULL escape rules static

Various stupid issues in value.c

Unused

More stupid warnings

Fix stupid vla warning

Pedantic

Used arg

Just toggle the individual bit

Fix stupid -Wassign-enum warning

Bad function cast

Fix bad terminator in par test arrays

More zero length array struct members

Unused macro

Fix -Wmissing-noreturn

Dead code

Actually used

Extra semi

Mark cap table len as static

Not inline

Unused args are used

Fix dead code in tacacs encode

Capital W

Fix arguments inside of preprocessor condition warning

Typo

Fix various pedantic warnings

variable is used

remove unused macro

shut up stupid clang

variable is used

Always provide a session state

This won't matter when we use a pool of them

# Conflicts:
# src/lib/server/state.c

Apparently we have leaks

Don't leak sate ctxs and store the request ctx

check for child ctx

shouldn't have been comitted

Fix more talloc parenting issues

Brackets!

Swap out the list heads for pairs

Rework the support for child session state to make it simpler

Fix a ton of pair parenting issues

clean up final uses of dict_attr_size

the struct validation code now checks if (a) the da contains
a known fixed size, or (b) it's a FR_TYPE_VALUE.  That covers
the same situations as before.

It still doesn't allow for variable-sized fields like dns labels,
but that will have to be another step.

now that dict_attr_size isn't used, it's removed.

fix data types

call fr_value_box_from_network()

instead of re-implementing a subset of that functionality

and add a test case for it

use the actual length of the value box, instead of the dict size

which is more protocol-specific

use network sizes instead of dict sizes

and put things into macros like commit 74d0467, for simplicity

tests for paircmp() against strings and attributes

remove references to v3.1

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