Don't double advance buffer

It's not an error if we don't have any WITH_ macros

Always build with udpfromto

More reorganising

Reorg argument order

Only use src_ipaddr if it's not a wildcard address

Remove more ippool-dhcp references

Formatting

Typo

Set the protocol for sockets

Formatting

Not used

Remove useless wrapper

Reorganise

Fix typo

tacacs: Fix uninitialized memory issue (#3722)

* :gear: tacacs: Remove unecessary UNUSED attribute

* :gear: tacacs: Typo

* :gear: tacacs: Fix uninitialized memory issue

The issue appeared only agains gcc >= 7

* :gear: tacacs: Add missing dbuff calls

It is helpful to clears any markers handled by dbuff.

Typo

Remove dhcp_sqlippool, qualify attribute references in sqlippool

Treat requested_address as an expansion string, not an attribute

Better name for attribute_name

Merge ippool and dhcp-ippool configurations

Add back bulk_release_clear

and remove DHCPv4 specific text

Pull the requested address and gateway from the sqlippool configuration file

Address should be 43 bytes to allow for IPv6 prefixes

fix doxyen @file

move decode_proto API and callers to use fr_pair_list_t

define fr_pair_list_init() and use in internal decoder

update and clarify

regenerate from config

s/fr_pair_t**/fr_pair_list_t*/

s/fr_pair_t**/fr_pair_list_t*/

and the compiler is smart enough to not care too much about
passing "fr_pair_t**" to a function which now takes
"fr_pair_list_t*"

remove old uses of fr_pair_list_t

remove more fr_pair_list_t, and replace with fr_pair_t**

make internal encoder use fr_pair_t** instead of fr_pair_list_t*

until such time as we redefine fr_pair_list_t*

remove unused functions

look at vp->vp_group, not vp->children.slist

because that's the correct name

Owner and gateway should be 128 in sqlite too

Typo

Fix random gateway and owner lengths

15, 30 and 64 are too small.

sync redis ipool terms with sql ippool

s/pool_key/owner/g

Use a more generic term for the owner of the lease

Correct the table named in the MySQL stored procedures

I guess this was never tested

Prefix index names in the mssql schema

s/dhcpstatus/fr_ippool_status/g

Use the same table prefix for ippool statuses

s/fr_dhcp_/fr_ippool_/g

Store procedure prefixes should be the same as the table prefixes

Formatting

s/dhcpippool/fr_ippool/g

dhcpippool isn't correctly namespaced and doesn't provide a hint as to what's using it.  If were going something generic it's not good to use a protocol name either.

s/FramedIpAddress/address/g

FramedIpAddress is a RADIUS specific term.  There's no need to use it for a Generic IP pool allocation module.

More pair list fixes

Formatting

Move challenge lookup to outside encode function

More pair list fixups

s/RADIUS_PACKET/fr_radius_packet_t/

More macro conversions

Unused

Fix some magic numbers in DHCPv6

Remove dbuff encode wrappers

Not an rcode, an slen...

Fix talloc steal

Don't run IMAP or SMTP unless we've explicitly setup the test server

Make rlm_icmp work under docker.

Print arguments when we're trying to open icmp sockets

Should use euid not uid as changing from root removes root's caps

gcc defines things that clash in unexpected ways

rearrange checks in the proper order

so that the correct message is produced

Revert "v4: Add a "substring" xlat keyword (#3684)"

Uses legacy API, and disagree with syntax

Tighter scope for reply_ok, reply_fail

Formatting

hard-code Relay-Reply to avoid admin stupidities

the reply to a Relay-Request is _always_ a Relay-Reply.

See RFC 8415

update for new name

we use brackets

v4: Add a "substring" xlat keyword (#3684)

* Add a "substring" xlat function to rlm_unpack

Return a substring of the string representation of a string or attribute
using the syntax:
"%{substring:<string:attribute> <start> <len>}"

Negative `start` or `len` values count from the end of the string

* Tests for substring xlat

* Correct return of zero length string

Convert DHCPv6 encode_value() to use dbuffs

Docs: Fix SQL Howto markup and tidy

Make fr_radius_encode() a layer around fr_radius_encode_dbuff()

Make fr_dhcpv4_encode() a wrapper around a dbuff version

Both functions are visible; the dbuff version calls the
now-exposed fr_dbuff_dhcpv4_encode_option_dbuff() and
takes advantage of dbuff functionality.

Add fr_dhcpv6_encode_dbuff()

:gear: arp: Update fr_arp_encode() to use dbuff

:gear: vmps: Update fr_vmps_encode() to use dbuffs.

:gear: tacacs: Update TACACS+ fr_tacacs_encode() to use dbuffs

don't rely on accidental behavior for checks

have an explicit flag, instead of relying on "vp" being left over
as a pointer to something

use merge to merge lists of cursors

initialize the reply, for nested relay packets

Formatting

Hack subrequest so it does what we want

We really need some way of specifying "all children of"

Add pair list debug function we can call from lldb

Invert argument order of unlang callback functions

Update the docs for call

Fix explosions when we hit the unlang stack limit

Asserts are no longer correct

Have call operate on the original request

Simplify unlang_module_push

More cleanups

Unused

The new call code will do almost the same thing

More struct name fixes

Move redundant struct around

Use the same names for frame process functions

Whitespace

Unused

Not used