Hello!

Just now upgrade to 1.0.0. All works fine except check-radiusd-config
and rlm_perl.

1. check-radiusd-config  doesn't work with 1.0.0.
because -p option is deprecated.
why? IMHO, check-radiusd-config is userfull, when need to check new config
while working instance is running. So option -p have to be (IMHO).


2. There is memory leak in rlm_perl.c again.

I had memory leak before Jul,12. After installing rlm_perl
from snapshot leak leaves me.
I don't know what version is exactly.
there was line
 * Version:    $Id: rlm_perl.c,v 1.13 2004/02/26 19:04:34 aland Exp $
in source. The line is the same in 1.0.0 verion, but files looks differ.
diff -u rlm_perl.c-jul-12 rlm_perl.c-from_1.0.0
shows:


=====================================================================================
@@ -40,6 +40,10 @@
 #undef INADDR_ANY
 #endif

+#ifdef INADDR_NONE
+#undef INADDR_NONE
+#endif
+
 #include <EXTERN.h>
 #include <perl.h>
 #include <XSUB.h>
@@ -48,39 +52,6 @@

 static const char rcsid[] = "$Id: rlm_perl.c,v 1.13 2004/02/26 19:04:34 aland Exp $";

-#ifdef USE_ITHREADS
-
-/*
- * Pool of Perl's clones (genetically cloned) ;)
- *
- */
-typedef struct pool_handle {
-   struct pool_handle  *next;
-   struct pool_handle  *prev;
-   enum {busy, idle}   status;
-   unsigned int        request_count;
-   PerlInterpreter     *clone;
-   perl_mutex      lock;
-} POOL_HANDLE;
-
-typedef struct PERL_POOL {
-   POOL_HANDLE *head;
-   POOL_HANDLE *tail;
-
-   int     current_clones;
-   int     active_clones;
-   int     max_clones;
-   int     start_clones;
-   int     min_spare_clones;
-   int     max_spare_clones;
-   int     max_request_per_clone;
-   int     cleanup_delay;
-   enum {yes,no}   detach;
-   perl_mutex  mutex;
-   time_t      time_when_last_added;
-} PERL_POOL;
-
-#endif
<skip>
=====================================================================================

When i run radiusd, it consumes about 60MB of RAM.
after 15 minutes - about 200MB. And it grows.
previous version consume 55-70MB even after few days.
So i think, there is memory leak again.

Sincerely, Mike.





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

Reply via email to