driver_inst can't be NULL here.  CID #1414433

close socket on error.  CID #1419878

we don't care if write() fails here.  CID #1414420

use correct number of arguments.  CID #1419884

check both workers... CID #1419881

handle dup() failure.  CID #1419880

app_io must be set.  CID #1419879

fix args to printf.  CID #1419885

check correct values.  CID #1419876

This might explain why some events are odd..

channel recv_time cannot be NULL.  CID #1419871

fix assertion. CID #1419870

set time to now.  CID #1419868

check before dreference.  CID #1414436

fr_scoket_client_tcp() can fail.  CID #1414430

driver_inst is already checked, and can't be NULL

delete lock after using it.  CID #141442

use success / fail instead of empty sections

document new detail file reader

add "send fail"

notes on retransmissions for failure

clean up and double-check

if there's a good reply, run send { ... }

if there's a bad reply, run send Do-Not-Respond { ... }

add synonym

remove notes about v2

ifdef out variables for assertions

set the backlog from the parent who has it

zero out memory so the event handler doesn't get excited

over garbage memory

more sanity checks in unlang_resumable()

backlog can be NULL if the request wasn't detached

detach before freeing child

child requests are never in the runnable queue

clean up unlang_resumable()

one more assertion

check for thing before freeing it

more assertions and checks

assert that the request isn't in the runnable queue

more assertions

remove "runnable" requests from the runnable queue

set default for "track" and "poll_interval"

clarify how thngs work

child created detachable is always detachable

clean up children immediately on "return"

add configurable poll_interval

cast usec to int, too

use state transition function for u, too

do polling for detail files

assert that packets which have replies were actually sent

i.e. once we have a reply, the packet is no longer sent,
it's done.

set work_io_instance=NULL in file, not worker

because many things can cause the file to be deleted...

more cleanups on error, and deletion of detail.work

alloc a worker and run it

add decode & write

which bounces through to proto_detail_work, which does the
actual work.

We should probably have a way to pass the fr_listen_t to proto_detail,
so it can use it's app_io, instead of the one in the proto_detail
instance

free the worker if we're closing the FD

and if the flag tells us to

minor cleanups

add "self" for linking issues.
parent is not "const"
move fr_schedule_t to proto_detail_t, not proto_detail_work_t

and initialize the entries

clean up to allow proto_detail_file to call us

minor cleanups

filename.work is no longer mandatory.  If it isn't defined,
we'll just create it.

If detail.work does exist, watch for it's deletion

more than one NOTE can be returned

open RDWR if tracking, else RDONLY

clear out el->events[] for an FD when the event is deleted

use -, not a-b

don't use t->pending, just rely on heap size

if there are two things showing the same information, it's possible
for them to get out of sync.  So... one way to rule them all.

set t->pending only if there's pending packets

handle 'close' as a separate function

so we can catch it, and inform our parent instance that it
needs to rescan the directory

instance isn't 'const' in close

auto-create "work" configuration

to simplify the users life

note recent changes

/ may occur in dictionary names, too

so we can't put it into the comments in the file

cast to long double

%f is double

cast to double

simplify the translation rules

port relay fixes from 3.0

sort by FD first, not filter.

there are really only 2 filters used, but multiple FDs

don't free children in parallel state destructor

they're already either parented from the request, or linked
to the request via request_data_add.  And will therefore be
freed as necessary.

Plus, if they're detached, we definitely DON'T want to free them
when the parallel section exits

use correct enum value

parallel clones by default

but can be empty via 'parallel empty'

which likely needs a better name

free children on DONE signal

delete the reply after receiving it

connection may be NULL

remove unused variable

dereference correct variable

make ourselves active if we read and there's pending packets

this is likely imperfect, but it's not entirely wrong

double-check idle handling

more cleanups and sanity checks

go over status-server retransmission checks

go over zombie timeout checks

and more cleanups

the basics work.  But (again) the corner cases need to be checked.

more cleanups

start of cleanup and change to better state transitions

Merge pull request #2091 from bmork/unisphere-vsas-for-v4.0.x

add new Juniper/Unisphere/ERX VSAs

remove active_requests

we can calculate it from the queued / sent heaps

travis fix

clean up connection state on failure

always add Event-Timestampt to status packets

and always update it when sending those packets

don't update Acct-Delay-Time for status check packets

enforce limits on retransmissions

and make status check packets use the same retransmission path
as normal packets

use local retry variable

we can send reply to network after max_request_time

do proper state changes on retransmit

add new Juniper/Unisphere/ERX VSAs

Importing new attributes from
https://www.juniper.net/documentation/software/junos/junos161/radius-dictionary/unisphereDictionary_for_JUNOS_v16-1.dct
using the exact attribute names Juniper use, following the precedence set
by the original conversion from dictionary.erx to dictionary.unisphere

Yes, Juniper *do* use four different attributes prefixes here:
 Unisphere, Pcp (sic), Jnpr and ERX(!)

Quite impressive.  And quite a mess.  But at least it's not *our* mess
if we just got with their names...

Signed-off-by: Bjørn Mork <bjorn@mork.no>

Return -1 from fr_event_filter_insert if fr_event_build_evset fails (instead of going into an infinite loop)

track active packets in a connection