radius: Don't need file/line for fr_assert_fail anymore

Don't need fake expr

event: Better to make it an assert

Missing header

More compiler issues

Define macros for NDEBUG

Quiet compiler

Remove all calls to exit() and _Exit() except where hey're absolutely necessary

Fixup assert macros

# Conflicts:
# src/lib/unlang/tmpl.c

minor clean ups and bug fixes so that it works

kevent() will return an error if the process has already exited

so we just call waitpid() to reap its status

return the child_fd to the caller

because fr_exec_child() doesn't do it

it helps to wait for the child...

add unlang_module_yield_to_tmpl()

cleanups and simplifications

turn the output buffer into an output value box

and allow exec-wait without an output FD

Fix rlm_cache_ sanity check for crazy people (#3370)

This PR fixes an incorrect size argument passed to `strncmp` which resulted in smaller prefix matching than intended.

use kevent to wait for the PID

it should all be working.  Next is to add tests

more framework for exec-wait

we still need to use kqueue to wait for the child PID.  But the
basic parts of the framework are now here.

add fr_exec_waitpid()

so that child processes will be cleaned up if we do exec-wait,
but there's an error.

start of framework for exec-wait

with reading the output.

And some "todo" notes for the future.

exec-wait API using new methods

fix typo

stupid OSX leaks memory on fork().  Let's ignore it.

check for tainted flag on first argument

precompile TMPL_TYPE_EXEC in unlang

into vpt->tmpl_xlat
Update tmpl_verify to allow this.

In preparation for having precompiled xlat's everywhere.

preparation for signal handlers

minor cleanups

allow modules to push async calls onto the stack

i.e. triggers.  Which are fire and forget, but which also need
to do async expansions.

don't exit on full stack

instead just cancel the request

rlm_radius: Fail the first message not the last message

rlm_radius - Remove function names, not needed anymore

There's a logging mode which prints log lines that'd be better to use

Always reset repetitions counter

log: Fix repetition check

rlm_radius: Free packets before retransmission

radius_udp: Simplify cancellation function

trunk: More documentation

test: lets test the 'radsniff -a' (#3367)

It will prevent to detect issues with pcap.c API used to fetch and list
all network interfaces.

rlm_radius: Add conn_release callback and cleanup release logic

track: Enable "free by ctx"

trunk: Add callback that is called any time a request is remove from a connection

Typo

trunk: Do tconn assignment earlier

trunk: Zero out log entry structs

Typo

clean up and allow async xlat's to work again

disable this for a day until we finish the async tmpl expansion

pcap: Fix multiple issues with fr_pcap_device_names - Closes #3363

- Strip trailing separation char correctly.
- Don't over allocate string buffer
- Don't use snprintf (it's slow).
- Don't use is_truncated... For some reason it doesn't work correctly here.

trunk: Record info about the tconn the request is assigned to

rlm_radius: Fix potential double-free (#3366)

%{1} is async-safe

make `foo` work again in the config files

this isn't strictly necessary and should probably be deleted.
But for now, it's a simple way to test UNLANG_TYPE_TMPL and
the async exec functionality

set our async_safe value, based on the childs value

UNLANG_TYPE_TMPL is allowed to yield

walk over the list of child xlat nodes, too

warning fixes

remove UNLANG_TYPE_XLAT_INLINE and replace with UNLANG_TYPE_TMPL

which has the same functionality, but will also allow for async
template expansions.

start of code for async template evaluation

add more 'const'

we shouldn't be modifying instructions as they are being interpreted.
they're essentially static after the server starts.

radsniff: Check if fr_radius_init() started with success (#3362)

radius_udp: Skip free ID allocations

trunk/radius: Print out detailed state log

request: Record where allocation occurred

Generalize the "hack" in dhcpv4_encode_xlat()

Add three functions:
- fr_cursor_filter_head()
- fr_cursor_filter_next()
- fr_cursor_filter_current()
to let one select items for which an "evaluation function" passed
the item and a context value returns true.

add and expose fr_exec_nowait()

which is much simpler than the previous code.

add and expose fr_value_box_list_flatten_argv()

which takes a list of value box groups (e.g. from xlat_eval)
and turns them into an argv-style array

constify things that don't modify their inputs

add and expose tmpl_async_required()

add and expose xlat_async_required()

break string across lines

radius_track: Non-debug version shouldn't have file/line

rework so CHILD_GROUP is no longer necessary

re-do commit 798ef3a9b

return something

track: We still probably want this check for non-debug builds

schedule: Fix debug messages for workers

trunk: Remove asserts limiting states we can enter the failed state from

connection: Allow nested signal pausing, and prevent duplicates

trunk: Pause connection signal delivery in more places

track: Disallow double allocations

radius: Add check for treq being freed without releasing its tracking entry

probably not a valid assert given the way the rest of the public API works

trunk: Assert if we attempt to requeue a request in an invalid state

fixup evaluation of XLAT_CHILD nodes

put the result into state->rhead
on resume, merge that result into the parent group

parent string value from value_box we just created

Fix memory leak in radsniff (#3359)

We should call pcap_freealldevs() always after pcap_findalldevs()

add async_exec method for testing async execs

not done, and mostly for debugging the framework.  Will be
extended as the framework is fixed.

fix typo

allow things like --foo=%{bar}

for argv expansion.

stop on "p == end", too

terminate string

forbid duplicate sections

rlm_radius: better assert

rlm_radius: Assert if we still have tracking entries when the connection is being closed

more debug messages

return "read error" when we're done reading

which then ensures that the sockets are closed.

a blocked worker is now a non-fatal error.

The network side just tries another worker.  And if all workers
are blocked, it suspends all network reads.

Then when a worker replies, it unsuspends the network IO

pass "inlen" to the various functions

which mostly just pass it along to the functions they call,
and don't check it as much as they should.  But it's needed for
later work.

scheduler: Don't need to destroy the network mutliple times

fr_network_exit now destroys the network too.

Calling it again on line 666 results in use-after-free issues with the message set.

network: Pop messages before signalling workers to exit

fix problems introduced in commit 3e7e6970

u->code != code.  They are different variables with different
meanings.