fix typos

fix same typo in comment, in different files

fix typo in comment

fix typo in comment

fix typo in comment

fix misplaced parenthesis

tolower(cmd_str[1] != s ) does not actually turn anything into lowercase, since it is evaluating a number and not a char

fr_value_box_cmp() is used in stable sorts

and therefore can't call fr_digest_cmp().

use MEMCMP_FIELDS() in paircmp() too

and add note about the secret flag

define MEMCMP_FIELDS() and use it

which is a little clearer (and less wrappers) than MEMCMP_RETURN

just call cache_expire() instead of doing it locally

which handles more corner cases.

also other minor tweaks

try to clear out 3 cache entries for every 1 lookup

which lets us clean up old entries more quickly after a spike of
traffic

restore write errno after ftruncate in partial header write path

Signed-off-by: ethan-thompson <ethan.thompson@networkradius.com>

fix int truncation of ssize_t return from fr_concatv

Signed-off-by: ethan-thompson <ethan.thompson@networkradius.com>

skip NULL request pointers when marking batch entries runnable

Signed-off-by: ethan-thompson <ethan.thompson@networkradius.com>

use fr_hash_string for file table lookup to match filename_hash

Signed-off-by: ethan-thompson <ethan.thompson@networkradius.com>

always return "updated" if we updated something

cache xlat can succeed if it returns no data.

i.e. "no data in cache" isn't an xlat failure, it's an xlat
success with no data.

set_ttl with TTL=0 is a noop

fix cache it logic

so that it follows the documentation

pass correct pointer to handle

return 0 on count error, not negative.

because the return value is unsigned.

allow tests without the full re-run of all tests

cache_foo depends on rlm_cache, too

Correct section name

minor cleanups in thread code

move fd management to narrower scope

and use fileno(fp), so that we have fewer concerns with managing
the fd

move common code to fr_thread_start / fr_thread_exit

rename fr_thread functions for clarity

move thread information to a common data structure

move thread create && wait to common functions

move thread setup to common function

move thread instantiation to common function

simplify the names

less shouting is good.

Initial baseline multi-server tests for load-generator -> 5 homeserver and load-generator -> 1 proxy -> 2 homeserver scenarios.

tweaks, and use fr_dlist_foreach() in more places

wrap multi-line macro in do/while

free dbuff on encode failure

minimize includes

Pacify Coverity (CID #1686327)

Check return (CID #1686328)

Check return (CID #1686329)

da can't be NULL here

return failure on error

check the return code of adding the compile list

typo

unify code which warns about unused processing sections

spelling mistakes

fix error message typo

check return value

if pthread_mutex_init does not succeed,
then trigger_mutex is not intialized,
so return -1

typo in comment

fix typo in comment

fix insert/remove from free_request with preprocessor macro

instead of using dlist functions and manually tracking head/tail,
use wrapper functions named for the list being updated, with
clearer and more descriptive names.

fix comment and debug message typos

fail connection with proper variable values, send error message

set state.last_failed to now,
decrement state.pending to avoid misleading "already at max" error message,
give information in error message when mem alloc fails for a new connection.

fix memory leak

close(fd) before returning

parser functions return ssize_t

no need to do head && then remove.  Just do pop_head()

insert new trunc requests at head
because we free from the tail

fix typo in comments

fix typo in comments

to to, in in, is is

fix typo in comments

the the

fix typo in comments

is is

fix comment

fix whitespace stripping that never executes

buff[2][slen] is set to '/0', which is not a whitespace character, so we need to check buff[2][slen - 1] to actually remove trailing whitespace

fix error message typo

correct error path when failing to open file

remove extra semicolon

initialize var

we can enter closed from shutdown && timeout, too

_connection_free() and connection_signal_shutdown() both enter
the closed state from the timout state.

Clean up on failure

Clarify comment

Control planes can be multi-producer, single consumer - so mulitple
threads can send / push a message.

Add coordinator startup / shutdown into scheduler

Add coord_attach callback to modules

Add support for pair list worker <-> coordinator data

On the coordinator side the pair list is then converted to a request and run through an interpreter.

Add API for coordniator callback instances

Allows callbacks to register an instance create function to create instance data and a set of callbacks to run as part of the event loop.

Add coordinator frame work for sending data messages worker <-> coordinator

Add fr_control_wait() to wait for a control message

Use a semaphore to ensure master thread has started before workers

Use mmapped memory for POSIX semaphores

In preparation for using separate processes in place of threads

Dynamically allocate the number of control plane callbacks

To allow the control plane to be used for an arbitrary number of message
IDs

Move control message IDs into headers relating to their use

print out packet type and ID too

clean up printing for regexes

print out strerror, too

flush the output, just in case

run scripts/build/min-includes.pl

to minimize the number of duplicate includes

remove duplicate headers

perl -ni -e 'print unless (/<(stdbool|stdint|stddef|string).h>/)' $(git grep -l stdbool.h src | grep '\.[ch]' | egrep -v 'build|autoconf' )

Include stdbool.h in default headers

Return on negative value (CID #1684172)

only call marker() functions once.

one the second pass, call set()

changed error message
replaced hardcoded value with the more useful library->autoinit->name

fix error message typo

fix debug message typo

fix warning message typo

fix warning message format specifiers
%d is for signed ints. index is uint_32 so it should use %u, which is for unsigned ints.

fix error message typo

initialize var

move function needing sbuff out of talloc.c

because otherwise we have circular includes of talloc to sbuff
to talloc, and things don't always work as well as they should.

minimize the number of includes for a core header file

fix uses of fr_sbuff_marker() vs fr_sbuff_set()

fr_sbuff_marker() is run once, and only once.  after that,
fr_sbuff_set() is used to update the marker