pull OSS-FUZZ patches in.

And rework them to follow our normal processes

use *p_result for "current interpreter result"

request->rcode should really be a cached version, and should
maybe not even exist at all.

typo

sample tacacs client and configuration

TACACS+ client connections always start at 1, and increment by 2

track outstanding status, not u->packet

because once we write the packet, we throw away all of the encoded
data.

track packets in the written buffer a bit better

various minor fixes

initialize h->send after setting send_buff_actual

fix typos

comment out until we double-check it

typo

allow it to run for tacclient

typo

allow for empty or non-existent secrets

it helps to parse the io_submodule

Packet-Type does not map to codes in the packet

it's a synthetic type created during the packet decoding

first pass of fixing statistics

They still don't work, but at least there's a path forward

Debian sid and Ubuntu >= 22.04 use OpenSSL v3

Work round for eapol_test not compiling on debian sid

Bump hostap to latest released version

remove unused assignment

it would help if the local compiler was as picky as CI

more NDEBUG

remove unused variable

clean up "todos"

allow for and initialize retries

do retries

we don't actually resend the packets for TCP, but a retransmit
timer allows the packets to fail over from one connection to another

reset can also reset the timer

ctype macros should take explicitly unsigned input

to avoid chars with high bits being converted to negative numbers

perl -p -i -e 's/(tolower|toupper|isupper|islower|isdigit|isalpha|isspace|isxdigit)\(\s*\*/${1}((uint8_t) */g' $(find . -name "*.[ch]" -print)
perl -p -i -e 's/(tolower|toupper|isupper|islower|isdigit|isalpha|isspace|isxdigit)\(\(int\)/${1}((uint8_t)/g' $(find . -name "*.[ch]" -print)

add TCP output which is compile-tested.

The basic logic should be mostly correct.  The timers haven't
been double-checked, so they're likely off.  But it's a good start

Explicitly ignore return code (CID #1519000)

Remove useless code (CID #1519003)

Use MEM() to add safety check (CID #1519004)

Use print_hex() to dump hex data

Use accessor function for PyFrameObject members on python >= 3.10

Fix comment

Add function for set global library directory

grumble

remove unused variable

add and use fr_tacacs_packet_log_hex()

Which makes my head hurt, because the TACACS+ packet format
was created by drunken orangutans throwing darts at a pumpkin.
The resulting Jackson Pollock eyesore was the published as
a work of utter genius.

Revert "fix decode_args() function"

This reverts commit 0aa0f42d90b15366ab79ac364d55b45cffae7a60.

run busy loop over 1msec.  Fixes #4846 in a clearer manner.
 Pleas  enter the commit message for your changes. Lines starting

first draft of "front end" for TACACS+ client

move commonly used functions to the trunk code

There's no need to reproduce these in multiple client IO modules.

fix decode_args() function

the arg_list does *not* start at the body of the packet.  Instead,
it starts after the fixed arguments have been decoded.

no need to update the output ptr

print packet name, too

so that the _msg argument to the macro is used

allow parsing expressions on the RHS of edit instructions

the compiler doesn't do anything with them yet, but the parsing
seems to work.

we don't support "filter" any more

remove extraneous brackets

make the skip condition code a bit more generic

don't resolve enums here, the caller should do that

hoist rcode parsing

so that tmpl_tokenize functions don't need to parse bare words
as enums

Include talloc headers in atexit code and fix erroneous uses of include <talloc.h>

Punctuation

Add  generic talloc free function for the atexit code

Just create two fr_atexit_global_once macros

One that returns values, one that doesn't

...as demonstrated by these

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