store the result somewhere

it helps to check the result of an xlat

except that this error check never catches an error, because
something in the interpreter isn't passing the results back to
us.

push a function before running the trigger

the interpreter needs the stack to be bootstrapped with a function
or module that is something other than a tmpl.  The tmpl expects
to be a sub frame.

Without this, the trigger subrequest never runs anything after the
first instruction

don't append output book on failure

allow non-operators for "limit" and "timeout", too

use macros for consistency with the rest of the code

rename fields for consistency with the rest of the code

set priority, too.

instead of having it left over from the previous instruction

Don't add FR_TYPE_NULL boxes to xlat output

They currently can't be cast to anything so cause issues when assigning
to a target attribute

Remove meaningless debug

Simplify trigger handling.

Remove trigger xlat, we just use the request like.

Use tmpl_afrom_substr instead of custom parsing logic.

Initialise triggers later, move trigger name into an attribute to make serialisation easier

add %test.null() which always returns a value box of FR_TYPE_NULL

Revert "slight tweaks"

This reverts commit 8f2845ebe9feeb010c086ac20fc329e44104ea95.

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

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

slight tweaks

use macro for common case

some instructions didn't return on timeout.  So using a common
initializer is beneficial

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