forbid %{func:...} and do the last few updates on the tests

typo

be more careful about freeing head

this doesn't need to be public

whitespace

sql_inst -> sql.

As part of cleaning up redundant names

Hoist mutex lock above check of client->ht (CID #1551708)

Other checks of client->ht are so guarded.

don't use the same function typedef name for two different things

return correct data type

try moving to the new syntax again

Handle subst == NULL (CID #1551707)

document clang 16+ suppressions

-Wunsafe-buffer-usage is useless for C code, as it complains
about essentially every use of pointers.

-Wcast-function-type-strict complains about casting functions
which take (void*) to functions which take a typed pointer.  This
pattern is used extensively in the dlist wrappers.

-Wreserved-identifier complains about identifiers which begin
with "_".  This is used all over the place in our code, but is
generally forbidden in C.

-Wc2x-extensions complains about initializers like "foo = {}"
which is fine, but which isn't C99.

-Wextra-semi-stmt complains about ";;" at the end of lines.
Our code is clean, but some system header files are crap.

return correct value

use correct data types, and check for overflow

clean up and check for subst==NULL

we don't care about the results of dhcpv4_decode

and print the correct pair list we just decoded :(

add support for uncompressed dns labels

ignore temporary editor files

allow for password longer than 128 characters

and update aruba dictionary for encrypted attribute

Manual port of commit 12a2cf4a6f8

Rename CONF_PARSER to conf_parser_t to match call_env_parser_t

typo

move more functions to new syntax

more move functions to new syntax

fixes

go back until we can do more tests

quote things

more move to new function syntax

move to new function syntax

skip whitespace for comma-separated argv

mark up when we parsed an xlat with an xlat wrapper

so that the print routines can print something sane

Clean up modeling file (#5222)

Remove models that don't work, replace our leading comment with
one that is commonly used in open source modeling files (slightly
edited).

we can't have an empty xlat expansion.

update CC_HINT for removed argument

remove alternation from defaults, too

Revert "Copy, don't link, schemas"

This reverts commit 823ad7a63550597682582c4d70010f536db4b7be.

this breaks CI

move xlat printing to new function syntax

remove XLAT_ALTERNATE from the source

remove old comment

enumvs always have names.  Look for ones with values

Rename some call env structures in preparation for additional changes

Copy, don't link, schemas

This lets us add extra custom schemas later

run through codespell, and regenerate antora

there isn't much point in updating the antora (outout) docs when
the raddb (input) files haven't been updated

use local dictionary

use local variable

more "remove Tmp-*"

Remove needless annotation. (#5220)

Fix typos in documentation (#5219)

Misspellings found by codespell.

Avoid fr_nbo_to_uint16() to get EAP packet length (CID #1243443)

Moved to talloc_array_length() instead, which worked much better
after I looked at the source and saw it's a macro that uses
the type of the argument, so it requires casting to pointer to
the type it was allocated with.

cache: Move key resolution to callenv

...weird intractable problems trying to get an old DHCPv4 config, easier to fix the code than debug it.

Less confusing error

Copy ldap config over to base dir

Don't immediately overwrite the specified socketpath

Allow overriding the socket path to work around weird permissions issues

touching the socket just produces another error

Pre-create ldap socket file

Remove comment

Build and scan with all available cores

ldap: Add extra schemas for tacacs

json: Add nested encoding tests

use radius_pairmove(), which does the same thing and more

in an attempt to get rid of old / duplicate APIs

Remove autosave file

Inst is const when used in the xlat function

Can't have more than one local variable!

Add test for using local attributes in eval

Allow local attributes to be referenced in eval xlats

Remove dict from xlat_exp_head_t

It's already in the nodes it needs to be in

Remove async from name of instantiate set functions

Rename xlat instance registration functions

Bootstrap is a distinct phase for modules, and calling the registration phase "bootstrap" is confusing as there's no bootstrap function available to xlats...

Disable performance tests on FreeBSD

It's a VM inside a VM so not going to be consistent / fast

Switch hypervisors for FreeBSD tests

Don't print undefined string

luajit 2.0 is deprecated, 2.1 is current

lua tests: pairs is a field in fr.request

We don't marshal structured attributes to lua yet

lua ffi.load doesn't search multiple paths

Fix compiler warnings on FreeBSD

Use cross-platform actions for FreeBSD tests

This action appears to be under more active development / maintenance than vmactions/freebsd-vm

Check for node == NULL in xlat_validate_function_arg() (CID #1548367) (#5213)

No point evaluating SQL-User-Name after the query has been built

To use SQL-User-Name in an SQL map, it needs to be populated before the
value box list of the query is created.

Use taloc_typed_strdup when xlat_eval returns no values

Otherwise using the resulting string in
fr_pair_value_bstrdup_buffer_shallow() or other functions which check
the talloc type will fail.

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

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

remove manual fr_extend_lowat()

add missing function

remove unnecessary ; after function

xlat: Don't leave dangling marker

Initialise max

sbuff: When performing prefix matches, always extend by the longest needle

typo

Revert "move extend out of users_file.c to sbuff.c"

This reverts commit 02ec7cdac5cbaa13305eb2503a23210d3af9084d.

it turns out extending things in the sbuff code causes other
problems.  So we leave some hacks in the users_file parser.

move to RADIUS attributes

add test for large users file

the default buffer size is 8K, so we add a file larger than that.
If the sbuff extension only looks for one byte at a time, then
we will run out of data in the buffer while there is still pending
data to read.  The users file reader will then return an error.

By updating the sbuffs to read more than 1 character, we ensure
that there's enough data in the buffer for most parsing purposes.

move extend out of users_file.c to sbuff.c

If we're processing substantial amounts of input in a loop,
it doesn't help to extend the buffer by 32 bytes at a time.
Doing so will help decrease the calls to "extend", and ensures
that the application no longer needs to manually extend the
buffers.