Pacify Coverity (CID #1649515, #1649514 and #1649512)

Ensure detached requests don't try to access results in the parent

Add allow_core_dumps = yes to more test server configs for debugging

frame->state is not always populated

if xlat fails, then we return fail, too

revert commit 4086074515

remove unused structure

Pacify clang scan

Reject users correctly in TACACS tests

Set allow_core_dumps in module tests to allow debugger use

Mark frame as repeatable when pushing tmpl

Allows for muplitple tmpls to be pushed on the stack before returning
UNLANG_ACTION_PUSHED_CHILD, and all of them will be evaluated.

state->func_name and state->func are both NULL at this point

If the frame is cancelled, don't run next

Condition result is an unlang_result_t

Use unlang_result_t for edit xlat evaluation

Use correct return macro

Switch rlm_crl to use unlang_result_t

More LDAP debugging

Overwrite rcode if result->priority is equal to frame_result->priority

Always set repeatable if we have a repeat function

This means the start function will run as the stack is burned down

Don't check for operation failures in ldap_check_userobj_resume

Print list of attributes we're retrieving

Use correct field in map ctx

Don't do exclusivity check if we're checking the config

Output what group attribute we looked for

Make the query available immediately after resumption

Shouldn't default to returning ok in mod_authorize_resume

Add debugging functions to dump LDAP messages

Add sbuff array concat function

tmp;s failing should unwind the current frame

regexes should return fail if they can't find a group

Add the ability to print instruction priorities to stack_dump

Explicitly check for failure

add rctx into user_modify_resume if we're calling it directly

Use correct check for ldap_initialize success

Duplicate a bunch of code in unlang/function.c *sigh* and allow it to work when the function yields

Assert _before_ allowing talloc_get_type_abort to trip

Fix order of yield and search in  LDAP mod_modify

LDAP_OPT_X_TLS_PROTOCOL_TLS1_3 not available in every libldap...

Fix const

Fix minor issue in rlm_lua

p_result may be NULL

Add variant of functions which do not produce a result

Can't use stack variables for p_result

Only resolve priorities if the rcode is actually set

Frames always have an instruction

More p_result fixes

Allow p_result to be set for rlm_ldap_find_user_async

Switch ldap modifications to using process functions

Add sanity check in unlang_module_yield

Why?  Because if the yield is done _AFTER_ a function has been pushed onto the stack, we get undefined reuslts

Run the ldap.authorize method as module methods, not functions

Audit more uses of unlang_function_push

Results for call_env expansion are written to env_result

Notes on unlang_function_push calls in virtual servers

Nothing seems to use the rcode result from ldap_trunk_query_results, so remove the return codes as they're confusing

Return UNLANG_ACTION_FAIL from RETURN_UNLANG_FAIL

Explicitly store the result of edit get password operations

Explicitly store the result of bind operations in the calling frame in LDAP

Add note on using unlang_interpret_result to make unlang functions transparent

Make the eap_virtual_server_resume function transparent in terms of rcodes

Handshake resume function writes out results to eap_tls_session->state so we don't need to pass through the rcode

Make the logging destination resumption function transparent in terms of rcodes

Make the result pointer for the current frame available

s/unlang_interpret_result/unlang_interpret_rcode/

We discard the return codes from triggers because they're fully async

Add better notes on sigmasks

Check unlang_module_yield return code in ttls.c

Notes on how subrequests are handled in peap.c

peap: Cancel child request's frames

Pass unlang_result_t to modules instead of rlm_rcode_t

Populate this frame's result with the results of the resolutions

Fix scan issues

Use a dummy instruction in frame[0]

Allow the rcode from virtual servers to be redirected

Make discarding function rcodes official

Check we can find slapd

Fix arg order to unlang_subrequest_child_push

Only assert if the child request is runnable

Set priority of the process_reply function high enough to bubble up

Help tracing the crazyness

Yielded makes more sense than resumable

Ensure that if the child hasn't completed, then its parent must be yielded if it's signalled

Don't crash on request timeout

Use section result from rctx in eap_aka_sim state machine

Use unlang_map_yield for LDAP maps

Fix header include in dict.h

Remove useless calls to unlang_function_push and just use unlang_module_yield

Adding new stack frames is expensive, and there's no need to do this as we can just mutate the module's frame

Add better support for async maps

We really don't want to push additional frames if we can help it

No reason to use a function to represent module state when processing accounting queries

Reduce stack dumpage

No need to use functions to resume in the rlm_files module

try-no-sibling test needs to set the rcode back to ok

Now that rcode works correctly, having OR or AND fail when their children fail causes tests to fail

Have xlat push functions take a result_p

Print what's being expanded in stack_dump

Transactions need to operate on the result of the child section, not the current frame result

Rework interpreter to make handling of rcodes sane, and to discard rcodes entirely when a frame is popped

Use common function to print closing section debug

Correct calculation of overall result for parallel keyword

Look in the correct place for transaction results

Fix auto-added headers

Use new rcode location in catch