return failure on error case

fix fr_nbo_to_int64v()

use / return int64_t, not uint64_t

set the high bits to zero starting at 0, not part-way through

initialize variable before doing recursive call, not in recursion

check current pointer, not packet start

use real parent context

use DEBUG for non-error message

increment pointer after \r, \n, etc.

copy va_list instead of re-using it,and other cleanups

check each individual entry, not only the first one

free on error

fix typos

use correct talloc type

fix minor issues in rlm_mschap

use distinct name per test

remove auth_fail tests from Postgres until we determine why they fail

the rows are inserted, but the SELECT returns no data

tweak lhs/rhs handling rules, and require well-formed strings

use different rules for local LHS / RHS, which ensures that
there's no aliasing, and that the local rules updates don't
mangle each other.  There were corner cases where the LHS rules
would get over-written in the tmpl_needs_resolving() condition
near the end of the file.

Complain if quoted strings on the RHS aren't well formed.
Add a test for this case, too

use correct talloc hierarchy, and clean up more on error

minor fixes to map.c

remove unused variables, tweak cleanup paths to always clean up
intermediate lists.

minor tweaks to connection handling

conn->shutdown may not exist, and other small changes for
consistency

use vb_foo instead of datum.foo

and update comments to note that '%l' and '%c' are NOT dates,
but instead integers.  If they were of type 'date', then they
would be printed as dates, and not as integers.

use MEM in more places, and free buffers on error

various minor bug fixes in compile.c

various bug fixes for corner cases in rlm_isc_dhcp

check for invalid MS-CHAP attributes

zero out binding in destructure, before calling release

which avoids a double free

ensure that frame->retry always has the same talloc parent

various minor bug fixes in interpret.c

more tests for corner cases of TACACS+ encoder

more tests for corner cases of DHCPv4 encoder

more tests for corner cases of DHCPv4 encoder

removed redundant free
fclose(f) also frees oc, so talloc_free(oc) is not necessary
(lib/util/fopencookie: fix double-free of cookie in fseek error path 159)

return code (-1) when invalid data type
(value.c: fr_value_box_ipaddr_from_network returns 0 instead of -1 for invalid type 302)

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