Add test for less than and greater than to rlm_files

Add test for overwriting attributes in rlm_files

Add test for prefix search in rlm_files

add patricia trie support

pretty much nothing needs users_file.h

move PAIR_LIST_LIST to using fr_value_box

move DEFAULT handling to it's own list

When we have htries for different data types, then we cannot
insert DEFAULT into the same htrie as for integers, as DEFAULT
doesn't parse as an integer.

move rlm_files to using htrie code

does not (yet) handle patricia tries.

move OSX leak suppression to code

...and another

Remove FreeBSD case

Remov errant commas

The CRYPTO_THREAD_lock_new leak isn't OS specific

Suppress minor/startup leaks reported by heaptrack

Accidentally removed...

Fix error

Remove default_eap_type from test configs and update doucmentation

Add support for type_identity to EAP-SIM/AKA/AKA'

Remove cruft from rlm_eap

Add docs for cf_log_*_by_child

Allow submodules to determine if they should be set for a given identity

Make outputs optional for sim/aka id functions, add conversion function to extract the eap type

Remove duplicate/unused header

Move the CSV module to using htries

update for latest behavior

Fix unitialized value (#4068)

Index is special

Initialise output

Add test attribute instances

Makes it easy to duplicate a set of test attributes, giving each one a unique name and identifier

Start of changing how list_perf_test operates

Add instance arg to fr_pair_find_by_da

Quiet GCC

Fix random whitespace issues in pair_list_perf_tests.c

cpp indentation

Remove duplicate code from pair tests

Make fr_rb_node do what it says

v4: Add tests for rlm_rest - both module and xlat (#4065)

* Add setup script for openresty instance to test rlm_rest

* Install openresty in CI

* Add module tests for rlm_rest

* Test for %{rest: } xlat

Remove another memcpy

Disable the impressively useless "super-linter"

Just remove the damn assert.  The rest of the code doesn't care about integer overflow, it'll just stop the advance.

don't check the results of delete, there may be no data to delete

Revert "Revert "Fix SQL tests checking properly the query results (#4064)""

This reverts commit da11cc71763c4c21c6dc70d28d923346ad109dbd.

Is the compiler really this stupid?

Revert "add cast to shut up compiler"

This reverts commit dfeadbdc389881fe541ced8ed7736202930272fd.

Revert "shut UP compiler"

This reverts commit 8fc06a0d0e5b4cdd937b0a7e8726bc41836dc7c3.

Revert "Fix SQL tests checking properly the query results (#4064)"

This reverts commit d8626c17363b83d6329f5321ea67c84ca548ed59.

shut UP compiler

clean up encode_concat()

slen is for signed return values, not for unsigned data lengths,
among other things.

add cast to shut up compiler

return dbuff values, not intermediate variables

Fix SQL tests checking properly the query results (#4064)

Use  macros, so the tests don't have huge lines of text

Revert "decode VSAs and vendors as nested attributes."

This reverts commit e906e3d73ff649d16916405165ed7736fb496372.

s/attr_shift/attr_fragment

...and make it use dbuffs properly

Allow the dbuff copying macros to work on dbuffs or markers

fix up for extended VSAs

update description of function arguments

and note that the function simply doesn't work for extended VSAs
<sigh>

remove unused variable

More doc cleanups

fix up attr_shift() so it isn't completely broken

convert it to just use pointers, and clean up the code so that
it's readable.

And add these magical things called "tests"

let's check for errors, if it's not too hard

The WiMAX header is 9 octets, not 2.

decode VSAs and vendors as nested attributes.

Update the tests accordingly.

Note that WiMAX fails, as it does not correctly decode
"continued" attributes.  But it turns out that didn't work
previously, either.

split on total length of attribute, not on encoded value length

fix typo

comments and renaming

use encode_attribute() instead of encode_tlv().  This function
encodes attributes in the standard 1/1 TLV format.  It does not
encode an attribute of type TLV.

In contrast, encode_vsa() will encode an attribute of type VSA,
encode_extended() will encode an attribute of type "extended",
etc.

unify encode_rfc_format() and encode_tlv()

there's just no reason for them to be gratuitously different

fix encoding of TLVs

turns out encode_value() should be calling encode_tlv_children(),
and then we can get rid of more if / then / else checks scattered
throughout the code

we can hard-code this

increase "depth" when encoding

so that the encode_foo() functions get passed a DA of the type
which they're encoding

hoist encode_wimax() into encode_vendor()

which then supports Vendor-Specific = { WiMAX = { ... } }

Additional debuff docs and renaming arguments

Add note on fr_dbuff_move

Allow replacement of inline rbnodes

Much more efficient that a normal replace!

clean up encode_extended()

remove unnecessary duplicate variables, extra dbuff copies.

use consistent names

there's no reason to make this signed

clean up and de-dup FR_DBUFF_MAX_NO_ADVANCE()

which was being called multiple times, depending on the call flow.

It now just passes the input buffers through, and any function
which knows it can only encode X bytes will do

work_dbuff = FR_DBUFF_MAX_NO_ADVANCE(dbuff, X)

There may also be issues with dbuffs, as the encode_extended()
was being called _always_ with max 255 bytes.  Yet it somehow
managed to encode more than 255 bytes in the tests

use names, not numbers

use different way of setting the length field

only use one byte

remove double quotes

Remove node_ctx from rb_inline functions

Mark tree arg in fr_rb_find as const

Remove fr_pair_find_by_num

Remove fr_pair_list_free_item

Add test for dlist_move

Add fr_dlist_entry_move

Add function for checking if a node is in an rbtree

simplify encode_wimax()

similar to the previous commit.

simplify encode_vendor_attr()

there's no need to have a large number of markers.  2 will do.
and don't call them src/dest.  Those names are meaningless.

move vendor-specific encoding to encode_vendor_attr()

which is now more in line with encode_wimax(), which does the
same thing

remove redundant code

rename functions

If they're all named _hdr(), then that's redundant.  We know that
VSAs and TLVs are not values.

Use more descriptive names than foo_internal().  All of these
functions are "static", and therefore internal.  Instead, give
them names according to their purpose.

ensure that fr_radius_decode_pair_value() always returns OK

it only returns -1 on programmer error, or OOM.  Otherwise,
it always returns a VP.  i.e. a real one, or a raw one.

add ASAN / LSAN options as appropriate

so that "make test" on OSX doesn't have endless complaints about
leaks in libc

Fix pair_list_perf_test.c

Typo

Type validation... generally a good thing

Use conversion macros where appropriate in the dlist code

Minor issues...

Add "old" argument to fr_*_replace to return the old data

This is to avoid multiple lookups in pair.c