s/vp_map_t/map_t/g

Simplify debugging

Weaken the loop tests

Only push the section if we have something to execute

Allow call to run with no children

Formatting

Formatting

subrequest instruction needs to be talloced

Correct test_dbuff_no_advance() (#3718)

fr_dbuff_end() just returns the end pointer; actually moving to
the end requires fr_dbuff_set_to_end().

unlang: Use unlang keyword specialisation structs instead of separate allocations

Test that operations on a dbuff created with FR_DBUFF_NO_ADVANCE() don't affect its parent's position. (#3714)

rlm_python: Add comments for -frecord-gcc-switches #3693 (#3702)

add descriptive error messages

remove filter from value_box

remove 'abinary' data type.

Document the changes.  Update the RADIUS encode / decode tests.

Remove FR_TYPE_ABINARY from everything.

remove / fixup abinary tests

in preparation for moving "abinary" data type to "string",
and making it radius-specific

update comments to match functionality

left over from changed behavior in commit 323f9c88a3

remove debug stuff :(

Fix issues with errors when a length is passed to pton functions

set the async flag as appropriate

fix for async.  left over from changed behavior in commit 323f9c88a3

print the correct quotes around FR_TYPE_TMPL, too

no need to use public functions for internal values

we're already using internal values everywhere else in the parser

convert "header" to using tmpl

which now lets the header be taken from an attribute, if the
user so desires

add FR_TYPE_NON_BLOCKING

for use with tmpl_eval().  We can mark configuration items as
allowing tmpls / xlats, BUT also mark them up with a requirement
for non-blocking operation.

This lets us convert much of the code to using tmpl_eval() instead
of xlat_eval().  At the same time, the admin is prevented from
breaking the server by using asychronous operations in places
which do not support them.

Once all of the callers of foo_eval() have been converted to
async, we can remove the FR_TYPE_NON_BLOCKING flag.

correct documentation and behavior

to match the name of the variable, and how all of the code
uses the variable

now that abinary is a pointer, treat it as such

comment out abinary tests until they're fixed

print out string values with correct quotes and escaping

so that the output is as close as possible to the input

no need to quote the string

use cp->attr internally, instead of cf_pair_attr(cp)

and clean up debug printing for FR_TYPE_STRING

we cannot call fr_value_box_from_str() on any structural types

We cannot assign hex strings to attributes of type "vsa"

Fix the test so that it has the correct error message

remove unused macros

Fix pair type checks to allow all nested types not just FR_TYPE_GROUP

sbuff: Set const depending on start pointer

pass the value_box directly to print_abinary

move print_abinary() to use sbuff internally

cleanup ascend filters

make it a bit more forgiving with respect to input size, by
requiring only as much data as we need.  Instead of mandating it
is 32 octets.

Remove unused definitions.  Convert the union to anonymous.

Fix talloc ctx for group attributes

Add nested subrequest tests

Add functions and macros to read values and copy bytes from dbuffs (#3709)

parse Juniper filters with IPv6 addresses

move ascend filters to pointer, and fix use of vb_length

vb_length _was_ inside of the union for variable-sized types,
but the code also set it for fixed-size types.  Only the size
of the ascend filter prevented various over-writes.

changing ascend filter to pointer instead of an array highlighted
this issue, and causes us to move the "length" field out of the
union in order to stop the over-write

abstract datum.length to vb_length

abstract datum.length

use vb_length instead of mangling things manually

add more tests for ascend data filters

Correct the fr_net_to_int*() functions

use fr_cursor_tail() instead of vp->next

move VMPS to using cursors, instead of VALUE_PAIR**

more xlat tests

Fix vendor specific tests

add tests for \%

which aren't correct right now, but the tests pass...

set server_cs, too

Fix programatically generated subrequests

Add missing vendor attribute

update to match new behavior

s/REQUEST */request_t */g

REMOVE ANOTHER LOUD TYPE

s/VALUE_PAIR/fr_pair_t/g

FIX ONE OF THE LAST ALL CAPS TYPES IN THE SERVER

Shut up clang scan (#3705)

Quiet clang scan

Add subrequest src/dst evaluation code

This allows us to produce nested attributes in unlang

Don't assume the first ref is unparsed

Revert "Write bytes as hex"

This reverts commit 4b3623661c930200a95cc9c2acbf757763863408.

Fix filtering functions to work with nested attributes

\n is EOF, too

Write bytes as hex

Fix typo in process_subrequest

allow unlang in subrequests

whoops, let's initialize everything

reword process_subrequest() to take many arguments

Start of process subrequests

shut up clang scan.  Fixes #3704

pass original packet, not a crappy offset from a vector

Default to fr_value_box_to_network

We don't need to list all the valid types for RADIUS either,

It likely makes the code slower as well... switches aren't magic, it's up to the compiler to implement them and it may well choose a binary search for this number of elements.

don't mangle things until AFTER we've freed the children

recurse into all children for structural types

there's no reason to specify the full path of the files

hoist tag code earlier, so we don't need to talloc_steal()

fr_value_box_to_network already has an "encodable" check

There's no reason to restrict protocols to a subset of types when there are vendor specific attributes involved...

Simpler function

make packet_ctx.vector[16] instead of a pointer.  Fixes #3703

Make sure we never merge flags by mistake

Pass all standard types to fr_value_box_from_network

Don't init the boxes twice

fixup option/len macros in DHCPv6

Revert "minor tweaks"

This reverts commit 44cb7e539f7e52199a30318be980fc2752d2f8f4.

pair: Pairs should free their children when they're freed

Whitespace

Mark certain attribute references as 'resolve_only'

This means when we're attempting to build out structures we know that if we didn't manage to resolve the ar using the existing pair tree, we need to abort.

Remove white space...

Typo

Unused

comment why the option is removed

use "text" instead of "varchar".

there's no reason to fix these fields at a small size

rlm_python: Anchor with a space so we don't mangle the middle of flags

See #3693

rlm_smtp with unit test (#3547)

* rlm_smtp with unit test

* Fix typo, remove extraneous info.

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

Make the underlying dbuff version of fr_radius_encode_pair() visible

This makes it possible to switch users of fr_radius_encode_pair()
over to dbuffs individually rather than all at once.