as posted to the list

note recent changes

reduce scope of variable.  Found by PVS-Studio

request by be NULL.  Found by PVS-Studio

move shutdown calls into check for ssn->ssl.  Found by PVS-Studio

check sizeof(*packet).  Found by PVS-Studio

check ptr before dereferencing it.  Found by PVS-Studio

remove redundant declaration.  Found by PVS-Studio

move assertion to correct place.  Found by PVS-Studio

remove redundant assignment.  Found by PVS-Studio

move assertion to top of function.  Found by PVS-Studio

fix wrong assertion.  Found by PVS-Studio

check for OOM.  Found by PVS-Studio

check before dereference. Found by PVS-Studio

don't assign wrong enum to variable.  Found by PVS-Studio

remove redundant check.  Found by PVS-Studio

don't use i for inner and outer loop.  Found by PVS-Studio

remove duplicate checks.  Found by PVS-Studio

Merge pull request #1989 from Sp1l/v3.0.x

Fix build with LibreSSL

use correct packet for channel binding.  Closes #1990

create string only if it's needed

use RDEBUG

remove always-false condition

Fix build with LibreSSL

LibreSSL does not have X509_get0_extensions and was forked from 0x1000200fL

See also: https://bugs.freebsd.org/218225

Merge pull request #1988 from alejandro-perez/v3.0.x

Fix typo in previous commit

Fix typo in previous commit.

re-order old names.  New names come second...

convert assertion to run-time check.

convert assert to run-time check.

revert debian packages to 3.0.12 versions

Added missing Patton Vendor Attributes

Added some new Patton Vendor Attributes to the list.
These attributes are now supported on the newest Patton device running on Trinity software version 3.11.2.

Merge pull request #1974 from alanbuxey/patch-3

fixed variable to use the "&" prefix

Merge branch 'v3.0.x' into patch-3

add aliases for well-known names

set statment to NULL.  Fixes #1983

Merge pull request #1985 from alejandro-perez/v3.0.x

Fix memory leak in trustrouter.c

Fix memory leak in trustrouter.c

In the trustrouter.c file, servers were being created using
talloc_zero() instead of tls_server_conf_alloc(). Thus, the
destructor _tls_server_conf_free() which frees the SSL_CTX
object was not being called.

Merge pull request #1982 from alejandro-perez/v3.0.x

Some fixes to the trustrouter related code

Set idle_timeout to 5s to all the dynamic home servers

Dynamically generated home servers get eventually replaced.
We want sockets using these servers to close as soon as possible, to make sure that whenever a pool is replaced, sockets using old ones will not last long (hopefully less than 300s).

Increase the amount of time a pool spends in the garbage list

Under specific circumstances (e.g high authentication load) a client might keep using an old pool since the socket did not expire. 60 seconds seems too low.
Increased to make sure we do not delete it while it is still being used.

Remove unnecessary check to update REALM

Existing code precluded a REALM from being updated if there were traffic within the last 5 minutes.
This is an error since when the TLS keys expire, the home server will reject client’s attempts to establish a connection, leading to up to 5 minutes of denied user authentications.

Merge branch 'v3.0.x' into patch-3

note recent changes

disable internal OpenSSL cache

8 and 9 have tags, too

set S_IWUSER when creating the file, not later

added one more attribute

update detail reader documentation

Closes #1973

fixed variable to use the "&" prefix

removal of yellow warning when running with this enabled

Patch from Jeff Gehlbach

The problem is that "radiusObject" is defined with an OBJECT-IDENTITY
macro, but it needs to be done with OBJECT-TYPE (i.e. a leaf node) to be
eligible for use in the OBJECTS clause of a NOTIFICATION-TYPE macro.
I've gotten jsmiparser happy by making that change, declaring the
object's syntax to be SNMP-FRAMEWORK-MIB::SnmpAdminString and setting
its max-access to "accessible-for-notify". These changes are reflected
in the attached version of the MIB definition.

Merge pull request #1964 from alanbuxey/v3.0.x

stop rotation of the session database files

stop rotation of the session database files

you really don't want to be rotating these under the server - they are
not normal log files but are stateful session files (used by various
utilities). these were removed from the logrotate some time ago but
appear to have crept back in.

terminate && check VP

account for trailing zero.  Closes #1960

Revert "these messages don't need to go to the main radiusd.log"

This reverts commit 1f1a02baae35080b4037af88c709ef6c0ccdd2d7.

note recent changes

these messages don't need to go to the main radiusd.log

Merge pull request #1961 from alanbuxey/patch-8

corrected some types and grammar in comments

corrected some types and grammar in comments

Don't crash on unexpected regex.  Closes #1959

check for request->packet.  Closes #1935

Fix typo.  Closes #1955

note recent changes

these attributes are byte, not integer.  Closes #1954

start of peapv1

Merge pull request #1952 from spbnick/rlm_ldap_segfault_fix

Handle connection error in rlm_ldap_cacheable_groupobj

Allow utc.  Patch from Peter Lambrechtsen

Handle connection error in rlm_ldap_cacheable_groupobj

Closes #1951

cf_log_err(), not fr_strerror_printf()

map_cast_from_hex() does not produce error messages

note recent changes

re-add SSL wrappers for freeing VPs and Certs.

because OpenSSL caches things at it's own pleasure...

more cisco VPN attributes

Add Attribute 3076/85 (CVPN3000-Tunnel-Group-Lock)

http://www.cisco.com/c/en/us/support/docs/security/ios-easy-vpn/117634-configure-asa-00.html

note recent changes

Search from the beginning for altname.  Closes #1946

Allow no cert when psk is configured

remove outdated link

Ensure that error is always initialized

Remove always-false condition from cf_item_parse

Remove always-false condition in rlm_eap_fast

return RLM_MODULE_FAIL for default switch statement

close open FDs on error, and use error path in more situations

remove unused variable

Merge pull request #1941 from spbnick/openssl_1_1_cert_perms_fix

Relax OpenSSL permissions for default key files

Relax OpenSSL permissions for default key files

Recent versions of OpenSSL appear to create keys with owner-only
permissions. Allow owning group to read the created default key files
in raddb/certs, so that they stay the same as with older OpenSSL, and
that the server can read its key.

port ranges haven't been supported for years

request->packet cannot be NULL. Helps with #1935

Allo session resumption for RadSec connectins.  Closes #1936

Coverity.  Closes #1937

more checks for client certificate expiration

Remove microseconds from %S.  Closes #1934

note recent changes

enforce TLS client certificate expiration on session resumption.

Merge pull request #1933 from spaetow/patch-1

Add enhanced checks to avoid targeted_id_salt leakage over %, {, and } in the salt

Update moonshot-targeted-ids

Merge pull request #1931 from sjbronner/patch-1

Fix command for linking modules in mods-enabled.

bump for 3.0.14

radtest should use Cleartext-Password for EAP

Fix command for linking modules in mods-enabled.

Running `ln -s mods-available/foo mods-enabled/foo` will result in a dead link: `mods-enabled/foo` will point to `mods-enabled/mods-available/foo`, which doesn't exist. The link is relative from its location, not from the current directory from which it was created.

The easiest method that allows using tab completion is to link from within `mods-enabled`. The second parameter to `ln` can be left off in that case, as well. This is the change I have proposed. Another alternative would be to run `ln -s ../mods-available/foo mods-enabled/foo` from the `raddb` directory.