OK, I'll buy that. :) The problem is when it calls rad_check_ts, _that_ function seems to be checking a radutmp file - whether or not I'm using radutmp or not for session checking - and several variables come back undefined, and hence checkrad still doesn't get fired up. So between the two, this seemed a more expedient solution to my particular problem (I'm only using sql, not radutmp). I'll dig over in rad_check_ts and go from there.
Thanks for the help, at least eliminating sql_check_ts from this makes the troubleshooting a little more obvious. Chris Kalin ----- Original Message ----- From: "Alan DeKok" <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Monday, August 26, 2002 4:50 PM Subject: Re: rlm_sql.c and sql.c fixes for Simultaneous-Use > "Chris A. Kalin" <[EMAIL PROTECTED]> wrote: > > Should be simple enough to understand without much comment. It's against > > last night's CVS snapshot. > > OK... but you changed it so that it doesn't call rad_check_ts() any > more, and instead calls sql_check_ts(). This is wrong. > > The issue is that SQL may not know that the user is logged in twice, > but another module may know. So hard-coding the SQL module to only > check the SQL functions, is wrong. > > The sql_check_ts() function should be deleted. See: > > http://lists.cistron.nl/pipermail/freeradius-devel/2002-August/003376.html > > Coincidentally enough, also from today. > > Alan DeKok. > > - > List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html > > - List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html
