Recent cleanups

Alan DeKok aland at deployingradius.com
Mon Feb 12 17:06:54 CET 2007


Joe Maimon wrote:
> This sounds too good to pass up. Sigh. Gotta rebase and rebuld off cvs 
> head again now.....

  branch_1_1 has the same fixes, IF you wait for all of the requests to
be cleaned up.  I haven't yet gotten around to having it clean up the
request list on exit.

   Again:

$ radiusd -m -Xd ./raddb

  Means that you can run the server, and once it's been idle for 5
seconds, hit ctrl-C.  Anything you see using "valgrind --tool=memcheck"
other than ltdl.c is a memory leak that needs to be fixed.

> root      5122  5122  0.0    8  1.2 116796 13340 ?        Ssl  Feb09 
> 1:32 /usr/sbin/freeradius

  Maybe you have a busy server?  Without more information, it's hard to
tell.

  Most of the changes I just added are for proper cleanup at exit.  So
they don't affect the normal running of the server.  They do, however,
make it a lot easier to read the output.  There's less noise, which
means the real leaks are obvious.

  Alan DeKok.
--
  http://deployingradius.com       - The web site of the book
  http://deployingradius.com/blog/ - The blog



More information about the Freeradius-Devel mailing list