fix radrelay

Updating Nomadix dictionary with missing attributes.

Fix dictionary test

Fix typo in alcatel dictionaries

Better error message

When checking to see if the attribute is a duplicate we need to check the parent too

This used to work because the TLV nesting was part of the attribute number.

Shouldn’t free attributes when the name is freed

It’s OK to have attributes in the dictionary where the name has been subsumed by a later definition

use fr_strerror_printf() instead of ERROR

more removal of pair_make

use numbers, not names

check ptr

close FD on one more path, too

remove unnecessary check

comment-out generic code

Update copyright year

more ignore discard

set FALL-THROUGH

more "ignore discard"

check data length both ways

check pointer before dereferencing it

note fall-through

ignore discard return code

ignore return code from discarding a packet

ignore return code we can't use anyways

ignore return code

we don't care about this return value

don't assign in an assert

cast to "uint8_t *" before adding byte offsets

return on error

remove unused variable

it's good to close files we've opened..

update hash based on client port, too

a few more checks on length

Split TLS caching operation

OpenSSL now receives tls_cache_serialize as the callback
for storing session resumption data, and the EAP-TLS code calls tls_cache_write separately when sending an EAP-TLS success, to cache the session data.

remove unused macro

remove unused variable

remove extraneous check

add missing 'break'

initialize salen

we don't care about the return value when discarding packets

check status of pthread_create

lower bound on maximum stack depth

reference correct field

add missing 'break'

add missing va_end

add missing 'break'

Avoid memcpy when obtaining session ID

remove unused assignment

initialize variable

use file.data for data

remove unused assignment

move dict_tacacs_root to libfreeradius-tacacs

initialize variable

Add TACACS+

systemd syslog.target is obsolete

Hmm, maybe we shouldn't create the database

Removed unused variable

Update rlm_sql.c

Use the actual field lengths when creating the result array

# Conflicts:
# src/modules/rlm_sql/drivers/rlm_sql_mysql/rlm_sql_mysql.c

Trim whitespace before searching for operation type

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

Create the database by default...

made more pretty

update for prettiness

update kibana dashboard so it doesn't have to be imported twice

define the search before the visualisations

Use fr_dict_oid_component in fr_dict_unknown_afrom_oid_str

one more 'const'

one more 'const'

return "known" unknown attribute if it exists

after creating unknown attribute, look it up by name.  If it
exists, return that one.  This lets us use unknown attributes
in the config files, and then have the decoder automatically
reference then when they're seen in the packets.

make fr_dict_unknown_afrom_fields() return const

Prefix references

Shut up doxygen?

Doxygen and other minor fixes

Add checks in more places parent should not be NULL

(everwhere)

Fix clang scan issues

Simplify unknown attribute OID parsing/printing

As tmpls are no longer stack allocated, we can hang additional talloced memory off the structure.

This means unknown attributes no longer need to be placed in a buffer which is part of the tmpl, but can be allocated on the heap.

It also means that where an unknown attribute hierarchy contains an unknown vendor, the unknown vendor can form part of the hierarchy too, without needing a special field.

References #1883

UNUSED -> used

more debugging

and print out packet in decode routine, not in recv routine

don't write empty packets to the detail file

Formatting

Write type directly to the tmpl

Use a dynamically allocated buffer for unknown attribute names

allow EVS for unknown attributes, too

Remove references to tmpl_from_attr_substr

Remove stack based variants tmpl_attr functions

A major limitation of using stack based allocations was that nested templates couldn't be used.

This meant syntax like Tmp-Attr-1[&Tmp-Integer-0] wasn't possible, as there was no way to include the structure required for the index reference without allocating it.

tmpl_from_attr_substr is now private

use tmpl_afrom_attr_substr() here, too

arg.  missed in previous commit.  Stupid git

make node->attr a talloc'd pointer

And move alternate into a "union" with attr.

And add a "regex_index" field to the union, because we can't
just steal node->attr->tmpl_num.  Which was always a hack.

use tmpl_afrom_attr_substr, not the stack-based version

s/new/n/  "new" is a bad name for a debugger which knows about C++

da may be NULL

Remove "Vendor-%d-Attr-%d" and "Cisco-Attr-%d" syntax.

v3 never produced those formats, but did parse them.  it's better
to just disallow them entirely.  That makes parsing much simpler

Fix assert

Doxygen doesn't like @fixme

Fix call to value_box_asprint

Add type field to value_box_t

... and modify all the hundres of places which get pair type from vp->da->type, to get it from the boxed value instead.

Add additional validation functions to check for places where the box type isn't set correctly.

References #1883
Closes #1012

Const to the left

Fixed raw VSA test

comment out failing tests until we can fix them

more unknown vendors

more unknown attributes