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

Start of relative dcusor init

Simplify tmpl dcursor init by just using VPs

shut up analyzer

use comparison operators for RHS list of &request -= { ... }

for RHS temporary attributes, just update the value-box in the VP

allow comparison operators for removal from list

set parent map where possible

clean up markup

correct docs for RHS list

allow removing from list by attribute and value.

With documentation and tests.

allow edits to be grouped in a transaction via group { ... }

ensure that list is empty before merge / union / etc.

Those operations sort their inputs *without* tracking those changes
in the edit list.  So if there are already edits in the list, the
sort will break the ability to undo things.

As a result, we forbid behavior which we know will result in wrong
outcomes.

try to quiet clang scan

remove --preprocess-first

Coverity doesn't like it

fix list_eval to return current from the cursor

and not what was passed in.  This mirrors the child_eval function.

And remove the next_peek from NUM_LAST, which was unnecessary
and was causing breakage

separate out tests individually

make it work with the new edit sections

and a "todo" because

&Tmp-String-1 = "%(concat:%{request[*]} ', ')"

ends up as a group:

"{bob, hello, ab c, de fg, 123}"

print the offending filename and line number for error messages

allocate temporary VPs in the current context

and talloc_steal() them when we're keeping them.  This is so that
they will be automatically cleaned up on error.

zero out more fields.

more fixes and updates

use new syntax and checks

&Foo = ... fail

doesn't get

&Foo == ""

it just doesn't create &Foo

use new edit syntax with correct operator

0 isn't an error

and we might as well print a meaningful error, too

don't bother checking operators here.

The fr_edit_list_apply_list_assignment() function will check it
anyways.  Not checking it here means that we can later add new
operators without changing this code.

print errors

assignments which fail don't change the destination

and don't eat the Module-Failure-Message

typos

use correct operators

commented-out code to use dcursors for list removal

We'd like to do:

&reply -= &Reply-Message

except that means changing the default list context of the RHS
from "request" to "reply" (i.e. the LHS list)

And it means NUM_ANY is really an equivalent for NUM_ALL.

Using dcursors means we have to instead do:

&reply -= &reply.Reply-Message[*]

which is a bit surprising.

whitespace

Put --preprocess-first where cov-build will see it, not make... (#4636)

allow &Foo = { bar, baz, bag}

compile it, evaluate it, and add a test for it.

add map_list_afrom_cs()

which parses

&Foo = { bar, baz, bag }

add fr_edit_list_pair_delete_by_da()

which mirrors fr_pair_delete_by_da()

just resolve entire LHS if required

quiet compiler

no need to check for "update" at every level

fix typo

just use TALLOC_FREE, which sets it to NULL

don't check if it's NULL before calling talloc_free(), that
function can take NULLs.

don't assign NULL to the variable just before exiting the function.
It does nothing

more "update -> edit"

handle list_as_attr when resolving virtual attributes

no need to resolve anything here.

just track "state", and not the individual vp

there's no need to add / update the vp via request_data_add()
on every iteration through the loop

use more consistent names

remove unused header

return FAIL on fail, not YIELD

map_afrom_cp: unescape double-quoted and back-quoted RHS prior to parsing it (#4607)

Simplify tmpl_cursors by using a dcursor at all nested levels (#4634)

Update platform for FreeBSD tests

Treq is no longer our problem

1) fr_ldap_trunk_search(): prevent fr_trunk_request_signal_cancel() from (#4628)

being called twice (from _ldap_search_sync_timeout() and ldap_trunk_query_cancel())

2) fr_trunk_request_enter_cancel_complete(): allow FR_TRUNK_REQUEST_STATE_CANCEL

This is dealt with by trunk_request_remove_from_conn

Because NUM_ANY confuses everyone

Try to make coverity see annotations in SBUFF_PARSE_*INT_DEF() (#4631)

I asked whether coverity sees annotations in invocations of
macro definitions containing annotations, and got the suggestion
to add the "--preprocess-first" option to the cov-build command,
and if it made no difference, submit a support case with more info.

Cross compile fixes (#4632)

* configure.ac: add option for libcap

Signed-off-by: Jackie Huang <jackie.huang@windriver.com>
Signed-off-by: Changqing Li <changqing.li@windriver.com>
Signed-off-by: David Gouarin <david.gouarin@thalesgroup.com>
Signed-off-by: Matt Weber <matthew.weber@rockwellcollins.com>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
* configure.ac: allow cross-compilation

The checking OpenSSL library and header version consistency will
always fail in cross compiling, skip the check and give a warning
instead for cross compiling.

Signed-off-by: Jackie Huang <jackie.huang@windriver.com>
Signed-off-by: Yi Zhao <yi.zhao@windriver.com>
[update to new version 3.0.17 to fix patch warning]
Signed-off-by: Changqing Li <changqing.li@windriver.com>
Signed-off-by: David Gouarin <david.gouarin@thalesgroup.com>
Signed-off-by: Matt Weber <matthew.weber@rockwellcollins.com>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
* configure.ac: add option for pcap

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
* configure.ac: add option for collectdclient

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
* configure.ac: try execinfo in libc before searching libexecinfo

execinfo is provided by glibc, so the usual case is that it is part of
libc. Do this before starting a redundant search for libexecinfo.

This is especially relevant for cross-compilation, where
FR_SMART_CHECK_LIB can find an incompatible library.

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Co-authored-by: Changqing Li <changqing.li@windriver.com>

Initialise all the fields in find Closes #4633

Allow cancel->complete transition here too

Deal with edge case in LDAP cancellations

Allow the transition from CANCEL to CANCEL_COMPLETE

Add VS code debugger config for gdb

notes for the future

call init_ssl(), not init_crypto()

init_ssl() calls init_crypto(), too.  And we need both in order
to pre-load all error strings.  Helps with comments in #4624

Fix CANONICAL_PATH when $(top_srcdir) has symlinks (#4611)

* Fix CANONICAL_PATH when $(top_srcdir) has symlinks:
in this case $(CURDIR) contains real path (with symlinks resolved)
and thus is not equal to $(top_srcdir)

This make most of changes in rev c2a6977a551acea0d927daa324e0353fa64ba221 not necessary

* Revert most of rev c2a6977a551acea0d927daa324e0353fa64ba221
(all changes related to $(CURDIR) != $(top_srcdir))

move to using new edit syntax

undocument "update", and document conversions to new syntax

always free cc->pool, to avoid leaks

add and use fr_pair_list_copy_to_box()

as we can't call fr_value_box_copy() on vp->vp_group.

set fail return code

move reset of indent to "stop" function

which is arguably where it belongs.

create LHS vp on merge RHS and union, too.

nothing MERGE-RHS FOO == foo
nothing UNION FOO == foo

rewrite merge, union, intersection to use fr_pair_next() etc.

to avoid pulling the rug out from under the dcursors.

Also correct logic in a few places, and make union check values, too

error if the VP to add is already in a list

and remove "b" from the old list before adding it to the new one

more debugging functions

replace equal pairs on merge RHS

add and use fr_list_assignment_op

Check strrchr() return in link_fixup() (CID #1504305) (#4627)

Don't check a value we know isn't NULL (CID #1503909) (#4618)

inst->filename will not be NULL. The check for that (after it's
already been used in open()) makes coverity complain, so we should
have the courage of our convictions.

Placate coverity re fixed size destination libpath (CID #720444) (#4625)

libpath has size PATH_MAX, so program should fit.

Correct error check on fr_base16_decode() for octets (CID #1503992) (#4623)

network_max_size() returns ~0 for octets, which when stored in ret,
a ssize_t, becomes -1. (Presumably this reflects that its max length
can vary.) Thus it makes no sense for it to appear in the test of
fr_base16_decode()'s return value.

Check curl_mime_filedata() return value (CID #1503934) (#4597)

Annotate uses of read preq pointers in test_demux() (CID #1469174) (#4614)

Trunk tests are written so that the data sent for mux/demux are
test_proto_request_t * values, which coverity considers tainted.

Don't do EV_SET for fr_event_fd_noop (#4626)

clarify docs

"no value for assignment" is possible

and needs to be treated as a separate result.

resolve all children, not just the first one

just resolve any unresolved tmpls

all of the code paths were the same, and the bare unresolved
data tmpls weren't resolved.

remove "update" from conf files

reset to dict_def if we're at the root of internal

which lets us do &request.Reply-Message, where normally "reply"
is in the internal dictionary.  But we want "Reply-Message" to
be in the RADIUS dictionary.

get the correct request, too

don't call tmpl_list when list_as_attr is set

shut up static analyzer

tests for nested edit groups

almost final cleanups