Quiet coverity (CID #1503969, #1504008) (#4475)

timestr_match() converts time from the beginning of the week in
a struct tm to minutes. Coverity doesn't consider the bounds
on those values (tm_wday and tm_hour), and thus warns of possible
overflow.

silence coverity complaint about possible overflow (CID #1469150) (#4474)

Update link to Cisco website (#4473)

Correction to description of function return values (#4472)

Add ciena dictionary

set head->flags correctly, and other minor cleanups

don't set the flags repeatedly in xlat_tokenize_argv()

merge head->flags into input flags

just call flatten_argv()

and update it's API to re-parent the nodes and free the input
list, so that there's no confusion about who owns what.

parent things properly

Add covaro dictionary

Fix potential leak (#4471)

filter out for CFLAGS, too

filter out warnings for local targets, we don't care

use HOST_CC

Revert "Don't try and build the make support libraries if this is a WASM build... we'll figure this out later"

This reverts commit 932bf41d78ab0c463446cdf46a8ac82812f0e2fc.

use HOST_CC for jlibtool

as it's a local binary

move xlat_exp_head_t to it's own data structure

fmt and flags are not yet filled out

node->fmt is already set above.  Re-setting it mangles things

Alternative source of sockaddr_ll for emscripten/musl

If we use the cache, then we don't populate CPPFLAGS on the cache run...

Ignore wasm build products

Add support to jlibtool for emscripten, it's basically the same as Linux, right?

Emscripten provides mallopt, but not the same knobs

Don't try and build the make support libraries if this is a WASM build... we'll figure this out later

It involves using two compilers in the build process, and as EMSCRIPTEN mostly uses static linking, using dlopen to figure out symbol presence probably isn't going to work.  We'll need some higher level function that execs the wasm linker like configure does.

Minor fixes for emscripten

Remove needless check for NULL fp (CID #1503900) (#4468)

Remove spurious debugging

Add simultaneous use queries for MS SQL

Fix decoding of single TLV

- use fr_dcursor_next_peek to check for multiple children so that the
cursor doesn't move
- extract the pair from the source list to append to the destination

New config.sub and config.guess

fix locking.  Helps with #4465

release mutex before destroying it.  Helps with #4465

Fix potential leak on error

Correct UB_EV_PERSIST check in _unbound_io_service_writable() (CID #1469157) (#4466)

crossbuild: update image prefix to remove clashes with v3

docker: tidy unused jenkins dockerfiles

Add args back

Just use fopen...

radmin isn't thread safe

Passing back port via variables is not thread safe, so set a unique variable containing the port for a given test suite

MODULE_TEST_UNLANG is a path

not yet for this

Horrible hacks to get EAPOL tests to run in parallel

Fix radclient tests for parallel execution

Allow the secret and base port to be overridden from the shell

Use the PID file to identify unique instances of FreeRADIUS if it's set

Get rid of superfluous test in loop in command_encode_dns_label (#4463)

By virtue of CC_HINT(nonnull), in, and hence p, will be non-NULL
the first time through the loop. To take another iteration, next
must be non-NULL, so that p, set to next + 1, will also be non-NULL;
hence the "while (p)" is effectively "while (true)".

typo

use correct data type

move encode_array() to common function

which further shrinks and simplifies the protocol-specific
encoders.

none of this is used, so change it to void*

catch the case of zero returns

it's either allowed, and we do nothing, or it means that we didn't
decode any data, and the input is malformed

options can have zero data

add fake xlat_exp_head_t, which is just xlat_exp_t for now

and update function signatures, etc. to use it.  In preparation
for moving to tlists

don't do N^2 checks

we just need to check if it's a value box,
xlat_is_literal() will check all subsequent nodes, and we don't
want that.

Fix parallel tests for sqlite, and enable parallel testing

Check fr_event_fd_insert() return value. CID #1469154 (#4461)

Add temporary thread local dbuff buffers too

typo

for length fields, just limit work_dbuff

instead of encoding "too much" and checking for it later

don't need release if we also dump the parent dbuff

These are technically linker flags

use marker instead of entire dbuff

more cleanups to use macros instead of hard-coded fields

move to macros for more things, instead of directly accessing fields

in preparation for moving to tlists.

clean up and rename

Errant \

Always print out the config.log

move function args and groups to their own members

in preparation for moving to tlists

move XLAT_ALTERNATE children to type-specific fields

check lengths, too

add verify_tlvs() function

so we don't go crazy creating structures, only to throw them
away when we discover that something bad is going on

minor cleanups

and create raw attributes, instead of just unknown ones for
things we don't understand

add option to print out VPs it's decoded

minor rearrangements

remove unused assignments

shut up compiler

increase bounds

initialize slen

fix function signature to shut up compiler

decode all extended attributes at the top level

remove "concat" flag from Digest-Attributes

it's not really "concat" in the same way that EAP-Message is,
as we can't split it in the middle of sub TLVs.

start of rewrite to be more like the other protocols

add fr_pair_dns_labels_from_network()

and make various protocols call it.

fix test.

When we encode something, the decoded version should NOT be "raw"
unless something has gone seriously wrong.

move decode_tlvs to common API

with a flag to see if we decode the TLVs as nested (or not)

add notes for DNS validation

move to using fr_pair_raw_from_network()

Insted of multiple local functions

make fr_pair_raw_from_network() closer to other decode functions

move / rename "pair raw from network"

so that it's in a better place, with a better name

break out some more test cases for clarity

move decode_array() to generic fr_pair_array_from_network()

align fr_value_box_network_length() with to/from-network code

align encode_array() with other protocols

and remove yet another protocol-specific "get attr length" function

clarify documention in to/from network functions

remove attr_sizes table and usage

only the type-specific decoders need to care

remove dead code.  CID #1469168

try to shut up coverity

which thinks that fr_token_t can be between 0..63, instead of
0..48?

remove fr_radius_attr_len() and related checks

we can just either (a) rely on the input dbuff to see when the
data is too long to encode, or (b) just check vp->vp_length

just call fr_value_box_to_network()

remove unused code