fr_strerror_clear() needn't bother with fr_strerror_init() (CID #1503892) (#4486)

fr_strerror_clear() already refers to fr_strerror_buffer, and might as
well use it rather than the return value from fr_sterror_init().

Move use of out after check for NULL (CID #1503900) (#4482)

Wrap malloc() in MEM() to reassure coverity (CID #1469117) (#4485)

Add check for NULL from fr_value_box_alloc_null() (CID #1469164) (#4487)

Add another check of fr_event_fd_insert() for trunk tests (CID #1469165) (#4492)

Correct memory leak in add_dynlink_noinstall (CID #1504301, #1504306) (#4493)

The single check for either install_path or current_path being NULL
will leak if it passes; load_noinstall_path() doesn't have an explicit
return NULL and lt_malloc() exits if malloc() returns NULL, so
current_path is guaranteed non-NULL if load_noinstall_path() returns.
To avoid the issue, check install_path immediately after it's set.

(If coverity doesn't recognize that current_path will never be NULL,
it may need MEM().)

At the end of add_dynlink_noinstall(), both are freed.

rename variables to something which makes sense

Remove needless check for FR_TYPE_NULL (CID #1503975) (#4488)

set target-specific C compiler via SRC_CC

which should usually be $(COMPILE.c)

More minor jlibtool fixes

Pass through our compiler and ranlib defs when building jlibtool

prctrl definitel isn't posix

LIBRARY_EXT is really TARGET_LIBRARY_EXT which isn't what we want

Set HOST_CC TARGET_CC HOST_RANLIB TARGET_RANLIB from their respective environmental variables

If the target is empty or set to "default" just use the host target

This makes it a little easier in the build framework

Don't build semaphore code under emscripten, it's not available

Do partial target matching in jlibtool

Typo

Avoid spurious output

Fixup jlibtool to allow crosscompilation without needing multiple versions of the binary

Control verbose flag passed to jlibtool

print out build for jlibtool, etc.

add "counter" flag for unsigned integers.

to enable more SNMP, and internal stats, along with automatic
aggregation of stats.

show more structs

Revert "Fixup jlibtool to allow crosscompilation without needing multiple versions of the binary"

This reverts commit 83892e9c37da2de186afdc775e6438b02d5a35ad.

export VERBOSE

FreeBSD uses the .so extension too

Fixup jlibtool to allow crosscompilation without needing multiple versions of the binary

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

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

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

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

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

Make minmax_heap_burn_in() exercise all operations (CID #1503996) (#4478)

random choice of operation wasn't hitting all cases

Move assignment out of fr_assert_msg() to pacify Coverity (CID #1504180) (#4479)

remove needless check of parent in parse_input() (CID #1469153 (#4480)

Earlier in the code, parent->allow_unlang is tested; if parent
is NULL, it will break thre, so the later check is unnecessary.

Move use of name after NULL check in dict_by_protocol_substr (CID #1503920) (#4481)

Remove needless NULL checks in cf_pair_parse_internal() (CID #1469127, #1469144) (#4476)

array is checked for NULL, and an error returned if it is, before
the loop that fills the array, so the checks of it in the loop (and
also of entry, which can't be NULL either).

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