if we allow %() and %{}, then parse both

remove check for now

mark up a bunch of functions as PURE.

Nothing uses the flag yet, but that's coming.

Note that any function which uses 'request->foo' at runtime
is NOT pure.  However, the xlat functions MAY use RDEBUG, REDEBUG,
or other R macros

pass xlat_flags_t to xlat_register()

so that we can pass both needs_async && pure

pure functions cannot require async resolution

they must operate only on things in memory

rename XLAT_LITERAL to XLAT_BOX

we already have XLAT_VIRTUAL, so we might as well avoid
XLAT_VALUE_BOX.  But that can be changed at any time.

Updated comments and functions to make it clear which things
are strings, and which things are literal values (i.e. boxes)

add value_box to xlat node, and make XLAT_LITERAL use it

right now we don't pass (or therefore use) any tainting flag.
That should arguably be fixed.

now that we don't mangle things, put these into a union

because there's no reason to have all of the field available
for all of the xlat types

we don't need to store node->attr for virtual attributes

the function itself defines what to do

minor reformatting, and use common function

minor cleanups, and add calculations for booleans

it's still not clear what to do for IFIDs and ethernet addresses.
IFIDs are site-local integers, and likely should just be treated
as opaque blobs.

we may want to allow editing ethernet addresses, by splitting
them into OUI, or by merging OUI and trailing bits.  But for now
the caller might as well just convert them to octets, do the
opertions, and convert them back.

add preliminary support for and/or

we likely also want to allow

ipaddr AND integer -> ipv4prefix

tho that' a little more work to figure stuff out

typo: don't NUL terminate octet strings

add cast to shut up warnings

this is only for clang

try to shut up clang

we only need to clear values, not the whole thing

subtraction is the inverse of addition

clear the right thing

add unary operation function

add tests for assignment operations

add binary op array

add function to manage assignment operations

add MUL and DIV

and rename errors due to floating point issues

add and use fr_value_box_cast_to_float()

set output on success, not on error

rename function

we will later have unary / self-modifying functions.  So it's
best to have consistent names from the start

create and use T_ADD, T_SUB, T_MUL, and T_DIV

note that we don't (yet) add them to fr_tokens_table[]

That's because attribute names have "-" in them.  Adding "-" as
a token means that we no longer parse attribute names in the
same way.

The solution is likely to update the various pair parsers to
avoid the 1999-era gettoken() APIs, and instead use more modern
equivalents.

For now, we just define and use add/sub/mul/div for the various
calc tests.

move T_OP_PREPEND

so that it's with the other assignment operators

update use of T_TOKEN_LAST

it's not part of the enum, it's the number of elements in the enum

use token-dependent initializers.

don't depend on implicit ordering

T_OP_ADD / T_OP_SUB -> T_OP_ADD_EQ / T_OP_SUB_EQ

because they have an equals sign, too.  This change is in preparation
for allowing many more tokens

Revert "add debugging to see if we can find out where it's crashing"

This reverts commit 3baea0df017f1b3ccd47472071517bab2e1fef05.

suspicions about the source of the issue were correct.

doesn't hurt to free this

return the VP only if the allocate_update was successful

more used / unused

when we cast to a fixed-size leaf type, we never allocate
memory.

GNU Makefile can't handle directories (#4314)

rearrange to have integer tests first

remove duplicate flags merge.

it's already done at the bottom of the loop

prepare for pure xlats

might as well make this larger

add debugging to see if we can find out where it's crashing

once we find it, this commit will be reverted

shut up clang scan

catch more corner cases

about the only time we can't handle overflow is when we do things
like

   uint64 foo - int64 INT64_MIN -> uint64

arguably the RHS should be converted to uint64.  Perhaps we'll
do that as a later step.

more checks and add calc_integer_size

more tests and cleanups

add integer to integer math

with upcasting of intermediate results to the largest integer
type which can hold the value.  And then finally cast to the
final type when we have the result.

now that the base errors are better, rely on them

instead of doubling up on errors

Revert "shut up clang on OSX"

This reverts commit c9e90a0435e271e397edbbdb8971a905cff0f108.

nope.  causes all kinds of other errors

shut up clang on OSX

return correct error

give humanly readable error message

use sbuffs and check sbuff usage

advance the input buffer, so we know what was parsed

remove unused assignment

reorder types

the ordered table in value.c didn't match the fr_type_t enum.

We now have all integer types together.  All unsigned types
together.  And all signed types together.

parse only as much as necessary for IP addresses and prefixes

add rules for ip addresses, hostnames, and prefixes

we don't need host name lookups

inet_pton() on OSX doesn't accept "127.0" as 127.0.0.0

despite what the documentation says.

But getaddrinfo() does accept it.  So when we turn off hostname
lookups, IP address parsing fails.

Work around it by just doing the work ourselves.

start of calculation and comparison API

fix typo: left not right

use correct name

move "parse type + value" into common function

in preparation for adding new functionality

ensure tokens table are the correct size

and use better initializers, so that we can reorder the tokens
without having to manually reorder multiple arrays

add more data types to "cast to bool"

T_HASH is the end of equality operators

this change means we can add new operators without changing this
definition

add function to insert list after another one

better error messages for inline attribute editing

add fr_edit_list_free_children()

which will clear the children of a structural type

"edit value then delete" gets moved to the tail of the edit list

add insert/delete tests

add "edit value then delete" tests

re-init the deleted pairs list on abort

because the pairs will be moved to their correct place by
edit_undo()

don't re-insert edit structure on "value then delete"

add replace_value() and tweak API

add hint for edit list size

so that all of the edits can (generally) live in a contiguous
region of memory, and then be freed all at once.

if we free on abort, we might as well make commit into free

no need to call talloc_free() after aborting the edits

Scheduled fuzzing: Merge and push pack the corpus (#4313)

remove dead store

we can't edit values of structural pairs

insert_after tests

minor notes

don't free VP, pair_delete() does that

rearrange / rename API, and add fr_edit_list_replace()

remove INSERT_BEFORE, and update API

we only need INSERT as insert after

try to shut up clang scan

add edit lists and tests

so that we can edit lists "in place", and then undo the edits
if anything goes wrong.

functions return fr_pair_t*, not void*

check correct variable, and allow "pos" to be NULL

which lets us insert things at the start of the list.

Tidy up ephemeral module_ctx_t struct creation

Correct fix for rlm_sql_cassandra

Unbound needs the inst name for xlats

Sync up module and instance data name

Fix arguments to cassandra

Use correct mctx fields in rlm_rest

Mark command_encode_dns_labels as nonnull

Use structures to pass arguments to instantiation and detach functions