Correct format specifier

Catch bad secret values

fff

Disable foreign attribute check

Check rad_debug_lvl correctly, so the messages are enabled/disabled based on request specific debug levels

Reduce manual calls to hex printing functions

Code should always execute.  WARNings are always emitted, and code applies fixup to VPs that shouldn't just be applied if debugging is enabled.

clarify message

remove unused assignment

word smithing

Use newer ephemeral struct init syntax

Typo

Cleanup minor issues with attribute search functions

Probably not a useful test anymore

Make it easier to see the ref count in a debugger

Convert pairs xlat to new async API

Update concat xlat to use separator

Bug fix in fr_value_box_bstrsnteal.
'len' is not updated when talloc_realloc_size is called.

Fix memory leak in xlat_base64

Replace base64tohex with base64decode using new async API

doxygen

don't increase refcount on first autoload

because the manual load of the dictionaries doesn't result in
a manual unload.  so the autoload has to auto-unload it for us

Fix NAS port struct

Free resources on failure in unit_test_map

Anonymise structs

make sim/base.h

clang scan fixes (hopefully)

if `dict` can't be NULL, no need to check for NULL

doxygen

Markup nonnull arguments of fr_dict_attr_add

fix auto-creation of Vendor-Specific

it was created and added by number, but not by name.  That's wrong.

And arguably we should only do this for the RADIUS dictionary...

suppress duplicate attribute definitions

We don't want to replace them, as they may have values, children,
etc.  Once an attribute is replaced, the old children, etc. cannot
be found.

update comment and add "type" to list of things we check

this should not have been committed

this dictionary is for RADIUS

removed duplicate / conflicting attribute definition

mark up autoloaded dictionaries vs manual ones

So that dictionaries can be manually loaded from raddb/dictionary,
and we can later still autoload the various protocol dictionaries

The only downside is that src/tests/Makefile still fails.
The server loads the test dictionary and parses the config files
correctly.  radclient loads the test dictionary and sends the
attributes correctly.

But somehow (magically) when the server receives the packets, it
can't find the test vendor, and prints out all of the attributes
as raw ones...

capture stdout && stderr to the log file, too

print out correct arguments

we don't need to regenerate this any more

when refactoring, copy code instead of re-typing it.

and building it is good, too

BSD fixes

Merge pull request #2378 from manu0401/master

Fix startup crash when running outside of gdb on BSD systems

Fix startup crash when running outside of gdb on BSD systems

On BSD systems, ptrace(PT_DETACH) uses a third argument for
resume address, with the magic value (void *)1 to resume where
process stopped. Specifying NULL there leads to a crash because
process resumes at address 0.

We introduce an OS-dependent _PTRACE_DETACH macro to specify
third argument as NULL on Linux and (void *)1 on other systems.
Always using (void *)1 could be another solution, since basic
tests suggests passing (void *)1 as third argument on Linux
does not cause harm.

Doxygen headers in rlm_eap_ttls

Move EAP and SIM libraries into src/lib

use top_srcdir for references to source files

print "AUTH-TEST" to make it clear which test is what

use radius as root for decoding TTLS, not internal dictionary

Remove freedhcp dictionary Closes #2376

Quotes

Rearrange so we don't make validating pairs conditional on running in debug mode

MS-CHAP-Password is a FreeRADIUS thingy

MS-CHAP stuff is in radius dictionary, not freeradius

'&' for the win

dictionaries are now autoloaded

don't manually include any of them

print name for duplicate

add AUTO here, too

better fix for previous patch

Revert "add '&' if it's not already there"

This reverts commit 8966bfdf901c06c84762ae8aaa0d37084f9eb0b4.

print out VP names if they exist, too

rearrange so that we're only doing debug work if we're debugging

handle the case where the LHS isn't a properly formed attribute

which isn't *strictly* needed due to the previous patch.
However... it should do the Right Thing even if the data in LDAP
doesn't start with '&'

add '&' if it's not already there

as a hack so that the user doesn't have to put '&' in all of the
LDAP entries.  We KNOW they're supposed to be attributes, so we
might as well FORCE them to be attributes.

use the correct parenting for the attributes

fix looping code so that it doesn't skip dictionaries

Typo

Typo

EAP-AKA-Subtype is in the EAP-AKA dictionary

Autoload sigtran dictionaries and attributes

Update systemd units to depend on network-online

Due to the use of SO_BINDTODEVICE, FreeRADIUS requires the address to be
up when binding to a specific IP address. In systemd, depending on
network.target is not sufficient to require the network to be up, so
FreeRADIUS can still fail to bind; instead, network-online.target is
required if the network is required to be online.

This manifests in errors such as:

Error: Failed binding to auth address x.x.x.x port 1812 bound to
       server default: Cannot assign requested address

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1637275
Signed-off-by: Alexander Scheel <ascheel@redhat.com>

Merge pull request #2375 from horsitis/patch-2

Add support for Downloadable user roles

Add support for Clearpass Downloadable roles

if it's a raw attribute, don't look it up in the global dict

it's known, but badly formed.  So it's not really known.

set unknown_da, too

Add HP-User-Role.  Fixes #2372

Fix typo.  Fixes #2374

allow Attr-oid.str here, too

Doxygen

Typo

typo

# Conflicts:
# src/modules/rlm_mschap/opendir.c

simplify ntlm_auth error output checks

In most cases, the error output will contain "0xC0000", in which case
we can take a fast path

# Conflicts:
# src/modules/rlm_mschap/rlm_mschap.c

Check for error conditions when loading test points

Use constants for errors

Use more compact, inline variable initialisation syntax where appropriate

don't over-write pragma once

tr fixes for Linux

'-' only means itself when it's at the end of a sequence

Revert "include dictionary for FR_USER_NAME"

This reverts commit 3df2a92b04b2d256bf77dbdd8bd99c063b57de4f.

include dictionary for FR_USER_NAME

autoload SNMP attributes, too

don't need to set this

search over all of the protocols for attributes

by using the new has iteration function.

This search is only done if `fallback = true`.  The search also
skips `dict_def`, as that has already been searched.

set fallback=true

so that we can look up key fields in the internal dictionaries, too

Hash iteration functions

Make the data in the hash entry unconst

Minor cleanups

skip '.' after protocol name

dictionary is radius

more cleanups... hope this works

add dict to pairlist_read()

add &

typo