Doxygen and pedanticism

Merge pull request #2109 from mattrose/v4.0.x

fix rpm build errors

fix rpm build errors

Use new value box based function for one letter expansions

Add inline fr_cursor_free function for removing and freeing the current and subsequent items

s/XLAT_PERCENT/XLAT_ONE_LETTER/ s/XLAT_MODULE/XLAT_FUNC/

PERCENT didn't really mean anything.

MODULE is a misnomer, because the majority of the xlat functions are not from modules.

Revert "one more fr_event_timer fix"

This reverts commit 33499ae0ecc1314bbaa679d45e3d29918cf208ae.

one more of let's see if this works

Revert "if we're leaking in event link ctx, don't link ctx"

This reverts commit ba91811370c1f48e3bb2f194734f4cd7047d0f87.

let's see if this works

ordering issues in free

don't do double frees.  Assign correct pointer to link->self

redo link ctx stuff to remove leak

one more fr_event_timer fix

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

Remove execution bit for systemd service conf file.

Remove execution bit for systemd service conf file.
This is to avoid 'Configuration file /usr/lib/systemd/system/radiusd.service is marked executable. Please remove executable permission bits. ' log messages

if we're leaking in event link ctx, don't link ctx

Stupid stats...

Helps to link the radius client against the radius library...

Explicitly link rlm_chap to libfreeradius-radius

and another fr_packet_codes reference

Use the Packet-Type attribute to resolve packet codes to names

Only a dependency if we're building with OpenSSL

Functions now available in libfreeradius-server

Explicitly link libfreeradius-io to libfreeradius-server

Explicitly link against the RADIUS library

Doxygen

Remove dependencies on main_config from modules.c and move to libfreeradius-server

correctly put spaces into dependency_libs

note one more todo

check cleanup_delay

Build unlang and virtual servers as part of the server library

Remove dependency on main_config from unlang_compile.c and virtual_servers.c

Move tls support functions into their own library

Move max_attributes and tunnel_password_zeros out of the global config

This means we no longer have to link radiusd to the radius library, which is worth the extra hassle.

Limit FreeRADIUS memory to 2G

remove unused assignment

clean up conflicting packets

i.e. toss the new one, but don't send a reply for the old one.

just rely on src_dst information

As per commit 293ce7e991843f

we don't need to track ID / code separately

distinguish conflicting packets from dups, from new ones

disallow src_dst_size == 0

Revert "Build tls functions into libfreeradius-server not radiusd"

This reverts commit 6087844e40959be171062dde235294810954bca1.

this breaks the build.  It needs to be re-done once it builds
correctly.

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