added test for malformed attributes

malformed VSAs get decoded as raw attributes

account for 4 bytes of PEN and 1 byte of length, too

ASAN poison regions of the buffer which we're not using.

Fix bad search/replace

Add src/dst address swap function

Unify socket address structures

Restore amount to advance by to fr_dbuff_marker_advance() (#3682)

Convert selected "dbuff" operations to macros that take dbuff or marker (#3674)

Aside from fr_dbuff_current(), which will use the marker's current
position pointer when passed a marker, these will, when passed a
marker, retrieve the value from its associated dbuff.

We leave the current marker functions in place so code using them
can change at will rather than forcing a flag day.

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

Add fr_dbuff_move() and tests (#3675)

use correct API to get 'octets' data type

malformed options are decoded as raw options, and don't return an error

s/request->state/request->state_pairs/

s/request->packet->vps/request->request_pairs/

s/request->reply->vps/request->reply_list/g

s/request->control/request->control_pairs/g

Add macros for pair lists

allow attribute references in subrequest

and update docs and tests

regenerate from config

check for overflow on each PEN + len + options

decode ipv4addr as ipv4addr, not as ipv4prefix

rename to DHCP-Network-Subnet

if it's data type "ipv4prefix", then the name should NOT be
DHCP-Network-IP-Address

Remove useless copy operation

Remove useless code in fr_dhcpv4_decode_proto

:gear: dhcpv4: Fix wrong memory context

It must be the received ctx instead of local allocation.

:gear: dhcpv4: Fix the data type value box

Ignore fuzzer crash files

statically define Tacacs-State

Fix UBSAN in fr_value_box_cast()

move run-time checks to compile-time checks

return on error

initialize the VPs list

more tests for multiple VSAs

typo

add more vendors, and reformat

encode multiple VSAs into option 25

remove unnecessary comment

add tests. and decode multiple VSAs in option 125

skip all of the VSA, even if we didn't read it all

Allow vendor-specific options in DHCPv4

make error message more descriptive

'Attr-' is supported

remove duplicate "inner-tunnel" virtual server

the methods which need it should provide it.  The methods which
don't need it shouldn't miss it.

output goes to log file, not %.ok

escape _our_ use of {}

add fr_dhcpv6_reply_initialize()

which recurses to create the nested structs / groups necessary
for nested replies.  This way we always know that the nested
packets in the request contain nested replies.

count the number of tests, too

remove unused variable

quiet scanner

don't check if values are valid attribute names

The VALUE parser in the dictionaries just strdup's the name.
So we shouldn't have more stringent checks here.

Embedded zeros, etc. should be taken care of by passing an
explicit length to fr_dict_enum_by_name()

it helps to advance the buffer

the tests still fail because of issues with fr_value_box_from_str()
noted in the previous commit.

start of encoding Relay-Message

The tests for encoding relay packets don't yet work, for a few
reasons.

* fr_value_box_from_str() is passed dst_enumv, and it then
  checks if the value is a valid _attribute_ name?  Which
  doesn't work for IPv6 addresses, as they contain ":"
* the fr_pair_list_afrom_substr() function doesn't treat
  "Attr-" as special, so it tries to look it up in the various
  dictionaries, and fails.
* the fr_dhcpv6_encode() function is failing for nested packets,
  and doesn't return a useful error message

check for end _before_ dereferencing pointers

set a descriptive error if the packet is not OK

parse nested groups

remove erroneous assert

now that we have duplicate detection, the assert is not needed

get16 already looked at an offset of p + 2.  It's less confusing to use two macros

don't run packet_expiry_timer twice

don't use "agent." prefix

now that we have full OIDs for names

relay packets may be nested to almost any depth

Englishify

Remove two uses of tmpl_preparse

Minor tweaks

remove unused assignment

forbid smashing flags.

once a flag is set, it shouldn't be overwritten

remove FR_TYPE_EXTENDED

it's now just a TLV with magic flags, and only for RADIUS

remove FR_TYPE_EXTENDED from the RADIUS dictionaries and code

change it instead to data type "tlv", with flag "extended"

remove duplicate names in dictionary flags

clean up and recursively verify Relay-Message

fix fr_dhcpv6_option_find() to check lengths
and verify to/from client now checks inside of Relay-Message
for the various packet requirements

Unknown parents shouldn't have any children

only one error per unit test

Better errors

Fixup tests for non vendor definitions

Typo

Add Nokia SR DHCPv6 options

<sigh>

skip option header, too

decode relay options, not relay header

fix the tests for new decoder and updated dictionaries

clarify error

decode it as a struct

Parent unknowns correctly

mark attributes as type 'group'

insert the relay-message into the output list

Print the child name when decoding structs

pass the correct length to value_box_from_network

Add failing test for relayed packets

Expose loaded attributes to encoder/decoder

Decode relay message options

typos.

slower is faster :(

fix return code from fr_dhcpv6_ok_internal()

check max_attributes for nested Relay-Messages

RFC 8415 Section 19.3 allows for nested Relay-Messages

don't allow Relay-Message in normal packets

recurse to check contents of Relay-Message

forbid duplicate virtual servers

fix fr_dhcpv6_ok() for relay message

add cleanup_delay to dhcpv4, dhcpv6, and ensure it's >0

Pass up the failure if onload fails

Fix relay message type