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

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

Realm ?????
by S.I 25 Aug '05

25 Aug '05

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. Can you make a patch for custom suffix waitng for your solution. Best Regards S.I

2 1

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

25 Aug '05

CVS log entries from 24.08.2005 (Wed) 08:00:01 - 25.08.2005 (Thu) 08:00:00 GMT ===================================================== Summary by authors ===================================================== Author: aland File: radiusd/src/main/auth.c; Revisions: 1.150, 1.149, 1.136.2.4 Author: nbk File: radiusd/src/modules/rlm_pap/rlm_pap.c; Revisions: 1.15.2.2 File: radiusd/src/modules/rlm_ldap/rlm_ldap.c; Revisions: 1.153, 1.122.2.6 File: radiusd/src/main/mainconfig.c; Revisions: 1.36.2.6, 1.36.2.5 File: radiusd/src/lib/misc.c; Revisions: 1.41.2.3 File: radiusd/src/modules/rlm_mschap/rlm_mschap.c; Revisions: 1.59.2.2 File: radiusd/src/include/libradius.h; Revisions: 1.76.2.1 File: radiusd/raddb/sql.conf; Revisions: 1.49, 1.48, 1.41.2.2 File: radiusd/src/modules/rlm_unix/cache.c; Revisions: 1.25.2.1 File: radiusd/src/modules/rlm_expr/rlm_expr.c; Revisions: 1.7.2.1 File: radiusd/src/modules/rlm_eap/types/rlm_eap_md5/eap_md5.c; Revisions: 1.11, 1.10.2.1 File: radiusd/src/modules/rlm_eap/libeap/fips186prf.c; Revisions: 1.3.2.1 ===================================================== Combined list of identical log entries ===================================================== Description: Fix compilation warnings with gcc 4.0 Patch from Steven Simon <simon.s(a)apple.com> Modified files: File: radiusd/src/include/libradius.h; Revision: 1.76.2.1; Date: 2005/08/24 14:37:51; Author: nbk; Lines: (+2 -2) File: radiusd/src/lib/misc.c; Revision: 1.41.2.3; Date: 2005/08/24 14:37:51; Author: nbk; Lines: (+3 -3) File: radiusd/src/main/mainconfig.c; Revision: 1.36.2.5; Date: 2005/08/24 14:37:51; Author: nbk; Lines: (+7 -7) File: radiusd/src/modules/rlm_eap/libeap/fips186prf.c; Revision: 1.3.2.1; Date: 2005/08/24 14:37:52; Author: nbk; Lines: (+9 -2) File: radiusd/src/modules/rlm_expr/rlm_expr.c; Revision: 1.7.2.1; Date: 2005/08/24 14:37:52; Author: nbk; Lines: (+4 -3) File: radiusd/src/modules/rlm_mschap/rlm_mschap.c; Revision: 1.59.2.2; Date: 2005/08/24 14:37:52; Author: nbk; Lines: (+13 -11) File: radiusd/src/modules/rlm_pap/rlm_pap.c; Revision: 1.15.2.2; Date: 2005/08/24 14:37:52; Author: nbk; Lines: (+3 -3) File: radiusd/src/modules/rlm_unix/cache.c; Revision: 1.25.2.1; Date: 2005/08/24 14:37:53; Author: nbk; Lines: (+4 -4) ===================================================== Log entries ===================================================== Description: Use %S instead of NOW() because the value is wrong when post-processing the query. (radsqlrelay) Modified files: File: radiusd/raddb/sql.conf; Revision: 1.49; Date: 2005/08/24 16:50:31; Author: nbk; Lines: (+2 -2) ------------------------------- Description: Use the same case-insensitives queries as postgresql.conf Modified files: File: radiusd/raddb/sql.conf; Revision: 1.48; Date: 2005/08/24 16:34:37; Author: nbk; Lines: (+65 -44) ------------------------------- Description: Pull from CVS head: nas_table definition case-insensitives queries %S instead of NOW() Modified files: File: radiusd/raddb/sql.conf; Revision: 1.41.2.2; Date: 2005/08/24 17:02:04; Author: nbk; Lines: (+54 -42) ------------------------------- Description: Dang typos Modified files: File: radiusd/src/main/auth.c; Revision: 1.150; Date: 2005/08/25 00:10:46; Author: aland; Lines: (+6 -6) ------------------------------- Description: Don't de-reference a NULL pointer if the auth-type is unknown Modified files: File: radiusd/src/main/auth.c; Revision: 1.149; Date: 2005/08/25 00:08:09; Author: aland; Lines: (+11 -4) ------------------------------- Description: Pull fix from the head Modified files: File: radiusd/src/main/auth.c; Revision: 1.136.2.4; Date: 2005/08/25 00:10:28; Author: aland; Lines: (+11 -4) ------------------------------- Description: Still warnings, fixing them. Modified files: File: radiusd/src/main/mainconfig.c; Revision: 1.36.2.6; Date: 2005/08/24 15:38:26; Author: nbk; Lines: (+7 -7) ------------------------------- Description: Delete the assertion (reply->value_size < 256) because an unsigned char is always < 256. Modified files: File: radiusd/src/modules/rlm_eap/types/rlm_eap_md5/eap_md5.c; Revision: 1.11; Date: 2005/08/24 14:23:05; Author: nbk; Lines: (+1 -2) ------------------------------- Description: Pull from CVS head: Delete the assertion (reply->value_size < 256) because an unsigned char is always < 256. Modified files: File: radiusd/src/modules/rlm_eap/types/rlm_eap_md5/eap_md5.c; Revision: 1.10.2.1; Date: 2005/08/24 14:23:11; Author: nbk; Lines: (+1 -2) ------------------------------- Description: Fix compilation warnings in code from Novell. Modified files: File: radiusd/src/modules/rlm_ldap/rlm_ldap.c; Revision: 1.153; Date: 2005/08/24 09:21:58; Author: nbk; Lines: (+5 -5) ------------------------------- Description: Pull from CVS head: Fix compilation warnings. Modified files: File: radiusd/src/modules/rlm_ldap/rlm_ldap.c; Revision: 1.122.2.6; Date: 2005/08/24 10:12:19; Author: nbk; Lines: (+9 -9) ===================================================== Summary of modified files ===================================================== File: radiusd/raddb/sql.conf Revisions: 1.49, 1.48, 1.41.2.2 Authors: nbk (+2 -2), nbk (+65 -44), nbk (+54 -42) ------------------------------- File: radiusd/src/include/libradius.h Revisions: 1.76.2.1 Authors: nbk (+2 -2) ------------------------------- File: radiusd/src/lib/misc.c Revisions: 1.41.2.3 Authors: nbk (+3 -3) ------------------------------- File: radiusd/src/main/auth.c Revisions: 1.150, 1.149, 1.136.2.4 Authors: aland (+6 -6), aland (+11 -4), aland (+11 -4) ------------------------------- File: radiusd/src/main/mainconfig.c Revisions: 1.36.2.6, 1.36.2.5 Authors: nbk (+7 -7), nbk (+7 -7) ------------------------------- File: radiusd/src/modules/rlm_eap/libeap/fips186prf.c Revisions: 1.3.2.1 Authors: nbk (+9 -2) ------------------------------- File: radiusd/src/modules/rlm_eap/types/rlm_eap_md5/eap_md5.c Revisions: 1.11, 1.10.2.1 Authors: nbk (+1 -2), nbk (+1 -2) ------------------------------- File: radiusd/src/modules/rlm_expr/rlm_expr.c Revisions: 1.7.2.1 Authors: nbk (+4 -3) ------------------------------- File: radiusd/src/modules/rlm_ldap/rlm_ldap.c Revisions: 1.153, 1.122.2.6 Authors: nbk (+5 -5), nbk (+9 -9) ------------------------------- File: radiusd/src/modules/rlm_mschap/rlm_mschap.c Revisions: 1.59.2.2 Authors: nbk (+13 -11) ------------------------------- File: radiusd/src/modules/rlm_pap/rlm_pap.c Revisions: 1.15.2.2 Authors: nbk (+3 -3) ------------------------------- File: radiusd/src/modules/rlm_unix/cache.c Revisions: 1.25.2.1 Authors: nbk (+4 -4) -- Automatic cron job from /web/pages/us.freeradius.org/bin/new_makelog.pl

1 0

memory leaks and other problems with rlm_ippool
by Peter Nixon 24 Aug '05

24 Aug '05

Hi Guys I am consistently finding that if you create more than one IP Pool bigger than a class B network that FreeRADIUS fails to start with an error saying that it cannot fork. It can however be run in debug mode with this config. I have tested this on a number of different servers with a number of different versions of FreeRADIUS and on SUSE Linux Enterprise 9, SUSE Pro 9.2, SUSE Pro 9.3 and some random version of Turbo Linux that a customer had. Also, one customer who had only a single class B IP Pool with about 10,000 clients connecting and disconnecting on a regular basis (Around 300 Auth requests and associated Accounting packets per minute between 8am and midnight) is seeing their FreeRADIUS grow over the course of a couple of days until the Linux OOM killer kills it. (Making their customers very unhappy until they restart their server) Their server is fully patched SLES9 running FreeRADIUS 1.0.4 Does anyone have any ideas? Surely someone else must have seen these problems. I can't be the only person running multiple, large IP Pools... -- Peter Nixon http://www.peternixon.net/ PGP Key: http://www.peternixon.net/public.asc

4 6