remove the "Juniper-" prefix

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

Unused macros

Fix uninitialised fields

Make NULL escape rules static

Various stupid issues in value.c

Unused

More stupid warnings

Fix stupid vla warning

Pedantic

Used arg

Just toggle the individual bit

Fix stupid -Wassign-enum warning

Bad function cast

Fix bad terminator in par test arrays

More zero length array struct members

Unused macro

Fix -Wmissing-noreturn

Dead code

Actually used

Extra semi

Mark cap table len as static

Not inline

Unused args are used

Fix dead code in tacacs encode

Capital W

Fix arguments inside of preprocessor condition warning

Typo

Fix various pedantic warnings

variable is used

remove unused macro

shut up stupid clang

variable is used

Always provide a session state

This won't matter when we use a pool of them

# Conflicts:
# src/lib/server/state.c

Apparently we have leaks

Don't leak sate ctxs and store the request ctx

check for child ctx

shouldn't have been comitted

Fix more talloc parenting issues

Brackets!

Swap out the list heads for pairs

Rework the support for child session state to make it simpler

Fix a ton of pair parenting issues

clean up final uses of dict_attr_size

the struct validation code now checks if (a) the da contains
a known fixed size, or (b) it's a FR_TYPE_VALUE.  That covers
the same situations as before.

It still doesn't allow for variable-sized fields like dns labels,
but that will have to be another step.

now that dict_attr_size isn't used, it's removed.

fix data types

call fr_value_box_from_network()

instead of re-implementing a subset of that functionality

and add a test case for it

use the actual length of the value box, instead of the dict size

which is more protocol-specific