update to use structures

remove map_cast_from_hex() as it is no longer used

make map_afrom_attr_str() a wrapper around map_afrom_sbuff()

remove unnecessary code

Change function signatures so that module methods can signal the interpreter directly

Restore line that shouldn't have been deleted

Let caller decide what it wants

Correct fr_dbuff_in() for floats and doubles (#3735)

Appropriate tests are added.

Don't run the normal CI workflow on the coverity_scan branch

Correct branch name

Formatting

Add coverity scan Action on push to coverity-scan branch (#3734)

Fix more magic numbers

Fix dbuff verb order to be the same as sbuff

Fix old calls to dbuff API

Formatting

Change internal encoding to use markers in anticipation of extensible dbuffs (#3632)

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

Convert EAP-AKA encoding to use dbuff (#3526)

* Convert EAP-AKA encoding to use dbuff

* Correct comment to reflect VALUE_PAIR -> fr_value_t

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

Add fr_dbuff_uint64v_out(), and float/double options for fr_dbuff_out() (#3732)

Unlike fr_dbuff_uint64v_in(), one must pass a length between 1 and 8
inclusive to specify the number of bytes to read, as well as a dbuff
or marker and a pointer to a uint64_t. Bytes read are interpreted as
an unsigned integer in network order, converted to host order, and
stored in the pointed-at uint64_t.

Install libjson-c4 for 20.0.4

Disable 20.04 until we have a source for all deps, and have package caching

slen is signed

and it helps to return the map to the caller...

make map_afrom_fields() static.

It's only ever called from map_afrom_attr_str()

add map_afrom_sbuff()

don't colourise when we send to non-terminals

even if the configuration file says "colourise = yes"

Last few doxygen fixes

Reformat dependencies

Delete .travis.yml

Change the build badge to GitHub actions

Add GitHub Actions CI workflow

s/travis/ci/g ... ish (#3728)

Add initial version of sbuff.doc (#3647)

* Add initial version of sbuff.doc

* Fixups and clarifications

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

Add tests to exercise fr_dbuff_out() with marker as source (#3730)

We also include the correction that the added tests turned up.

Try and quiet documentation warnings

Lots of doxygen fixes

Formatting

sbuff: Have bstrncpy_until take fr_sbuff_unescape_rules_t

dbuff: Structure documentation

dbuff: Fixup failing test, probably broke when fr_memcpy_out stopped advancing output dbuff

dbuff: Fix fr_dbuff_out to and associated functions to work with markers

dbuff: Have "move" functions extend the input and output buffers

dbuff: Doxygen

dbuff: Fix SIZE_MAX to work correctly with memcpy_in

dbuff: Use safecpy in more places

dbuff: Add extra integer types for fr_dbuff_set

A simpler fix...

don't run icmp on Linux.  it requires root or setcap

document setcap for Linux

don't clear cursor if we didn't initialize it

Amend fr_pair_list_tainted to take fr_pair_list_t *

Amend fr_pair_list_verify to take fr_pair_list_t *

Amend fr_pair_list_copy_by_ancestor to take fr_pair_list_t *

Amend fr_pair_list_copy_by_da() to take fr_pair_list_t *

Amend fr_pair_list_copy() to take fr_pair_list_t *

Amend _pair_list_sort_merge() to take fr_pair_list_t *

Amend _pair_list_sort_split() to take fr_pair_list_t *

Amend fr_pair_list_cmp() to take fr_pair_list_t *

Amend fr_pair_find_by_child_num() to take fr_pair_list_t *

Amend fr_pair_find_by_num() to take fr_pair_list_t *

Amend fr_pair_find_by_da() to take fr_pair_list *

Replace fr_pair_t ** with fr_pair_list_t * in pair_legacy.c functions (#3726)

More dbuff doxygen content

More minor length fixes in fr_radius_ascend_secret

update Oracle for changes as per commit 42229e776

Calculate fr_radius_ascend_secret() as per previous commit

The original code was fine, and avoided extra memcpys and intermediary buffers.  There was just an issue writing to the wrong buffer.

Better name...

fix fr_radius_ascend_secret(), and add tests

if the input data is smaller than the digest, pad it with zeros.
Copy digest to work_dbuff, not dbuff
use MD5_DIGEST_LENGTH or sizeof(array) in more places.

Fix fr_dbuff_uint64v_in to work with markers

Remove wrapper

Reduce direct buffer access

Fix strange fr_radius_ascend_secret use of dbuffs

dbuff: Fixup *in* functions to work on markers

encode message-type

and add decode test

no need to print trailing junk at the end of a packet

Remove dbuff_marker macros

Fix bad calls to fr_dbuff_remaining

fix encoder and add test packet

* dhcp opcode is uint8, not uint32
* NEVER leave fields unset.  Instead, initialize them to zero
* put hwtype and hwlen into header, instead of inside chaddr
* when copying data into fixed-width fields, fill in the rest with 0

add test point for encode-proto

use main "print hex" routine instead of a local one

remove extraneous "3" which breaks DHCPv4

clean up the output of hex printing the packet

don't do PROTO printing in non-debug mode

stop decoding fields at padding

Pass bytes in as hex

Operate directly on the dbuff

Addresses already inverted here

Swap src/dst before we call udp_send

Try using tmpl_cursor_init in map_to_request

whitespace

use memcpy, as IPs are stored internally in network byte order

Allow swapping out the fr_pair_list_t based on whether USE_DOUBLE_LIST is defined

v4: Swap all uses of pair_cursor.c to cursor.c (#3724)

* Add filter function for fr_cursor_filter_next() to match da

pair_matches_da() can be used with fr_cursor_filter_next() to return the
next pair in the list pointed to by the cursor with the matching da.

Use case:

  fr_cursor_filter_next(&cursor, pair_matches_da, da)

where da is fr_dict_attr_t *

* Replace fr_pair_cursor_ with fr_cursor_ in map.c

* Replace fr_pair_cursor_ with fr_cursor_ in pair.c

* Replace fr_pair_cursor_ with fr_cursor_ in radsniff.c

* Remove un-used includes of pair_cursor.h

* Remove pair_cursor.c from make file.

* Remove un-used vp_cursor_t definition

* Remove pair_cursor.c

* Update filter function eval_eq to take const parameters

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

fix this, too for "const"

Fixup udp_send to just take packet->socket

Simplify udp_recv so that all data is written to an fr_socket_t

There's a type for that...

Formatting.

add "SELECT true" as fail-over for on/off queries

If there are no rows to update, then the on/off queries will
return "no rows updated".  The server will then fail over to the
next query, which says "that's fine"

fr_cursor_eval_t should take "const"

evaluation functions should be idempotent

print out packet before encrypting

:gear: tacacs: Fix typo (#3723)

* :gear: tacacs: Better formatting

* :gear: tacacs: Fix typo