summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLinus Nordberg <linus@nordu.net>2016-11-01 10:25:02 +0100
committerFabian Mauchle <fabian.mauchle@switch.ch>2017-08-06 20:57:22 +0200
commite59a9fbd982a6ce9b00c45948444cc7acc2634b2 (patch)
tree76d77aea7d57e46560a538d66e02b23d47601f06
parentdbbf3d23b13fc37ca79e92ff310b8bb4b0c7e931 (diff)
Look at servers->dynamiclookuparg for deciding if a server is dynamic.radsecproxy-69
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.
-rw-r--r--ChangeLog2
-rw-r--r--radsecproxy.c2
2 files changed, 3 insertions, 1 deletions
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;
}