don't need release if we also dump the parent dbuff

These are technically linker flags

use marker instead of entire dbuff

more cleanups to use macros instead of hard-coded fields

move to macros for more things, instead of directly accessing fields

in preparation for moving to tlists.

clean up and rename

Errant \

Always print out the config.log

move function args and groups to their own members

in preparation for moving to tlists

move XLAT_ALTERNATE children to type-specific fields

check lengths, too

add verify_tlvs() function

so we don't go crazy creating structures, only to throw them
away when we discover that something bad is going on

minor cleanups

and create raw attributes, instead of just unknown ones for
things we don't understand

add option to print out VPs it's decoded

minor rearrangements

remove unused assignments

shut up compiler

increase bounds

initialize slen

fix function signature to shut up compiler

decode all extended attributes at the top level

remove "concat" flag from Digest-Attributes

it's not really "concat" in the same way that EAP-Message is,
as we can't split it in the middle of sub TLVs.

start of rewrite to be more like the other protocols

add fr_pair_dns_labels_from_network()

and make various protocols call it.

fix test.

When we encode something, the decoded version should NOT be "raw"
unless something has gone seriously wrong.

move decode_tlvs to common API

with a flag to see if we decode the TLVs as nested (or not)

add notes for DNS validation

move to using fr_pair_raw_from_network()

Insted of multiple local functions

make fr_pair_raw_from_network() closer to other decode functions

move / rename "pair raw from network"

so that it's in a better place, with a better name

break out some more test cases for clarity

move decode_array() to generic fr_pair_array_from_network()

align fr_value_box_network_length() with to/from-network code

align encode_array() with other protocols

and remove yet another protocol-specific "get attr length" function

clarify documention in to/from network functions

remove attr_sizes table and usage

only the type-specific decoders need to care

remove dead code.  CID #1469168

try to shut up coverity

which thinks that fr_token_t can be between 0..63, instead of
0..48?

remove fr_radius_attr_len() and related checks

we can just either (a) rely on the input dbuff to see when the
data is too long to encode, or (b) just check vp->vp_length

just call fr_value_box_to_network()

remove unused code

no longer need these

move string/octet length encoding/decode to value.c

The code already enforced some lengths (e.g. fixed-length fields,
with either truncation or zero-fill).

Since the dictionaries now support da_is_length_field() and
flags.array, we might as well support that in the functions
fr_value_box_to_network() and fr_value_box_from_network()

doing this allows a whole set of additional optimizations in the
rest of the code.  encode/decode array becomes a simple loop around
encode_value(), which means it's protocol agnostic.  And the
dhcpv4/dhcpv6 option_len functions aren't needed, and can just
go away.

Split macros out of acinclude

Use COMPILE_IFELSE in more places to speed up configure

Split out network byte order functions

WS

Cache pointer earlier

Do module mutex init later once we've loaded the library

More radsniff errors

Typo

Leave module_thread_inst_list around if thread instantiation fails

Initialise the module mutex during alloc

Typo

Remove vendor prefix

Make prefix format consistent for _FR_LOG_PREFIX_MARKER

notes for the future

port from v3

we now have a "real" 245.26.FreeRADIUS, so use 256 for tests

request can't be NULL.  CID #1504033

Check xlat_registered didn't return NULL before dereferencing. CID #1504055

Avoid null dereference. CID #1504060

Add adva dictionary

add MEM().  CID #1503919

shut up Coverity.  CID #1469139

more sanitization.  CID #1469146

move to struct initializer.  CID #1504050

request can't be NULL.  CID #1503955

*out may be NULL.  CID #1503963

remove dead code.  CID #1503987

shut up compiler.  CID #1504010

remove dead code.  CID #1504027

typo

call init instead of clear. CID #1504023

Check vb has been allocated. CID #1503940

remove dead code.  CID #1504073

Make POSIXification in exec_pair_to_env() notice lower case first char (#4458)

The loop as written always incremented p after the test to deal with
leading digits, so it wouldn't notice a leading lower case letter.

fix up encode_tunnel_password()

be a lot more rigorous around bounds checks, and aggressively
bounds check everything.

The tests change because we're using different amounts of output
of the random numbers.  But the encode/decode cycle yields the
same results, so that's OK.

initialize handle

bound "len".  CID #1503923

after rounding up, and adding room for the headers

sign / unsign issues

because the compiler is too stupid to notice that we check for
(ssize_t) foo < 0, and disallow it.

packet_len should be signed

close FD on error.  CID #1504032

Check vb has been allocated. CID #1469173

Correct operator. CID 1469166

use CMP()  CID #1503972

no need for intermediate variable.  CID #1504026

try to quiet Coverity.  CID #1469156

fr_tacacs_length() can return negative.  CID #1469155

free handle on error.  CID #1469135

remove dead code.  CID #1469125

Correct check for count of Februaries - tm_mon is 0 to 11

Correct check for invalid tm_mon. CID #1503944

Avoid array overrun. CID #1504011

check counter before accessing array. CID #1504030

Initialise variable. CID 1504065

we choose to shut up gcc instead of coverity

don't use variable out of scope.  CID #1503994

set inlen, too.  CID #1469118

remove dead code.  CID #1504044

return error if appropriate.  CID #1504006