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.

use inst->name where possible

these are just EAP things, and are common across all EAP modules

remove LOG_PREFIX_ARGS, and just use LOG_PREFIX everywhere

which simplifies things, in preparation for adding log IDs
everywhere, too.

use inst->name instead of dl_module_instance_name_by_data()

use static name with log_prefix and ID

instead of re-converting them every time

unify to inst->name, instead of inst->xlat_name

shut up compiler

return on error

Revert "Remove hot branch in rbtree find_node"

This reverts commit b19a7ac6737e7a0f0aefb20cd94d9441d0e0eb3c.

Remove hot branch in rbtree find_node

Suppress lgtm weak crypto warning

Include dhcpv6 state machine options to control status-code contents

Add idx versions of pair_find functions

Update test now we don't send status by default

Information requests are not directed

Don't add status on success

cache align TCP packets, too

remove unused functions

cache align UDP messages

not TCP message for now, as that requires memmove() on the
"leftover" bytes, which requires more expensive logic

CPU cache line sizes are generally 64 bytes

Revert "Pass dict root not dicts..."

This reverts commit 1503b48d7dfd64c303f7db9b9872f16f54bb7fdb.

Pass dict root not dicts...

Pass in protocol specific dictionaries to the packet debug functions

Invert args

special free function

We apparently don't cache this anymore

singular not plaural

Include our talloc header here

Typo

Aruba dict updates closes #4297

...one more

...and another

Should be signed

Move talloc header

Copy interface id back if it was included

Fix various dumb issues