Switch the compiler used for coverity builds

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

update comments to clarify

correct list checks

we're checking if the input VP is already in ANY list, and not
if it's in the TARGET list.

For insert after/before, we're checking if the "pos" item is
in the TARGET list, and the to_insert item is not in ANY list

expose "in A list" API, versus "in THIS list" API

and fix up tlist functions with better checks for "in a list",
versus "in this list", which we can't do for dlists.

comment out broken code which re-wrote OIDs to be broken.

We now trust the author to write the OIDs correctly, and don't
try to "fix" them.

add a few things, pretty-print the strings, sort alphabetically

change error message to match the types that can have explicit length (#4456)

add APIs to get parent list / vp from an input vp

which means that the caller has to do less tracking, if it wants

Remove unused variable

Define PAIR_VERIFY_WITH_LIST for non WITH_VERIFY_PTR build

don't "return" in function returning void

don't even bother checking for operators

argument 2 can be NULL

add NDEBUG_UNUSED

just hard-code '=' in output

op is always T_OP_EQ

no need to set vp->op in map callback functions

map_to_request() ignores it, and just uses map->op

always print '=' for vp->op

nothing here should need it or use it.

don't smash vp->op to T_OP_EQ

the rest of the code in the server should take care of either
not setting vp->op, or setting it to T_OP_EQ after using the
operator.

we don't need to set vp->op

it's set by fr_pair_afrom_da()

this is no longer needed

imove fr_pair_list_t to use tlists

and do various other cleanups and fixes as necessitated by that.

* doing actual work inside of the dcursor insert/remove callbacks
* adding fr_pair_list_t* to fr_pair_verify()
* adding PAIR_VERIFY_WITH_LIST() to verify that a VP is in a list
* ensure that pair_init_from_da() is always used, instead of
  manually doing things in multiple places

remove unnecessary struct/union

this doesn't change anything other than formatting

convert error-prone code to inline typed functions

Add VALUEs for Datum in Geoconf-Option and change test to match (#4454)

Use SET_FLAG(subtype) rather than explicitly writing the equivalent code (#4453)

It *may* be that the explicit version is there to avoid the compiler
kvetching about a needless bit++ at the end. If that's the case, one
can switch to initializing bit to -1 and preincrementing rather than
postincrementing.

Convert GeoConf-Option (code 123) to struct, with test (#4450)

Convert GeoConf-Civic to use struct and add test (#4449)

Add test for RFC 3495; this exercises nested TLVs (#4448)

Typo

Use the standard instantiate bootstrap callbacks for proto and proc modules

use offset from value box, not from datum

copy at the correct offset

might as well make this "const"

use API instead of poking fields directly

make the dlist offset point to the parent structure

instead of pointing to the fr_tlist_t.  That way we can just
use the dlist functions, and get the enclosing structure.

As a bonus, it's also a lot less code.

update TLIST macro functions

add function to get tlist from dlist head

because a lot of dlist callbacks give the dlist head, and we
sometimes wish to convert that back to a tlist

remove unused / unneeded functions

manually free the pair list, as done in the other test functions.

otherwise with the new tlist code, the pairs are freed via
the talloc_free_children() callback, and the PAIR_VERIFY() check
fails, because the pair is still in a list.

allow "attribute" command to parse/print multiple pairs

check for error before allocating the VP

which also makes the code a little cleaner.

can't re-init structural types

where would the children go?  Be freed?

Remove unused variables

fix relative OIDs here, too

fully qualify the dictionary OIDs

add fr_tlist_talloc_free_to_tail()

similar to fr_dlist_talloc_free_to_tail()

add fr_pair_list_dlist_head()

and use it so that no one looks at internal fields

remove unused lavel

update fr_pair_replace() and callers

it was previously never called.  And documented as freeing the
input VP, even though it didn't.  And documented as "finding"
a VP, even though it didn't.

Use fr_pair_replace() in pairmove() functions, instead of having
them poke the dlists themselves.

use fewer memset

new VPs are always initialized to zero

less code == better code

fix fr_pair_reinit_from_da()

it may be called with a VP which has been created via
fr_pair_alloc_null(), in which case vp->da is NULL.

use fr_pair_init_null() instead of memset()

add fr_pair_init_null()

because memset(vp, 0) is no longer appropriate

use "free to tail", and don't keep tmp_list

add "free to tail" function

Add apt retry configuration to second half of debian CI tests

Use correct apt option for retries

minor typos and fixes

add dlist_head function

typo

typo

remove unused macros

pair dlist functions are internal, not public

in preparation for changing the internal structure of pair lists

add note about variable length time specs

Fix handling of affected_rows for FreeTDS SQL driver

After all the required calls to ct_results(), the internal counter of
affected rows is reset.  So, retrieve it whilst still available and
store for later use.

Add AcctUpdateTime and AcctInterval to MS SQL accounting tables

To bring them in line with other databases

Try *really* hard to pull down packages

Scheduled fuzzing: Update src/tests/fuzzer-corpus/dhcpv4.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/radius.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/util.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/tftp.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/vmps.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/tacacs.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/dns.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/dhcpv6.tar

correct references and numbers

apparently the formatter isn't quite perfect

Bump the timeout

This'll temporarily fix CI, but we should try and figure out exactly what's happening.

Return NULL on error

run through the formatting script

which updates the copyright date, and changes whitespace

this encoded as DNS

Check for NULL heap pointers

Remove one layer of heap derefs and shrink heaps as elements are removed

update and rearrange

to reset TLV numbers

the first 2 fields are mandatory, so we might as well define them :(

these are "bool encode=exists", like Rapid-Commit