travis issues

don't rely on "echo" supporting \n

Don't leak the first unknown attribute in an unknown attribute hierarchy

Fix leak in anything that used fr_value_box_from_fmt

doxygen

remove unused variable

remove unused variables

%zd -> %u

doxygen

always one thing I missed..

Rewrite proto_radius to be not insane.

proto_radius_udp now ONLY does UDP reads/writes (and open/close)
it does not track packets, do dedup, etc.

All of the packet trackign has been moved to io.c, in proto_radius.
That function tracks pending packets, does dedup, etc.

The IO is bounced through io.c, which does the heavy lifting.
Then, if it actually needs to read/write packets to the network,
it calls the underlying app_io.

This means that RADIUS over TCP should be not much larger than
RADIUS over UDP.  There will need to be some additional work to
get accepted sockets in, but the base work for connections exists.

As a plus, you can now define a dynamic client and mark it as
using connected sockets.  Which means that NAT gateways can now
exist as dynamic clients, AND they can have connected sockets
behind them.  i.e. each connection through a NAT gateway can
have a different shared secret.

no longer look at client->network.

It's only used for proto_radius, and should never have been here.

use IP address as part of client name for debugging

undefined fields aren't assertions

copy the memory for the new entry

set packet event list AFTER inserting the FD into the event loop

so that the module can pause (or do whatever) to the FD as necessary

tell dynamic client code if the socket is connected

notes on low priority of detail file reading

constify lookup functions

don't pass a "free" function

the stats info is parented from the rbtree, so using a free
function results in a double free.

add provisions for private APIs

add connection tracking to clients.

magic for network write

pass packet_ct to NAK routine

Fixup odd output from format.pl

Probably something about fixing up the DHCPv4 dictionaries, will look when we have proper protocol dictionaries.

Exit cleanly from unit_test_attribute to avoid pages and pages of memory leak output

Use MEM to check allocation

Add IEC dictionary to help out with https://stackoverflow.com/questions/47052503/radius-and-iec-62351?rq=1

Format dictionary files

Autoload eap.c attributes

s/network_id/network_name/ to be consistent with RFC 5448

...and a few more

Once more pragma fixup

More sign fixes

Fixupup rlm_mschap Auth-Type enumv and the last few pair_make calls

Typo

Fix sign

Switch to #pragma once

Autoload rlm_mschap attributes

Formatting

Autoload rlm_logintime attrs

Use same logic as logintime

Autoload rlm_ldap attributes

rlm_unix: remove pair_make_config

rlm_winbind: remove pair_make_config

more rlm_yubikey dict updates, remove pair_make

if replicating, always return OK

Redundant condition

Autoload rlm_files attributes

Autoload rlm_expiration attributes

Acct-Unique-Session-Id is internal

Use %pV when printing dates

Correct type

Autoload rlm_example attributes

Autoload rlm_eap attributes

Load CA file into the RSA chain too (else we get validation errors)

Use common field names

Remove pre-proxy from rlm_chap.  This is now handled by rlm_radius.

SASL: Let configure SASL_SECPROPS per LDAP instance

Active-Directory does not allow to use GSS-API signing / sealing when
over TLS (to avoid double encryption?).
While SASL_SECPROPS (or maxssf) coul be set globally via system's
ldap.conf, it is not enough if you want one instead to work with
TLS and the other without TLS but with signing and sealing of GSS-API.

Add a directive sasl_secprops, to allow setting it per instance.

Autoload rlm_soh attributes

Autoload rlm_sql attributes

Autoload rlm_sqlcounter attributes

Autoload rlm_sqlhpwippool attributes

Autoload rlm_sqlippool attributes

documentation helps

add rules to build only libs / programs in a subdirectory

Autoload rlm_test attributes

Autoload rlm_unix attributes

Autoload rlm_winbind attributes

Autoload rlm_yubikey attributes

Bump to clang/llvm 6 (same as homebrew)

Set WITH_VERIFY_PTR during clang scan so that NULL code paths result in exit

s/VERIFY_DA/DA_VERIFY/

Fix scoping

fd not used so no need to set to -1

Add fr_pair_add_by_da

if ((typo) {}

Ignore missing prototype warning for __lsan_is_turned_off

Formatting

Leave events in the rbtree if they were not removed from kevent

Distinguish between deferred frees and kevent errors in fr_event_fd_delete.

Merge pull request #2210 from nchaigne/4.0.x-201804-dhcpv4-suboption-test

Test DHCP encoding of two sub-options which don't fit in a single option

Test DHCP encoding of two sub-options which don't fit in a single option

Added unit test following https://github.com/FreeRADIUS/freeradius-server/pull/2209

Merge pull request #2209 from nchaigne/4.0.x-201804-dhcp-encode-suboption

Fix to DHCP encoding error when sub-option doesn't fit

Fix to DHCP encoding error when sub-option doesn't fit

Ask FreeRADIUS to encode several sub-options too long to fit in one option.
Then the first option is not properly encoded. There are two octets leftover (the beginning of the second sub-option).

Here is a fix.

add CC_HINT nonnull

and remove "inline" which does nothing

da->parent may be NULL

check for duplicate NAK clients

If not running under valgrind AND not running under LSAN dlclose()

Add features for running under valgrind, lsan and gdb/lldb

Don't dlclose if we're running under LSAN

Cache fr_get_lsan_state results

At function to determine if we're running under LSAN

Doxygen

Disable stupidly pedantic warnings

fix assertion

Revert "remove unnecessary check.  CID #1433563"

Check is necessary.

Revert "sizeof(struct) not sizeof(struct *).  CID #1433557"

This reverts commit 24326dff862bab6561b43bc058792780d1a335c2.

they are intended to be pointers

Revert "sizeof(struct) not sizeof(struct *).  CID #1433559"

This reverts commit 57b3d362f4463195ccb53b1eaace37ab9bd7e528.

they are intended to be pointers

re-allow cast from uint64 to ethernet. CID #1409865

Unset ASAN_OPTIONS and LSAN_OPTIONS so they're not passed to scripts called from configure

Leaving them enabled messes with some of the tests