Add DHCPv6 test server

Take Packet-Type from the input file

We can't just default to Access-Request anymore

Slightly better way of initialising

move RADIUS virtual servers to radius.conf

export PROTOCOL for use in tests, if needed

no quotes needed

fix typo

add support for PROTOCOL in keyword tests

use internal attribute for tests, not RADIUS Filter-Id

remove unused attributes

don't manually skip internal attributes.

rely on the iterator for that.  And as a side effect, remove
support for Raw-Attribute

aggregate tags across the entire packet

not just one attribute

the previous tests were decoding lists of pairs, which worked.
updating the tests to decoding packets should the issue.

note that packet encode with tags is still failing.  That will be
fixed next.

remove end[], and just use sizeof()

we should probably add CC_HINT(__packed__) to the structures,
but that creates additional compiler warnings.  One thing at
a time.

none of the keyword tests use rlm_cache

Fix bad field access in json_object_from_value_box, and encode size

Remember parent's log lvl

Remove magic number

More ctx fixes (dhcpv4)

Parent DHCPv6 pairs correctly

Revert "At least _try_ to resolve unresolved attributes in xlats"

This reverts commit db9dd99009666a3df032964c3f2d00b0e44bc296.

Unknown attributes get marked up as XLAT_VIRTUAL_UNRESOLVED

Don't print unresolved - unresolved

At least _try_ to resolve unresolved attributes in xlats

Update dictionary.fortinet (#3862)

Add Host-Port-AVPair attribute.

some fixes

allow output directory to be set with FUZZER_ARTIFACTS.
The default is $(BUILD_DIR)/fuzzer/$(PROTOCOL)

change -timeout=x to -max_total_time=$(FUZZER_TIMEOUT)

allow FUZZER_TIMEOUT to be set, too.  Default is 10s
for "test.fuzzer".  For "make fuzzer.radius", there's no
timeout.  Perhaps we could unify that.

Add FUZZER_ARGUMENTS for extra arguments which may be needed
by some process.

Fix stupid argument name

add assertion that nothing is left unresolved

add test for Packet-Src-IP-Address

so it fails here instead of in the radclient tests

Don't build the trunk_test functions either

Split out virtual attribute resolution login into a separate function so we can enumerate the cases

Disable for now

free detached child on error

cleanups and fixing logic

clean up logic

clean up logic

add brackets so the && applies to the virtual attributes.

check for allow unresolved before checking for attr / virtual

do compile-time checks for unresolved attributes

Set the unresolved flag in xlat_tokenize_attribute for unresolved attribute references

We produce asserts on bad arguments

convert assertions to run-time checks.

with descriptive error messages, and tests

add "test.unit.PROTOCOL" target

for utter laziness

Update dictionary.juniper (#3857)

Add Juniper-CWA-Redirect to dictionary.juniper

remove the "Juniper-" prefix

wimax: Fix sql queries.conf (#3854)

eapol_test: Fix OSX build using clang >= 11 (#3855)

Fixes the eapol_test build using clang >= 11

../src/radius/radius_client.c:817:24: error: cast to smaller integer type 'RadiusType' from 'void *' [-Werror,-Wvoid-pointer-to-enum-cast]
        RadiusType msg_type = (RadiusType) sock_ctx;

Update dictionary.juniper

Add Juniper-CWA-Redirect to dictionary.juniper

Not used

Fix various spec file issues

Add clang format markers

minor cleanups

Missing parens

CI is slow

Fix issue in JSON bool evaluation

We'll fix this another way...

Better error messages in fr_pair_list_afrom_substr (#3852)

* Better error messages in fr_pair_list_afrom_substr

Helps seeing what we did wrong when using dhcpclient or radclient

* Not const

CI: Build with PCRE 2; Add "LIBS_ALT:yes" target; Tidier naming (#3851)

Now build using PCRE 2 by default.

LIBS_ALT: yes will replace default libraries as follows:

  PCRE 2       -> PCRE 1
  MIT Kerberos -> Heimdal Kerberos

Add custom names for the matrix entries so that the workflow output is easier
to interpret. Reorganise the entries so that the variable components are first.

Debian packaging: Use PCRE 2 in preference to PCRE 1, if known (#3850)

Turn floating point comparison into memcmp() to silence compiler (#3849)

In numerical methods, it is indeed a bad idea to compare
floating point numbers for equality, but this just tests
that fr_dbuff_in() and fr_dbuff_out(), barring lack of space
breaking fr_dbuff_in(), are inverses for float and double.
No floating point arithmetic involved, so it's reasonable to
compare the raw bits. We'll use memcmp() to (1) emphasize that
and (2) keep the compiler from warning about comparing floating
point values for equality.

No longer need to do anything special with state when detaching

Remove (now) invalid asserts

Don't leak session state

Bad pointer to terminal elements

re-enable this one again for now... they can often be fixed and it's causing issues with GCC

use clang diagnostics only on clang

clear out array when it's freed

use TEST_BIN_DIR instead of hard-coded paths

raw attributes have no tag

clear out data structures that we're using

dhcpv4 encode / decode does not support array of dns_label

once the code is fixed, we will update the dictionaries

don't hit assert if the dictionary parser is broken

we should also fix the dictionary parser...

zero-length attributes are raw, not asserts

this is a DNS label, not an array

check for empty packets

add "test.fuzzer" target in order to do fuzzer tests

be more forgiving about fuzzer flags

fix typo

remove extra semicolon

use pre-existing DIAG_OFF macros

turn off clang-specific complaints.  The code is fine

wrap macro in `do { ... } while (0)` to shut up compiler

better error messages

add missing "break".

so we don't hit unknown packet type

CI: Add MacOS build to the matrix

No fixtures, just ordinary "make test" for now.

We disable clang static analysis scan for now to keep the runtime within that
of the other jobs.

CI: More complete FreeBSD build and tests

Tests: There's no bash at /bin/bash on FreeBSD

shut up compiler.

tho arguably we should be using memcmp() i.e. *bit* comparison
instead of "==", which is more *semantic* comparison

shut up clang about system libraries

variable is used

Missing break

Revert back to previous way of unsetting  UNENCRYPTED flag even though it shouldn't be necessary?

Should be static

More float comparisons

More stupid flag set/unset warnings

Extra comma

Should be static

Shouldn't be marked as inline

Unused

Fix more optional args

Should probably be exported

Make more things static

Bad comma