Type and func macros don't use semicolons in other places

Another attempt at accepting NULL macro args

Allow global at exit init functions to return error codes

Too many semicolons

Fix -Werror=address complaints... maybe

Allow NULL init and free functions for fr_atexit_global_once

Tests for slab allocator

Define a talloc based slab allocator

Allows regularly re-used objects to be allocated from a slab once and
reused.

Just sneak this in there...

Fix rpmbuilding for Alma Linux

Remove confe file

rpmbuild really wants these flags together

Doesn't seem necessary to use perl here... and rpmbuild seems to sub it anyway

Fix typo. refs #4836 (#4843)

Force openSSL version check literals to unsigned long

Correct syntax of debug_ policies

rlm_rest: Prefer the Content-Type provided in a REST-HTTP-Header attribute (#4841)

* rlm_rest: Prefer the Content-Type provided in a REST-HTTP-Header attribute

* More than just SP is allowed between header attributes and values

* Simplify content_type use

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

eapol_test-build.sh log output should be to stderr

stdout is where the path to the final binary is output.

Update default delay for rlm_delay

Due to the option being parsed as a tmpl, if it is 1.0, it becomes a
float, which doesn't have a cast to fr_time_delta_t.

Adding the time unit, makes it a string which parses fine.

Handle corner case in using fr_sbuff_is...() functions on markers (#4839)

* Corner case fix for calling an fr_sbuff_is... function on a marker

Previously if fr_sbuff_is_...() was called on a marker and the buffer
had advanced past the end, then the function always returned false,
regardless of the character the marker was pointing at.

* Test to check fr_sbuff_is..() handles markers regardless of buffer position

* Correct parsing <num>.0 into fr_time_delta_t

rlm_rest: Don't skip alternate REST-HTTP-Header entries (#4838)

Only cleanup child processes if the exec has been populated

Add error message when delay value fails to parse

Scheduled fuzzing: Update src/tests/fuzzer-corpus/dhcpv6.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/dhcpv4.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/radius.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/util.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/tftp.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/vmps.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/tacacs.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/dns.tar

Revert "util: Add new fr_dict_global_set_lib_dir() function (#4835)"

This reverts commit 2a1a9df08f4baf7d76592325d39c9dd1dd704912.

util: Add new fr_dict_global_set_lib_dir() function (#4835)

It helps to add extra directories to be checked during library load.

Use correct value for last argument of delay_add()

Align worker bounds on max_request_time with docs

max_request_time is a fr_time_delta_t not uint32_t

eapol_test: Add new 'HOSTAPD_REPO' variable (#4834)

It allows us to build the 'eapol_test' using external repos like:

$ HOSTAPD_REPO="https://github.com/NetworkRADIUS/hostap" \
  HOSTAPD_GIT_TAG="feature/get_radius_reply" \
  ./scripts/ci/eapol_test-build.sh
$

use enums when upcasting for comparisons

Note that (for now), we're not doing it in tmpl_cast_in_place(),
which *explicitly* avoids passing enums when casting.

We're also not doing it for expression / math calculations, as
it doesn't make sense to do

Service-Type = Framed-User + 1

It might make sense to use enums for IP address calculations? e.g.

Framed-IP-Address = network & mask

but at that point, you might as well just use attributes or local
variables for everything.

This is in preparation for updating the parser to force the user
to use single-quoted strings for enum names, as in:

Service-Type == 'Framed-User'

Which then means we'll also have to update the printing routines
to always print quotation marks around enums, and then update the
users file, SQL, detail file, etc.

user-visible error messages shouldn't have function names in them

we can't define ENUMs for strings or structural types

notes for when we parse expressions

ascend.h is no longer used

fr_dict_attr_by_oid() first check if the 'err' is valid (#4830)

fr_dict_attr_by_oid() should parse the entire input string

if it doesn't, then the input string contains a valid attribute,
but then ends with an invalid character, or with an unknown
attribute.

allow unknown attributes to be TLVs

Fix up printing so that we can swap from known ancestors to
unknown da's, and still print things correctly.

Fix up pair verification to allow TLVs as unknown attributes.

Update the RADIUS decoder to check if unknown attributes can be
decoded as TLVs.  And if so, do so.  The odds of anything else
decoding correctly as a TLV are vanishingly small.

Add tests for encoding and decoding unknown TLVs.  Previously we
could represent them and encode them as OID strings, but the decoding
process would only decode them as a top-level "unknown", with
an opaque blob of data for contents.  It now decodes them as OID
strings, too.

Remove annotation inside macro definition (#4829)

Such annotations have no effect, so they're just clutter.

ensure that type_size and length are set for unknown vendors

that information is used to encode attributes.

this is no longer used

just allocate a vendor, and let the rest of the code figure it out

and fix unknown child allocation, which was using the wrong parent.

correct comment

use the correct API here.

There's no need to create a vendor intermediary

don't use talloc_parent()

passing a ctx is easy, and rather a lot faster.

remove needs_async flag

as it is no longer used

remove xlat_sync_required()

synchronous xlats now call unlang_interpret_synchronous(), and
thus can deal with async xlats

tmpl_async_required() already calls xlat_async_required()

remove FR_TYPE_NON_BLOCKING

as it's not really used or needed, now that the synchronous xlat
code can call unlang_interpret_synchronous()

add assert to match commit b5c9a3cd25121

Fix missing headers so clangd stops complaining

Print the size of fr_rb_tree_t in radsize

always set our_parent

opendir may fail.  Fixes #4827

still go to "next" attribute if there's no data encoded

RFC 2865 Section 5 days of length zero (0) MUST NOT be sent;
omit the entire attribute instead.

our_parent cannot be NULL here

Catch uses of uninitialised fields

Fix invalid test

Fix field alignment issue

Move size_t into a struct with the variable length value fields

Re-arrange fields in fr_value_box_t to take size from 104 bytes -> 64bytes

Shrink fr_dlist_head_t by packing the offset with num_elements

Move vb size field into a struct with the ptr types in fr_value_box_t

Print some extra size information in radsize

Unused macros

Only use vb_length for variable sized values

Make clangd happy when parsing unit_test_attribute.c

typo, so we do short-circuit

print full paths in debug mode if we have them

so we're printing the tmpl name which created this VP, and not
just the name of the VP

inherit internal flag from parent

so we can do

&reply += {
     &26.80.10 = 0xabcdef
}

and have it show up in a RADIUS packet

allocate real message only if we are sending a reply

use better names for function parameters

typo and clean up "stop" handling.

The main unlang_interpret_stop() should take care of interpreter
issues, so the worker "stop" callbac shouldn't do that.

set STOP_PROCESSING on stop

cancel used to do this, but no longer

CID 1517660

ippool_script(): fix EVALSHA with wait (#4826)

Add '-o coa_port'

extract condition pseudo-parsing into its own function

add fr_skip_xlat() to mirror fr_skip_string()

which now takes an "end" ptr, and is a bit more intelligent

make conditional parser use fr_skip_string()

in the interest of common code.

allow (...) in xlat arguments

list_as_attr fixes

move to v4 syntax

typo

better debug output

Fix arg order

proto_detail_file.c needs main_config.h too

Re-arrange some includes

We always need to initialise the request code so we have the list attributes

Lists should not be capitalised

No strlcpy in the make framework

CID 1503903

CID 1503983

Typo

This reverts commit 270e21e9ed5ed5358f5537627c27dff322669b79.

CID 1503986