From e59a9fbd982a6ce9b00c45948444cc7acc2634b2 Mon Sep 17 00:00:00 2001 From: Linus Nordberg Date: Tue, 1 Nov 2016 10:25:02 +0100 Subject: Look at servers->dynamiclookuparg for deciding if a server is dynamic. The dynamiclookupcommand member of the _config_ of the server is being set to NULL when it's copied in confserver_cb(), resulting in dynamic discovery being done for realms that already have a server. Patch from Fabian Mauchle. Addresses RADSECPROXY-69. --- ChangeLog | 2 ++ radsecproxy.c | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 715d69b..0ae86bc 100644 --- a/ChangeLog +++ b/ChangeLog @@ -27,6 +27,8 @@ Changes between 1.6.8 and the master branch - Tie Access-Request log lines to response log lines (RADSECPROXY-60). - Take lock on realm refcount before updating it (RADSECPROXY-77). - Fix a couple of memory leaks and NULL ptr derefs in error cases. + - Don't forget about good dynamically discovered (TLS) connections + (RADSECPROXY-69). 2016-09-21 1.6.8 Bug fixes: diff --git a/radsecproxy.c b/radsecproxy.c index 67e8839..ef1d710 100644 --- a/radsecproxy.c +++ b/radsecproxy.c @@ -707,7 +707,7 @@ int hasdynamicserver(struct list *srvconfs) { struct list_node *entry; for (entry = list_first(srvconfs); entry; entry = list_next(entry)) - if (((struct clsrvconf *)entry->data)->dynamiclookupcommand) + if (((struct clsrvconf *)entry->data)->servers->dynamiclookuparg) return 1; return 0; } -- cgit v1.1