Efficiently initialise unlang_module_event_t

Fix names to be consistent

Typo

Typo

Don't need server_cs field

re-arrange fields in request_t

Unused

Unused

Unused

Remove request->listener

Unused

Fix redundant declarations of talloc_const_free

Need signal.h

Replace talloc_autofree_context with our own version Closes #3916

The original talloc_autofree_context was also deprecated in Talloc 2.1.15

Allow disarming of atexit thread-local destructors

Just return NULL at the detection of padding

As written, it incremented where and continued. That makes sure it
leaves the loop, but since *only* where is changed, data[0] is still
0 the next time through, so it's an O(size) way to return NULL.
Might as well just return NULL.

talloc: Fix missing _talloc_pooled_object() configure check (#3935)

It fixes the typo in _talloc_pooled_object 762c3a72100e079d5918f140d0c516c406495672

Co-authored-by: Jorge Pereira <jpereiran@gmail.com>

Talloc memory limit is deprecated as of Talloc 2.1.15 Closes #3925 Closes #3924 Closes #3923 Closes #3922 Closes #3921 Closes #3920 Closes #3919 Closes #3918 Closes #3917

Jeremy Allison and the talloc team believe the  memlimt code is too complex and introduces too many corner cases.

None of our uses of memlimit are critical, so we're removing support in anticipation of talloc_set_memlimit being removed from the next major version of talloc.

Add dictionary pull request template

typo

more error messages

talloc_setmemlimit() is now deprecated on OSX.

homebrew installs a newer version of talloc, which has marked
talloc_setmemlimit() as deprecated.

This change is a temporary stop-gap until we get rid of
talloc_setmemlimit() everywhere.

Delete old issue template

master: Spawn tmate on failing package build and install jobs with ci-debug branch (#3929)

* CI: Simplify now that authorized_keys restriction merged into GitHub Action

* CI: Spawn tmate on failing package build and install jobs with ci-debug branch

Ensure DH parameters are created before stopping dovecot as root (#3927)

fix typos

don't copy listener

child requests should be internal, and should have no idea
about external networks

update copyright date

use the v4 dictionary format

remove BEGIN-TLV.

Failover-DNS-Removal-Info is a *group*, not a TLV.
The DHCPv6 number space is global.  So option 117 must be
defined globally, even if it can only occur inside of
attribute 116.

update

Updated AVPs based on the latest SmartZone 5.1.1 RADIUS data dictionary, see https://support.ruckuswireless.com/articles/000005220

Fixes #3907

remove request->async->fake

as it's mirrors functionality of checking request->async->listen

and add macros to check for internal / external status, which
is clearer

remove one more goto and break

by moving them into clearer control structures

Initialize version_byte to make static analysis happy

Use dbuff primitives to read/write from dbuffs. (#3912)

Allow fr_dbuff_in_memcpy() from marker to uint8_t * (#3906)

(and add a corresponding check in dbuff_tests)

Typo

Use old deconsting method

Unused

Fix various isuses

Start of defining module methods in the dictionary

Honour *_TEST_SERVER variables to exercise module feature tests (#3911)

* Respect individual *_TEST_SERVER variables if set

* Handle module tests in subdirectories

* Correct list references in SMTP tests

* Use correct group for Dovecot userdb

* Correct imap_setup script in light of linter comments

* Extend time for exim attachment delivery

* Parameterise IMAP test server settings.

* Parameterise SMTP test server settings.

Quiet compiler

Unconsting

TMPLs evaluated from LDAP responses are "at_runtime"

Only if we have a parent...

Use the request_alloc functions to setup logging in trigger_exec

So we don't do it wrong and trip up asserts elsewhere in the code... or double alloc log.dst

Even detachable requests should inherit log settings from their parents

Move into a loop, so it's clearer what's going on

For when you can't event get a valid FD

More helpful asserts

Whitespace

Rename fr_dict_attr_enum functions to fr_dict_enum

Unused

Check if we have  a trigger args list

Add fr_rb_node_t fields to structures that are inserted into rbtrees

This is being done ahead of removing malloc calls in the rbtree.

Use names field initialisers

Move shellcheck RC from root to workflow config; disable JSCPD (#3904)

Fuzzer artifact fetcher (#3893)

* Fuzzer: Make sure that the fuzzer is built before attempting to run it

* Fuzzer: Script to fetch fuzzing artifacts from GH Actions workflows

Requires a Personal Access Token with "repo:public_repo" scope to be set in
GITHUB_TOKEN environment variable to download artifacts.

* Shut up linter

POSIX-compliant methods of getting around this are fugly.

Revise dbuff_tests to avoid provoking jscpd (#3902)

Only run the free walker if we have a free callback

The rest of the nodes are parented off the tree

More rbmonkey fixes

Delete data as the _last_ step when removing or inserting a node

LLVM snapshot repository is broken; switch to LLVM 10 for now

Add Arista Captive-Portal and fix WebAuth start/complete values (#3900)

* Fix WebAuth start/complete values

* Add Arista-Captive-Portal

* Remove Arista- prefix

Typo

look at listener only if it exists

by pushing checks for size to code where it exists

Fix condition

Don't crash if env_pairs are NULL

Add Hitron dictionary

sqlcounter: fixed configuration for PostgreSQL #3867

ABSTIME is deprecated since PostgreSQL 7.0 and removed in PostgreSQL 12.
GREATER doesn't seem to be documented, but GREATEST is available
since PostgreSQL 9.

Add inverse type mappings for bool

More helpful error messages

rlm_python: The detach() function in example.py expects parameter

Add extra tests to rlm_attr_filter to exercise the different operators

radclient: Fix 'invalid attribute' test

It is necessary to match the output files built with/without -NDEBUG

Changed prematurely

thresh should be an fr_rb_test_node_t

Grr stupid generics

Avoid type conflicts in fr_dbuff_advance

radclient: Add tests validating invalid attributes

test: Add 'IGNORE_ERROR' option for radclient tests

Avoid backing up markers and make naming and usage consistent.

Modernize and clean up dhcpv4 encoding code to make better use of dbuffs and markers

Don't allow the length argument of fr_dbuff_advance to be a signed integer

Remove FR_DBUFF_RESERVE*() now that nothing uses it.

This allows a simpler, more direct expression of FR_DBUFF_MAX*()
(for now, at least; things will get interesting again when we
let FR_DBUFF_MAX*() take advantage of extensibility)

Fixup rbmonkey so it does not duplicate private structs

- Use a test structure with room for an rb node instead of just an integer.
- Minor style corrections.

ensure that we do adoc -> man conversions, too

because $(wildcard ...) will only pick up files which exist
in the directory.  the auto-generated "man" files might not,
so we need to add a dependency so that they are created, too

Correct path for man pages made from adoc files

We should always have list head pointer values now Closes #3889

Can't write to a const

CI: Allow one-off fuzzing runtime to be specified in the branch name

CI: Set explicit retention days for uploaded assets; name fixes

Default retention period is 90 days which is excessive for large
artifacts, especially now that this can be controlled.

Do not retain the packages built for post-install testing. We limit to
the minimum of 1 day. These assets are large and we don't want people to
use them.

Make attr_filter test pass

Remove pair from temporary list before adding to correct list

unify checks for buffer_len==1 and do_not_respond

always use protocol size, even for do_not_respond

we rely on proto_foo encode() to look at request->reply->code,
and then do "the right thing" to signal it's own write() layer
that we're not replying.

tell the IO handler that we're done

DHCPv4: Still need to copy pairs to a local list so we can search for options

add flag / assertion for request is finished

so that we don't erroneously refuse to clean it up

Use a local var for easier debugging