Clear the error buffer on successful tests, else it gets really confusing

Explicitly include packet.h in dhcpv4.h

Fix warnings in map.c

Tests! (for DHCPv6 encoder)

Just a few of the simple types, more needed...

Fix a few issues in DHCPv6 encoder

Now skips false bools properly and doesn't crash

Need to build base.c in dhcpv6 code

Ignore false bools too

Rename DHCPv4 test

add FALL-THROUGH for newer compilers

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

Fix when skipping record

Fix when skipping record

one more attempt

add rules to create / link directories

remove "util". It's not needed

Skip false boolean attributes in SIM/AKA/AKA' encoder

Fix stupid GCC warnings

Remove doxygen until we use the function

Add preliminary DHCPv6 dictionary

cache leftover data when retransmitting

Options not attributes

Add v6 encoder test point

Convert all encoders from vp_cursor_t to fr_cursor_t

Add (untested) code for DHCPv6 encoder

Describe how pair_encode/pair_decode functions *should* work and define some standard return values.

Add CHECK_FREESPACE macro

In general the internal encoder functions *should* return how much extra buffer they needed, so that their callers can make decisions about TLV nesting

Define SSIZE_MIN if missing

notes for dynamic clients and connected sockets

remove NAT configuration for clients.

Instead, if we have dynamic clients *and* we're using
connected sockets, THEN the clients should be connection-specific.

The remaining "todo" is to check for multiple packets from the
same source IP, but with different source ports... and to
track those separately.

remove erroneous NAT code

this should now be handled automatically by mod_read()

clarify check

send a 1 byte NAK back

update the right fields, and if_index, too

add debug3 message

clean up mod_read()

instead of having horrible if / goto stuff, split up the code.
There's now a mod_read_packet(), which gets a packet from one
of N possible sources.  This function also updates address,
address.client, and track.

The mod_read() routine then does checks on address.client and
track as necessary.

Test with UDP packets.  Untested with dynamic clients or connections.

hoist checks for connected socket to before client lookup

because if we have a connected socket, then we don't need to
look up the client.  Instead, we just send the packet to the
connected socket.

notes on removing the mutex lock

comments

document "work" subsection

remove worker configuration from the directory watcher

in files are annoying

rename for Windows.

src and dst may be the same

Remove unused arg

Convert JSON integers to the smallest signed or unsigned integer type

When the code was first writen we didn't have the full range of signed types.

Fix double free in acct_redundant

copy the leftover data if necessary

pass "leftover" explicitly

casts

Remove debugging fprintf in fr_inet_pton

Break out cast to unsigned integer into functions

Fixes issues like not being able to cast a bool to a uint32

clean up fr_inet_pton()

Walk over the *whole* string, checking
*all* of the characters.  The issue with the earlier code was
that it bailed too early, and forced IPv4 even when the address
was IPv6.

manual port of 5452b13cef

if the entry is too large for the buffer, discard the file

convert assert to run-time check

Wait to remove the candle from the mandle

TAG_ANY is ok too

If action is NULL return NOOP

Better errors in fr_event_list_alloc decides to fail

Print an error message for everything that could fail in fr_event_list_alloc

Because it's currently failing randomly and we have no idea why

Fix more files to have doxygen style headers

Rename files.c to users_file.c

create CHAP-Challenge if it doesn't already exist

s/FR_SIGNAL_DONE/FR_SIGNAL_CANCEL/

the request has been cancelled, but it might not be done.

Revert "make fr_network_socket_delete() thread-safe"

This reverts commit 5dc5d9b97f523607c7829bdeaeed448bac49d8ba.

fr_network_socket_delete() should only be called by the IO
handler, which is already in the network thread

Store current module, and set old_backlog correctly when calling unlang_interpret_synchronous

Doxygen

Only build proto_tls_cache.c if we have OpenSSL

Add stub proto_tls_cache module to compile tls_cache sections

Ignore all subdirectories in src/include

Because manually updating .gitignore every time we create a new library or protocol is going to get very tedious

this shouldn't be commited either

ignore DHCP things

this should NOT be committed

Call/compile TLS Session and OCSP server sections

This is technically async safe, but the calls are still potentially blocking.  More work needed to fix that.

There's no advantage to having them actually async capable until the cache module is async capable anyway...

Check permissions last, right before we call the panic action

add assertion that deletion succeeds

clarify debug message

Add the -f back into LN-SF

Doxygen

Update kqueue FD removal assert to print errno

Move all rad_cond_assert to fr_cond_assert

The distinction isn't that important, and they ended up being mixed in the source files anyway.

Boilerplate functions for DHCPv6 (hey it's a start)

make fr_network_socket_delete() thread-safe

call fr_network_socket_delete() before closing the FDs

allow listeners to remove themselves from the network

limit max_outstanding to 1

port #2175 from v3.0.x

clean up error handling

rely on parent function to insert retry timer

allow faster retries for empty files

deal with race conditions on duplicate packets

Untested as of yet. :(

do de-dup detection for accounting packets, too

more sanity checks

make code/id 8-bit values

that's all we need

I hate my life

rely on talloc to clean things up

don't pass bad values to the encoder

i.e. we have layers of checks, just in case one layer breaks

fail accounting == do not respond

move assert to run-time check

Don't leak message buffers

Don't leak xlat_thread_instances for epehemeral xlats

Doxygen

developer build should have debug symbols in it