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

always unlock.  Fixes CID #1681712

Assert if slab elements are not released correctly

attr.by_num is redundant and can be removed

allow dict.attr.by_oid to operate with foreign protocols

Remove unused argument in dict function

only remove it if it's in the dedup tree

add assertion to quiet static analyser

remove old entry from dedup tree

parse *= and /=.  Update tests to match

tweak error messages to produce actual error string

various typos in tacacs

make tcp respect max_attributes and require_ma

and also make it handle partial reads

allow packet_list_recv() to return error

so that we can close TCP sockets

various bug fixes in the RADIUS protocol handler

largely in things for radclient, or proposed code that isn't
used.

reset flag on free all

fix typos

check correct field

fix typo in comparison

close FPs on errors