Minor formatting issues in xlat_tokenize.c

xlat_copy should actually insert the nodes it copies

Redundant as xlat_exp_alloc_null already sets the pure flag

Improve xlat_debug by indenting nodes

Descend into unresolved xlat functions too

This fixes a bug where xlat functions calls that were children of an unresolved xlat function call were never instantiated, even if that call was later resolved.

It presented with crashes when using %{expr:} as that function is registered by a module and is initially unresolved.

the default validity for certificates is 60 days, from
raddb/certs/server.cnf

%(date:...) should return FR_TYPE_DATE

Revert "unify pass2_fixup_update_map() and unlang_fixup_map()"

This reverts commit 51c5635130ee088aab54dda3de3ee2399e7d0bf9.

Changes break the json and cache module tests.

Revert "set the default dictionary"

The internal dictionary isn't allowed as the default as the internal dictionary is searched in most cases.

Formatting

Add copy functions for tmpls and xlats

set the default dictionary

tets are in the RADIUS dictionary

add map_list_tail

unify pass2_fixup_update_map() and unlang_fixup_map()

and change callers of unlang_fixup_map() to pass tmpl_rules_t
as the ctx

copy the string values, instead of shallow copies

we're not allowing multiple refcounts for the same string, so
the simplest solution is to just copy it whole-sale.

shut up linker complaints on OSX

allow casts for negative numbers in strings

date / time_delta are _signed_ integers

update comments

add SSLKEYLOGFILE capability

and add "keylog_file" to the TLS configuration, so that we can
distinguish SSL keys for EAP versus incoming / outgoing RadSec

cast RHS, as the types might be different

rename functions and add a few more helper functions

update return code

add assertion

add recommendation from the coverity people

this time without extra stuff

Revert "add recommendation from the coverity people"

This reverts commit d5d2149b16656b9ed2fe29de33aef598b2526049.

nope.

remove assert

fix code broken in commit  3391575a5

we can get away without using "..."

This also serves as a test for the fixes in commit f0db834416

add recommendation from the coverity people

update pure flags

we default to pure, and various other things will over-ride
the flag if necessary.  e.g. impure functions will set the flag
to false, as will attribute references.

rand, etc. aren't pure

Fixes for mruby for new instantiate / detach structures (#4319)

Revert "reorder types"

There are so many uses of the old ordering it makes visually checking whether cases are covered annoying.

The old ordering orders things by types which relate directly to C types, and then more abstract types, and system types, which is generally far more useful.

The C types are usually the ones we deal with explicitly in martialling and unmartialling functions.  It's very rare we end up needing to convert between time_delta and a native database type, but very common we need to deal with the numeric and float types.

try a different way to shut up gcc

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