Work around faulty clang analyzer logic

Pass pointer to the handle

Init valuebox in overflow case

find_lm_password and find_nt_password report failure to find password attributes

Initialise output pointer

Initialise client in a way that Clang Analyzer understands

Use corrext context

Better encapsulate regex match data

Switch regex xlat to an async expression.

Make regex xalt ploymorphic to allow querying by name or by capture number.

Allow N capture groups for PCRE and PCRE2 regexes.

If we have REGEX_PCRE2 or REGEX_PCRE set the feature 'regex-pcre' to true

Add support for PCRE2 Closes #2315

Fix elsif (should be elif) in inet.c

Formatting and use talloc_bstrndup for binary safe copying of POSIX regex captures

Formatting

Formatting

Explicitly prevent overflow

move common code to fr_app_process_instantiate()

Length changes size and size between versions of json-c

call tokenize_cast only if there's a cast

move "bootstrap app_process" into common function

Fix potential NULL dereference for function with NULL argument list

Should be const too

Fix clang scan stupidity

Fix incorrect return code check for cond_tokenize_cast

Fix uninitialised return in input on error

Out may be NULL

Deal with NULL ptr value in dlist code

Markup other arguments as nonnull, so we get warnings earlier in clang scan

Move fr_command_register_hook call inside the HAVE_PTHREAD_H ifdef

We only spawn workers if we're built with threads, and the list is life uninitialised otherwise

Don't continue if we error out spawning worker threads, destroy the scheduler and return NULL

Silence various compiler warnings in Clang 7.0

move common code to master_io mod_instantiate()

move common code to master_io mod_bootstrap()

move dynamic_submodule into master_io

now that we have master_io bootstrap, etc. functions.
All of the duplicated work can be done there.

we no longer need master IO talloc context

parent things from the parent structure

ignore editor ~ files

typo

move fr_io_client_t handling to destructor

move "live" to it's own variable

more separation of variables

more separation of "const inst" and non-const thread_instance

make fr_io_instance_t "const"

clean up talloc parenting

in preparation for making fr_io_instance_t "const"

header levels

dictionary doesn't have a .conf suffix, include it anyway

convert local links to inter-document cross-refs

add comment to get auto-expanded in asciidoc

make sure asciidoc links are in inter-document xref format

So that when files are converted to HTML, they still link nicely
to each other.

fix links

convert to markdown (text still needs updating)

convert asciidoc to pretty HTML

remove "parent" from client

we have access to the relevant fields already

one more const issue

move mod_open() to using thread_instance

use thread_instance

const issues

The length of chap challenge isn't bounded

handle longer CHAP-Password when using debug level 3

move writable socket stuff to private structure

which means we're one step closer to using thread_instance

and we can now remove mod_detach(), as the per-socket stuff
is talloc'd, and we can use a talloc destructor for it

detach can only be called for the main instance

not for child connections

"open" shouldn't be called for connections

make app_io_instance "const"

in yet another step to making thread-specific data structures

don't call detach, the dl APIs take care of that for us

write to thread_instance, and then copy it to app_io_instance

Revert "move "free clients" to mod_close(), instead of mod_detach()"

This reverts commit a505ccc195fc49847166ad2bf3839e099282944b.

we don't yet have thread-specific data structures for sockets.
so the "alive_clients" are still in the app_io_instance, not the
thread_instance.  Once we have thread data, the alive_clients
can move there, and then be free'd on mod_close()

don't bind client tcp sockets

use listener, and not app_io_instance for NAK

Revert "start of work to allow workers to do their own IO"

This reverts commit 79c344b9333e8e7c0239e671392be135dd03cf20.

it's better to just add an fr_network_t to the worker thread,
as with single-threaded mode.  Then, have special markers
in the channel so that messages result in API calls, and not
in actual channel messages being sent

write to thread_instance, not app_io_instance

in preparation for making app_io_instance "const"

pass listener, not app_io_instance

do closing in mod_close(), not mod_detach()

close things in mod_close(), not mod_detach()

move "free clients" to mod_close(), instead of mod_detach()

since the dl framework calls mod_detach, we don't have to do that
for child listeners.

move the remainder of the functions to taking fr_listen_t

rearrange code

revert 8d15eee4 for proto_ldap_sync

add 'int fd' to listener structure

and remove "get_fd" functions and callbacks

remove most mod_close as unnecessary

and fix master.c to call close() if there's no app_io->close

more fixes for passing the correct parameter

more rename 'listen' to 'li', and fix const complaints

pass the correct variable over

need to close this, too

make mod_close() take fr_listen_t

rename variable "listen" to "li"

so that it doesn't conflict with the function listen()

make mod_open() take fr_listen_t

make fd_set take listener

make mod_fd() take fr_listen_t *

once all of the functions are converted, we can move more
variables to fr_listen_t.  e.g. fd, name, src/dst.

We can also add / use listen->thread_instance in order to
make listen->app_io_instance "const"

set thread_instance == app_io_instance for listeners

so that the new code can use the correct variable name, even tho
they point to the same struct.  We can then make them point to
different structures

create and use child listener

with all of the information we need about the socket

Revert "add thread_instance to fd()"

This reverts commit a2701b977fe1483e22fb36993f7998e2fde15672.

our own listen->app is always ourselves

Switch rlm_lua to using the thread instance framework Closes #2311

Check return code of i2a_ASN1_OBJECT Closes #2310

remove extra parameter to "open"

it turns out we're not going to use it

sc can't be NULL

simplify the code

move common code to fr_master_io_listen()

add thread_instance to listen socket

don't build these test programs

they're no longer needed

add thread_instance to fd()

notes on which entries are "const"

cleanups in preparation for multithreaded networking

we can't re-instantiate a module, that's bad.  And, the default
structs for a module should be the ones it's modifying.  the
main config should be "const", and rarely used