free group on error

shut up analyzer

when shutdown fails, enter failed state, not closed state

various bug fixes in limit.c

various bug fixes in hash.c

add locks when accessing entry

fix typo

fix name comparison callbacks to use correct values

return on failure

check before memcpy

use digest_find_vp() for qop, too

even though it's hex

point to correct attribute

typos in lengths

clean up cs on error

Add test of redis_ippool with requested address

Renumber lua script lines

Honour requested address in rlm_redis_ippool

Compare numbers rather than strings

Since 10 > 2 but "10" < "2"

Mutex is protecting dlist maniplation, not entry->expires (CID #1682066)

various bug fixes in pool.c

fix pipe bio to use normal read/write blocked/resume callbacks

just use our own buffer for the pipes

and tweak read/write logic to be better.

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