- Freeradius-Devel - lists.freeradius.org

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

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

28 Aug '05

CVS log entries from 27.08.2005 (Sat) 08:00:01 - 28.08.2005 (Sun) 08:00:01 GMT ===================================================== Summary by authors ===================================================== Author: nbk File: radiusd/src/modules/rlm_ldap/rlm_ldap.c; Revisions: 1.155 File: radiusd/src/modules/rlm_sql/sql.c; Revisions: 1.85 ===================================================== Log entries ===================================================== Description: Fix ldap_pairget() to copy characters after any space or equal found in the middle of the string from LDAP. (closes: #261) As a bonus, add support for the back-quoted strings, too. (for example Reply-Message := `Hello %{User-Name}`) Modified files: File: radiusd/src/modules/rlm_ldap/rlm_ldap.c; Revision: 1.155; Date: 2005/08/27 16:25:07; Author: nbk; Lines: (+81 -29) ------------------------------- Description: Fix typo line 109. (closes: #272) Modified files: File: radiusd/src/modules/rlm_sql/sql.c; Revision: 1.85; Date: 2005/08/27 12:37:13; Author: nbk; Lines: (+2 -2) ===================================================== Summary of modified files ===================================================== File: radiusd/src/modules/rlm_ldap/rlm_ldap.c Revisions: 1.155 Authors: nbk (+81 -29) ------------------------------- File: radiusd/src/modules/rlm_sql/sql.c Revisions: 1.85 Authors: nbk (+2 -2) -- Automatic cron job from /web/pages/us.freeradius.org/bin/new_makelog.pl

1 0

Automatic report from sources (radiusd) between 25.08.2005 - 26.08.2005 GMT
by Automatic cvs log generator 26 Aug '05

26 Aug '05

CVS log entries from 25.08.2005 (Thu) 08:00:00 - 26.08.2005 (Fri) 08:00:01 GMT ===================================================== Summary by authors ===================================================== Author: aland File: radiusd/src/modules/rlm_sql/sql.c; Revisions: 1.84, 1.79.2.3 File: radiusd/src/lib/rbtree.c; Revisions: 1.16 Author: fcusack File: radiusd/src/modules/rlm_otp/otp_rlm.c; Revisions: 1.5, 1.4 File: radiusd/src/modules/rlm_otp/otp_util.c; Revisions: 1.4, 1.3 Author: nbk File: radiusd/doc/ChangeLog; Revisions: 1.57.2.26 ===================================================== Combined list of identical log entries ===================================================== Description: typos Modified files: File: radiusd/src/modules/rlm_otp/otp_rlm.c; Revision: 1.5; Date: 2005/08/26 04:44:48; Author: fcusack; Lines: (+5 -4) File: radiusd/src/modules/rlm_otp/otp_util.c; Revision: 1.4; Date: 2005/08/26 04:44:48; Author: fcusack; Lines: (+3 -3) ------------------------------- Description: accomodate FR-1.0.5 removal of module init() and destroy() methods: - otp_rlm.c: remove global rnd_fd fd handle to /dev/urandom - otp_util.c: call lrad_rand() #if FREERADIUS Modified files: File: radiusd/src/modules/rlm_otp/otp_rlm.c; Revision: 1.4; Date: 2005/08/26 04:32:39; Author: fcusack; Lines: (+32 -40) File: radiusd/src/modules/rlm_otp/otp_util.c; Revision: 1.3; Date: 2005/08/26 04:32:39; Author: fcusack; Lines: (+12 -3) ===================================================== Log entries ===================================================== Description: Adding everything I found with the command: $ cvs log -N -S -rrelease_1_0 -d '20050618<20050825' Modified files: File: radiusd/doc/ChangeLog; Revision: 1.57.2.26; Date: 2005/08/25 16:22:04; Author: nbk; Lines: (+37 -1) ------------------------------- Description: Now that NIL is 'const', do more checks on pointers before setting ptr->Color == Black. This was causing core dumps occasionally. Modified files: File: radiusd/src/lib/rbtree.c; Revision: 1.16; Date: 2005/08/25 21:11:26; Author: aland; Lines: (+18 -14) ------------------------------- Description: Zero allocated memory. This should fix bug #271 Modified files: File: radiusd/src/modules/rlm_sql/sql.c; Revision: 1.84; Date: 2005/08/26 00:37:32; Author: aland; Lines: (+3 -2) ------------------------------- Description: Pull fix from the head. This closes bug #271 Modified files: File: radiusd/src/modules/rlm_sql/sql.c; Revision: 1.79.2.3; Date: 2005/08/26 00:37:47; Author: aland; Lines: (+3 -2) ===================================================== Summary of modified files ===================================================== File: radiusd/doc/ChangeLog Revisions: 1.57.2.26 Authors: nbk (+37 -1) ------------------------------- File: radiusd/src/lib/rbtree.c Revisions: 1.16 Authors: aland (+18 -14) ------------------------------- File: radiusd/src/modules/rlm_otp/otp_rlm.c Revisions: 1.5, 1.4 Authors: fcusack (+5 -4), fcusack (+32 -40) ------------------------------- File: radiusd/src/modules/rlm_otp/otp_util.c Revisions: 1.4, 1.3 Authors: fcusack (+3 -3), fcusack (+12 -3) ------------------------------- File: radiusd/src/modules/rlm_sql/sql.c Revisions: 1.84, 1.79.2.3 Authors: aland (+3 -2), aland (+3 -2) -- Automatic cron job from /web/pages/us.freeradius.org/bin/new_makelog.pl

4 4

i need init and destroy
by Frank Cusack 26 Aug '05

26 Aug '05

Looks like the init and destroy methods went away 5 days ago. My module (rlm_otp) needs them. - How was this tested? struct module_t rlm_otp still contains these methods. How can this even compile? - What are the chances of getting these back? I can do something like pthread_once(otp_init()) and add a refcount to cleanup in detach, but really, init and destroy are superior. -frank

2 7

log.c patch
by Valts Mazurs 25 Aug '05

25 Aug '05

Hello, Freeradius developers! Since I don't see any reason why we should cut out all the non-latin characters from freeradius logs I made a tiny patch for log.c. Some time ago me and Alan had a discussion about this topic. We came to conclusion that newlines (\n) and carriage returns (\r) should been cut out (due to security reasons) but not all the other non-latin characters. I care about tabs because I use them to format debugging output of my perl scripts (I use rlm_perl). I use also newlines but I have made a simple workaround in perl scripts. Making workaround for tabs would be too complicated. You can find patch (freeradius-1.0.4-log.c-20050803.patch) in the attachment of this e-mail. Best regards, Valts.

2 1

Rlm digest using non cleartext password
by Pezdetek 25 Aug '05

25 Aug '05

Hello! Has anyone implemented non clear text password for digest method? I would be nice if I could use A1 result and not cleartext password in my database. Br

2 1

Realm Help
by S.I 25 Aug '05

25 Aug '05

I debug radiusd with gdb but nothing happens. The gdb show me no debug symbols i compile radiusd with -g and -ggdb and i switch --enable-developer from configure script ,..... --------------------------------------------------------------------------------- S.I wrote: > Hi, I have a problem with freeradius-1.0.4 on FreeBSD 5.4-STABLE > > when realm user (*@blia-bluia-blia) or !@#$%^&*()_+@domain.com is login in my NAS . kernel show error: > > radiusd in free(): warning: junk pointer, too high to make sense. >Warning is not an error. > Can you make a patch for custom suffix > > waitng for your solution. > ---------- Valts Mazurs <valts(a)datatechlabs.com> wrote: >Debug radiusd, figure out what's wrong, make patch, send it to list (I'm >sure that Alan could explain better how to do it). That's the solution. >If You want to do nothing and get Your solution I suppose You should pay >someone to do the job. > >Regards, >Valts. ---------------

1 0

rlm_perl minor bug
by Alexei Chetroi 25 Aug '05

25 Aug '05

Hi, It seems to me there's a minor bug in "POOL_HANDLE *pool_pop()" function of rlm_perl module. Around line #429 perl_pool.current_clones gets incremented, which is incorect, since it already has been incremented by pool_grow() function. The rlm_perl.c file is: $Id: rlm_perl.c,v 1.13 2004/02/26 19:04:34 aland Exp $ See attached file for proposed fix. -- Best wishes Smile... Tomorrow will be worse. (c) Murphy's Law

1 0