remove unused assignment

remove unused struct entries

remove unused struct entries

move declaration to inside ifdef, where it's used

%zd -> %zu for size_t

do run-time check of rbtree_insert()

add assertions for RBtree checks

srcdst structure is const

src/dst addresses are const

Move tls_strerror_printf into its own header file

Add implementation of 3GPP ephemeral identities

Build tls functions into libfreeradius-server not radiusd

Expose the arrays used to encoding/decoding base64 directly

remove unused assignment

more preparations for eventual cleanup

destroy worker where we created it

note that we would LIKE to free the event list here, too.
But... the event list is cached in many modules, and therefore
cannot be freed until all of the modules have been freed,
and all of the threads have exited.

timer events are parented from the event list

so delete them manually, before calling talloc_free_children(),
otherwise we will try to access children that no longer exist...

notes on thread exiting

simplify use of circular list

Fix coverity issues CID #1420467, 1420478, 1420469

add assertion

set entry->reply = NULL as necessary

move polling message to debug3

it will still wake up and print "ready to receive requests",
but one less debug message is useful

do more memory cleanup on exit

free children before the queues they use

move debug message to higher level

note todo

coalesce stats across multiple threads

clean up schedule exit code

because of back & forth signaling issues, we have to be careful
how we clean up memory.

First, the network is signaled to exit.  When it does, the
scheduler thread sends a semaphore saying so.  That signals the
master thread to call fr_network_destroy(), which in turn signals
the worker threads that it's exiting.

BUT fr_network_destory() doesn't free underlying memory.  Instead,
we have to wait for all of the workers to finish.

The workers are signaled to finish, and again don't call
fr_worker_destroy().  Instead, they don't ACK the channel close,
but do signal the semaphore again.

When all of the workers are done, it's safe to free the network
memory.  And then it's also safe to free the worker memory.

... and then disable most of it, because apparently even
signaling the workers to exit causes problems.

allow storing and querying stats by src/dst/global

for now, it's thread-specific.  But it does work.

Next, merge all of the stats from other threads...

track statistics by src/dst IP address, too

merge thread stats into global stats on thread exit

first pass at having statistics in v4

Revert "add counters by packet code"

This reverts commit 2b75a31a05de9c6160b493dc60fb29ec141ec332.

in the end, this belongs elsewhere

remove redundant memset.  CID #1271306

clarify error message

remove unnecessary variable

Merge pull request #2103 from pwdng/v4.0.x

Bug fix to get radsniff working

Bug fix to get radsniff working

multi-stage docker builds

add counters by packet code

fix dictionary error printing

dict.c now pushes errors
and the caller now prints them line by line

also double-checked the various messages so that they all make sense...

use new RADIUS API

remove unnecessary debug message

track statistics in proto_radius_udp

Only global statistics.  Per-packet statistics should be done
in a module.  Sorry about that...

rework dockerfiles

mk-build-deps needs pkg called libkqueue-dev

debian 9 build dependency

force install in dockerfile

Merge pull request #2102 from herwinw/rlm_rest_value_box_rest_http_status_code

Fixed rlm_rest

Fixed rlm_rest

Add an explicit type to the value assigned to REST-HTTP-Status-Code.
Without this, the following error occurs:

CONSISTENCY CHECK FAILED src/lib/util/pair_cursor.c[478]: VALUE_PAIR
attribute 0xb6c639a8 "REST-HTTP-Status-Code" data type (int8) does not
match da type (uint32)

re-add deleted variable

Revert "remove rad_authenticate()"

This reverts commit 8f22ef299bb34a45bc9c10c70c675e21747417ad.

start the timer when we get the packet

not when we first send it.  If all of the connections are down,
the timers will never fire.

distinguish different kinds of stop messages

because it was already mostly doing that

more assertions

delete only if they're in the heaps

remove more unused functions

remove unused variable

reset "is_set" on every round through the loop

name2 can be NULL

remove rad_authenticate()

and move auth_name to proto_radius, which is now the only
user of it.

remove reference to unused function

remove prototypes for functions that no longer exist

remove more unused files.

If we need the code, it's in the v3 branch

no more need for rad_accounting() or acct.c

name2 may be NULL

fr_event_timer_delete() returns int, not bool

don't delete on conflicting packets

remove WITH_foo hacks

the base library should be able to do everything

avoid zero-change memcpy()

fix wrong assertion

clean up tracking entry in corner cases

dump use after free

active (maybe) only when returning

fd_active() may free the connection <sigh>

e.g. if there's an error activating it.

more assertions and cleanups

it's 4.0

Merge pull request #2100 from jsynack/virtserver-fix

Virtual server fixup

Update the example config file for virtual servers.

1. Remove reference to proxy.conf

2. Modernize the server config settings with namespace/listen/transport information.

3. Include enough configuration for a working virtual server for auth/acct on alternate ports.

Correct typo and expand an error message when the virtual server lacks the namespace setting

Merge pull request #2099 from jsynack/utfscrub

UTF Scrub

Replace a UTF char with the ASCII version

Update information in RADIUS-LDAP-eDirectory

1. Remove the registered trademark blip to make this file ASCII-only

2. Update the documentation URL with current version

Remove UTF ZWNJ/ZWSP combo in comments to make the file ASCII-pure

Replace UTF char with ASCII version

Replace UTF-8 spacing with ASCII spacing

Revert "manually set heap_id = -1"

This reverts commit bd5402a53b82e8b46ccb69f2073020da6d8dde65.

no longer necessary now tat we've tracked down the real issue

use new definition for Original-Packet-Code

clean up, and allow for (some) extended attributes

include RFC 7930

don't re-insert the connection if we receive multiple packets

update assertions

minor optimizations for status check

verifications for heaps

better cleanups for connections

better cleanups for status check packets

free only if it exists

Merge pull request #2098 from jsynack/radsniff-fixup

A few adjustments for radsniff

Merge pull request #2097 from jsynack/dictfixups

Dictionary Fixups