mash leading '.', too.

and set safe_for

fix: 1. initialise thread mutex before use
(rlm_stats: initialise thread mutex before use 415)

fix: 1. typo
(rlm_sqlcounter: fix copy-paste bug in find_prev_reset() for "never" reset 413)

fix: 1. function parameter typo fixed
(Fix copy-paste error: response_window never validated in rlm_tacacs and rlm_radius 416)

fix: 1. exact duplicate code removed
2. return 0  -->  return -1   (for invalid discriminator/state)
(protocols/bfd: fix duplicate detect_multi check and wrong return value in decode_proto 433)

fix: 1. missing 'false' argument when calling function
2. num_events == EINTR  -->  errno == EINTR     (typo)
3. if (*start > MY_ARRAY_SIZE)   -->   if (*start >= MY_ARRAY_SIZE)   (wraparound sometimes skipped)
(tests/util: fix missing 5th arg to fr_message_set_create, wrong EINTR check, and off-by-one 225)

fix: 1. calling wrong function, is now pop_tail (copy paste typo)
2. fix undefined behaviour
(lib/util/dlist: fix free_tail pop direction and sort-merge NULL deref 156)

fix: 1. pesudo --> pseudo (typo)
2. added outer parenthesis
(net.h/net.c: fix 'pesudo' typo in function name; add missing outer parens to IP_VHL macro 283)

just use sprintf to match the rest of the code

and other minor tweaks

greatly simplify function

as it should only be taking unsafe inputs

the function is only used by one caller

and doesn't have to be exposed

add callback for escaping value-box with sbuff erules

use real type, and not uintptr_t

remove functionality which is useless and never worked

add simple tests for value-box functions

use standard APIs for filename escapes, and check for errors

add value-box escape with erules

there are a number of places which need it.

continue after \r, \n, \t

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