fall back to component method if named method doesn't exist

prefer named methods instead of old-style methods

the PID can be zero if it timed out

save the processes exit status, not the raw one

sbuff: Simplify macros and reduce line count by using generics to make the same macros work with sbuffs and markers

sbuff: Unused char classes

remove unused variable

remove allocate_clear

it can cause issues when SKIP_LOCKED is used.  There is also no
real reason to clear IPs before allocating a new one.  The cleared
IPs had already been expired, so clearing them does nothing other
than remove MAC / NAS / etc. information.

This change results in slightly different behavior than in v3.
In v3, expired leases were more likely to be granted new IPs.
In v3, expired leases are more likely to be granted the same IP,
if it hasn't already been reused

looks like some versions of antora don't like this being empty

don't run radclient on "make test"

until we find out what is wrong with the acct_3 test

remove pre-proxy and post-proxy

mods-available/ and sites-available/ still reference pre-proxy
they need to be updated to use the new proxying methods

remove stats from old listeners

since the old listeners are completely unused, this is fine.

these changes are part of the general "clean up the server" fixes

remove old-old listener stats

these are unused even in v3.  So we remove them for v4

remove held_stats

which were updated, but never examined.

no one uses WITH_BFD

remove WITH_ACCOUNTING,  but keep all of the accounting code.

There were a number of cases where it would still build accounting
functionality even when WITH_ACCOUNTING was not used.

remove WITH_COA

which was a special hack for incoming CoA packets. Now that
all of the processing is done in standard methods, this feature
is no longer relevant.

removed WITH_ASCEND_BINARY

remove WITH_DYNAMIC_CLIENTS

remove WITH_DHCP

Formatting

Add control{} for performance testings (#3508)

It allows us to verify connection status using radmin.

ring_buffer_test: Fix typo (#3537)

Fix use of table name

Better initialiation of connections

Tables must not be NULL terminated

table: Record literal string length in tables to avoid using strlen during searches

sbuff: Allow parsing bools

Typo (#3548)

Fix bad table terminators

typo

fix up #ifdef's

use the correct tracking data structure

Create DHCPv4 and DHCPv6 LDAP attributes, and renumber OIDs

sbuff: Fix use of NULL pointer

remove many unused or unnecessary WITH_FOO

we should probably just get rid of all of them, TBH.  v4 uses
dynamic libraries for features which were compile-time changes
in v3.

sbuff: Typo in safecpy check

hex: Add allocating version of fr_bin2hex

don't need to take address of ipaddr

it's already a pointer

log text on failure, too

clean up and fix build

fix status code return

simplify compile_call()

so that it uses fewer special APIs

simplify compile_call()

so that it uses fewer special APIs

travis fixes

add missing ;

check length of data copied

actually set YIADDR.  Oops.

And rearrange code so that the success case is first, which makes
it a bit easier to understand

add missing parameter

move "add arp entry" to arp library.

clean up and enable updating ARP table entries.

ws

sbuff: Simplify macros

sbuff: Use preprocessor to figure out the length of terminal elements

Saves using strlen

sbuff: Add support for multibyte terminal chars

sbuff: Fix issue with buffer extensions

clean up and clarify the logic for sending DHCP replies

print out header attributes, too

fix fr_dhcp_print_hex() to work

clean up compile_empty()

so that it no longer needs fr_cond_type_t.

And clean up the various uses of "skipping if / elsif" so that
it makes more sense

sbuff: Fix integer output when next char is not terminal

sbuff: Add overflow checks to advance and simplify setting marker positions

ippool - mysql: Fix and comment SKIP LOCKED usage (#3544)

don't build VPs from NULL value box.  Fixes #3545

don't leave dangling pointers around

dbuff: fr_sbuff_set needs explicit cast for some operands

quiet clang scan

print: Deal with empty octets boxes

fix sign

hex: Fix clang scan issues

hex: Add explicit error checks

Quiet compiler

Can't set void to NULL

Not being able to alloc an array is an error

sbuff: Convert hex functions

typo

track lease queries by giaddr, not chaddr

sbuff: Switch to distinct in/out initialisers

sbuff: typo

sbuff: Quiet compiler

s/tmpl_unknown_attr_add/tmpl_define_unknown_attr/g

sbuff: Fix operations on overlapping buffers

value/tmp/redis/unit: Fixup uses of fr_value_str_unescape which now uses sbuffs

sbuff: Fix trailing num char detection

Revert "round robin among network threads"

This reverts commit 6f9baf3fa57a03ace86404cd747a1806bd6eee64.

sbuff: Check for double escapes

sbuff: Always produce buffer for alloc sbuff functions

libjson-c5 just landed in Debian

Handle SQLITE_BUSY as an error (#3539)

Exclusive locks can cause SQLite to return an SQLITE_BUSY error.
Treating this as an error causes processing of the current packet to be
abandoned complete with clearing up of the query.  Previously it was
being treated as a need for a reconnect which was causing a memory leak
as the existing connection was not cleared before reconnecting.

sbuff: Unescape function should return the number of bytes written

Like any well behaved function which writes out string data

sbuff: Trim talloced buffers

sbuff: Better formatting

sbuff: Add the ability to interpret certain escape characters, but not transform them

sbuff: Add the ability to process escape character sequences like \x<hex><hex> and \000

round robin among network threads

sbuff: Add support for parsing unsigned integers passed as octal or hex

sbuff: I suppose we should use standard names

sbuff: Add function to de-escape strings when copying them out of an sbuff

This makes dealing with config strings easier

sbuff: Protect against errant copies

sbuff: Verify length constrained functions terminate output on zero length input