line number is not order.

The "users" file may have multiple $INCLUDES.
We DON'T want a "user" entry at line 266 of "$INCLUDE 1" to conflict
with a "DEFAULT" entry at line 266 of "$INCLUDE 2".

So we add an "order", which tracks the entry number.  The "lineno"
is no longer used for ordering comparisons.  Instead, it's only
used for debugging.

remove extraneous variable

note recent changes

wrap udpfromto_init

if we encoded data, it's not an error

remove unnecessary function, and fix error message

use udpfromto.  Which should help.

don't munge src/dst IP/port.  May help with #2199

fr_packet_list_find_byreply() already sets the src/dst ip/port
as necessary.

set correct src/dst ip/port for TCP, too.  May help with #2199

make detail writer reopen file if filename no longer exists

radclient "-i" didn't work, so just remove it to save confusion

There's no current way to tell fr_packet_list_id_alloc to use a
particular ID, and fr_packet_socket_t isn't available to us to
change the ID after it's been allocated.

If anyone really needs to do this for testing it can be
temporarily hacked into send_one_packet().

Merge pull request #2191 from nward/update_erx_dictionary

Add new Juniper/ERX/Unisphere VSAs

Add new Juniper/ERX/Unisphere VSAs

Pull fixes from v4 for freeradius.spec

note recent changes

ubuntu Dockerfile fixes/cleanup

Backport Dockerfiles from v4 and simplify (a lot)

also see issue #2188

Merge pull request #2184 from uobnetops/redhat_systemd_spec

Red Hat systemd tmpfile

Merge branch 'v3.0.x' into redhat_systemd_spec

set state correctly

Merge branch 'v3.0.x' into redhat_systemd_spec

vp may be NULL

added softbank dictionary

remove unused typedef

Merge pull request #2187 from jimdigriz/issue-2186

add missing table name

add missing table name

note recent changes

don't add stripped-user-name to the reply

instead, add User-Name

stat the file to see if it changed under us

don't lock the file if it has been renamed

Merge pull request #2164 from frenche/ssl_read

TLS: Treat SSL_read returning zero as an error

Merge pull request #2162 from frenche/partial_chain

TLS: Allow partial certificate chain to trusted CA

Merge branch 'v3.0.x' into redhat_systemd_spec

Tweak paths to deploy to the right place

note recent changes

allow duplicate "authhost" for duplicate realms.

note recent changes

copy tag over from template

copy tag over from template

more error messages

skip empty accounting packets, not all empty packets

one last check

Add a tmpfiles.d entry on systemd based redhat systems, to stop systemd agressively cleaning up /var/run/radiusd at boot time

Merge pull request #2180 from spbnick/double_free_in_acct_redundant

Fix double free in rlm_sql acct_redundant

Fix double free in rlm_sql acct_redundant

Do not free "expanded" buffer twice in "acct_redundant" in rlm_sql.c.
This fixes a crash in the case of an accounting packet not matching a
Start entry in the database.

See also https://bugzilla.redhat.com/show_bug.cgi?id=1540580

Found and fixed by Benoit Welterlen.

clean up fr_pton()

strchr() from the previous commit didn't work, because it
ignored inlen.  Instead, 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.

note recent changes

hoist IPv6 raw checks outside of the look

"face:i..." is a valid IPv6 address, and the "f" portion
shouldn't cause it to do a DNS lookup.

return size of *output* data, not input data

typo

added expected / got text so we know WTF went wrong

updates

cast for correct value

check prefix length values, too

don't copy short data for complex types

re-add byte, ipv4prefix, ipv6prefix

note recent changes.

leave one byte of room for the trailing zero

and zero out the array, too

Revert "re-add SSL wrappers for freeing VPs and Certs."

This reverts commit 63559782c19507cbe9e616d783e51b11c9e6dc72.

Now that we've disabled the internal OpenSSL cache, we can
rely on talloc to clean up memory.

It turns out that OpenSSL doesn't call our cleanup handlers
when removing SSL sessions.

check lengths for these, too

allow casting from integer to date

zero out memory, and copy only as much as necessary

talloc_free is useful, even for temporary things

Merge pull request #2177 from qnet-herwin/patch-1

Typo fix in README

Typo fix in README

s/dialip/dialup/

Merge pull request #2175 from frenche/may_be_attr

parser: relax may-be-attr logic

parser: relax may-be-attr logic

we check for too many hyphens later

change RuntimeDirectory to a location

as per systemd docs and various other blogs, this value is the directory name relative to the parent path, not the fully defined path
with this change /var/run/radiusd will be created and used

typo.  fixes #2169

rhel7 libwbclient RPM dependencies

add and use "find home server using src_ip" function

allow stats to specify 'src <ipaddr>'.  Helps with #2169

The underlying "find home server using src IP" isn't there yet,
but this is the start.

note recent changes

use "raw" string values.  Fixes #2168

Merge pull request #2167 from nchaigne/3.0.x-2018-rest-cainfo

3.0.x - rest - curl option CURLOPT_CAINFO

3.0.x - rest - curl option CURLOPT_CAINFO

As proposed on the ML, this patch adds support for curl option
CURLOPT_CAINFO (through a new parameter "ca_info_file").
This new parameter can be used instead of "ca_file" (which sets curl
option CURLOPT_ISSUERCERT).
(They can also be used both at the same time.)

CURLOPT_CAINFO is useful to validate a chain of certificate authorities.
If this option is not set, curl will try to validate the CA chain using
a default location, and it may fail.

CURLOPT_ISSUERCERT is useful to check the issuer of the server
certificate.

For reference, see:
https://curl.haxx.se/libcurl/c/CURLOPT_ISSUERCERT.html
https://curl.haxx.se/libcurl/c/CURLOPT_CAINFO.html

home_server may be NULL

note recent changes

TLS: Treat SSL_read returning zero as an error

As according to the doc, as well as other usage
of it in our code.

TLS: Allow partial certificate chain to trusted CA

This lets for example to only trust a local sub CA
without having to trust the whole hierarchy.

Merge pull request #2161 from mcnewton/v3.0.x

don't call process_proxy_reply twice when proxying to a virtual server

don't call process_proxy_reply twice when proxying to a virtual server

don't try to build rlm_eap_fast if it can't be built

Merge pull request #2158 from mcnewton/gccnodt

autoconf updates

autoconf typo

Check for -Wno-date-time to pacify ancient compilers

these attributes are "string".  Fixes #2130

remove LN-S from warnings

bump for 3.0.17

note recent changes

don't re-generate `configure` unless asked to

Merge pull request #2155 from nchaigne/3.0.x-2018-dhcpcli-raw

3.0.x - dhcpclient - raw socket fixes

3.0.x - dhcpclient - raw socket fixes

Following the commits by Alan to reintroduce the raw socket code to
dhcpclient, I have:
- fixed a few compilation warnings
- added autoconf for HAVE_LINUX_IF_PACKET_H

I'm not so sure of myself for the autoconf part, can you check it's done
properly?
Thanks!

include dhcpclient binary in freeradius-dhcp debian package

don't fail in removing freeradius-config .deb pkg if dir no longer exists

also don't pull in boiler.mk whilst doing make deb

allow "make deb" before running "configure"

as "make deb" runs "configure" itself in a fake root

make sure install continues even if the daemon isn't running

First install fails if freeradius package and module packages are
installed at the same time.

remove groupname from radacct

it's not used, and it was confusing too many people.

note recent changes