return actual offset, not bumped up by 1

close on error.  Fixes #4518

close on error. Fixes #4518

log: Fix fr_log_close() 'dst' check

log: Fix fr_log_init_file() 'fd' set

Ensure module loading errors are printed out

add MEM.  CID #1504430

typo

shut up clang scan

don't free too much

move xlats to dlists.

clean up in preparation for moving to lists

clean up so that there's less magic.

remove flags from xlat expr tokenize functions.

sanity check for flags

the caller should't set run-time flags.

remove flags from xlat tokenize functions.

The flags are now only used for xlat registrations.

start of API cleanup

minor clarifications

xlat tokenize functions now only need head

not ctx, out, and flags

set flags correctly.

it's probably easier to do this correctly if we have a better API

xlat_resolve() no longer needs flags

ensiure that xlat_resolve() can always be passed NULL flags

no need to pass flags here

remove xlat flags from tmpl structure

start of removing xlat_flags_t from tmpl

sometimes go a bit sideways to make progress.

remove flags from xlat some functions.

expose an API to get at the few flags which are needed.
move tmpls to use the API

set head flags, too

schedule: Fix potential memory-leak (#4510)

Check fr_value_box_from_network(), fr_value_box_cast() returns (CID #1469115, #1503931) (#4514)

re-add flags removed erroneously

Add explicit paths for OSX

because apparently some of the C compilers aren't built with the
default paths for everything on the system.

fix up for OSX

newer versions of clang can't find things.  So we need to manually
add the paths in.

Catch invalid status values in tls_cache_session_ticket_app_data_get (#4509)

Make load_noinstall_path() free what it doesn't return (CID #1504304) (#4500)

v4: Changes needed for fr_pair_list_..._by_da macros (#4508)

Add checks of fr_lst_insert() calls (CID #1503950, #1504014) (#4512)

Check fr_minmax_heap_alloc() return value (CID #1504069) (#4513)

Define macros for adding a pair to a list and assinging its value (#4507)

Just adjust ext when removing "mod_" (CID #1504300) (#4506)

One need only adjust ext by 4 when, in truncate_dll_name(), one
removes "mod_". Then if len exceeds 8 ext still points at the
extension, and coverity can't kvetch about not checking the value
strchr() returns (Using strchr() the second time means
"mumblefrotz.foo.dll" won't behave as expected; does that ever
turn up in practice?)

Switch to memmove() for truncate_dll_name() (CID #1504298) (#4495)

with overlapping src/dest, strcpy() gives undefined behavior

Remove the needless done_init variables from tests (#4505)

This change should settle CIDs 1504383-150439, making clear to
Coverity that rand_ctx is indeed initialized.

Use fr_fast_rand() to select operation in burn-in (CID #1503935, #1504015) (#4503)

Convert tests to use fr_fast_rand() rather than srand()/rand() (#4502)

Coverity considers rand() risky, but data structure tests don't
require cryptographically secure pseudorandom numbers. This should
take care of CIDs 1469120, 1469179, 1503927, 1503935, 1503941,
1503967, 1503977, 1503990, 1504015, 1504048, and 1504054.

Check value returned by fr_pair_value_mem_alloc() (CID #1503929) (#4498)

Correct MEM() check of fr_pair_value_mem_alloc() return value (#4497)

Make python configure script check that it can actually link to the python libraries

Fix atomic queue alignment on 32bit systems

stderr and stdout are macros and conflict with these fields

Allow building without semaphores

Build build system libraries with HOST_CC

Print out canonical target names and add "darwin"

Allow toolset to be set explicitly

remove unused variable

minor cleanups

add "ignore this list" entries.

If we're inserting a new structural member, then we don't need
to undo edits to any children in it's lists.  Those edits will be
undone automatically when the parent pair is undone.

We keep a separate list of "lists to ignore", and check that
before doing anything.  If the list can be ignored, then we
discard delete / clear / etc.

In inserts, if the VP is structural, then remember its child list,
so that we can ignore edits to it, too.

rename, as there are too many "list" things here.

Remove initial FR_TYPE_NULL check from fr_value_box_from_substr() (CID #1504016) (#4496)

Better to let it be handled in the switch statement.

Free result of file_name_stripped() when needed (CID #1504303) (#4494)

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.