forbid update in most places

except tests which actually use update

The bext step is to take all of the update tests, and run them
twice.  once as normal, and once with "-S rewrite_update=yes"

add migration configuration to forbid the use of "update"

auto-convert update sections.

via some pretty disgusting methods.  But it's not a lot of code,
and it catches the common use-cases.

better error messages

better error message

temporary hack to set name2_quote

quit coverity.  Fixes CID #1516908

add main_config_migrate_option_get()

so that various things in the code can query what they should do,
without having global variables all over the place

add cf_item_insert_after()

for use with automagically rewriting edit sections

Package process_dhcp*.so with dhcp module

Ensure module specific process_foo*.so are not packaged in freeradius package

rlm_sql_foo goes in package freeradius-foo not freeradius-sql-foo

Add recv_buff option to proto_ldap_sync

Add more ldap sync stuff to debian packages

Package libfreeradius-ldap

Add XLAT_ARGS helper macro to pull xlat arguments from the input list

Add type specific dcursors for value boxes

Doesn't perform any refactoring, just defines the types

Fail on internal exec error

reorganization

typo

add documentation for limit and timeout

only insert instructions which have thread alloc / instantiate

More verbose sanity checks

Use type safe functions for value box lists

add "limit" keyword

insert instructions into the unlang_instruction_tree

add unlang_thread_instance()

which for now is only called from the "limit" keyword.

The thread instance data could arguably go into the stack frame,
but 99.9% of the stack frames won't use it.  So for now we just
make the users call it manually

thread_instantiate may be NULL

but we may still have thread instance data, which is just
counters

remove unnecessary debug message

close sockfd on error.  Fixes CID #1516897

allow tmpls as arguments for timeout

add timeout keyword

Always initialise the output PID even on error

Fix the exec legacy header to stop vscode complaining

Ensure we use all the arguments

Quiet deprecated warnings for macos

More uncommitted sbuff notes

Check return from fr_dict_unknown_attr_afrom_num() (CID #1503891) (#4748)

Make correct changes to deal with toctou issues (CIDs below) (#4793)

1271307, 1445217: use *at() calls to work around toctou

Don't compare with bounds in SBUFF_PARSE_[U]INT_DEF (CIDs 15040045, 1504004) (#4794)

Explicit comparison with the bounds makes sense but when you hit
the type the underlying conversion function uses, it's pointless,
and when it's part of a larger expression, coverity considers it
a possible coding error.

So... rather than doing an explicit comparison, we use the following
characterization: it's an error if the cast to the desired type changes
the value. To distinguish between overflow and underflow for signed
integer types, overflow happens if the cast value is less than the
return from strtoll(), underflow if it's greater.

allow digits, in name2, too.

mostly for "timeout"

move counters to callbacks

where they're correct, rather than using ad-hoc and in-line things

expose frame_signal for use in timeout keyword

if max_rtx_foo is defined, it has to be >0 time duration

document actions and retries

flag TBD

don't use "update"

Fix redis_ippool behaviour on DHCPv6 solicit

Align trigger_exec() prototype with function definition

Ensure rate limited triggers are fired on first occurrence

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/dhcpv6.tar

Handle timeout from ldap_results() when performing persistent searches

Adapt RFC4533 ldap_sync tests to validate new cookie store behaviour

Fix typo

CI: Bump freebsd vm action version

Ensure Python 3.10 used on MacOS until 3.11 supported

There are deprecated functions which break the build with Python 3.11

rlm_python is Python3

On some deb systems python-dev installs python-dev-is-python2
and hence python2-dev

Correctly detect working python when using gcc

gcc wants -l options after the source file - so populate $LIBS

Send correct cookie

sync->cookie is the latest received.
sync_packet_ctx->cookie is the one for the checkpoint we've reached.

Explicity ignore return code (CID #1516561)

Minor fixes

Slightly better messages

Print as much information as possible about why we failed waiting for a process

Don't print confusing PID, this is exeptected

Revert "Ensure there's no way linelog tests can conflict"

This reverts commit fb9e85ad6a41b323c19ded3b4e242a1dd5a6fe61.

Ensure there's no way linelog tests can conflict

Add -Wdeclaration-after-statement where supported (#4762)

* Add -Wdeclaration-after-statement where supported

This enforces one of the FreeRADIUS coding standards.
NOTE: I had to do a "make force-reconfig" to get the changes
made to configure.ac and m4/ax_cc.m4 to have an effect in
configure. This caused changes in thirty-five other configure
files that are unrelated to this change. I am including only
the changes to the files relevant to adding the above option
to CFLAGS, namely the files I explicitly changed and configure
itself.

We also add changes to fix the declaration after statement
instances this addition turns up.

* Don't declare vars in loops

* Don't declare vars in groups

Co-authored-by: Arran Cudbard-Bell <a.cudbardb@freeradius.org>

v4: Add functionality to rate limit LDAP sync store Cookie (#4784)

* Simplify arguments for ldap_sync_cookie_store

* Add config items for cookie store interval

* Add module instance to sync_state_t

And pass it to sync initialisation functions to populate the
sync_state_t

* Define structure for tracking sync packets

* Add list to sync_state for tracking pending packets

* Create sync packet ctx to track progress of each change

* Add counters for batching cookie storing

* Store received cookies in list of pending packets

rather than sending immediately

* Use counters to send cookies based on number of processed changes

* Use timer to send cookies on a periodic basis

We need to have cyrus-sasl-devel installed when building the LDAP packages otherwise we don't get SASL support

CI-DEB: 2nd instance of use "sid" in place of "bookworm"

Try to shift up to sbuff start as current sbuff can be child sbuff with no used space (#4761)

* Try to shift up to sbuff start as current sbuff can be child sbuff with no used space.

Co-authored-by: Arran Cudbard-Bell <a.cudbardb@freeradius.org>

Sync openssl version checks with tls/version.c

Only run self-hosted cleanup job on main repo

track the correct list, and still insert leaf attributes

Revert "ignore the inserted child list, not the parent one"

This reverts commit 0806bb103d9ac5840e0113ea89c920ac3474f06d.

ignore the inserted child list, not the parent one

remove dead code

CI-DEB: use "sid" instead of "bookworm" for NR debian repos

CI: Run on GitHub unless it's the main FreeRADIUS repo

manually enable / disable new conditions for tests

SQL uses SQL-Group, so use the old conditions here, too

upcast to 64-bits for integers

typo

CI: run sanitizers in a separate non-Docker job on GitHub

CI: fixes for when running directly on GitHub images

Fix dangling pointer issue (#4786)

clang may not realize that memcpy() doesn't keep the pointers handed
to it around, so there's no risk of the address of an auto being kept
past the caller's return, hence the "dangling pointer" warning.

Instead, head it off at the pass by immediately returning if len
is zero, so in needn't be set to the address of an auto; the only
explicit passing of NULL for in passes 0 for len. (Comments explain
how if len == 0, nothing changes in the context.) One could argue
for checking in rather than or in addition to checking len... OTOH,
shouldn't passing NULL and a non-zero len break loudly?

CI: disable sanitizers until we can work out why it deadlocks in Docker

CI: fix comment

Only run the main CI build on our infrastructure

Remove unused

push xlat condition parsing until later

to avoid having unused xlats hanging around when we parse
policies.

register function arguments if available

so that the parser can do the correct thing

add "don't use new conditions" for LDAP tests

so that when we "flip the switch" to always enable them, nothing
should break