Not needed

More conversions

Missing uint32_t?

Talloc type safety for rbtree

Check the type of talloced heap elements

A heap size of INT32_MAX elements is OK

More type fixes

Check for overflow when expanding

Provide errors
Check for realloc failure
More type fixes
Formatting

allow moving of only one packet

look up address, not AF.  And print descriptive message

Change how fr_heap_create is called to allow talloc type validation

make heap_id be int32_t

clean up dynamic client expiry

- we now have idle_timeout, not lifetime
  - this is because dynamic clients can't be connected (for now)
- simplify and unify the "clean up packet" code
- don't have an "expired" flag in the socket.
- don't have an "expired" flag in the client.
- track client->outstanding only for dynamic clients
- decrement client->outstanding only when the packet is cleaned up

Add heap_id field to fr_pool_connection

Take offset of correct structure for connection pool last_released/reserved_cmp

Make sure the field storing the heap_id agrees with the rest of the heap code

Deal with error writing the delimiter

stop gcc 7 complaining

c_rehash the correct dir, and do the same for ecc certs

Use GCC7

document --allow-mschapv2

expand the *unescaped* string, not the raw `...` string.

Manual port of commit 5f8d05da

Ensure -Wimplicit-fallthrough is enabled

Formatting

Merge pull request #2200 from nchaigne/4.0.x-201803-dhcp-fixes

A few fixes in fr_dhcpv4_packet_decode

A few fixes in fr_dhcpv4_packet_decode

1) Options 26 and 57 are short => use vp_uint16

2) Field "opcode" is byte => use vp_uint8

3) Value 3 does not exist for field "Opcode". Client request is 1.
Option 63 should have been 60 (Vendor Class Identifier).
That said, I don't think this "MSFT 98" code ever worked - unless MS does some really weird stuff with DHCP. So maybe it's worth considering removing this altogether ? I'll let you decide. :)

add missing "break"

add missing "break"

add missing "breaks"

Add the client's TLS alert number as an attribute

Fail if we can't parse the virtual servers

New line

Set string encoding in ocsp.cnf too

Fix leak in unit_test_attribute

Don't strip the first letter from the cipher description

Correctly nul-terminated empty strings have length 1

This would trip up regex compares where the subcapture didn't
match anything, so had length 0. Then trying to assign it to an
attribute the RHS was FR_TYPE_INVALID as fr_value_box_strsteal
exited early and never set it to FR_TYPE_STRING.

set string_mask to utf8.

OSX uses "nombstr", which is non-UTF8.  That's not what we want.

generate ecc and rsa.  Update Makefile to match.

this isn't needed

typo

Print index starting at zero

Display cipher list and priority at dbg lvl 3

It's nice to know what cipher suites are actually in use

Check that there are some virtual servers defined

Don't allocate defaults with name2 if its the magic wildcard value

make certs/{rsa,ecc}/ directories, too.

Both for raddb and src/tests

clean ups

.PHONY needs a target.  Intemermediate targets should be
directories, and not phony targets.

tv_usec / 1000 will always fit within "int"

iline 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.

Manual port of commit bee85afcc61

remove unnecessary variable

Create ECC version of all certificates

Revert "Fail if we leak memory (maybe)" - Can't do that until libkqueue is fixed

This reverts commit c83748c58d77d41748e7407fd73187adfc64aeb0.

Fail if we leak memory (maybe)

Stop the leak checker complaining

Some editors trim trailing whitespace...

Regression tests for list existence

Fix memory leaks in xlat fmt parsing and map processor registration

Fix leak in module_find_with_method

Fix some debug messages in fr_worker_get_request

*actually* deregister all the xlat functions when the tree is freed

One set of LSAN_OPTIONS...

Bad tab

Try and get more malloc ctx info

More address sanitizer flags

reopen if we can't stat it

remove junk files

When TMPL_TYPE_LIST, vpt->tmpl_da is NULL if the list is empty

Microseconds part can be 0 to 999999 so 32bits is fine

s/FR_TYPE_INT8/FR_TYPE_UINT8/g

Print system error on localtime_r/gtime_failure

Use string not array so we don't expand random junk

Tidy code into a slightly neater order

Being consistent is nice whether numbers or strings

If values need to be zero padded they should be fed into the padding expansions

It's not ok to mess with the underlying data type just to get a particular presentation format.

The whole point of reworking the xlat code to take discreet boxed values is that upstream consumers of the output can use real types to make proper casting and comparison decisions.

By forcing these values to strings there's a high chance that they'll be fed into a lexicographic comparison function instead of the server just using integer comparisons (they would be now anyway, but we're moving towards something better).

give rlm_date ability to return request/now time in different formats

rlm_date: separate logic from encoding/conversion

and get rid of all the gotos in the process

add utc option to rlm_date

Tidy time expansions

Make it easier to produce dates in particular formats by always
using fixed length strings; add ISO format; fix milliseconds.

What was %T is now %S.%M (or %S.000000 to be exact...)

Merge pull request #2198 from nchaigne/4.0.x-201803-dhcp-encode

Do not encode htype and hlen by default for DHCPv4

Do not encode htype and hlen by default for DHCPv4

DHCP fields "htype" and "hlen" should not be encoded to 1 and 6 (respectively) by default.
They don't need to, because they are automatically set if there is an Ethernet address in field "chaddr".

For a Lease Query request, as described in RFC 4388, it is possible to query: 1) by IP address, 2) by MAC address, 3) by Client-identifier option.
In cases 1) and 3) "The values of htype, hlen, and chaddr MUST be set to zero.".

So I think that if no "chaddr" is provided, it's better to leave "htype" and "hlen" to 0.

This proposed pull request implements this change.

Remove '.' from dictionary names, in preparation for marking it as an disallowed character.

Remove invalid assert

Typo

don't stat meaningless data off the end of the list

Both referencer and referencee must be TLVs

Doxygen

Add missing INTERNAL_IF_NULL

Doxygen

PENs are explicitly 32bit

s/vendorpec/pen/g

It's Private Enterprise Number (now?)

Reorganise dict.c

clang scan

Reorganise attribute adding/validation functions and add support for internal dictionary references.

References allow one attribute to internally reference another TLV or a dictionary root.

This allows us to create "group" attributes which allow any attribute to be encoded within the group and foreign protocol attributes which allow RADIUS to be encoded within DHCP (for example).

Attribute references are also needed to mix protocols in the same virtual server, as the template validation functions can retrieve the foreign dictionary context from the talloc hierarchy.

Inline some string format functions

Doxygen

Re-order dictionary functions

Strip global prefix from internal dict.c functions

fr_dict_print_attr_oid is a public function

Remove duplicate declaration

Add protocol dictionary load functions

Fix scan warnings