O/H Alan DeKok έγραψε:
> Claudiu Filip wrote:
> ...
>   
>> Second:
>> 8x-----8x------
>> rlm_sql_postgresql: Status: PGRES_TUPLES_OK
>> rlm_sql_postgresql: query affected rows = 3 , fields = 5
>> rlm_sql (sql): Read entry nasname=1.2.3.4,shortname=nume,secret=secret
>> rlm_sql (sql): Adding client 1.2.3.4 (nume) to clients list
>> Segmentation fault (core dumped)
>>     
>
>   OK.  I don't put clients into SQL, so I haven't tested that portion of
> the code.
> ...
>   
>>      So, we free the same location..
>>      I guess the problem is in the clients_parse_section which doesnt
>>      return a new address space.
>>     
>
>   A better solution is this:
>
>   - remove the "read clients from SQL" code in src/modules/rlm_sql.
>   - add configuration to the "clients" section, e.g.:
>
> client 192.168.0.0/16 {
>       query = "%{sql: SELECT ....}
> }
>   
Hmm that would mean stil having to add client entris in the 
clients.conf. We 'd like to avoid that when using sql.
Something like:
clients.conf:
per_socket_clients {
clients_query = "%{sql: SELECT .... }
}

>   And have it do the SELECT, and parse the result at run time.  It will
> take a bit of work to add that, but it's a much better solution.
>
>   Alan DeKok.
> --
>   http://deployingradius.com       - The web site of the book
>   http://deployingradius.com/blog/ - The blog
> - 
> List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html
>   

Kostas Kalevras
Network Operations Center - National Technical University of Athens
http://kkalev.wordpress.com/
- 
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html

Reply via email to