[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freeradius-devel
Subject:    Re: sql_check_multi and sql_check_ts
From:       "Alan DeKok" <aland () ox ! org>
Date:       2002-08-26 21:51:39
[Download RAW message or body]

Kostas Kalevras <kkalev@noc.ntua.gr> wrote:
> /src/cvs/radiusd > egrep -l --recursive '(sql_check_multi|sql_check_ts)' *
> src/modules/rlm_sql/rlm_sql.h
> src/modules/rlm_sql/sql.c
>
> Is there a reason for these functions to even exist?
> rlm_sql_checksimul should be able to handle everything.

  I agree.  The SQL code should call rad_check_ts(), which Does the
Right Thing, including any calls to the SQL checksimul function.

  Calling 'checkrad' from the SQL code is *bad*.

  Alan DeKok.

- 
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/devel.html
[prev in list] [next in list] [prev in thread] [next in thread]