remove duplicate print

more tests for the edit code

FR_EDIT_CHILD doesn't have a vp

move the check for it to before the assertions that VP is sane

comment out talloc_get_type_abort_const() until we fix all callers

typos in calc.c

dup the correct FD

typo in cbor

fix typos in time code

better checks for corner cases when parsing fractional seconds

various bug fixes in timer.c

remove duplicate code

add MEM()

use vp->da instead of da

clean up for alignment, overflow checks, etc.

arguably this code should just be removed, as pretty much every
modern OS doesn't need these wrappers

use value-box API rather than hand-rolled code

use talloc_strlen() instead of talloc_array_length() - 1

use talloc_strdup() now that it's a define for talloc_typed_strdup()

outputs depend on the libraries that we're using

use correct callback for copy, instead of recursing

typos in pair_tokenize.c

set type for talloc_strdup() and friends

The talloc string functions set the type of the string to the
string contents.  Which makes it very difficult to do type checking
on them.  As such, we wrap the various functions in ones which set
the name to something reasonable.

don't pass "const" to talloc functions

secret may be NULL, and make code more consistent

parent filename off of thread, not inst

use correct attribute for duplicate checks

warn if they're using a secret other than 'radsec' for TLS.

@todo - also when we do DTLS, the secret should be different.
And when ALPN is used, we don't have a secret

clean up on failure.

which doesn't matter in practice, as we don't dynamically read
users files.  But it's good practice

clean up on corner case of failure

and use "goto fail" in more places, to ensure a common cleanup
path

typo in name match

increment count as per docs

don't leak FDs on corner cases

typos

multi-thread fail is different from single-thread fail

clean up on error, and insert into list only on success

remove duplicate code, and tweak order of existing code to match

fix various bugs in unit_test_map

don't re-alloc memory, and always close fp on exit

fix typo - don't call init twice

add tests for corner cases of keywords

check extensions, too

typo - use & instead of modulo

typo in error messages

track memory with talloc

TBH, this module should either be discarded, or re-written from
scratch.  It's pretty terrible

add tests for cf_util, etc.

various bug fixes for tmpl_dcursor.c

don't return freed pointer

minor fixes to rlm_eap

use SBUFF_CHAR_CLASS instead of UINT8_MAX + 1

and fix a few issues where it wasn't using the "+ 1"

add tmpl tests

to avoid regressions, and sanity check the code.

complain on invalid input

return on failure

use correct variable

various bug fixes in password.c

more error checks for OOM, etc.

various bug fixes in connection.c

reset processing_signals on freed, instead of returning.
otherwise it's locked forever.

leave one-shot watchers enabled until they're finished, otherwise
connection_deferred_signal_process() will access a freed watcher

allow more states in the handlers, as they're all valid.

return error on OOM

Free connected dynamic client parents when their idle time expires

Without this, the "parent" client which is built when a connected
dynamic client is first created never expires, so the client is never
reloaded.

add missing enum.

Why not just set TRUNK_CONN_ALL = ~0?

typos and use dlist API properly

fix typos

quiet static analysis

Need inline for always_inline to work

various tweaks to tmpl_tokenize.c

check for !fips here, too

typo

docs-v4: moved network security best practices page to `Introduction >> Best Practices`.

docs-v4: Added more details to Developer >> Contributing section including git commands, PR submisssions. Rearranged hierarchy, added subsections, and formatted accordingly to style guide.

docs-v4: Correct nav link for `unlang >> List of Data Types`

correct check for FIPS mode, and re-init free list

when changing digest functions, we have to clear and re-initialize
the free list, otherwise the contexts are associated with the
wrong ctx_free() function.

better error messages

when local variables are defined in the wrong place

various bug fixes in map.c

don't leak, and make "-=" work a bit better

various bug fixes in rlm_mschap

set result, too

more checks for corner cases and invalid data

typo

remove PID file on error, too

status checks can expire, too

add more checks and complaints

remove old from htrie, too

various bug fixes to pair functions

check for memory allocation failure, tweak macros, etc.

use hyphens

use consistent formatting for copyright notices

add slab tests for commit 331aa05

and use TEST_ASSERT instead of TEST_CHECK + if (!foo) return

loop only while we have 4 bytes in the buffer

fix order of %ull vs %ul

fix typo

remove test content

various bug fixes in EAP

eck for rare failure on init

hoist failure check before setting destructor

always save a copy of u->packet

avoid strlcpy(), and use memcpy() instead

check return code of function

check for IP in pcap receive path

produce correct error message, using the configured length

various bug fixes in rlm_unix

pass remaining data, to avoid repeated strlen()

tweaks to comments and assertions

various bug fixes in winbind

more checks for malformed vendor options

free buffer on all error paths