Radius CVS bug

classic Classic list List threaded Threaded
2 messages Options
| Threaded
Open this post in threaded view
|

Radius CVS bug

Dinko Korunic-2
I believe that rlm_sql.c in current CVS has got a small typo with
mainconfig.client_trees. Trivial fix follows:

Index: rlm_sql.c
===================================================================
RCS file: /source/radiusd/src/modules/rlm_sql/rlm_sql.c,v
retrieving revision 1.145
diff -u -r1.145 rlm_sql.c
--- rlm_sql.c   18 Jul 2005 20:26:39 -0000      1.145
+++ rlm_sql.c   21 Jul 2005 14:44:58 -0000
@@ -381,7 +381,7 @@
                DEBUG("rlm_sql (%s): Adding client %s (%s) to clients list",
                      inst->config->xlat_name,
                      c->longname,c->shortname);
-               if (!client_add(mainconfig.client_trees, c)) {
+               if (!client_add(mainconfig.clients, c)) {
                        client_free(c);
                        return -1;
                }

Cheers,
D.

--
NAME:Dinko.kreator.Korunic      NOTE:Standard.disclaimer.applies
URL:kreator.esa.fer.hr   IRC:kre   ICQ:16965294   PGP:0xea160d0b
-
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html
| Threaded
Open this post in threaded view
|

Re: Radius CVS bug

Nicolas Baradakis
Dinko Korunic wrote:

> I believe that rlm_sql.c in current CVS has got a small typo with
> mainconfig.client_trees. Trivial fix follows:

I've seen the error, too. It was corrected a few hours before your
post. Just "cvs update" or download a new nightly CVS snapshot
tomorrow.

The code compiles correctly but it's still untested, though. You're
welcome to report problems if there's anything wrong.

--
Nicolas Baradakis
-
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html