8 Jan
2007
8 Jan
'07
11:16 a.m.
On Tue 02 Jan 2007 00:41, FreeRadius Devel wrote:
hello,
in case of a fatal error that occurs on the postgresql database server the sql accounting module returns "ok" so the "redundancy" feature of freeradius does not work. in file src/modules/rlm_sql/drivers/rlm_sql_postgresql/sql_postgresql.c, function sql_check_error I see that for non "SQL_DOWN" errors it returns 0, should it be -1 ? a non zero value ?
You Are correct. Fixed now. Look out for a further monster (60+ line) patch to sql_postgresql.c in the next few hours. I am just putting it through testing at the moment. It fixes some other edge cases and a memory leak reported in Bug 413 Cheers -- Peter Nixon http://www.peternixon.net/ PGP Key: http://www.peternixon.net/public.asc