use "text" instead of "varchar".

there's no reason to fix these fields at a small size

rlm_python: Anchor with a space so we don't mangle the middle of flags

See #3693

rlm_smtp with unit test (#3547)

* rlm_smtp with unit test

* Fix typo, remove extraneous info.

Co-authored-by: Arran Cudbard-Bell <a.cudbardb@freeradius.org>

Make the underlying dbuff version of fr_radius_encode_pair() visible

This makes it possible to switch users of fr_radius_encode_pair()
over to dbuffs individually rather than all at once.

typo

don't list all structural types, just use the pre-existing definition

minor tweaks

if there are errors decoding the packet, we might as well leave
pre-existing VPs in place.  The caller will then generally
free all of the VPs.

Missed include

Signed-off-by: Arran Cudbard-Bell <a.cudbardb@freeradius.org>

Not just for dictionary attributes any more

Make the underlying dbuff version of fr_dhcpv4_encode_option() visible

This makes it possible to switch users of fr_dhcpv4_encode_option()
ver to dbuffs individually rather than all at once.

Make fr_sbuff_is_hex() use isxdigit() (#3700)

Satisfy clang static analysis

Got rid of assignment to a not-thereafter-used variable.

Remove vp->next from decode.c

.

Remove vp->next from users_file.c

Correct doxygen comments for rlm_ldap and rlm_winbind

Add tidying to rlm_ldap and rlm_winbind

Correct doxygen

Remove unused VALUE_PAIR** from paircmp functions

move fr_dhcpv6_decode() to use fr_cursor_t

Englishify

s/__offsetof/offsetof/

Add generic extensions for different dictionary structures

fr_dbuff_extend_lowat() returns a length, it never returns 0

use macro instead of repeated code

cast to allow generics to work

Add initial version of extensible dbuffs (#3589)

Convert RADIUS encode_value() to use dbuffs. (#3631)

Make the underlying dbuff version of fr_dhcpv6_encode_option() visible (#3691)

This makes it possible to switch users of fr_dhcpv6_encode_option()
over to dbuffs individually rather than all at once.

move fr_dhcpv4_decode() to use fr_cursor_t

added more tests

fix up dhcpv4 decoder.

Options are allowed to have zero length.  In which case they
MUST be "octets" or "string".  Or for other data types, they MUST
be converted to "raw".

Fix up the decode_value_internal() function to skip calling
fr_value_box_from_network() if there's no data.

Fix the callers to allow it to return 0 data.

fix fr_dhcpv4_decode_proto() to return the total number of bytes
decoded, instead of "0" for success.

Update and add more tests for error conditions

copy child_struct, too

tighten limits on VALUEs and auto-generated VALUEs.

We need to update the dictionaries to fix them.

The "Auth-Protocol' field in DHCPv6 is not really a 'key' field.
Instead, the key is a combination of 3 fields.  And IANA has
assigned values for them, but it's not clear what are the
corresponding structures...

Deal with cached vendors correctly when copying

Formatting

Fix logic issue in fr_dict_attr_by_qualified_name_substr

we don't use this

move sql_getvpdata() and sql_fr_pair_list_afrom_str() to use fr_cursor_t (#3690)

Temporary fix for child struct copying (i.e. don't do it)

Disable memlimit on mprotected pool

This is an upstream issue with realloc and memlimit on pools

More information on realloc failure

Whitespace

Fix enum debug printing

Add function for debugging attribute extensions

Provide a fixup pool

We don't always have extensions

Fix merge issue

Unused

Add better support for copying extensions

Move enumeration value lookups into the context of the attribute

Shuffle functions around

it helps to initialize hash tables before dereferencing them

check return codes, and print useful error message

move to fr_cursor_t API

now that we don't need to magically initialize src_list

initialize the src_list

it is used later in the code, BUT on some code paths, the only
initialization of it is in a debug section which prints the VPs

move fr_tacacs_decode() to use fr_cursor_t

Not there yet

More fixes

Check the talloc chunk before reallocing

Allocate 'name' as part of the da

Update assert

Fix assert

No children for groups

minor hash table cleanups

Minor tweaks

Move children into extensions

Add mechanism for adding type specific dictionary extensions

allow for struct[n] in the dictionaries

to declare that a structure is fixed-size.

Then if it's set, check the MEMBER sizes against it.

This is in preparation for allow STRUCTs to be embedded into
the middle of other STRUCTs.  If they're variable sized, that's
hard.  If they're fixed size, it's a lot easier.

use correct parent

validate the flag field even if it's zero

because the validation function also sets flags->length, among
a few other things.

This change lets us calculate the correct size of a structure

typo

Tests: Stop and report when cursor test fails

fix typo

fix typo

Proper error message

Give in to static analysis stupidity

return "false", not anything else.

and add bool/uint64 to allowed data types for bit fields

unknown attributes are now OID 0

Convert fr_dict_print_attr_oid to sbuffs

make fr_struct_from_network() use enumv->child_struct[0]

which now means that it doesn't care about the data type of the
key field

cache child struct da in fr_dict_enum_t

in preparation for adding unions

allow "ref=..." for ATTRIBUTEs of type "tlv"

so that we no longer need to duplicate atributes which have common
children.

don't allow over-write of subtype if extra bit is used

quiet static analyzer

typo

groups cause child refs to be followed

use da->da_stack for known attributes

add and use fr_dict_walk() function

we will need this when each structural attribute has hash tables
of child names / attributes

add dict_attr_can_have_children()

because there are multiple checks for da->children, and we would
like to generalize / abstract these checks.

VALUEs cannot be defined for structural data types

add ALIAS support

which is a simpler way of defining multiple names for one
attribute.  This keyword allows aliasing of the name, without
the need to repeat number, data type, etc.

At some point we will forbid duplicate ATTRIBUTE definitions,
and instead require the use of ALIAS

Use the correct parse function for attribute references

Otherwise it might produce an "unparsed" tmpl and the code will crash

Support parsing src/dst references for subrequest

add assert to shut up static analyzer

we can't talloc_steal() the same "da" to multiple parent "vp"s.

remove use of vp->next

don't access vp->next

Split fields for the various keywords into separate structures

unlang_group_t has gotten too confusing

Fix dhcpv4/raw.c

Shorten fr_socket_addr_t to fr_socket_t

We don't always have asan...

Use fr_socket_addr_t in packets too