use xlat_debug_attr_list

export functions to debug attrs and lists

support list_as_attr in tmpl_get_list()

shut up clang scan

typo

more rough-in for getting child lists to work in edit sections

we need some changes / additions to the dcursor API in order
for this to be finalized.

Add missing MEM() guards to fr_value_box_alloc_null() calls (#4619)

One, in debug_attr_vp(), will silence CID #1507353. The
other is not associated with a coveriy issue, but should
be there. With this change, all fr_value_box_alloc_null()
call returns will either be checked by MEM() or have an
explicit check in the code.

manually unwind indentation on cancel.  Fixes #4622

comments and docs

separate out leaf / structural edit apply

added fr_pair_list_steal()

don't print RHS when it's a list

this has to be updated later to do the Right Thing

more leak suppressions for recent OSX

start of parent / child relationships for nested edit sections

do less copying when editing lists

move edit state to sub-structure

in preparation for allowing nested lists

allow TMPL_TYPE_DATA on RHS of map

Constrain rcodes mapped to eap codes (CID #1503933) (#4620)

Coverity notices that there are rlm_rcode_t values that
exceed the bounds on rcode_to_eap_code[].

xlat_func_subst: do not add an additional self-reference (#4621)

Fixes the following talloc error when %(subst) is used:

ERROR: talloc_free with references at src/lib/util/value.c:3425

        reference at src/lib/util/value.c:3982

Second fix on changing /etc/apt/sources.list

print nested pairs, too

allow edits of structural attributes.

this should work, as the underlying infrastructure now supports it.

don't record NOOPs

we now have head->flags, so just use that

add %{paircmp:LDAP-Group foo} API and tests

so we can hack up "&LDAP-Group == 'foo'" in the short term

rearrange code

hoist common code to common function

in preparation for doing paircmp() in the new xlat framework

Annotate strcpy() in make_vlog; log_keyword is always short (CID #1503899) (#4615)

All calls to make_vlog are done via macros that pass one of
"error", "warning", or "info" as log_keyword, all much shorter
than the 256-byte buffer that accumulates the log entry.

Keep attr_protocol_encapsulation specific to one gctx

hoist static analysis checks to before when they're used

unify resolution of FUNC_UNRESOLVED and FUNC

by turning FUNC_UNRESOLVED into FUNC, and then just falling
through and using the normal FUNC resolution process.

set more flags correctly

more asserts for sanity checks on needs_resolving

free conditions and children of "if" sections which were skipped

and ensure that cf_section_free_children() doesn't free CONF_DATA,
as that's still needed.

don't merge flags quite as yet

the node may be re-written.  We only merge flags when we're adding
the node to a function argument list

regex now loops over input

use fr_value_calc_list_cmp() for comparison operators

add fr_value_calc_list_cmp()

which is O(N^2), and implements v3 functionality.

We arguably need a real list comparison, which checks each member
for equality

Uniform way of dealing with static analysis (#4613)

Static analysis doesn't necessarily infer all properties of code
and thus may give false positives. Since we use at least two static
analyzers, clang's and coverity, we define STATIC_ANALYZER if and
only if some static analysis is running, and check it instead of
a macro specific to a particular static analyzer. It's defined,
or not, in src/include/build.h, so that adding a new static
analyzer only requires adding code, not changing it.

Debian sid docker image no-longer has /etc/apt/sources.list

Use a child vp when encoding children of a group

Add VS code intellisense configuration

remove unnecessary calls to fr_dcursor_current()

don't create value-boxes with NULL ptrs for 'octets'

simplify checks

print out which function has issues

fix bug and add test for logical ||

  (&Framed-IP-Address || &Packet-Src-IP-Address) == (ipaddr) 127.0.0.1

should work when one of the attributes doesn't exist

more docs

shut up clang scan

don't free vp before using it

auto-remove the auto-added references

Fix when filename starts with a directory separator (is an absolute path). (#4601)

Fix UNIT_TEST_KEYWORD_ARGS when BUILD_DIR is an absolute path name (#4602)

* Fix UNIT_TEST_KEYWORD_ARGS when BUILD_DIR is an absolute path name

* Fix the following error when eapol_test test fails:

/bin/sh: line 8: ": command not found

* Fix map, modules and radclient tests when BUILD_DIR is an absolute path name

annotation typo (CID #1504436) (#4596)

Annotate false positive in _raddict_report() (CID #1505146) (#4588)

coverity doesn't know that if children == NULL, len will be
zero, so children won't be dereferenced.

Annotate false positive sizeof mismatch (CID #1504018) (#4599)

Check tmpl_find_vp() return value (CID #1503995) (#4600)

Now that tmpl_find_vp() returns -2 as advertised, we can check
the return value rather than checking vp, placating coverity.

Accomodate coverity in tls_cache_session_ticket_app_data_get() (CID #1503904) (#4604)

Make the default appear for coverity as well as for clang static
analysis, for the same reason: otherwise, an unexpected status
could pass a NULL request to tls_cache_app_data_get().

Annotate false positive on curl_escape call (CID #1503903) (#4605)

The NUL terminator isn't counted in the length parameter of
curl_escape():
1. You can pass 0 and curl_escape() will itself use strlen().
2. The example code from "man curl_escape" gives a length
   that doesn't include the NUL terminator.

We therefore annotate the call to placate coverity.

tests for %{string:} with other data types

update documentation on string escaping, etc.

removed duplicate page, and merged content

Start adding common module methods

fixes and tests for regexes

Revert "zero-length strings are allowed to have NULL ptrs"

This reverts commit a49569534534c33bc0dad3505f012767dd241b85.

added new RFCs

Update python version on FreeBSD CI

call xlat_init() manually, before registering any xlats

so that we don't need to check xlat_root every time we try to
register an xlat

resolve all tmpls which need resolving

pass the current dictionary to xlat_resolve()

don't leave shallow references to an sbuff

let's commit this, too

allow for encoding of other protocols inside of the internal dict

we can't just drop the other protocols into a packet along side
internal attributes, because we can't distinguish the internal
attribute "1" from the protocol number "1".

We therefore need an encapsulation layer.

The internal encoder / decoder still needs to be updated to handle
Protocol-Encapsulation as a special-case, ala Message-Authenticator

allow for 'ref=PROTOCOL' as a bare word

always assign t_rules.  Fixes #4594

t_rules is checked for NULL, but it later always dereferenced.
So just assign it to default_rules.

add cf_item_free_children()

for use with if (0) { ... }, so that unused xlats can be freed.
Otherwise they're stuck in the instantiation tree, and will cause
issues.

test for manual encoding of TLVs

typo

more casting comments

Typo

Report correct type in error message

Align docs with function definition

remove UNUSED

Revert "map_afrom_cp: unescape double-quoted and back-quoted RHS prior to parsing it (#4590)"

This reverts commit db35ae505b78437f1a726f0635cf6ef7a0d16ce3.

this breaks many unit tests.  We will need a better fix.

update casting rules

(string)&Foo now means "print to string".  This change only affects
'octets' types, as other types were already printed to a string

more docs

map_afrom_cp: unescape double-quoted and back-quoted RHS prior to parsing it (#4590)

ctx is used

Remove un-needed variable

remove unused variable

use print xlat for double-quoted strings

"foo%{Bar}"

produces "foo" plus the *printable* version of &Bar

"foo" + (string) &Bar

produces "foo" pluse &Bar *cast* to a string.

Both are useful.

add "print" xlat, which produces a printable string

using double-escaping rules when we have tainted inputs

quote data type, to be clear that it's a data type

always escape 'octets' in fr_value_box_print()

fr_value_box_list_aprint() calls fr_value_box_print() to print
normal types.  And fr_value_box_print() prints 'octets' as hex
base16.

However, fr_value_box_list_aprint() calls
fr_value_box_list_concat_as_string() for 'group' types, and
that function just copies 'octets' to the output string.  Which
results in non-grouped 'octets' being printed as hex, and grouped
'octets' bring printed as binary.

the solution is to update fr_value_box_print() to enforce hex/octets
printing of 'octets'

At the same time, update the escaping logic so that if the octets
string is tainted, OR there are escaping rules, we just escape
the raw octets value.  The previous code printed it as hex, and
then escaped that, which doesn't make much sense.

resepct cast in tmpl_eval_pair_virtual(), too

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

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