fixed double-counting, when updating source and destination statistics
code was updating both stat[src_code] and stat[dst_code], once when updating source statistics and once again when updating destination statistics.
(rlm_stats: per-IP stats double-count both src_code and dst_code 229)

remove impossible assert
ret == RD_KAFKA_CONF_UNKNOWN already checked in previous if statement that returns value 0. so assert is redundant, always fails, and also prevents cf_log_err() from printing error message
(lib/kafka: remove impossible assertion in kafka_topic_config_dflt 106)

return code (-1) when out of memory
(proto_dhcpv4/v6_udp.c: return 0 instead of -1 on talloc_zero failure in mod_instantiate 305)

more tests for corner cases of RADIUS encoder

use correct size for poisoned region

instead of relying on "slen", which is then over-written before
un-poisoning the region.

clean up and tweak substr tests

more corner cases for internal encoder / decoder

more corner cases for DER

more corner cases for TACACS+

add corner cases / error tests for DHCPv6

more tests for dhcpv4

more corner cases for RADIUS

add more tests for error cases and bad packets

add tests for more corner cases

disallow empty input files

while it can be nice to do nothing, it's best to do something.

return on error

rhel: Disable dwz (dwarf compression), so we get debuginfo usable with lldb

docker: Copy GPG key from the build phase to reduce spurious failures

count offers, not replies

check for end pointer, too

return on OOM

update macro

remove unused file

use same code as previous check

this should really be abstracted into a common function

free on OOM

check for failure when adding chain cert

close pipe after ferror() call

one more fix

Fix debuginfo in RPM builds

Always built with -g3 no matter if it's a developer build or not, otherwise state machine issues become impossible to debug.

don't use "index" as a variable name, it's a C library function

the compilers aren't smart enough to notice this and complain.

typo
(rlm_python: fix wrong variable 'index' in py_freeradius_build_parents error message 341)

rlm_sql_firebird: fix free() on talloc-allocated memory in fb_free_sqlda()

return after error, initialize var
(command.c: fix missing return false and uninitialized child_done 148)

return after error, deallocate variable
(rlm_sql_unixodbc: fix missing return after connection failure and ind memory leak 412)

return code (-1) when parse failure
(map_to_vp: returns success (rcode=0) on fr_pair_value_from_str failure 239)

typo
(lib/kafka: fix auto_commit property name typo (enable_auto.commit -> enable.auto.commit) 107)

add missing 'usage()' for consistency
(rlm_redis_ippool_tool: add missing usage() call after invalid prefix error 353)

allow queries to proceed in CONNECTING or CONNECTED states

instead of checking for a failure state, and missing some
failure states

use consistent names

'conn' is the #fr_connection_t

'c' is the DB connection handle

MAX is last valid plus one

various bug fixes and tweaks to the tests

clean up FDs and PID in all situations

and don't complain on cleanup if the program exited cleanly.

typo

use better dlist functions, which means less code

arguably functions calling talloc_free() should return int

and the callers should check the return code

typo in doxygen

use fr_dlist_talloc_free() instead of hand-rolled loop

just use pop_head(), which is a bit clearer

typo

use value-box escaping function.

mash leading '.', too.

and set safe_for

fix: 1. initialise thread mutex before use
(rlm_stats: initialise thread mutex before use 415)

fix: 1. typo
(rlm_sqlcounter: fix copy-paste bug in find_prev_reset() for "never" reset 413)

fix: 1. function parameter typo fixed
(Fix copy-paste error: response_window never validated in rlm_tacacs and rlm_radius 416)

fix: 1. exact duplicate code removed
2. return 0  -->  return -1   (for invalid discriminator/state)
(protocols/bfd: fix duplicate detect_multi check and wrong return value in decode_proto 433)

fix: 1. missing 'false' argument when calling function
2. num_events == EINTR  -->  errno == EINTR     (typo)
3. if (*start > MY_ARRAY_SIZE)   -->   if (*start >= MY_ARRAY_SIZE)   (wraparound sometimes skipped)
(tests/util: fix missing 5th arg to fr_message_set_create, wrong EINTR check, and off-by-one 225)

fix: 1. calling wrong function, is now pop_tail (copy paste typo)
2. fix undefined behaviour
(lib/util/dlist: fix free_tail pop direction and sort-merge NULL deref 156)

fix: 1. pesudo --> pseudo (typo)
2. added outer parenthesis
(net.h/net.c: fix 'pesudo' typo in function name; add missing outer parens to IP_VHL macro 283)

just use sprintf to match the rest of the code

and other minor tweaks

greatly simplify function

as it should only be taking unsafe inputs

the function is only used by one caller

and doesn't have to be exposed

add callback for escaping value-box with sbuff erules

use real type, and not uintptr_t

remove functionality which is useless and never worked

add simple tests for value-box functions

use standard APIs for filename escapes, and check for errors

add value-box escape with erules

there are a number of places which need it.

continue after \r, \n, \t

the sbuff escape / unescape rules should be const

move escape sbuff sequences to utility library

so that others can use them

check for initialize failure

typo

quiet static analyzer

fix:    1. return 01 --> return -1 (it's meant to be an error return val)
2. uid_str --> gid_str  (orelse it's checking the same condition twice)
3. uid_str --> gid_str typo (orelse it's running a gid function on a uid_str)
(fix(sem): octal return value, wrong NULL check variable, and wrong gid_t type in header 168)

formatting bug: changed '%zu' to '%zuk' to stay consisten in error message
(rlm_sql_sqlite: add missing 'k' unit to bootstrap file size error message 411)

protocols/der: fix fr_der_flag_has_default_value macro — trailing semicolon and spurious != NULL comparison

rlm_wimax: fix trailing dash overwriting null terminator in Calling-Station-Id reformatter

check proper return code

the function returns 0 for "more data", and 1 for "no data",
which is different from the other APIs

check for invalid hex digits

use correct IP when prefix=0

various bug fixes in command.c

remove unused macro

turn off more warning messages

ignore AI test files

add basic tests for calc.c

remove unused variables

not all packet codes are valid

check for errors and set flags

various bug fixes in eap-md5

we don't use "name".

check password length

add more sanity checks

ctime() can return NULL

tweak bounds

more fix typos

to finish the changes from bcf136dd0

typo in macro

use "goto fail" in more situations

minor tweaks and updates to the use of AI

Add client name to error message

Fallback to attribute parsing if cast doesn't work

This allows call_env parsing for types such as uint32 to support

foo = 10

and

foo = attr

with the value being a bare word in both cases.

manually do dir + filename

to get rid of all double '/', and to avoid walking over the
input strings repeatedly.

relax checks in corner conditions

assert that the entry is in the heap