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

Check for NULL, and free on error.  CID #1503946

check for NULL.  CID #1503997

remove dead code.  CID #1469132

strcmp() returns large values.  CID #1504056

reduce boilerplate code

fix typo in cut&paste.  CID #1504002

we don't care if the call fails.  CID #1504005

if we're going to "error" anyways, no need for an extra "if"

remove double unlock.  CID #1504063

add MEM().  CID #1469145

allocation can fail.  CID #1469123

uint8 is bit 2..8, not 2..7

rearrange to hopefully help the compiler and clarify

if it's inline, then separating the code paths for iter NULL (or not)
should help.

add more structs

we don't need a separate tlist_head offset field

just pass fr_tlist_head_t to item/entry functions

we don't need a separate type in fr_tlist_head_t

Switch the compiler used for coverity builds

Add the coverity button back

Fix pcre atexit function

More ndebug fixes

Match function to prototype

Don't print file loacations if this isn't a debug build

Allow atexit handlers to return error codes so we can track when they fail

Avoid infinite loop in module_root(), oops...

Cleanup more atexit stuff, and explicitly trigger atexit handlers to avoid ordering issues with libraries which install their own atexit handlers

Add the prefix of the root module onto the name of thread instance data

Use separate module lists for proto, process and rlm modules

Enforce that [length], if present, must be at the end of the type field (#4457)

remove cursor code

now that it's wholly unused

remove use of cursors

remove dependency on fr_cursor_t

we're not going to fix everything right now, but this is easy,
fast, and not wrong.

dcursor.c is only for testing