Skip to content

Commit

Permalink
Look at servers->dynamiclookuparg for deciding if a server is dynamic.
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
Linus Nordberg authored and Fabian Mauchle committed Aug 6, 2017
1 parent dbbf3d2 commit e59a9fb
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions ChangeLog
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down
2 changes: 1 addition & 1 deletion radsecproxy.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down

0 comments on commit e59a9fb

Please sign in to comment.