fr_perror should accept a NULL fmt string

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/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

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

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

Always pass in line numbers

Actually commit the strerror functions.

Convert more TLS functions to strerror style

try to quiet compiler

Add "strerror" variants of more logging functions

This means the TLS library functions can integrate better with the different logging facilities, allowing us to print chain errors in the context of a conf item (for example)

Not enabled if we don't have any request

The less awful method of getting src information from OpenSSL

Record the file and line for strerror calls

Only accessible via the debugger for now

Just use the printf functions to produce tls errors

Add 'v' variants to strerror logging functions

More "update {....}" to edit against src/tests/modules/* (#4735)

remove use of locate(1) from configure scripts

Remove extra semicolons at ends of statements and declarations (#4751)

Found one while looking through coverity defects, and decided to
take a few minutes with grep and an editor to get rid of them all.

refresh configure scripts

show optional module build features in configure report

use m4 macro to set module fail state

tidy and standardise format of module configure.ac files

rework all whitespace in module configure.ac files

module autoconf depends on toplevel m4 macros

update configure.ac and nice report for libraries, too

autoconf macros for tidying module config

Add a nice report at the _end_ of configure, so there's no excuse
of missing a "silently unconfigured" module any more.

expand missing variables

initial attempt at statistics handling

rename "is_counter" to "counter"

and add it to the flags being printed

add fr_value_box_memcpy_in()

which mirrors fr_value_box_memcpy_out()

time_delta can be counter, too

refresh configure.ac files suggested by autoupdate

refresh configure scripts

make target to force refresh of all configure scripts

force build autoconf files

so make sees updated timestamps, even if nothing actually changed

fix make reconfig so it rebuilds all.mk files

aclocal.m4 and acinclude.m4 are now gone

fix autoconf underquoted warning

adjust all AC_PREREQ

nobody should be using anything before 2.69 now, it's 10 years
old, and 2.71 isn't available on all systems yet

add FR_INIT_MODULE autoconf macro

helps with consistency and saves duplication

move module m4 macros to top-level m4 directory

split acinclude.m4 into separate files in m4/

re-add the rule that actually lets make reconfig work for modules

use aclocal to regenerate aclocal.m4 on the fly

this means we can also remove m4_include, as aclocal will search
for all required macros

shouldn't need this hack any more

Work around for debian sid not reporting codename with lsb_release

There is a known debian bug #1020893

Ensure consistent llvm version is installed on MacOS

Remove unnecessary ;

docs

Add type specific dcursors (#4747)

* Add macros to generate type-safe wrappers for dcursor functions

* Add tests for type-safe dcursors

Add tests of fr_dcursor_set_current() and fr_dcursor_free_item()

Annotated two false positive dereferences (CIDS #1506638, #1506635) (#4741)

Two overlooked cases in which the caller knows that the dlist
entries in fact exist.

Check fr_openssl_init() return in fr_aka_sim_init() (CID #1504435) (#4746)

Curiously, coverity is no longer noticing this defect, but it
seems like a real bug nonetheless.

track currently running / yielded instructions

rework perf calls

to catch cases of retry / yield / abort / etc.

don't shift by 64.  Fixes Coverity #1515536 and #1515537

do time tracking per frame

and only add it to the instruction totals when the frame is done

use fr_time_tracking for the time tracking.

if it's good enough for modules, it's good enough for the interpreter

Which now lets it track yielded time, too.

We still have some @todo, which is to track parent frames, so
that when a child yields, we mark the parent as yielded too

we don't need a parent for time tracking.

fix compiler errors

Add benchmark for fr_time

clean up integer calculations

simplify the process of hinting, and always upcast intermediate
integers to int64_t or uint64_t.  That way the result is much more
likely to be representable

Make sure we don't overrun implib_file (CID #1504299) (#4739)

def_file wasn't the only fixed-size buffer in jlibtool.

Check fr_unlink() return in radsniff (CID #1504435&) (#4740)

initialize buff to placate coverity (CIDs #1503942, #1504042) (#4738)

Coverity doesn't realize that fr_sbuff_out_bstrncpy_allowed()
will put something in buff, if only a NUL terminator. Until
coverity sees annotations in macro definitions, the only way
I know of to avoid the false positive "uninit_use_in_call"
defects in SBUFF_PARSE_FLOAT_DEF()-generated functions is to
actually initialize buff.

minor rearrangement

update operator precedence (and tests)

More convert 'update' to 'edit' (#4724)

* More convert 'update' to 'edit' against EAPOL tests

* More convert 'update' to 'edit' against bin tests

* More convert 'update' to 'edit' against TACACS tests

* More convert 'update' to 'edit' against AUTH tests

* More convert 'update' to 'edit' against DIGEST tests

* More convert 'update' to 'edit' against PERFORMANCE tests

* More convert 'update' to 'edit' against RADCLIENT tests

* More convert 'update' to 'edit' against VMPS tests

* More convert 'update' to 'edit' against XLAT tests

Annotate calls we know won't fail (CID #1503939) (#4722)

The individual tests in edit_tests all operate on a three-element
pair list they get from add_pairs(), freeing it after they're done.
fr_pair_afrom_da() only returns NULL if it runs out of memory--
unlikely in this context, so we annotate the calls that add items
to the list.

Annotate false positive tainted data in fr_udp_header_check() (CID #1504068) (#4728)

Coverity doesn't recognize the check that diff == 0 as a check of
udp_len.

Check def file name against def_file size (CID #1504299) (#4731)

The check needed to be there... if coverity doesn't realize that
we are now checking, it will still need annotation.

Check inst->filename before trying to use it (CID #1503909) (#4732)

It appears that all invocations of the open function of an
fr_app_io_t check for error, so this should suffice (apart
from possible error loggign) and should satisfy coverity
without annotation.

Annotate "tainted data" in fr_udp_header_check() call (CID #1504049) (#4727)

Coverity claims that the "downcast" of a uint8_t const * to
ip_header_t const * taints "the data that this pointer points to".
"this pointer" must be p, because coverity says not a word about
dereferences of ip. The pointed at data isn't changed, being const
qualified. p and the passed length are checked in fr_udp_header_check().

Use dummy LDAP handle to tidy code

Typos

we shouldn't be using a NULL talloc_ctx.

and always free the result_copy, as the debug level may change
in an xlt function

reset request->module only when the module is done

not when it returns.  Because if the module asks for an xlat to
be expanded, that is done on behalf of the module.  And any xlat
debugging should print the module name

add file / line for indent debugging

and move indent debugging to after the change, so it's clearer
what has been done where.

typo

and reformat to match the use a few lines above, to make future
typos easier to spot

manually ported from v3, with fixes

indent for nested xlat expansion

which makes the debug output rather a lot easier to read

save node->fmt as the name of the attribute we're looking for

we should arguably walk through the xlat after it's been parsed,
and replace node->fmt with the *printed* version of the node.
That way we don't have to resort to run-time walking through the
func+argc, etc. in order to print a name for the xlat we're
resolving.

use _s for structures

add visual bars for xlat expansion

which helps to visually separate them from unlang statements.
And also helps to line up nested xlats

Nokia SR dict: Cleanup and freshen against July 2022 reference (#4729)

move save / restore indent to the correct spots

more cleanups for debug printing

move luajit definitions to one place

and only use the definitions if libfreeradius-lua is built.

Apparently on an Apple M1, adding the various pagesize options
results in an invalid Mach-O executable

add and set "constant" flag for xlat

so that we can suppress printing xlat expansions for XLAT_BOX
and tmpl_t's which are data.  It's confusing to see

EXPAND 4
  ---> 4

assert that there's always a format for the node

tweak debug output, and add indentation

add RINDENT_SAVE / RESTORE

so that we can save indention, and then avoid REXDENT on all
of the error paths.

always set node->fmt

cleanups

more cleanups

we no longer create a tmpl from the RHS result.  Instead, we just
leave it as a value-box list, or as a pair list

minor cleanups and lots more comments