we only care that it doesn't crash.

We don't care what the results are

allow '=' for list assignment to leaf types, too

clear out old resume function and rctx

when we have two xlats in a row, the first one might need to be
resumed, so it pushes a resume ctx onto the stack.  It then returns
"done", and the xlat evaluator goes to the next xlat node.

However, the next xlat node might _not_ push a resume function onto
the stack.  So when the evaluator resumes, it will see the first
resume function and rctx, and call it again.  Cue boom.

The solution is on DONE, replace the resume function / rctx with
a NULL one which does nothing.

copy from v3.2.x

Fix typos

remove diag, as some compilers don't understand it

clean up for semicolons

move wimax tests to new framework

CI: use correct inputs

json: Another test

delay commas until necessary

we have a host of stopping conditions, which can be nested.
As a result, a simple `next != NULL` check isn't enough to
determine if we need to write out a comma.  Instead, we delay
the comma until necessary.

remove extraneous ,

clean up fr_pair_list_print() and make it work in more situations

remove extraneous assert

json: various fixes

json: Modify json_quote function to be more useful

Output can now be interpolated with other JSON text and will always produce valid JSON values.

initialise with zero length strings

it helps to parse psk_identity etc. in the client configuration

typo

add and use fr_pair_legacy_print_nested flag

so that we can update the tests to print flat attributes as nested,
without updating all of the tests.

The next step is to set the flag in the tests, so that the tests
will parse flat attributes, but will always print nested ones.

unify "attribute" and "pair" commands

the "attribute" command parsed a pair or a list of pairs, as did
the "pair" command.  But the "pair" command did it manually, which
meant that it was unable to correctly deal with structural attributes

this was unused by anything

Add Ruckus attributes from Cloudpath config

remove stray assert

Ensure temporary list is freed

CI: split some install dependencies to separate action

unify fr_pair_print() and fr_pair_list_print() functions

so that they call each other.  i.e. less code.

And clean up various other bits

Revert "Keep too-long lines from overwriting output_buffer (#5093)"

This reverts commit b69393030a90e01aa9ff592d0171333efa1c3498.

You can use "goto" to jump into another block, but that won't
cause the compiler to treat variables of the same name as identical.

add and use fr_pair_list_print()

so that we can hoist the "nested or not" debug printing into
this function.  Which also lets this function peek ahead, and
print attributes as if they were nested when they really aren't

CI fuzzing: install extras repo for latest libkqueue

CI fuzzing: show which commit is being tested

docs: antora web redirects

docs: ensure v4 antora docs are marked as development

docs: antora ui dir and site url

docs: minor updates

minor tweak and notes on connected UDP sockets

cache: Don't deadlock rbtree module if the cache entry isn't found and the cache xlat is called

Remove pointless casts (#5132)

json: Add test for consuming output from function like xlat

Initialize local value boxes for some coercion results (CIDs below)

CIDs: #1503917, #1503948, #1503o5o, #1503989

These are in calc_{string, octet}(), where coercion results stored
in locals must be cleared before returning. Currently coverity claims
that the value box's type is not set. One would think that modeling
fr_value_box_cast() to say it writes the destination on success would
suffice, but that might not be the case, so we will initialize the
value boxes just as we have done with some local buffers used in
sbuffs.

Improve sample TACACS virtual server

Add authentication flags attribute and set to No-Echo for TACACS GetPass

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/dhcpv4.tar

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

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

Change other kqueue ifdef, too

See 433970a3ed07

used / unused

typo

typos and word smithing

add tests for explicit_bzero() and memset_explicit()

move to using memset_explicit()

move Interpreter to top

Missed out init in test_bstrncpy() (CID #1504057) (#5127)

redisGetReply doesn't nullify the reply pointer when there's no reply...

redis: Expose read only toggle

redis: Don't prefix function loading with "read only"

Help coverity recognize the range check (CID #1503921) (#5125)

The code checks that offset is in [12, start - packet), and
coverity recognizes the lower bound check as constraining
offset, but doesn't recognze the upper bound check, hence the
TAINTED_SCALAR defect. We rewrite the check in an equivalent
form with offset by itself on one side of the relational
operator.

Enclose explicit loop case of fr_memset_secure() in block (#5126)

Done because of the initial if statement, which makes the
explicit loop declaratons not be at the start of a block.

Keep too-long lines from overwriting output_buffer (#5093)

This came up because of CID #1533664, but almost certainly won't
placate coverity. It just makes sure that it won't ask for more
than will fit in output_buff, at the extremely rare cost of not
printing all of the remaining part of the line.

add and use fr_memset_secure()

we will update the configure scripts later

Revert "securely wipe secret information in pairs"

This reverts commit d8ddfd4a1df33cfa21b942b20ce53af263b93c52.

nope.  Various Linux distributions don't have it.

securely wipe secret information in pairs

Check fr_sbuff_in_bstrncpy() in generate_from_header() (CID #1503908) (#5107)

Initialize sbuff_tests output buffers for now (#5098)

CID #1503912, #1503930, #1503945, #1503930, #1503945

This should quiet coverity while we figure out why the
sbuff function models don't communicate to coverity that
on success, they really do write to the output sbuff.

Check sbuff returns in header_source() (CIDs #150424, #150280) (#5116)

It's seriously unlikely, since conf_buffer can use all the
memory there is, but in theory the generation of the SMTP header
*could* fail, and that's enough to make coverity kvetch.

Make clear to fr_radius_ok() that totallen is validated (CID #1455247) (#5109)

Coverity thinks that calculating totallen via byte-swapping taints
the buffer holding the packet to be decoded. Modeling fr_radius_ok()
doesn't seem to have worked, so we'll try making it clear to coverity
that we validate totallen.

ldap: Alloc all resources at the start of the function

Add _ldap_connection_close_watch only after `queries' have been (#5123)

initialized and after the connection is known to be correctly
initialized (and not free'd after being added to close watch list).

minor documentation updates

Update FreeRADIUS web site Antora documentation

bump antora to 3.1 as 2.0 is now EOL

Add copying of secret flag to xlats which create a new value box

and produce output which can reveal the input

Use fr_value_box_contains_secret when securely printing lists

Lists can contain groups and the secret could be a child of a group

Set value box secret on all concat_in_place paths

Add sample "send ...Get..." sections to TACACS virtual server

Set the secret flag of User-Message based on the attribute it really holds

Add API for querying / manipulating value box secret flag

In anticipation of the flags moving to a structure.

TACACS User-Message can contain the password - so mark as secret

doc: man page asciidoc fixes

doc: fix nav for howtos and tutorials

docs: some small antora fixes

and remove unused list.adoc (replaced by all_modules.adoc)

doc: fix a load of broken xrefs

We don't need to qualify the enums in the calix dictionary

Record file destination from config into default_log

Needed so that each destination created from default_log knows what file
to use.

Typo

Keep track of secret flag when concatenating value boxes to string / octets

If suppress_secrets is set, use hide secret value boxes in %pM printing

Add fr_value_box_list_aprint_secure()

For printing lists of boxes in debug output when hiding secret values

Always parse log section of main config

There are other options in there besides the destination.

Whilst the fr_box() macros do something different to fr_value_box, it's still not appropriate to use strlen implicitly when boxing values

User-Password is secret here, too

in may be NULL

mark secrets as secret

Uase __coverity_writeall__() in fr_value_vox_init() model