use correct type

don't set packet->vector for non-RADIUS protocols

and most of those were wrong, too :(

we no longer use packet->vector for anything DHCPv4

don't smash the authentication vector

pass dbuff && packet_ctx to encode function

which makes it easier to add more functionality

point to common context, instead of local struct

make common context "const"

remove vector[] from common encode/decode context

remove duplicate API as part of cleanup

there isn't much point in switching to a new API if we don't
switch to a new API

clean up API and simplify

no need to pass buffer / size twice to the receive function,
it's already in the dedup_ctx

for simplicity, put rb node into dedup context

don't encode Message-Authenticator multiple times

use pctx for packet ctx

allow setting CoA filter attribute name

Revert "Add Coverity-only check to pacify it (CID #1604609)"

This reverts commit aa37659f220f4d0a338ab98ad4fd3110a6082fdf.

link in radiusd -X

point to new files

print out only at end, to avoid dups

point to correct link

script to cross-check antora files

this is a new module

moved to better location

Update index.adoc

Create Alcatel-Lucent

Create Huawei

Create HP

Update cisco.adoc

Create Alvarion

Update index.adoc

Create EAP PEAP

Create Disconnect Messages

Create Twitter

Create Stats with radsniff

Create Raduat

Create radiusd -X

Create Eduroam Configuration

Create Logging in an eduroam environment

Create EAP-SIM and EAP-AKA:

Create Troubleshooting

Create Red Hat FAQ

Create Git config management

Create FAQ

Create RADIUS Concepts

Create Wired Equivalent Privacy

Create WPA

Create RFC Compliance

Create RADIUS

Create NAC

Create NAS

Create Virtual Servers

Create Performance

Create UDP

Create Scheduler

Create Message API

Create Coding standards

Create Submitting patches via GitHub

update developer instructions

Create Configurable Module Fail Over

Create Building on macOS

Create Suse and OpenSuse

Create Solaris

Create RPMs with Oracle support

Create RedHat and CentOS

Document which OS's are supported

add documentation for building on Debian and Ubuntu

Create Overview and Features

use constant-time comparisons for secret values

use correct return code

Remove redundant CI action option

Read what libcurl has set as the default CA path

Annotate overflow_const in  fr_isaac_init() (CID #1604614)

overflow_const strikes again in a RNG. The shift is intentional, and
it's an unsigned type so the behavior is defined.

Use size_t where needed (CID #1604623)

Coverity correctly noted that the calculation of the number of
bytes in an error message can overflow unsigned int, so we use
size_t instead (with matching format change)

Add Coverity-only check to pacify it (CID #1604609)

Coverity does not realize in the FR_EVENT_FUNC_IDX_FFLAGS case of
event_fd_func_index_build(), the loop condition will keep pos from
ever underflowing. The check, which only shows up for Coverity,
should reassure it that this can never happen.

Documentation updates

rlm_rest no longer opens connectios at server start

Since the move to async

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

Scheduled fuzzing: Update src/tests/fuzzer-corpus/radius.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/dhcpv4.tar

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

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

Add notes on when actions are used

Correct references

We need the TLS headers in order to be able to manipulate the SSL_CTX *

We always want fr_curl_conf_t to be available

Add support for keylog_file in all curl based modules

update bugs link

Typo

Declare the local fflags consistently with the fflags field.

In the FR_EVENT_FUNC_IDX_FFFLAGS case in event_fd_func_index(),
the first loop directly passes entry->fflags, which has type
uint32_5, to fr_high_bit_pos(). but the loop that iterates over
the bits of fflags uses a local of type int. fr_high_bit_pos()
takes a parameter of type uint64_t.

C rules for widening from signed types to unsigned types differs
from that for widening from unsigned types to unsigned types. The
former will effectively sign extend the signed value, but the latter
zero extends. I suspect that there's no current fflags value with
the most significant bit set, so it makes no difference now, but

1. Should it happen, you'll definitely fall off the end of
   map->ev_to_func.
2. Why go out of one's way to give the local a different type,
   risking just this sort of issue and making whoever reads the
   code wonder about it.

This won't affect CID #1604609; it turned up while I was looking
at it.

Close sqlite handle when we fail to pre-poopulate the database.  Closes #5392

Use latest cross-platform-actions for FreeBSD

Which now recommends use of a linux host for better performance.

Use MariaDB connector for FreeBSD build

Print values and prepend timestamps to log lines in RFC3339 format

Prevent reinitialisation of sbuff scratch space once the thread destructor has been called

Use CS_SET to set things

FreeTDS driver uses fr_sql_query_alloc()

To build and run the "USE <database>;" query run at connect time.

Correct comments