API improvements needed by freeradius

In with the scabs, out with the tr_msg union!

The tr_msg union lead to a number of security issues because the code
tended to check to see if msg->msg_struct_name was non-null.  However
it was always non-null because the pointer was shared among all the
union members.  Instead, use accessors for everything.
LP: #1333734

ABI/API break: pas in TID_RESP * to handler

Previously, we passed in TID_RESP ** to the request handler.  However
the request handlers assumed that the response was allocated.  We
don't want responses allocated in the handler, so make it a single
pointer.

note that the existing handler interface is probably inappropriate for
an event-loop-based trust router.

Track num_servers correctly

TID_RESP: array of servers rather than linked list

Provide an array of servers rather than a linked list for easier sorting.

TID_RESP is now allocated by talloc.

Make tid types opaque

It is not a failure to have no constraints at all, although no authorizations are created

tr_constraints: constraint set members can have limited types

If a constraint set member has a domain constraint but no realm
constraint treat that as a universal realm constraint (*).

However, if no constraint set member has that constraint type then
access is denied; we do not fail open.

Include authorizations view in schema

Iterators also needed for tests

don't redefine json_t

Back port jansson iterators

tids: include constraints in database

new table authorizations includes constraints for domain and realm as
well as the COI and APC used for the connection.

tr_constraint_set_get_match_strings

New function to retrieve the wild card strings that match a constraint
type for an intersected constraint set.

As a result convert TID_REQ to using talloc.

Depend on talloc project wide.

  # Please enter the commit
message for your changes. Lines starting # with '#' will be ignored,
and an empty message aborts the commit.  # On branch master # Your
branch is ahead of 'origin/master' by 3 commits.  # (use "git push" to
publish your local commits) # # Changes to be committed: # modified:
common/tr_constraint.c # modified: configure.ac # modified:
include/trust_router/tid.h # modified:
include/trust_router/tr_constraint.h # modified:
include/trust_router/tr_name.h # modified: tid/tid_req.c # modified:
tid/tidc.c # # Changes not staged for commit: # modified:
include/trust_router/tr_versioning.h # # Untracked files: # "\a" #
cscope.out # db # dest/ # foo.c # trust_router-1.0.tar.gz #

tr_dh_pub_digest

Function to compute public key digest of client.  Use to store that in
sqlite3 database.  Update schema.

copyright update

Include constraints in tid_req messages

Makefile: enable tests and -Werror

Enable t_constraint tests in make check

Also enable -Werror since we pass with that.

tr_constraint_set_intersect

New function to intersect a constraint set and return a constraint
describing the domain and realm constraints that can be met by the
set.

Include tests for this.  The particular test cases are also designed
to test merge_constraints (included in this patch) and
tr_prefix_wildcard_match.

tid_req: Store json references

Support storing references to json objects in TID requests.

Move tr_prefix_wildcard_match to tr_constraint.c

We need tr_prefix_wildcard_match for merge_constraints and for
tr_filter.c.  Export it from libtr_tid even though it's in a private
header.  It's not part of the public API but is part of the library so
tr_filter can import it.

Also, fix bug; all strings were treated as wildcards.

fix keys creation

s:trustrouter:trust_router

Fix typo

Enable unit

Include tids service unit and schema sql.

Update spec file for 1.2

Remove need for remote def of TR_FLINE that won't compile on Centos.

Update trust_router version number to 1.2

Increment TID library version number to 1.

If port passed in to tidc_open_connection() is 0, use the default port.

Debugging printfs for trust router port number.

Remove API dependency on jansson for constraints.

Allow caller to set port number for tidc_open_connection().  Install
include/trust_router/tr_constraints.h, so that freeradius will build
with updated TID code.

Change name type passed to gss_import_name().

Avoid overwriting gss error before printing.

Allow the Trust Router's TIDS port to be set in the internal config.

Don't overwrite minorStatus before printing error.

Fix bug in previous commit.

Add hostname to service name in gsscon_connect().

Add files not commited for AAA Server IP Addr to Hostname change.

Configure AAA Server hostname, instead of expecting an IP address.

Completion of constraints code, not fully tested.

Merge branch 'master' of moonshot.suchdamage.org:/srv/git/trust_router

tr_tids_gss_handler: print auth name

Print the name  we authenticated to.

Configuration code for realm and domain constraints.

asprintf not sprintf

Added hostname to configuration and example code.

Add things missing from previous commit to add realm_name to the service name.

Changes to add realm name (from config) to end of service name for passive authentication.

new version

Fix bugs found in testing new filtering code.

Clean up compiler warnings (and likely bugs).

Full support for rp_permitted filters using new filter structures, etc.

Merge branch 'master' of moonshot.suchdamage.org:/srv/git/trust_router

Configuration for full filter structures.

specfile: bump release

Update makefiles to include tid/tr_resp.c.

Merge branch 'master' of moonshot.suchdamage.org:/srv/git/trust_router

Added access functions for TID_RESP structure.

Merge branch 'master' of moonshot.suchdamage.org:/srv/git/trust_router

temporary: gsscon_passive_authenticate: acquire trustidentity creds.

As discussin in LP: #1203159, the client always uses trustidentity as
a name.  We're running into problems because the server uses
GSS_C_NO_CREDENTIAL.  That means no service name is included in RADIUS
and unless there's proxy magic, then channel bindings fails.

For now, also acquire trustidentity credentials on the server.  This
still leaves the security issue discussed by that bug, but at least
the code works.

Access funcitons for TID_REQ structure, incl TID code reorg.

specfile: sqlite-devel not sqlite3-devel

buildrequires

Don't multiply define the same type

spec file updates

fixes LP: #1236768

Add accessors for TR_MSG structure.

Updated config file with complete, consistent config for Trust Router 1.0.

Fix forward reference that causes CentOS build to fail.

Fix logic error in previous fix for APC parsing.

Don't parse APCs in IDP realm if they aren't there.

Version 1.0

Fix order of parameters in wildcard call.

Clean up wildcard match code, add debugging printf

Fix logic error in gss name comparison.

Fix gss matching code to compare to the rp realm for the request.

Fix segfault in RP client lookup code.

Fixes to make gss_name checking code compile.

Merge branch 'master' of moonshot.suchdamage.org:/srv/git/trust_router

Add code to do gss_name check in trust router.

tid: implement GSS name callback

Add code for rp client lookups to the build.

Update config code and config files to fully parse rp_client
rp_permitted filters.

tids: build after gsscon change.

This callback always succeeds and needs to be replaced with a correct callback exposed through the tids interface.

gsscon: callback for client name validation

gsscon_passive_authenticate now takes a callback for client authentication.

Merge gsscon_connect() and gsscon_active_authenticate() calls.

Declare variable so the code will build.

Check rp_realm and target_realm membership in both COI and APC

Updates to the way configuration is merged.

Do not require server list in error response.

Do not send the request on community membership error.

Set socket options before bind().

Fix segfault where request was only partially memset().

Code to check community membership on TR.

Added function to lookup a community by name.

Merge branch 'master' of moonshot.suchdamage.org:/srv/git/trust_router

Added function to do prefix wildcard comparison for filtering.

Fix coding error (segfault) in community config code.

Make functions to find RPs and IDPs externally accessible.