August 2005 - Freeradius-Devel - lists.freeradius.org

Automatic report from sources (radiusd) between 26.08.2005 - 27.08.2005 GMT
by Automatic cvs log generator 03 Sep '05

03 Sep '05

CVS log entries from 26.08.2005 (Fri) 08:00:01 - 27.08.2005 (Sat) 08:00:01 GMT ===================================================== Summary by authors ===================================================== Author: aland File: radiusd/src/modules/rlm_ldap/rlm_ldap.c; Revisions: 1.154 File: radiusd/src/modules/rlm_sql/drivers/rlm_sql_unixodbc/sql_unixodbc.c; Revisions: 1.14, 1.11.2.1 File: radiusd/src/lib/radius.c; Revisions: 1.157 File: radiusd/src/main/session.c; Revisions: 1.28 File: radiusd/src/main/exec.c; Revisions: 1.42.2.1 File: radiusd/src/modules/rlm_exec/exec.c; Revisions: 1.3, 1.2 Author: nbk File: radiusd/src/lib/token.c; Revisions: 1.18, 1.17.2.1 ===================================================== Combined list of identical log entries ===================================================== Description: Use per-system max fd to close, rather than hard-coded number Modified files: File: radiusd/src/main/session.c; Revision: 1.28; Date: 2005/08/26 22:02:29; Author: aland; Lines: (+10 -2) File: radiusd/src/modules/rlm_exec/exec.c; Revision: 1.3; Date: 2005/08/26 22:02:29; Author: aland; Lines: (+15 -3) ===================================================== Log entries ===================================================== Description: Added comments Modified files: File: radiusd/src/lib/radius.c; Revision: 1.157; Date: 2005/08/26 18:12:19; Author: aland; Lines: (+14 -2) ------------------------------- Description: Fix an off-by-one error in the function getthing(). Modified files: File: radiusd/src/lib/token.c; Revision: 1.18; Date: 2005/08/26 12:53:55; Author: nbk; Lines: (+3 -3) ------------------------------- Description: Pull from CVS head: Fix an off-by-one error in the function getthing(). Modified files: File: radiusd/src/lib/token.c; Revision: 1.17.2.1; Date: 2005/08/26 12:54:00; Author: nbk; Lines: (+3 -3) ------------------------------- Description: Don't add too many attributes. Modified files: File: radiusd/src/main/exec.c; Revision: 1.42.2.1; Date: 2005/08/26 19:17:45; Author: aland; Lines: (+7 -2) ------------------------------- Description: Don't add too many attributes Modified files: File: radiusd/src/modules/rlm_exec/exec.c; Revision: 1.2; Date: 2005/08/26 19:18:48; Author: aland; Lines: (+7 -2) ------------------------------- Description: Escape more characters Modified files: File: radiusd/src/modules/rlm_ldap/rlm_ldap.c; Revision: 1.154; Date: 2005/08/26 22:01:40; Author: aland; Lines: (+10 -7) ------------------------------- Description: Account for space AND trailing NULL Modified files: File: radiusd/src/modules/rlm_sql/drivers/rlm_sql_unixodbc/sql_unixodbc.c; Revision: 1.14; Date: 2005/08/26 18:02:12; Author: aland; Lines: (+1 -1) ------------------------------- Description: Pull fix from the head Modified files: File: radiusd/src/modules/rlm_sql/drivers/rlm_sql_unixodbc/sql_unixodbc.c; Revision: 1.11.2.1; Date: 2005/08/26 18:02:47; Author: aland; Lines: (+1 -1) ===================================================== Summary of modified files ===================================================== File: radiusd/src/lib/radius.c Revisions: 1.157 Authors: aland (+14 -2) ------------------------------- File: radiusd/src/lib/token.c Revisions: 1.18, 1.17.2.1 Authors: nbk (+3 -3), nbk (+3 -3) ------------------------------- File: radiusd/src/main/exec.c Revisions: 1.42.2.1 Authors: aland (+7 -2) ------------------------------- File: radiusd/src/main/session.c Revisions: 1.28 Authors: aland (+10 -2) ------------------------------- File: radiusd/src/modules/rlm_exec/exec.c Revisions: 1.3, 1.2 Authors: aland (+15 -3), aland (+7 -2) ------------------------------- File: radiusd/src/modules/rlm_ldap/rlm_ldap.c Revisions: 1.154 Authors: aland (+10 -7) ------------------------------- File: radiusd/src/modules/rlm_sql/drivers/rlm_sql_unixodbc/sql_unixodbc.c Revisions: 1.14, 1.11.2.1 Authors: aland (+1 -1), aland (+1 -1) -- Automatic cron job from /web/pages/us.freeradius.org/bin/new_makelog.pl

4 3

Release 1.0.5?
by Alan DeKok 03 Sep '05

03 Sep '05

It has a few fixes that are very nice, like MAC OS X support. I'll see if I can fix the thread pool issue soon. Other than that, any reason to hold off on releasing 1.0.5? Alan DeKok.

5 18

Re: [Bug 269] Many compiler warnings with gcc 4.0
by Alan DeKok 01 Sep '05

01 Sep '05

bugzilla-daemon(a)dump.segv.org wrote: > ------- Additional Comments From nbk(a)sitadelle.com 2005-08-24 09:58 ------- > Many thanks for second patch! I've added it to the CVS branch 1.0, > so your fixes will be in release 1.0.5. > > There're still problems of incompatible pointer types, but I need > Alan's opinion about bringing so many casts in the > source. (personally I don't like the idea) Neither do I. They make the code harder to read & maintain. Alan DeKok.

4 14

Fw: question about realms file format
by Thanh Lim 31 Aug '05

31 Aug '05

Oops. Sorry, sent to the wrong list. -thanh ----- Original Message ----- From: "Thanh Lim" <limt(a)corp.earthlink.net> To: <freeradius-devel-request(a)lists.freeradius.org> Sent: Tuesday, August 30, 2005 2:46 PM Subject: question about realms file format > How much longer will we need to support the old realms file format? I'm > looking into updating all code dealing with the REALM structure, and > improving performance for lookups, etc. and am unsure if I should be > updating that code as well. > > Also, I notice within the REALM structure, every single time, a linked > list iteration occurs, many times, the active and account active flag is > checked, and set, even if it's not the ip address and port that we are > looking for. Why was it done this way, and shouldn't we just update the > active flag only when we find the right one(s)? > > Thanks, > -thanh >

2 1

how to change free radius attributes like (type, value )pair dynamically
by 6ackpace 31 Aug '05

31 Aug '05

hi group, i wannt to change the replys from freeradius to the switch.can anyone help me? i have tried rlm_files module and used pair_add function to change responses but failed. please guide me thank you,

1 0

Automatic report from sources (radiusd) between 30.08.2005 - 31.08.2005 GMT
by Automatic cvs log generator 31 Aug '05

31 Aug '05

CVS log entries from 30.08.2005 (Tue) 08:00:01 - 31.08.2005 (Wed) 08:00:01 GMT ===================================================== Summary by authors ===================================================== Author: aland File: radiusd/src/lib/radius.c; Revisions: 1.158 File: radiusd/src/include/libradius.h; Revisions: 1.94 File: radiusd/src/lib/dict.c; Revisions: 1.70, 1.69 Author: fcusack File: radiusd/src/modules/rlm_ldap/rlm_ldap.c; Revisions: 1.156 ===================================================== Combined list of identical log entries ===================================================== Description: We don't have differences between vendorpec & vendorcode any more Modified files: File: radiusd/src/include/libradius.h; Revision: 1.94; Date: 2005/08/30 21:45:07; Author: aland; Lines: (+2 -9) File: radiusd/src/lib/radius.c; Revision: 1.158; Date: 2005/08/30 21:45:07; Author: aland; Lines: (+6 -21) ===================================================== Log entries ===================================================== Description: More sanity checks on the dictionary. VSA's can't be tagged, and only string & integer attributes can be tagged Modified files: File: radiusd/src/lib/dict.c; Revision: 1.70; Date: 2005/08/30 23:05:18; Author: aland; Lines: (+33 -2) ------------------------------- Description: Add "internal" structs for attributes, etc. Use a hash for them, which makes lookups much faster. Hmm... we should probably just go to a hash table for ATTRIBUTEs and VALUEs, as that would be even better.. Modified files: File: radiusd/src/lib/dict.c; Revision: 1.69; Date: 2005/08/30 23:03:21; Author: aland; Lines: (+332 -197) ------------------------------- Description: fix bug #278: connection status must not be tested until after acquiring mutex protecting it Modified files: File: radiusd/src/modules/rlm_ldap/rlm_ldap.c; Revision: 1.156; Date: 2005/08/30 22:27:00; Author: fcusack; Lines: (+8 -3) ===================================================== Summary of modified files ===================================================== File: radiusd/src/include/libradius.h Revisions: 1.94 Authors: aland (+2 -9) ------------------------------- File: radiusd/src/lib/dict.c Revisions: 1.70, 1.69 Authors: aland (+33 -2), aland (+332 -197) ------------------------------- File: radiusd/src/lib/radius.c Revisions: 1.158 Authors: aland (+6 -21) ------------------------------- File: radiusd/src/modules/rlm_ldap/rlm_ldap.c Revisions: 1.156 Authors: fcusack (+8 -3) -- Automatic cron job from /web/pages/us.freeradius.org/bin/new_makelog.pl

1 0

per-module per-thread constructors and destructors
by Frank Cusack 30 Aug '05

30 Aug '05

I need to add a thread local variable to rlm_otp. It's a per-thread fd which connects to another program running locally. rlm_otp makes requests of it and it returns results. The semantics of that program does not guarantee sequencing of results unless each request thread has its own connection. That is, if I have a global fd, the response received by one thread may not be to the correct request: t1 t2 get_state(foo) get_state(bar) -> write(foo) -> write(bar) -> read() -> read() -> bar -> foo The other program may return bar first, and even if it returned results in the order received, because both threads are blocked in read(), either might pick up the result for the other. An fd per thread fixes this. It also fixes the problem that writes() might interleave (atomic writes are only guaranteed in certain cases which won't be met here), but that is just icing for the cake. The problem, though, is that on thread exit no cleanup is done. I want to add two functions to threads.c: register_constructor() and register_destructor(). When a module is instantiated, it can call these to register functions to be called on thread creation and destruction. (rlm_otp will register a destructor which closes the fd.) It's fairly trivial, but it touches the "core" of freeradius, and it affects thread behavior, so I wanted to be sure there were no objections before adding it. -frank

2 8

rlm_sql post-auth variable bug?
by Fernando Schapachnik 30 Aug '05

30 Aug '05

Hi, (Haven't had any reply on the users mailing list, so I'm giving it a try here, please excuse if this is OT). In my AAA chain I have a module that *sometimes* adds a custom attribute (defined in my dictionary as ipaddr) called Auth-NAS. This module is called in the authorize section. Later on, in the post-auth session I have a SQL-module that has: postauth_query = "INSERT INTO ${postauth_table} (...) \ VALUES (.... \ '%{check:Auth-NAS:-0.0.0.0}', ...)" When run, and Auth-NAS is not present, it outputs: rlm_sql (sql-xxx) in sql_postauth: query is INSERT INTO table (...) ??VALUES (..., '=01', ...); Sometimes the '=01' is replaced by '255.255.255.255'. Is this a bug in some place or am I doing something wrong? Any ideas? I'm using 1.0.4 (also seen on 1.0.2). Thanks in advance. Fernando.

2 9

Automatic report from sources (radiusd) between 29.08.2005 - 30.08.2005 GMT
by Automatic cvs log generator 30 Aug '05

30 Aug '05

CVS log entries from 29.08.2005 (Mon) 08:00:01 - 30.08.2005 (Tue) 08:00:01 GMT ===================================================== Summary by authors ===================================================== Author: aland File: radiusd/src/modules/rlm_ldap/rlm_ldap.c; Revisions: 1.122.2.7 File: radiusd/src/main/xlat.c; Revisions: 1.72.2.7 File: radiusd/src/modules/rlm_sql/drivers/rlm_sql_unixodbc/sql_unixodbc.c; Revisions: 1.15, 1.11.2.2 Author: bjordanov File: radiusd/raddb/experimental.conf; Revisions: 1.31 File: radiusd/src/modules/rlm_perl/rlm_perl.c; Revisions: 1.25 File: radiusd/src/modules/rlm_perl/example.pl; Revisions: 1.3 ===================================================== Combined list of identical log entries ===================================================== Description: Patches from bug #267 Enhancements for rlm_perl example.pl - added functions in example for pre_proxy, post_proxy and post_auth rlm_perl.c - added cleanup for pre_proxy, post_proxy and post_auth Modified files: File: radiusd/src/modules/rlm_perl/example.pl; Revision: 1.3; Date: 2005/08/29 12:44:44; Author: bjordanov; Lines: (+25 -1) File: radiusd/src/modules/rlm_perl/rlm_perl.c; Revision: 1.25; Date: 2005/08/29 12:44:44; Author: bjordanov; Lines: (+11 -8) ===================================================== Log entries ===================================================== Description: Patch from bug #267 Enhancements for rlm_perl Modified files: File: radiusd/raddb/experimental.conf; Revision: 1.31; Date: 2005/08/29 12:42:17; Author: bjordanov; Lines: (+43 -15) ------------------------------- Description: Check bounds Modified files: File: radiusd/src/main/xlat.c; Revision: 1.72.2.7; Date: 2005/08/29 16:19:41; Author: aland; Lines: (+5 -3) ------------------------------- Description: Escape more characters Modified files: File: radiusd/src/modules/rlm_ldap/rlm_ldap.c; Revision: 1.122.2.7; Date: 2005/08/29 16:09:56; Author: aland; Lines: (+10 -7) ------------------------------- Description: Don't call rad_malloc() from sql_error(), it leaks memory Modified files: File: radiusd/src/modules/rlm_sql/drivers/rlm_sql_unixodbc/sql_unixodbc.c; Revision: 1.15; Date: 2005/08/29 17:03:11; Author: aland; Lines: (+2 -2) ------------------------------- Description: Don't call rad_malloc from sql_error(), it leaks memory Modified files: File: radiusd/src/modules/rlm_sql/drivers/rlm_sql_unixodbc/sql_unixodbc.c; Revision: 1.11.2.2; Date: 2005/08/29 17:01:58; Author: aland; Lines: (+2 -2) ===================================================== Summary of modified files ===================================================== File: radiusd/raddb/experimental.conf Revisions: 1.31 Authors: bjordanov (+43 -15) ------------------------------- File: radiusd/src/main/xlat.c Revisions: 1.72.2.7 Authors: aland (+5 -3) ------------------------------- File: radiusd/src/modules/rlm_ldap/rlm_ldap.c Revisions: 1.122.2.7 Authors: aland (+10 -7) ------------------------------- File: radiusd/src/modules/rlm_perl/example.pl Revisions: 1.3 Authors: bjordanov (+25 -1) ------------------------------- File: radiusd/src/modules/rlm_perl/rlm_perl.c Revisions: 1.25 Authors: bjordanov (+11 -8) ------------------------------- File: radiusd/src/modules/rlm_sql/drivers/rlm_sql_unixodbc/sql_unixodbc.c Revisions: 1.15, 1.11.2.2 Authors: aland (+2 -2), aland (+2 -2) -- Automatic cron job from /web/pages/us.freeradius.org/bin/new_makelog.pl

1 0

Automatic report from sources (radiusd) between 28.08.2005 - 29.08.2005 GMT
by Automatic cvs log generator 29 Aug '05

29 Aug '05

CVS log entries from 28.08.2005 (Sun) 08:00:01 - 29.08.2005 (Mon) 08:00:01 GMT ===================================================== Summary by authors ===================================================== Author: nbk File: radiusd/raddb/oraclesql.conf; Revisions: 1.6 File: radiusd/raddb/sql.conf; Revisions: 1.50 File: radiusd/src/modules/rlm_attr_rewrite/rlm_attr_rewrite.c; Revisions: 1.31 File: radiusd/raddb/postgresql.conf; Revisions: 1.28 Author: pnixon File: radiusd/suse/freeradius.spec; Revisions: 1.16 ===================================================== Combined list of identical log entries ===================================================== Description: Move the "read_groups" and "read_clients" items to the top. This way, all query definitions are at the bottom, while config items are at the top of the file. (closes #276) Patch from Thor Spruyt <thor.spruyt(a)telenet.be> Modified files: File: radiusd/raddb/oraclesql.conf; Revision: 1.6; Date: 2005/08/28 12:14:15; Author: nbk; Lines: (+8 -6) File: radiusd/raddb/postgresql.conf; Revision: 1.28; Date: 2005/08/28 12:14:15; Author: nbk; Lines: (+10 -10) File: radiusd/raddb/sql.conf; Revision: 1.50; Date: 2005/08/28 12:14:15; Author: nbk; Lines: (+11 -9) ===================================================== Log entries ===================================================== Description: Fix a typo in function name. (closes #274) Patch from Thor Spruyt <thor.spruyt(a)telenet.be> Modified files: File: radiusd/src/modules/rlm_attr_rewrite/rlm_attr_rewrite.c; Revision: 1.31; Date: 2005/08/28 11:27:22; Author: nbk; Lines: (+7 -7) ------------------------------- Description: Add SUSE version tag to rpm packages Modified files: File: radiusd/suse/freeradius.spec; Revision: 1.16; Date: 2005/08/28 17:58:44; Author: pnixon; Lines: (+12 -8) ===================================================== Summary of modified files ===================================================== File: radiusd/raddb/oraclesql.conf Revisions: 1.6 Authors: nbk (+8 -6) ------------------------------- File: radiusd/raddb/postgresql.conf Revisions: 1.28 Authors: nbk (+10 -10) ------------------------------- File: radiusd/raddb/sql.conf Revisions: 1.50 Authors: nbk (+11 -9) ------------------------------- File: radiusd/src/modules/rlm_attr_rewrite/rlm_attr_rewrite.c Revisions: 1.31 Authors: nbk (+7 -7) ------------------------------- File: radiusd/suse/freeradius.spec Revisions: 1.16 Authors: pnixon (+12 -8) -- Automatic cron job from /web/pages/us.freeradius.org/bin/new_makelog.pl

1 0