On Mon, 7 Jan 2002, Chris Parker wrote:

> At 11:51 AM 1/5/2002 +0200, Igor Chen wrote:
> >Seems that radius can't reach sql-user-name when updating session with
> >Alive record...
> >---
> >rlm_sql: Reserving sql socket id: 4
> >query: UPDATE radacct SET FramedIPAddress = '217.12.196.221' WHERE
> >AcctSessionId = '00000003' AND UserName = '' AND NASIPAddress=
> >'217.12.196.130'
> >rlm_postgresql Status: PGRES_COMMAND_OK
> >sql_postgresql: affected rows = 0
> >--
> >Why UserName = '' ? :-\ strange, because first query INSERT get's username
> >:-\
> 
> What do you have set in 'sql.conf' for 'sql_user_name'?
> 
> Also, check the 'sql.conf' to be sure that it specifies '%{SQL-User-Name}'.

cvs files:

sql.conf:

accounting_update_query = "UPDATE ${acct_table1} SET FramedIPAddress =
'%{Framed-IP-Address}' WHERE AcctSessi
onId = '%{Acct-Session-Id}' AND UserName = '%{SQL-User-Name}' AND NASIPAddress=
                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
'%{NAS-IP-Address}'"

rlm_sql.c:

 case PW_STATUS_ALIVE:

                        radius_xlat(querystr, MAX_QUERY_LEN,
inst->config->accounting_update_query, request, NULL);
                        query_log(inst, querystr);


whilst for ACCOUNTING_START/STOP packets:

case PW_STATUS_START:

                        /*
                         * Set, escape, and check the user attr here
                         */
                        if(sql_set_user(inst, request, sqlusername, 0) < 0) {
                                sql_release_socket(inst, sqlsocket);
                                return RLM_MODULE_FAIL;
                        }

                        radius_xlat(querystr, MAX_QUERY_LEN,
inst->config->accounting_start_query, request, NULL);
                        query_log(inst, querystr);


We *really* need to do an sql_set_user for Update packets or change sql.conf to
read 
UserName = '%{User-Name}'

--
kkalev


> 
> -Chris
> --
>     \\\|||///  \  Chris Parker    -    Manager, Development Engineering
>     \ ~   ~ /   \       WX *is* Wireless!    \   [EMAIL PROTECTED]
>     | @   @ |    \   http://www.starnetwx.net \      (847) 963-0116
> oOo---(_)---oOo--\------------------------------------------------------
>                    \ Without C we would have 'obol', 'basi', and 'pasal'
> 
> 
> - 
> List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html
> 


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

Reply via email to