return OOM on oom

we have talloc_const_free(), so just use that

clarify code so that we can find uses of a variable

remove unused function

pair_afrom_da() already copies the unknown da

try to quiet static analyzers

clean up in preparation for further raw cleanups

fr_dict_unknown_afrom_oid_substr() now continues from where the
previous parser left off.  The raw handling is now essentially
all out of the dictionaries

allow unknown attributes to be relative, too.

As part of a general cleanup and rearrangement, fix the code,
have less code, and make it more functional

switch encoders to use vp->vp_type

because vp_type could be different from da->type in some cases

fr_pair_to_unknown() -> fr_pair_raw_from_pair()

because all of the callers are creating a raw pair from malformed
data, and aren't creating an unknown pair.

more debugging, and return OOM on OOM

only free the the vp on the error path

if the first byte is non-zero, then the result is non-zero,
and we don't need to check it on all paths

don't bother checking chaddr size during decode

fr_dhcpv4_ok() checks if htype==1 && hlen==6, and fails if that
isn't true.

So either that test passed, or the caller didn't call fr_dhcpv4_ok()
to verify the packet.  In either case, there's no reason to do
additional validation here.

clear the value, to be consistent with other uses of free_value

don't clear the box, clear only the value.

hoist TLV checks to before creating the VP

because there's no point in creating the VP and then freeing it.

use fr_pair_raw_from_network() instead of hand-rolling it

define and use fr_dict_unknown_copy()

because we want to get rid of fr_pair_to_unknown().  The caller
should instead have created a raw attribute

don't need to check is_raw flag

because we now always set is_raw for unknowns, so it's enough to
check the is_unknown flag

hoist setting of "is raw" out of protocol decoders

and into the dictionary code so it's more centralized.

Another side effect is that unknown attributes are now always
"raw".  Which is likely what we should always have done anyways

hoist "is_raw" to be more public

There was already an "is_raw" field in the unresolved sub-structure
so just hoist it out, and use it instead of da->flags.is_raw

fix unused

use macros / functions instead of accessing fields directly

define and use vp_raw

In preparation for moving the "raw" flag out of the dictionaries

fix corner case  CID 1544258

use the correct name for the parnt type

Simplify fr_ldap_edir_get_password

Return an unlang_action_t from ldap bind auth functions

Ensure we free memory on errors

Handle rcode from previous async call

No need for p_result in async LDAP functions

It was only set on failure, and that is handled by the
UNLANG_ACTION_FAIL return code

Add max_request_time timer to unit_test_module

Typos

Ensure auth bind resume functions clear up after failed requests

Add callback for failed ldap auth binds

No need for request_free callback

bind_auth_ctx is parented of the trunk request so is freed automatically

Typo

always decode tacacs as nested

allow tacacs to encode nested attributes

typos

it's smarter to have clear-text unit tests

instead of encrypted ones.  We should also add unit tests for
encrypted packets, too

make pair_legacy_nested=true the default

EAP-SIM and TACACS+ do not yet handle nested attributes, so they
manually set the flag to false.

remove flatten / unflatten tests and keywords

they are no longer necessary, as we are moving to nested attributes

don't run unbound tests on OSX

it fails in CI, and we don't care enough to fix it

json: Add json escape

minor tweaks

tests for nested types

loop over nested extended types

VSAs can only contain internal attributes and VENDORs

the RADIUS protocol cannot encode attributes of type 'group'

check return for CID 1544245

Realistically, unlang_xlat_yield() _always_ returns
XLAT_ACTION_YIELD, and the only reason it returns anything
is so that we can do "return unlang_xlat_yield()"

decode TLVs nested

nested tests for wimax

note that giving flat lists results in us creating nested ones

Add 'Net.*' pairs to TACACS

VSA types are decoded as groups

note that we will have to audit all of the "decode raw" routines.
the current behavior is for the _called_ function to create a
"raw" parent if the decode fails.  If the parent is itself a group,
the called function will instead have to return an error, and then
the calling function will turn it into a raw attribute

coalesce VENDOR attributes at the same level

VSA types are decoded as groups

coalesce VSAs at the same level

Insert new pair where we were asked to

decode as nested

except for FR_TYPE_GROUP, which gets put into the group VP, but
the attributes are decoded from the dict root

@todo - we don't coalesce vendor / VSA data types, and we should
probably do that.

fix group decoding

on failed decode of group, create a raw attribute instead of
returning an error.

return in the group handler, so that we don't set vp->vp_tainted
for a structural data type.

Don't leak memory

ldap: Quiet clang

ldap: Accept profile in either URI form or DN form

ldap: Fix auth tests

ldap: Add profile xlat to allow application of arbitrary profiles

ldap: Take a ctx in fr_lap_map_expand

ldap: Return the result code from the LDAP operation in rlm_ldap_map_profile instead of messing with the rcode

ldap: Make type names more explicit

ldap: Replace confusing macro with static inline

Split out and abstract ldap profile code

also use local libraries for unit tests

Initialise more boxes correctly

Revert "always call fr_pair_tlvs_from_network() as nested"

This reverts commit 53d37c2b16802920a6d6ebcba568859e48dc5789.

update tests, too

this is now nested

always call fr_pair_tlvs_from_network() as nested

always call fr_pair_tlvs_from_network() as nested

more peephole optimizations for logical operations

mark up "@todo" with more comments and classifications

decode DHCPv6 structs as nested

set mutable recursively

typo

Remove unused variable

Reconnect trunk connection if the error is FR_LDAP_BAD_CONN

Correct passing of fr_ldap_connection_t in async functions

The comment about changing the connection due to auto re-connect is
wrong - that was the old sync code behaviour.

Use standard routine to proces LDAP errors

Allows distinguishing between e.g. syntax errors and connection errors.

Handle LDAP query result codes correctly

Add callback to tidy up if LDAP trunk requests fail

notes on future peephole optimization

pass in the correct operator

if aavailable, check the enum name at parse time

only set fixed length for leaf types which have fixed length

make sure the value-box is initialized

If we have a LHS leaf, pass it to the RHS tmpl parser

which means that we catch more load-time errors for typos in
the configuration files.

and add test case.

The RHS resolution is done in pass2, so there's no issue with an
enum name being defined after the configuration files have been read.

add unknown only if the LHS is unknown

and assert that the LHS is an attribute, which it should be for
being a bare word

use attr_net_*, and suppress Net.* if they exist in the list

Add ldap test using SASL proxy authorization for admin bind

Pacify coverity