ldap: Check for the presence of cache_da

Fix clang scan errors

Rework flag, ref, and alias parsing

Move duplicates to aliases, and disallow duplicate attribute names and numbers

Break up attribute initialisation into phases

fr_dict_attr_by_name_substr sets its own errors

Check we didn't run out of parents

Add fields to store attribute location

Rework protocol flag types

Can't produce errors using name beause it might not be set yet

Formatting

Use "namespace" consistently

Add utility functions for applying alias fixups

Explicitly mark up attributes as name only instead of using magic numbers

Alias the "Compatible" vendor to cisco.vpn5000 to avoid conflicts

We don't need the length passed to dict_fixup_clone

Remove double declaration of KDF

Have the tests load their own custom dictionaries

Formatting and minor fixups in unit_test_attribute

Fix obvious, yet hidden, numbering error in dictionary.h3c

Alias Vendor-Discrimination-Str to Ethernet-Interface... they're both octets types

Fix obvious error in Site-Specific numbered attributes

...which was hidden previously by the fact we allowed numbering conflicts

For why not can sentence sense make

WS

internal encoder: ACTUALLY skip name only attributes, instead of just infinitely looping

If the input arguments were invalid... error out, don't just fail silently

eap-aka: Use enumv values from KDF instead of definitions

timeouts are perhaps not quite errors

MRC of 1 is really MRD

i.e. a simple duration, and not a repeated count

Better handling of SQL rcodes in xlats

If RLM_SQL_ALT_QUERY is treated as an error, then redundant SQL xlats
will try the same query again on each database host (with the same
result).

To handle switching to a different query if an INSERT fails due to key
contraints, then the return value of the xlat, which is the number of
affected rows should be checked instead.

No data returned is not a failure

Update MS SQL default schema

When using FreeTDS driver, `char` fields are returned with trailing
spaces, which causes "=" in the [op] field to become "= " - which fails
to parse.
Using `varchar` ensures only the real data length is returned.

Align rlm_sql_freetds NULL behaviour with other drivers

Remove unused function

FreeTDS doesn't return the number of rows from a SELECT

Update rlm_sql_freetds to use trunk connections

Unfortunately FreeTDS doesn't have the Async support that the Sybase
client library does.

Remove needless function

Add basic test of redundant SQL modules

Closed SQL connections can't be used for escaping

The handle gets freed when the connection is closed

Set return type for escape functions to ssize_t

So failures can be reported back

Move tests to be used by all SQL flavours

Set safe_for child xlats inside a redundant xlat

Correct comparison for end of retry period

Revert "Don't load the RADIUS dictionary in the xlat code"

This reverts commit 6cf9cd8db3a700b1ceb056cc14a0d03201966ed8.

Breaks CI tests of foreign protocol decoding

Use mod_accounting in send Accounting-Response

There's very little point in logging accouning responses in detail
files, and the sample config calls "detail" in the send
Accounting-Response section.

Add test of NULL SQL value handling

Align Firebird results handling with that of other database drivers

Handle NULL return values in PostgreSQL

Handle NULL return values in MySQL

Handle NULL fields returned in unixodbc

ODBC uses the data length indicator to report NULL return values.

Protect against NULL return values from SQL queries

Won't happen with the default schemas, but always possible with custom
schemas

Handle NULL returns from SQL when parsing map values

add this, too

this is static

remove call to write_flush

we don't need write_flush

add and use explicit verify_ctx for BIO

so that bio->ctx can always be application data

Don't load the RADIUS dictionary in the xlat code

We're not a RADIUS server... and this appears to be unused

Compiler is correct... if bio is nonnull, this should be do {} while ()

Support resetting the dictionary root

Don't automatically mark up defines as internal, as this breaks the DER encoder

Trunk "request specific" settings go in request, not requests

rename "activate" to "connected"

to match the connection / trunk API

as taken from PR #5428, with edits for v4

add fr_bio_flush() API.

which returns whether or not the BIO was flushed.

Ideally the write callbacks will also be signalled if the write
was blocked, too.

allow RADIUS clients without retries

document more authorization replies

add API to allow signals to run retries

i.e. on DUP signal.  We don't want to have a separate code path,
we just want to run the normal retry mechanism.

remove unlang_module_timeout_add()

and clean up the API for module timeouts / retries

add unlang_module_yield_to_retry()

hoist module timeout into main module state

the yielded ptr is associated with the stack frame, not the request

more cleanups on delete timers

parent the timeout from the frame state

so it is automatically cleaned up when the module exits.

remove unused code

request_ctx is a macro used by the server for other things

cap "next" at "end".

So users of the API don't have to manually check "end", they can
just look at "next"

rename to be clearer

and add comments about what the functions do

ensure that the retry struct is always initialized

before calling the sent() function

add tracking for timers to the radius client bio

Correct comparison of module xlats

Add Firebird mods-config to RHEL packaging

Firebird is not compatible with MS SQL queries

It is a fork of InterBase rather Sybase

Remove vague comment

Report what we're connecting to

Set Firebird connection timeout

Firebird can detect key conflicts

Update rlm_sql_firebird to use trunk connections

Allow for Firebird UPDATE ... RETURNING behaviour

when no records are updated, UPDATE ... RETURNING returns a row with all
fields set to NULL.

Set new allocated results size after allocation

Don't print results in fixed width fields

No other database drivers do this - and "     1" != "1"

SQL driver specific config is driver not dialect related

Firebird doesn't return the row count for SELECT

Formatting tidy ups

Do clearup in finish_query

Rather than when preparing the next query

No need for a mutex

Exclusive use of a connection is handled by pool / trunk connection
code.

Use talloc rather than malloc in rlm_sql_firebird

Don't call query finish when not needed

sqlippool_result_process is already calling query finish

Keep a list of all files we saw during loading the dictionary

This will be used later to record the files attributes were defined in, with buffer lifetimes that are equal to the attribute lifetimes

Field not used

Corrections to new table macros