sanity check request->async, too

Revert "disable exec wait and ignore child signals"

This reverts commit 5432dc9359752cea74ed29b249696474e0ddb4b7.

Now that exec.c tracks 'nowait' children, after the previous
commit.

track "nowait" children we've exec'd.

So that they can be cleaned up appropriately.

show the line number where the tests failed

disable exec wait and ignore child signals

until such time as "wait" is integrated into the event loop.

whitespace

doxygen

remove unused variable

instruction may be NULL?

Install signal handler to destroy EAP sessions

Explicitly free rctx

Else we'd leak memory into the REQUEST *

Don't need this anymore

Try and emulate Linux symbol binding behaviour

Don't bake module.c into unit_test_module

It's provided by libfreeradius-server, and building with it here gives us a chimaera effect, where half the symbols resolve in the local version, and half resolve to libfreeradius-server.dylib

Sanity check for unlang_signal

Resume frame should repeat too

Pass a repeat function for going back up the stack

Case where q is NULL

Switch to two level namespace in macOS to fix OpenSSL/BoringSSL conflicts

Set/unset module names when calling signal and resume functions

Not just cosmetic fixes, we hit asserts if the RINDENTs() and REXDENTS() are done with an inconsistent request->module pointer.

This is because there's a separate indentation counter for when the request is being processed by a module, and for when the request is running outside of a module.

Formatting

Fix low level debug output in xlat_register

Fix ocsp.cnf

So we can generate fake certs again...

Start work on making rlm_eap async

resolve symbols now, and use dlerror

typo

doxygen

"documentation" is "useful"

hacks for LD_PRELOAD, where we just load it ourselves...

we don't use checkrad in v4

or, at least, in the same way as it was used in v3

parse ENV subsection before loading modules or threads

so that we don't have to set LD_PRELOAD on the command line
for stupid libraries

check for address->client

Allow unlang OPs to push generic functions onto the stack

This allows functions deeper in the C call stack to set up resumption points.

This makes submodules and even internal module functions far easier to integrate, as they don't have to communicate with callers further up the C stack, to get them to push the resumption functions they need.

Move more of the module specific code out of unlang_op.c

There's still a bunch of module specific event handler functions which either need to be made generic or moved into module_unlang.c

Always initialise quote

Fix handling of NULL RHS in update sections

Reset alternate on an additional code path

...an xlat can yield multiple times

Improve xlat evaluation debugging

It should be about equivalent to the synchronous debug output, with the added bonus feature of printing the function fmt string and the expanded arguments.

Add function for copying value_box lists and for checking list taint

more hacks for tmpl_verify

Doxygen

Fix inifinite loop in xlat evaluator where both expansions in an alternate expansion evaluate to NULL

Use ephemeral xlat instantiation in unit tests

Add cancellation function for rest xlats

Extra sanity check

if vp is NULL.... don't dereference it

Make debug message clearer

Don't leave stale REST-HTTP-Status-Code attributes around

talloc_get_type is only in debug builds

doxygen

...and another

Typo

Print out more of the error stack

...and fix EAP AKA/SIM identity parsing so it says exactly why it thinks the permanent ID is invalid

add const

Fix talloc_list_get_type_abort when get_type_abort is disabled

Fix input to map processors to be async

Typo

Don't double alloc the output box in xlat_bin

Formatting

Move private structures into a private header file

Organisation

Check for list taint

Add incr_ref argumet to fr_value_box_copy_shallow

We don't want to increment reference counts when shallow copying globally allocated boxes

Add a value box 'meta' type

Add fr_value_box_alloc_null

To allocate null value boxes which are then populated with another function in the value.c API.

This means you don't have to pass the taint/enumv flags in twice (which looked stupid).

don't re-define macro

update track, too

Don't crash if an async function is used in the legacy xlat functions

Add generic talloc_list_get_type_abort function

...and use it to verify the state of the result xlat eval result list

it's now allowed to delete entries with 0 timestamp

because we may receive packet A (slow).  The NAS gives up and
sends packet B, which we reply to.  After that, packet A is done.
At that point, the tracking entry has timestamp=0, uses=1, so
we can now delete it.

reference count tracking table entries

if we found an entry in the tree, don't re-insert it...

pass recv_time to tracking entry delete

so that it can do all of the logic necessary for cleanups

Zero out input value box

Else we get a garabage enumv field and that breaks everything else

Don't try and use result after passing it into xlat functions

Compule recv Access-Request in TLS verify virtual server

If no input...

Deal with empty xlat expansions in a sane way

We actually want a pointer to a NULL pointer because the value box code gets excited

Concat input arguments into URL string

Fix empty result in xlat_frame_eval

Both variants?

Pass empty result when calling xlat_frame_eval_repeat internally

Need to pass in **result so xlat_frame_eval and functions can indicate when they've consumed result boxes

Set Accouting-Response code on handled

check map, too

Revert "Target needs to be absolute for the rule to run in the correct order"

This reverts commit e46e496911ae57ce8843d8434bb257bbbc7996e1.

which breaks the build on most systems

temporary hack

assert that the pointer is of the right data type

memdup the packet we're injecting

Target needs to be absolute for the rule to run in the correct order

Fixup more tests

Adding [0] in conditions and & before attribute references

rhel7 libwbclient RPM dependencies

Doxygen

Use generic talloc list free function

Doxygen

clean up pointers on clone

and only free in the master

retransmits start at 1, not at 0.  Fixes #2166

The "total packet" counters are maintained separately.
So we need to count retransmits starting from 1, not 0.

Which also means we *print* retransmits starting from 1, not 0

Hack dynamic LHS functionality back in

...I will fix this mess when there's not an impending deadline