(RADIATOR) RHES 3 & Net::Daemon problem and LogSQL.pm fix

[Mike McCauley]

Hello Jacob,


On Wed, 12 Nov 2003 03:47 am, Jacob Leaver wrote:
> Has anyone had any success with Radiator 3.7.1 on RedHat Enterprise 3 ?  I
> can't seem to get Net::Daemon to compile and test without failing the forkm
> test.   I went ahead and installed it anyway, but radiator is failing with
> no reason given, even at trace level 9.  One can only assume that this is
> related.

I have compiled and tested Net::Daemon fine on RH ES with this version:

uname -a
Linux xxx.open.com.au 2.4.9-e.27 #1 Tue Aug 5 15:27:02 EDT 2003 i686 unknown

The forkm test ran and succeeded.

Is this the same version as your RH Enterprise 3?

If Radiator is dying without leaving a message in the Radiator log file, you 
might see an error message produced on stdout. Try running it by hand from a 
shell.

>
> On another note, I can't remember if I mentioned this before or not, but
> LogSQL seems to have some non-functional code in it: around about line 54
> of LogSQL.pm, it reads:
>     # Catch recursion
>    return if $in_log++;
>
> which I think should always evaluate as true.   It seems to work better
> reading something more like: # Catch recursion
>    return if $in_log > 1;
>     $in_log++;
> which I believe will prevent the recursion that is mentioned in the
> comment.

The intention of this code is to let the body of the function execute at most 
once, even if it recurses afterwards (ie even if the logging internals 
produce a new log message). I believe the code as delivered works correctly 
on all platforms, but if you know of a platform where that is not true (and 
the code either fails to execute at all or fails to detect recursion, then we 
would like to see the Trace 4 log file, please).

>
> Jacob Leaver
> ReachONE Internet

-- 
Mike McCauley                               mikem at open.com.au
Open System Consultants Pty. Ltd            Unix, Perl, Motif, C++, WWW
24 Bateman St Hampton, VIC 3188 Australia   http://www.open.com.au
Phone +61 3 9598-0985                       Fax   +61 3 9598-0955

Radiator: the most portable, flexible and configurable RADIUS server 
anywhere. SQL, proxy, DBM, files, LDAP, NIS+, password, NT, Emerald, 
Platypus, Freeside, TACACS+, PAM, external, Active Directory, EAP, TLS, 
TTLS, PEAP etc on Unix, Windows, MacOS etc.

===
Archive at http://www.open.com.au/archives/radiator/
Announcements on radiator-announce at open.com.au
To unsubscribe, email 'majordomo at open.com.au' with
'unsubscribe radiator' in the body of the message.