Turn fr_value_box_to_network() into wrapper around dbuff version (#3425)

:gear: Fix 'make test.modules' verbosity

Lets keep the same behavior for all tests/modules/$name targets
whitout that 'OK' message.

fix typo.

We want to re-run the tests if the radmin binary changes

move "copy data from request" to arp encoder

which is really where it belongs

Fix 'make test.digest' build

Fix 'make test.radmin' build

Fix 'make test.radclient' build

radiusd.mk: Fix missing .PHONY for 'make test.${name}.radiusd_start'

tmpl: Have tmpl_value "return" a pointer to the box

Makes it behave more like a function

tmpl:  Switch to "getter" style tmpl macros in preparation for nested attribute references

SQL IP pools howto: Reference the no-skip-locked SP for MariaDB and old MySQL

Fix digest dictionary attributes

Typo

make fr_value_box_cast more forgiving

i.e. if you request a cast to a bad type produce a run time error instead of asserting.

Add top level lists to the internal dictionary

fill in the reply fields from the request

so that the admin only needs to set Sender-Hardware-Address
in the reply, and everything else is filled out automatically

document "active"

and add notes on filtering only ARP request packets

export constant names for packet codes

add filter option

Add fr_dbuff_net_from_uint*() and associated tests (#3423)

Remove explicit frees

MariaDB sqlippools: New SP that emulates SKIP LOCKED using "user locks"

If we're careful we can emulate the SKIP LOCKED pragma by using user locks with
names based on the framedipaddress.

Testing shows greatly improved throughput largely due to preventing thread
starvation under highly concurrent load.

radclient: Add test suite

radclient: Fix the '-s' option

We can't use printf() here, because the stdout is being changed in
fr_log_init()

radclient: Add '-C <client_port>' option

It allows the user to specify port number.

update to use the new v4 virtual servers

make it compile and work better

define ETH_TYPE_ARP

make it mostly work

compile the processing sections
add proto_arp_ethernet which (for now) just reads packets
no reply is currently sent
various other cleanups and fixes

whoops.  Opcode is 2 bytes

expose virtual_server_section_register for public usage

remove lower limit on ring buffer messages

ARP packets are fixed-size at 28 bytes

Make sure _fr_dbuff_init() returns a parentless dbuff (#3422)

This also adds a test for that property.

rlm_digest: Don't need 'Digest-' attributes here anymore.

As we have the 'Digest-*' attributes as TLV and ready in
dictionary.iana, so we don't need that anymore.

unit_test_module: Don't need 'Digest-' attributes here anymore.

As we have the 'Digest-*' attributes as TLV and ready in
dictionary.iana, so we don't need that anymore.

radclient: Don't need 'Digest-' attributes here anymore.

As we have the 'Digest-*' attributes as TLV and ready in
dictionary.iana, so we don't need that anymore.

dictionary.iana: Add missing 'Digest-HA1' attribute

digest: Update 'Digest-' attributes as TLV.

radclient: Fix '-x' option (#3420)

fix format string

add proto_arp_process

'conf' is used

first draft of renewed "proto_arp"

It still needs proto_arp_ethernet to open the socket.
and proto_arp_process to process the packets

rearrange

use request log for request errors, not global log

added ARP encoder / decoder, and tests

sbuff/dbuff return should hold the result in a signed var

use frame-local talloc ctx

Which means that there's no need to free the temporary structures,
as they are cleaned up when the frame exits

add frame-specific talloc_ctx API for modules

so that modules can use a talloc ctx which is associated with
the frame, which automatically goes away when the frame is done.

Otherwise, the only option modules have is to use "request".
Which means either they have to be careful about freeing the
allocated memory, OR the memory sticks around until the request
is done.

radclient: Fix the output messages (#3418)

return exit status of EXEC to the caller

and use it in rlm_exec

make mod_exec* fully async

note that it ignores status2rcode right now.  That needs fixing.

fix and expose fr_pair_list_afrom_box

sbuff: Advance stacked sbuffs

dict: Move a subset of dict functions to use sbuffs

sbuff: Bring more inline with dbuffs

Start adding support for stream parsing

dbuff: Don't automatically reference compound literals

Means we can't use the macros to initialise structures on the stack.

make exec async when there's no output pairs

:mute: shut up compiler (#3417)

add missing argument

use variable...

tmpl: Fix minor leak (#3416)

Let's release all allocated memory when something goes wrong.

Remaining is better than freespace as it works for printing and parsing

stronger validations for nested maps

value: Simplify

add input VPs to asynchronous tmpl expansion.

It's only used for TMPL_TYPE_EXEC, but it's still useful

always preparse EXEC templates

so we don't need to do it in the unlang compiler
and we can run async xlats in the map_proc interpreter

we don't need this code

clarify comments

sort the files, so that they are (mostly) run in order.

There is still a "depends.mk" file which tracks inter-file
dependencies.  But it's much easier to read the output when the
files are in non-random order.

shhh gcc

value: More fixes

Use the correct hton function for 64 bit dates

value: Use integer range macros

move async map code to its own source file

radclient: Fix '-s' summary option

radclient: Some fixes (#3414)

* radclient: Just use 'ipproto'

* radclient: Typo

* radclient: errors should be 'return -1'

* radclient: The '-c' value should be >= 1

If we pass '-c 0' causes an infinity loop.

* radclient: Fix issue with '-c N' parameter

So, if the (request->id == -1), let's create the packet.
It fixes the current issue to send multiple packets correctly.

* radclient: Fix the timestamp checks for handle the packets

Fix non-debug builds in event.c

event: More dbugging

rlm_always: xlat for setting module status via unlang (#3404)

* rlm_always: xlat for setting module status and rcode via unlang

Parity with: radmin -e 'set module status db_status
{alive,ok,fail,notfound,...}'

* resource-check virtual server for triggering control flow updates

Convert internal protocol encode to use dbuff (#3413)

* Convert internal protocol encode to use dbuff

Ultimately, we will switch generally from passing around ptr/length
to using dbuffs. Doing it in stages means the following:

- the outside interface (and inside via test points) will only
  change once, at the end
- because of that, and because of common functions called by
  various protocols' code (e.g. fr_value_box_to_network()) means
  there will be wrappers that ultimately go away or become the
  official versions (e.g. fr_value_box_to_network_dbuff() and
  fr_internal_encode_pair_dbuff())

The layout and time's arrow combine to give internal_encode()
two alternatives:

- copying data only when it's known, at a cost in time and memory
  for copying some data twice
- modifying and/or moving already-written data (e.g filling in
  fields in the first encoding byte when they're known, or moving
  encoded data when it turns out to be longer than 255 bytes so
  that the length can be stored)

internal_encode() has done the latter in the past, and this version
sticks with it by noting dbuff->p's value before fr_dbuff_mem*()ing
in things it may want to overwrite or move.

* Wrap with parenthesis

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

typo

value: Children of the argv array should be parented by the array

Or at least the caller seems to assume that...

value: Temporary boxes can be on the stack

event: Pass back opaque pointer for fd event

NULL is better than zero

initialize variable

if there's no TLS, skip tests which need TLS

make exec async on the RHS of maps

And fix unit test to be correct.

typo

unescape quoted strings which will be passed to exec()

Add destructor so we can set a breakpoint for state ctx freeing

constify session-state name and add comments

instruments

Assert we've not previously allocated status resources

Be more verbose about why we couldn't send packets to workers

Print the onload return code in debug builds

Print any errors on the error stack if onload fails

Raise debug level for "optimizing" debug statements

add and document "repeat" functionality