Start using modeling for Coverity (#4853)

This involves
* making the coverity workflow build the model
* adding model source file(s)

Use found terminal character to determine how many URI parts to advance

not the character found after skipping `extra_skip`

Correct module config for rest module test

Replace != with !( == ) in rest module tests

cleanups and fixes

re-initialize the iterator for every module we delete

because deleting one module may cause it to delete child modules
in the tree, which confuses the iterator and makes it sad.

skip the '/'

automatically set DEBUGGER_ATTACHED when executing gdb or lldb

because the debugger detection on some OS's is broken and doesn't
work.

typo

more checking for corner cases

it helps to set the value for State

use VALID macro

rename TACACS+ packet types

for increased clarity, now that we're actually using it.

and do a number of other cleanups.  Fix the state machine to match
the new packet types.  Fix the state machine to track state based
on synthesized information.  Add authentication logging ala RADIUS

fix up macros to be sure there's no possibility of error

Resequence to avoid potential double free.  Fixes #4863

Ensure LDAPMessage is freed on errors. Fixes #4864

there should have been no reply yet

force reject if there's no auth-type

and don't set default accept if "recv Access-Request" returns "ok"

typo and cleanups

not needed

update docs

Remove dead code (CID #1519001) (#4854)

_event_build_indexes() always returns 0, so fr_atexit_global_once_ret()
will always set ret to zero, and hence ret won't be negative.

pap: Hoist &control:User-Password check so that it can be found (#4866)

Since loop is rooted at &Password.

fix unlock and return code.  Fixes #4862

Don't override an error return code with a successful one.
Instead, set the success code first, and then override it if
there's an error.

Now that that's cleaned up, always unlock the mutex

use FR_TACACS_PACKET_CODE_VALID

Only leaf vps have the data object

Add unit tests launcher to launch.json

Don't set the path in the spec file

Placate coverity about day-to-second calculation (CID #1520503, #1520502) (#4859)

Coverity doesn't realize that day of week or month is constrained
such that multiplying by seconds per day won't overflow int32_t.

install libtalloc && libpcap, too

Clean up versioning for libtalloc and libkqueue, add rule to install libpcap (#4858)

Signed-off-by: Stephen Blackwell <sblackwell@networkradius.com>
Signed-off-by: Stephen Blackwell <sblackwell@networkradius.com>

change assert to run-time warning

until we track down why it occurs in CI but not locally

script to build (hopefully) prerequisites on RH

wrapper scripts to run local binaries

tell people about "xcrun foo" on OSX

assert that we don't leak memory on each packet

change name from SARL to SAS

whitespace

it's a day for typos

clean up and regularize names

 perl -p -i -e 's/fr_vmps_codes/fr_vmps_packet_names/g' $(git grep -l fr_vmps_codes)

clean up and regularize names

perl -p -i -e 's/fr_dns_packet_codes/fr_dns_packet_names/g' $(git grep -l fr_dns_packet_codes)

Clean up and regularize names

perl -p -i -e 's/fr_dhcpv6_packet_types/fr_dhcpv6_packet_names/g' $(git grep -l fr_dhcpv6_packet_types)

clean up and standardize names

perl -p -i -e 's/fr_request_types/fr_radius_request_name_table/g' $(git grep -l fr_request_types)

clean up and dedup FR_RADIUS_PACKET_CODE_VALID

perl -p -i -e 's/is_radius_code/FR_RADIUS_PACKET_CODE_VALID/g' $(find . -name "*.[ch]" -print)

clean up RADIUS names

perl -p -i -e 's/fr_packet_codes/fr_radius_packet_names/g' $(find . -name "*.[ch]" -print)

clean up names

perl -p -i -e 's/fr_tacacs_packet_codes/fr_tacacs_packet_names/g' $(find . -name "*.[ch]" -print)

clean up definition of FR_TACACS_CODE_MAX.  Fixes #4856

The CODE_MAX definition should be one more than the last valid
packet code.

The DO_NOT_RESPOND value should be a value which will never show
up in a real packet.

The PACKET_CODE_VALID macro should be used instead of relying on
manual checks of CODE_MAX

typo

typo to rearrange code

don't call frame_pop() if we have REQUEST_STOP_PROCESSING

clean up descriptions and names

so that the names and functions are descriptive and correct.

use CLOCK_MONOTONIC_RAW if it exists

otherwise use CLOCK_MONOTONIC.  The RAW version is unaffected
by frequency or time adjustments

remove mach functions.

OSX now (and for a long time) has had clock_gettime().  The old
code using the mach functions didn't even compile when we tried
to not use clock_gettime().  So... nuke the old / broken / unused
code.

clarifications and typo fixes

add functions to get gmt offset and daylight savings flag

we should note that the server tracks times in UTC, so

(date) foo % (time_delta) 1d

will return the offset from the UTC day, not from the local time.
This could perhaps be more prominent.  But tracking times internally
in UTC is the only way to make sure we're not screwed up by time
zone and DST changes.

update for expressions terminating before EOL

the admin must manually add backslashes for continuation

add week / month / year time resolution

which are vaguely correct for months, and correct to the second
for years.  The main purpose here is to allow for at least rough
parsing of years / months

Get rid of double unlock in fr_atexit_global_once_ret() (CID #1518999) (#4855)

add Tmp-Time-Delta-*

we really need local variables :(

date % time_delta --> time_delta

so we can round dates up / down as appropriate.

Note that this really only works for fixed periods of time_delta,
such as seconds / minutes / hours / days.  It doesn't work for
months.

add magic flag for expressions

so that they always get parsed via xlat_tokenize_expression().

Because if we pass an expression to tmpl_afrom_substr(), it will
happily return a tmpl for the entire string, which is a series of
string concatenations.  And that's not what we want.

print out what we're assigning before we assign it

add map_afrom*_edit() for edit sections

because the RHS of edit sections can now be expressions.

API to get log destination by name

if we can't parse a token, it must be an expression

first pass at multiple log destinations

for now, only files.  And there's no way to change log destinations.
but it's at least a start

use more consistent names

partial parse is only if at eol, and depth==0, and terminal character

set an error string

add tmpl_set_xlat()

for use with edit expressions

only sections can have ident2 name trees

i.e. CONF_PAIR and CONF_DATA don't have children which use
ident2.

Or currently even ident1 for that matter, but it's theoretically
possible and useful.

print out sizes of configuration things, too

rearrange to decrease size

more typo

typo

set global search path, so that the fuzzer can find things

typo

treat time_delta as floating point for multiplication

if there's no precision, time_deltas are evaluated in seconds

remove extraneous "break"

add and document %(time:now)

which returns a _boxed_ time, and not a stupid integer.

casting date to time_delta does NOT change the value

it only changes the precision, and that's only used for
printing / parsing.  Internally, all different-precision dates
and time_deltas are stored as 64-bit integers in nanoseconds

and the same goes for date --> time_delta

special-case  (date - date) --> time_delta

note crappy code && todo's

use normal APIs to print the reply packet

and make sure that the contents all go to the same FILE* handle

add fr_log_init_fp()

for pre-existing FILE* handles

typo

we use SRC_CFLAGS, not TGT_CFLAGS

the documentation lies.

The build output shows

CFLAGS=-O1 -fno-omit-frame-pointer -gline-tables-only -DFUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION -fsanitize=address -fsanitize-address-use-after-scope -fsanitize=fuzzer-no-link

So let's try that.

regenerate as per previous patch

Fixed misplaced bash syntax when adding the debug flag to CFLAGS in developer mode (#4852)

MAX code is invalid

check that the sequence numbers match

as a "duct tape" kind of security.

check that clients send odd sequence numbers

use random session ID for connections

minor cleanups

include the header in the packet length

print decode error, too

the EOF callback may free and close the socket

so check for is_registered before calling ef->error(), as the
event_callback() in the EOF handler may have closed and freed
the socket.

don't smash CFLAGS when doing --with-developer