Fix typos and option 82 handling in DHCP sort changes

remove unused macro

note TODOs

added replies to RADIUS tracking table

Simple RADIUS packet tracking API

fix APi for fr_message_localize

added fr_ring_buffer_start()

Which gets "pointer + size" to the data at the start of the
ring buffer.

cleanups, and fr_message_done() doesn't need ms

m_* -> mr_*

which is a better name

document -t

Assignment to tmp not used

Add fr_message_alloc_reserve()

For protocols like TCP, where we reserve a large chunk of memory,
and then read multiple packets into it.  If we want to have a
unique messages for each unique packet, we need to be able to
allocate a message, while reserving the rest of the buffer.

%zd -> %zu

use src_type.  src->type isn't (yet) used everywhere.

clang complains about this, and src_type is used everywhere else
in this function, so we should probably use it here

remove unused variable

Fix typo

Reduce scope of custom in fr_vasprintf

travis warnings

split out allocate message from allocate ring buffer

fix comparison checks.  Fixes #1851

Fix initialization errors

fix inner-tunnel policy

This policy replaces "use_tunneled_reply".  It's better to use
if (0) { ...} to disable blocks of code, instead of commenting it out.

Also, update the local reply before copying it to the outer
session-state list.  That makes a lot more sense.

set reserved

add fr_ring_buffer_reserve_split()

Which lets the caller fix a partial packet, and move it to
a new ring buffer.

Overload %p printk style

Add custom fprintf style function for talloc binary buffers and value boxes

Fix previous commit

Fix DHCP attribute ordering

Extra space between cache options and colon

set reply code

reject on no auth-type

added "todo" note about UDP protocols

Added "protocol" field to each message.

Don't copy the entire datum union

Formatting

re-order fields in value_box to match PW_TYPE

Add missing field sizes

Correct src offset in _tmpl_to_atype

Formatting

Formatting

Use the correct source fields in value_box_hton

Rename cond tokenization and cond evaluation functions

parser.c and evaluate.c really‽

s/tmpl_data/tmpl_value_box/g

Don't use an anonymous union inside value_box_t

Too many other things in there now...

Revert "s/tmpl_data_value/tmpl_data_box/g"

So i can apply the large patch set I was working on cleanly...

Merge pull request #1848 from herwinw/rlm_winbind_const_fixes

Use const consistently in rlm_winbind

Use const consistently in rlm_winbind

This saves us a compiler warning.

s/tmpl_data_value/tmpl_data_box/g

doxygen

add context pointer to event list status function

In preparation for scheduler idle loop work.

Rename value_data_t to value_box_t

value_data_hton() should take value_data_t for src

add priority to request messages

so that the worker knows how to prioritize them

move the union back to the start of value_data_t

because it's just too hard to separate the pointers to the data
from the pointers to the value_data_t.

use union.datum as a generic pointer

track pointer instead of complex struct

pass the contents, not the value_data_t

mark value_data as tainted on copy / steal

don't assume that the struct is packed in any particular order

Part 2.

LOTS of references to &vp->data.  Some of which are to the union,
and some of which are to the data inside of the union.

Separating them isn't trivial.

include linked list and worker stats into time tracking.

So we have one API call to track time && yielded list,
instead of two.

don't assume that the struct is packed in any particular order

i.e. if we need sub-element "foo", refer to sub-element "foo".

filters might as well be uint8_t.

We don't access them as anything else.

pass the correct length to hex2bin

set all of the message to zero on allocation

Don't create the symlink every time we run make

mark User-Name from EAP packets as tainted

VPs coming from packets are tainted

VPs coming from packets are tainted

VPs coming from external programs are tainted.

VPs coming from packets are tainted

function to mark VPs as tainted

Move fields around, and add "type" and "tainted" flags.

So that value_data_t becomes a bit more like a real boxed value
structure

Merge pull request #1844 from herwinw/rlm_rest_json_prefix_buf

Use stack based char array to generate JSON keys

Use stack based char array to generate JSON keys

This saves us from having to allocate memory in a loop. It simplifies
the code a little bit too, by removing an if statement that was far away
from the code that generated the requirement to call it.

allow us to put the channel into the message

clean up link and order dependencies

Case

NO is 0. YES is 1.

return good value

preliminary code for new / close channel

return which channel needs servicing

names are better than magic numbers

errno is already defined, and CANNOT be re-defined

don't put business logic into an assertion

tweak signal logic

travis fix

try to build rlm_lua, too

Merge pull request #1842 from herwinw/gendh_dhparam

Use 'openssl dhparam' to create DH-params

const-ify it

don't need this

Use 'openssl dhparam' to create DH-params

'openssl gendh' is removed in newer version of OpenSSL, and using
dhparam appears to work since at least release 0.9.8

Link libfreeradius-util to dependencies

Else it doesn't link to kevent

Fix doxygen and argument order for module_find_with_method

Split out virtual_server functions from modules.c

Constify more inst and ctx pointers

Add thread pointer for thread specific instances

doxygen for kev

comments

Merge pull request #1841 from herwinw/cppcheck

Fixed some cppcheck warnings

fr_redis_cluster_node_addr_by_role returns an ssize_t

Removed impossible to reach else statements

We had "if (<int> < 0)" else "else if (<int> >= 0)", which cover all
possibilities