dns: Use box initialiser instead of doing it manually

concatenate multiple results from child.  Fixes #3466

add test for things after the expansion

initialize heap_id

allow adding to an empty heap when heap_id==0

initialize heap_id, and don't assert on cleanup

if we can't insert the ev into the heap, then _event_timer_free()
shouldn't assert that it is in the heap.  We fix this by just
setting the talloc destructor to NULL.

use struct initializers instead of manually setting fields

as this should ensure the other fields are NULL

typo

fix build rules

OUTPUT is only defined by $(eval $(call TEST_BOOTSTRAP)), so it
can't be used before that function is called.  And it can't be
used if that function isn't called.

Hoist TEST definition to a place where it's available, even if
we're not running the tests.

tmpl: Fix direct box value assignments (for most cases)

conf: Initialise temporary boxes correctly/completely

value: Ensure talloc buffer types are always explicitly set

redis: Use boxing macros, and add support for 64bit signed types

ldap: Don't modify boxes directly, use assignment functions

cache-redis: Use value box assignment functions

cache: Remove unused type assignment

rlm_redis_ippool: Don't touch boxes directly, use assignment functions

rest: Don't touch intermediary boxes or martialed boxes directly, use the assignment functions

value: Allow bools to be boxed

json: Use VB boxing macros instead of doing assignment manually

Fix _CONST macro position in dict.h

more *cpy to *dup

global: Correct function names, should be *dup not *cpy as we allocate memory

pair: Remove more code by using the value box assignment functions instead of re-implementing them

pair: Make fr_pair_value_snprintf use our interanl fr_vasprintf function via the value box API

print: Fix the talloc buffer type emittted by fr_vasprintf

Fix const issues and initialisation issues in pair_legacy.c

Temporarily revert changes to remove strsteal functions

remove fr_value_box_bstrsteal() and fr_value_box_bstrnsteal()

they are no longer used, needed, or wanted.

switch to fr_value_box_bstrassign()

switch to fr_value_box_bstrassign()

switch to fr_value_box_bstrassign()

allow for text immediately after expansion.  Fixes #3466

Sometimes less code is better

use fr_value_box_bstrassign()

which can be done with only minimal code rearrangements

add fr_value_box_bstrassign()

which avoids talloc_steal().  That function is O(n) in the
number of chunks in the parent's talloc ctx.

move GNU make magic && shell scripts to more GNU make magic

the rules to create $(OUTPUT)/%.attrs didn't work.  This failure
is shown by the fact that the make file has 3 different ways
to produce the same files:

* a set of BOOTSTRAP variables
* $(OUTPUT)/%.attrs: %(DIR)/%.attrs rule
* shell script if / cp rules

So we delete all 3 methods, and replace them with a simple 'cp',
along with a $(if) and $(wildcard), to select files A or B.

We would _like_ to do:

$(OUTPUT)/%: $(DIR)/% $(OUTPUT)/%.attrs

except GNU make it too stupid to allow that.

remove one more bstrsteal by reordering code

remove one more use of talloc_steal()

along with getting rid of snprintf.

Which ends up being simpler and less code

Reduce scope of ret closes #3465

May as well skip the strlen

xlat: Use assignment functions when setting vb values, closes #3464

event: More detailed asserts

event: remove redundant assert in _event_timer_free

The heap will catch this condition and return an error, along with a bunch of other invalid heap_id issues

heap: Remove duplicate/redundant asserts, and quiet clang scan

if we remove something from the heap, it must be in the heap.

fix typo

if it's not in the heap, don't extract it

don't extract things which aren't in the heap

the timer is frame-local

when the frame goes away, so should the timer.

tests are good

make functions take explicit TALLOC_CTX

which avoids talloc_parent()

xlat: Make %{pack:} work on boxes not attribute references

log: There are many types of error

heap: Shutup pedantic compiler

allow attributes of type "struct" with flag "internal"

These attributes don't do anything right now, so they can't
be problematic.

The hope is to allow structs for pack / unpack xlats

document %{pack:...}

heap: Cleanup get/set code

global: Fixup incorrect uses of fr_strerror

talloc: Extend link_ctx functionality so that we can run custom destructors

re-add test.radclient

ensure that the heap_id is always -1 before insert

more "shut up clang analyzer"

remove debugging messages

radclient tests fail for unknown reasons.

disabling so that we can get travis back to a working state.

if a number of things fail in travis, they should be disabled until
such time as they're fixed.  Otherwise new code may introduce new
travis failures, which won't be seen until the other issues are
fixed.

remove debugging line

add %{pack:&Attr-foo &Attr-bar} as a parallel to %{unpack:...}

typo

suppress things which are known to not work under travis

request cap_net_raw on Linux

print out strerror from thread instantiate

so that we don't need to do it in each module

asserts to keep clang scan happy

rlm_rest should depend on libfreeradius-curl

tests for ICMP

allow unit_test_module to use the parent event list

so that it can run modules which yield

try to run tests in alphabetical order, instead of random order

s/create/alloc/

also check that node 0 isn't already in the heap

remove redundant code

more sanity checking

on extract, we set heap_id = -1

on insert, we check that heap_id == -1 or 0

0 is still technically wrong, but we allow it as a special case
for nodes allocated via talloc_zero().  And changing the heap code
to start at offset 1 instead of 0, will require many changes.

The current code is a good compromise

Unused

there's no reason to have `test` depend on the binaries

each individual test should have its own dependency on the
binaries that it needs.

dict: More detail in lineage assert

Undo some assertion changes in proto_radius_udp.c

stop radsniff tests running until they work...

shutup clang scan

allow apt https installs in jenkins travis dockerfile

make the jenkins travis dockerfile readable

remove more unused variables

cleanups and fixes

- just use "counter" to loop packets
- don't create per-request data / ident.  These should be
  per-thread.  Checking them per thread allows us to avoid
  rbtree lookups when the packets aren't for us
- clean up other debugging

tested for IPv4 and IPv6

tmpl: Fix tmpl_attr_resolve_unparsed so it doesn't trash num or tag

...and works for nested attributes correctly.

remove unused variable

event: Fix assert msg

rlm_icmp: Fix icmp6 support

The documentation says that sockets() created with IPPROTO_ICMPV6 will receive packets
without the IPv6 header nor IPv6 extension headers.

In this case, we don't have a way to extract the source address. That is the reason for
we are using the ICMP requests 'ident' field instead of the IP address.

rlm_icmp: Print the icmp-reply payload

rlm_icmp: Just keep 'ping6'

cond: Print operands correctly

remove unused variable

sranddev() is FreeBSD only

remove redundant function

heap: Convert heap tests to acutest tests