the sbuff escape / unescape rules should be const

move escape sbuff sequences to utility library

so that others can use them

check for initialize failure

typo

quiet static analyzer

fix:    1. return 01 --> return -1 (it's meant to be an error return val)
2. uid_str --> gid_str  (orelse it's checking the same condition twice)
3. uid_str --> gid_str typo (orelse it's running a gid function on a uid_str)
(fix(sem): octal return value, wrong NULL check variable, and wrong gid_t type in header 168)

formatting bug: changed '%zu' to '%zuk' to stay consisten in error message
(rlm_sql_sqlite: add missing 'k' unit to bootstrap file size error message 411)

protocols/der: fix fr_der_flag_has_default_value macro — trailing semicolon and spurious != NULL comparison

rlm_wimax: fix trailing dash overwriting null terminator in Calling-Station-Id reformatter

check proper return code

the function returns 0 for "more data", and 1 for "no data",
which is different from the other APIs

check for invalid hex digits

use correct IP when prefix=0

various bug fixes in command.c

remove unused macro

turn off more warning messages

ignore AI test files

add basic tests for calc.c

remove unused variables

not all packet codes are valid

check for errors and set flags

various bug fixes in eap-md5

we don't use "name".

check password length

add more sanity checks

ctime() can return NULL

tweak bounds

more fix typos

to finish the changes from bcf136dd0

typo in macro

use "goto fail" in more situations

minor tweaks and updates to the use of AI

Add client name to error message

Fallback to attribute parsing if cast doesn't work

This allows call_env parsing for types such as uint32 to support

foo = 10

and

foo = attr

with the value being a bare word in both cases.

manually do dir + filename

to get rid of all double '/', and to avoid walking over the
input strings repeatedly.

relax checks in corner conditions

assert that the entry is in the heap

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