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

Keep a pointer to the module instance in the thread instance

Remove mctx->instance

It's annoying having the same data twice.  There's also tons of places this data is cached

use appropriate error functions

which mean we don't need LOG_PREFIX here

Pass the private version of dl_inst down in the mctx

Make TEST_CHECK_*() macros display stringified _got (#4310)

Conditionalize the definition of ALLOC_REQ() (#4309)

shut up compiler

be a bit more forgiving around registering xlats

for module instances like "foo { ... }" versus "foo bar { ... }".
if the xlat is already registered, don't do anything.  If it isn't
registered, then register it

no need to cache CONF_SECTION

no need to print the name

the module isn't going away until after it's data has been freed.

add 'const' here, too

add 'const'

remove rlm_ldap_t from fr_ldap_thread_t

the only use of thread->inst uses thread->inst->config, which is
already cached in thread->config

remove unnecessary pointer indirection, and make config 'const'

no need to call it mod_instantiate()

everything else uses instantiate()

remove unused prototype

Remove redundant comparisons (#4307)

make sure we catch 0 iterations in the $PBKDF2$... format, too

Working trunk tests (#4306)

* Enable working trunk tests

All time function usage is updated to use the new API.

* Change to make test_connection_start_on_enqueue() work

The code added causes the event to handle the backlogged request
to run, the absence of which throws off all the following checks.

don't bother using "rlm_" prefixes everywhere.  They're not needed

use "foo - bar" for names, as that's being used elsewhere, too

and the output looks cleaner.