SQL uses SQL-Group, so use the old conditions here, too

upcast to 64-bits for integers

typo

CI: run sanitizers in a separate non-Docker job on GitHub

CI: fixes for when running directly on GitHub images

Fix dangling pointer issue (#4786)

clang may not realize that memcpy() doesn't keep the pointers handed
to it around, so there's no risk of the address of an auto being kept
past the caller's return, hence the "dangling pointer" warning.

Instead, head it off at the pass by immediately returning if len
is zero, so in needn't be set to the address of an auto; the only
explicit passing of NULL for in passes 0 for len. (Comments explain
how if len == 0, nothing changes in the context.) One could argue
for checking in rather than or in addition to checking len... OTOH,
shouldn't passing NULL and a non-zero len break loudly?

CI: disable sanitizers until we can work out why it deadlocks in Docker

CI: fix comment

Only run the main CI build on our infrastructure

Remove unused

push xlat condition parsing until later

to avoid having unused xlats hanging around when we parse
policies.

register function arguments if available

so that the parser can do the correct thing

add "don't use new conditions" for LDAP tests

so that when we "flip the switch" to always enable them, nothing
should break

Amend Active Directory persistent search tests for parallel runs

Amend persistent search tests to use different output files for parallel runs

Amend persistent search tests to use new entries

Add extra entries to 389ds test directory

CI: clean up docker cruft after job runs

Make the same fix for mkdirat

If the directory already exists then we need to provide a handle to it

Return the actual number of bytes processed, and the actual negative failure offset in fd_mkdir

Returning path - p where p is NULL results in a positive value

Unclear at this point why mkdirat is returning EBADF

Allow FR to start its LDAP persistent search before starting tests

Delete correct output file

Revert "Re-work ldap setup script"

ss is not available on self-hosted runners, and further
observation shows this is not the cause of the random
failures

Need to check for both...

*sigh* the real issue is path is not const

const

Fix comment on status line length in rlm_rest (#4783)

Don't return garbage when failing to create the top level directory

Don't lose the context for mkdir errors

Remove one fork() per utility invocation

It's fine if the dir we're making already exists

syserror isn't useful here, whereas our thread local buffer will be

Catch commands returning invalid codes

Pass mode when creating output files

Need sys/file.h

Prevent several race conditions in unit_test_attribute

Make writing as robust as possible

Move to node16 version of github actions

Remove deprecated set-output

Re-work ldap setup script

To check server is listening before attempting to add data

...and no one actually put the sneaky alias in FreeBSD

Except when a test sneakily creates a wrapper script

but then not so sneakily fails to catch all the previous sysctl instances

signal the request to cancel, too

nproc doesn't exist on freebsd

test: Quick wins by performing some tests in parallel (#4782)

Update tests to use different output filenames for parallel running

Amend tests to use new entries

Add extra entries to LDAP directory for parallel tests

Ignore returns from certain sbuff calls (CIDs #1504025, #1504070) (#4750)

These are cases in which sbuffs are either created from C arrays
or are allocated from the heap but given initial and maximum sizes.
The author(s) of the code presumably chose sizes intended to suffice,
but despite that, coverity will notice that the sbuff functions
used return a value, and hence will report check_return barring
either explicit cast to void or annotation. We therefore cast to void.

Check return of fr_pair_list_copy_by_da() (CID #1469148) (#4752)

Explicitly ignore the curl_mime_type() return (CID #1504071) (#4764)

* Check curl calls that allocate memory (CID #1504071)

This includes the curl_mime_type() call that gives rise to the
coverity defect, but also checks the mime and mime part allocations
that must work for body_init() to work.

* Formatting

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

Check fr_event_timer_in() return value (CID #1503926) (#4774)

* Check fr_event_timer_in() return value (CID #1503926)

* Assert and error if we can't insert timer events

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

tests: Ignore spurious output from jlibtool with VERBOSE=1 (#4781)

Add ability to set static headers to rlm_rest

fr_value_box_from_network: Fix NULL pointer dereference when truncating (#4779)

Annotate fs_check_call (CID #1271307) (#4778)

The unlink() call (the use of the toctou) does check its return
code. Also, it's not liwted among the UseSet functions in "TOCTTOU
Vulnerabilities in Unix-Style File Systems: An Anatomical Study",
https://www.usenix.org/legacy/events/fast05/tech/full_papers/wei/wei.pdf

Ignore fr_sbuff_in_strcpy() return in fr_pair_list_log_sbuff() (CID #1504278) (#4775)

sbuff is emptied each time through the for loop, and down this
path it will only contain the output of fr_dict_attr_oid_print(),
possibly preceded by "raw.". fr_pair_list_log_sbuff() is only
called from one place, which passes in an sbuff using an array
of 1024 characters. This should suffice to make appending " = "
always work.

Ignore fr_event_timer_in() return here, too (CID #154036) (#4773)

Like 1503936, this is in a callback and will reuse the idle
timeout event.

Deal with time of check/time of use complaints (CIDs below) (#4768)

1400053: the call coverity complains about is just there to
         determine why the dlopen() failed.
1445217: the fopen() return is checked, so any changes between
         the stat() call and it should be detected.
1503910: here, the unlink() return is checked

only NDEBUG

Add username to the REDIS auth command

typo

cleanups and fixes for #4777

Move common macros and includes to a common header file.

Create a macro which wraps OpenBSD's closefrom() in a macro,
so that the source code isn't littered with ifdef's

don't run this in parallel

update request_clone() so that it works

do perf updates before freeing the bits from the frame

add "interpret" section with instruction limiting pieces

add maximum instruction count for failure testing

CI: bump timeout

see if that's causing random failures

Fallback to `uname -n` on missing `hostname` in radtest (#4771)

This should work without inetutils being installed, so add a fallback.

Signed-off-by: Christian Hesse <mail@eworm.de>
Signed-off-by: Christian Hesse <mail@eworm.de>

CI: remove debugging

CI: install git-lfs early, too

Explicitly ignore fr_event_timer_in() return value (CID #1503936) (#4769)

fr_event_timer_in() (actually _fr_event_timer_at()) reuses events
where possible, and being in the idle timeout means that there is
such an event available for reuse.

CI: Tidy home directory in Docker

CI: ruby/setup-ruby broken in GitHub container right now

CI: for the time being run on both GitHub and self-hosted

CI: fixups to use 389ds in Docker container

CI: use vars to set llvm/gcc versions

CI: move fuzzer tests to separate composite action

CI: move main tests to separate composite action

CI: move build process to separate composite action

CI: prepare Docker image more like GitHub runner

CI: run in Docker and add test service containers

- mariadb
- postgres
- redis
- 389ds

CI: add matrix.os to set where jobs run

CI: move macOS to a separate job

CI: actions/setup-ruby is deprecated

CI: explicitly enable developer mode

CI: fix YAML whitespace

rlm_sql_oracle: Fix typo (#4765)

Suppress MacOS ld warning from native compilation (#4766)

Add the option '-Wl,-w' as suggested in https://issues.guix.gnu.org/issue/57849

Improve queries for processing radacct into periodic usage data

Removing `OR` from the query allows better index usage

Typo

Fix typo (#4763)

eapol_test: Add paths for Brew on OSX/M1 (#4757)

Needed for the Brew running on OSX/M1 due to the new base path is in /opt/homebrew
as described on https://earthly.dev/blog/homebrew-on-m1/

Make plain 'autoconf' 'autoheader' 'autoupdate' invocations work again

Reading the autoconf docs the preferred way to do includes is to set `AC_CONFIG_MACRO_DIR([m4])` then run aclocal to figure out which .m4 files need including.

Check the size of time_t so we can determine if we're going to overflow it...

fr_perror should accept a NULL fmt string

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

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