Actually run on our runners

Attempt to move sanitizers to local infrastructure

Fix typos and bad docs

don't assert on FR_TYPE_NULL

in preparation for allowing it as a real type

Add additional file check flags

Add specific flag for unix sockets

must always check the return code of fr_value_box_cast

add test of "last loop returns the rcode"

use consistent names

$ git grep -w result_p  | wc -l
      42
$ git grep -w p_result  | wc -l
     860

move MOD_ACTION_NOT_SET --> 0

add macros for valid values.  sprinkle assertions throughout the
code.  Move the action names to a fixed-size array.  Update the
printing to print out the names in all cases (not the numbers)

allow ::enum for 'case' statements

Correct docs on rlm_radius return codes

correct behavior of parallel

it helps to actually set p_result when pushing the child.

Then, update the tests to match the documented behavior.

it helps to track the child result

use macros for priorities, not hard-coded numbers

this helps with transitioning the actual values

give expr.rcode() a better name: interpreter.rcode()

the old one is still around for compatibility reasons

print rcode if we don't get passed options

Simplify call_env expanding

Pointers to source tmpls are always set in call_env_expand_start, so
don't need handling in call_env_expand_repeat.

Parse only envs are never pushed as tmpls, so will never be processed in
call_env_expand_repeat.

If a tmpl is data, no need to push it for evaluation

use macro to set p_result

which lets us initialize the structure with the correct values

Use netlink API to insert ARP entries on FreeBSD

Comparable to how FreeBSD's arp command adds entries using netlink.

Set the request->rcode to the default section rcode when we enter a section

Default rcode is now noop

Sync up perl return codes with server return codes

add signal.raise("break"), this lets us stop the program after an unlang keyword executes, which makes it easier to debug certain issues

RLM_MODULE_NOT_SET=0 so that zeroed memory is invalid.

use macro to initialize struct with UNLANG_RESULT_NOT_SET

Make the compound literal part of the macro

The result of the interpret should be the result_p of the frame not the section_result

This caused synchronous evaluation of xlats to return reject

move redundant to its own function

it's not really a group.  Instead, it should share the redundancy
functionality of redundant-load-balance

Remove unused function

run dsymutil if we have it and libbacktrace

The pid file may reference a process that doesn't exist

add p_result to unlang_tmpl_push()

Regression test for internal encoder

Add support for libbacktrace

Add utility functions to trim paths

Don't pass a pointer to the password value box...

Add libbacktrace as a submodule

remove radius_packet_t

structs require alignment.  But packets can come from anywhere,
and aren't always aligned.

regularize pool configuration for tests

it makes absolutely zero sense to have "uses = 2".  This means
that the connections will get closed in the middle of the unit
tests.

show when we hit boundaries

so that the poor admin reading the debug output can find out
why the connection was closed

add and use automatic re-init on connection failure

Some SQL modules use a thread-specific connection hamdle for
escaping. However, that connection may fail for some reason.
When that happens, the connection should be reopened.

Otherwise, the connection will go down, and will never come back
up again.

Fix unaligned access in DHCPv4 "get_option" on MacOS

rework load balance

it now supports all data types via fr_value_box_hash()

more sanity checks and simplifications

Update docs from raddb

Whitespace

Update section names referenced in sample modules

Update sample rest module

Update rlm_rest test config

Use call_env to detect which sections rlm_rest is called in

And find conf sections based on the section name where the module is called.

Make call_env_parse a public function

So we can use a custom parser to point at a specific CONF_SECTION rather
than having them hard coded in to the parser config.

add comments and macros for mod_action

add error message to xlat_redundant when it fails

add tmpl_dump

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/bfd.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/tacacs.tar

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

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

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

Correct #ifdef wrapped logic

If we send using pcap, use pcap to receive

Calling recvfrom() on the pcap fd returns EWOULDBLOCK or EAGAIN

Correct UDP checksum calculation

Move variable initialisation (CID #1655307)

Correct return value for comparison (CID #1655308)

Add libpcap option for sending DHCP replies

Which will be used on non-Linux systems.

Also, can be used on Linux for testing, using the hidden listener conf
option of use_pcap

No need for dhcp_packet_t

Since we're just sending data - not decoding the header.
Using dhcp_packet_t also can cause spurious alignment complaints from
sanitizers.

let's take the result back up the stack

better debug output and clarifications

Use module name in custom call_env_parse functions

So call_env_parse() can be made public

Add checks that tmpl expansion created a value

If the query tmpl did not expand, fail the module call

More consistent handling of dcursors in xlat args

xlat_frame_eval doesn't know if the arg is required, so if no pairs are
matched, allow a cursor to be returned.
Then xlat_process_arg_list can do the right thing depending on whether
the argument is required or not.

Spelling

it turns out that we care if the xlat fails

so we return an error when that happens

query must exist

otherwise it means that one of the expansions failed, and the
interpreter did not pop the stack.  Instead, it just kept going

failed cast is fail, not OK

docs: Add guidelines for documentation for contributors, coders, and me. Update Terminolgy.adoc standardizing on UK/British spelling. Update accessibility section & included more details in other sections.

Added documentation for trigger_exec

update docs for parallel for intended behavior

hoist "set default actions"

so that it's done in only one place

and then since we're cleaning things up, do a bit more work
on unlang_compile_children()

unlang_register() no longer needs a type argument

the type is now in the unlang_op_t.  And even that will be removed.

also reformat for consistency.

and add unlang_t size and name for each instruction

remove debugging

move keyword compile_foo() to foo.c

mostly. compile_edit(), compile_tmpl(), compile_module(), and
compile_function() are still in compile.c

Many helper functions are exported in interpret_priv.h.  Some of
those should be cleaned up / renamed / un-exported.

The function names could likely be cleaned up.  Much of the code
uses unlang_foo() for private functions.  It should arguably use
private function names.

The unlang_register() function takes a type as both a function
argument, and as a field in unlang_op_t.  Fixing that will require
changing instruction->type to instruction->op in the interpreter,
which should be a next step.

The hash function is just fr_hash().  We should arguably come up
with a perfect hash for the keywords, which will help simplify
the lookups.

point to correct function

link in "next"

Update and format.  Manual merge of #5618

notes on multiple return codes

more trigger fixes.

allow back-ticks for exec, and run the exec tmpl type

allocate a main loop, so that triggers can use it

assert when we enter the tokenize function, not later

when we call xlat_finalize().  This helps to hoist errors earlier
in the call flow.

"frame" is not talloc'd

TMPL_TYPE_EXEC can have xlats, too

use the local thread interpreter here, too

Add note on SQLite WAL

Only the top level bio needs to be shut down

Ensure request is runnable after error