Notes on stdin cleanup

Fix spurious warning

Fix minor build errors

pid is signed, so give it a default value of -1

Deal with race between EVFILT_READ and EVFILT_PROC

If EVFILT_PROC is delivered first we may never end up reading pending stdout/stderr data.

Deal with EWOULDBLOCK gracefully in log_request_fd_event

Calling __builtin_clzll with a 0 value is undefined (and it does the wrong thing)

Allow users of event.c to get at the callbacks registered

This is mainly for the exec code so it can manually call the read callbacks before processes exit.

Add atexit.h macro for doing automatic initialisation and freeing of global resources

Should be much cheaper than pthread_once

Adjust initial LST pivot stack size in accordance with initial size (#4213)

It's now initially max(1, log2(initial array size)).

order requests by sequence number, too

pass fr_io_track_t to track_create function

so that we can free old tracking entries by just talloc_free'ing
them

pass instance / thread instance to track_create

in preparation for doing more aggressive cleanups of tracking
entries

Fix error marker check

Make lst_validate() consistent with current lst functions and types (#4212)

Remove LOCAL_PID hacks, we'll reimplement it in libkqueue

Fix event_timer_location_cmp

Add init parameter to LST creation closes #4210

The parameter determines the initial capacity choice: 0 will get
the default initial capacity, while any other value will be rounded
up to a power of 2 and used as the initial capacity.

Meant to revert the other commit

This reverts commit f1760b1302a00524c3a8cf6298b0bb0b5ea53524.

it's now fr_unix_time_from_tm()

all of the uses of it either set the time zone, or used mktime()
and then called fr_unix_time_from_timeval().  So to simplify things,
we just use fr_unix_time_from_tm()

change to passing CONF_SECTION, and publish it

update initializer

pass actions to compile function

in preparation for making it public

parse "retry" subsection of "actions"

add "retry" subsection to actions

macro to initialize data structure

More fr_time_from_utc fixes

Add safety check  for mod_post_auth

Remove perl keyword test

Move perl xlat tests into the module module

Optional modules shouldn't be in keywords

Fix sign issue with unix time conversions

Fix unbound timeouts

Allow xlat calls without instance data

More helpful TLS alert messages

Move tls/utils functions into a new header

Add cf_log_pwarn

Our version of unix time can't be negaitve

Simpler way of getting the serial

Correct uses of fr_time_from_utc

Verify certificates are valid

Check certificate validity periods explicitly for lib/tls users and rlm_cipher

Add %(<inst>_certificate:notBefore) and %(<inst>_certificate:notAfter)

move actions to struct

better error messages.  Fixes #4204

move actions to struct

use typedef to simplify things

this is "abinary"

ported manually from v3

Fix typo (#4208)

Add assert to reassure (or shut up) "make scan" (#4206)

Use CMP() in  _trunk_connection_order_by_shortest_queue() (#4205)

enforce response_window only for synchronous requests

remove unused variables

rework to respect response_window, etc.

add and document response_window

fix typo

cleanups and corner conditions for deferred transitions

add support for multiple deferred state transitions

add signal handler

Data usage for MySQL: Run SP as invoker not definer (#4201)

Usage schema extension for SQLite: Typo (#4199)

Typo in 4c67d412956d2efed8b6ab4f216eb49abf3119c5 component of PR #3635.

Fix fr_machine_s

Revert "LSTs likely slower when used here"

This reverts commit 6a88d68e3e5c4398e7feee501a855ea84a57633d.

Free the heap

Needs to be alloced in hp not ctx

Shouldn't need to specify ctx... but maybe this will fix the leak?

Move array pop code to function and prevent it being inlined

Not sure how that got there...

Record first pop

"init" parameter isn't in LST alloc functions yet (#4195)

More fixes

Less confusing way of representing struct fr_heap_s **

LSTs likely faster here

Fix heap tests

LSTs likely slower when used here

This reverts commit c59a612b93a55b130ea5fea768c76f6c661b17e0.

Fix up queue cmp tests

Higher memory locality gives significant performance gains for small numbers of heap elements

force link errors on OSX for unknown symbols

When linking binaries.  So that we get errors at compile time,
and not at run-time, or even later in CI tests.

Link to the crypt library

Minor tweaks

cast function pointers to void*

because we're using them as placeholders.  We don't actually
care what they are

use [] instead of [0]

Add lst/heap comprison test

Typo

initial stab at generic state machine handler.

should not have been committed

Move crypt code into rlm_pap

Unused

Fix indent

Switch to unsigned type for fr_lst_index_t (#4190)

* Switch to unsigned type for fr_lst_index_t

This involves adding one to the true index when storing it in an
item, and subtracting one when reading it for use.

* Switch to unsigned type for fr_lst_index_t

This involves adding one to the true index when storing it in an
item, and subtracting one when reading it for use.

* Switch to unsigned type for fr_lst_index_t

This involves adding one to the true index when storing it in an
item, and subtracting one when reading it for use.

Re-enable support for TLS 1.0 fixes #4191

Allocate the stack as part of the lst structure

Control the initial number of elements in a heap

Sprinkle more unlikelys

Sprinkle unlikely

Pre-allocate LST stack memory

This is mostly to ensure good locality

Do talloc type checking if we have one set

Use a consistent type for representing stack indexes

No reason for stack indexes to be signed.  There are no negative indexes.

Add explicit bracketing

Minor style fixes

Doxygen function headers must start with /**

Mark up iterators as nonnull and move doxygen comments to source file

Number of elements and size tracking should be unsigned integers