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

Continue splitting dictionaries by RFC (#4445)

Remove prefixes

Add netelastic dictionary

freeradius-snmp "proxy": Fixes and cleanup (#4446)

Accelerate walks with a "next OID" hash.
Enable collection of per-client stats by default.
Populate accounting client table with real data.
Avoid race by hoisting radius_stats_init() out of updater thread.
Have radius_updater thread terminate on SIGINT.
Refresh RADIUS-AUTH-SERVER-MIB and RADIUS-ACC-SERVER-MIBs.
Add IPv6 client support.
Assume default value (0) for non-existent stats attributes.
Whitespace and style.

Try and fix the random debian packaging failures

add back Centos 7 support, apparently that was still passing in the scheduled builds somehow

Typo

Use another source of centos stream

Strip support for fedora and centos-7

See if this fixes the centos 8 package build...

Fix failures in OpenSSL 3.0 with custom logging BIOs

Apparently the create callbacks need to call BIO_set_init() otherwise all operations on the BIO will fail.  This isn't documented anywhere.

If we're building OpenSSL make sure it's a debug build

We still need the request for state reset

Hack in stack over-allocation for OpenSSL async contexts

Bump OpenSSL for alt libs

i->ifa_addr may not be aligned

on OSX it fails with alignment issues, as apparently the alignment
requirements are different for sockaddr and sockaddr_storage?

Or at least ubsan complains on my Mac.

more sanity checks on parsing dates / times

move private macros to private header

add cf_item_has_no_children()